急急急！！请教链表的问题

zhanner

creatoutlist为建立含一个节点的链表的函数。经gcc编译，程序执行时
输出
malloc of 24 bytes success!
saddrip=210.76.120.6,length=32342
segmentation fault(core dumped)

在strcpy(s->;saddr,saddrip);   开始出现故障。

#define LEN sizeof(linkoutlist)

typedef struct outnode
{
  char saddr[16];
  unsigned long length;
  struct outnode * next;
}linkoutlist;


void *emalloc(unsigned int n)
{
    void *p;
    p = malloc(n);
    if(p=NULL)
       printf("malloc of %u bytes failed:\n",n);
    else
       printf("malloc of %u bytes success!\n",n);
    return p;
       
}

linkoutlist *creatoutlist(char *saddrip,unsigned long length)
{
    linkoutlist * head,* s;
   // head = (linkoutlist *) emalloc(LEN);
   
    head = NULL;  /* outlist链表开始为空 */
     
    s = (linkoutlist *) emalloc(LEN);
  
   // s->;saddr = saddrip;
    printf("saddrip=%s,length=%lu.\n",saddrip,length);

    strcpy(s->;saddr,saddrip);   //出现故障？？？
     s->;length = length;      //出现故障？？？？
   
    printf("333333\n";
    s->;next = head;
    head = s;
    return head;
   
}


请教：
1。  malloc of 24 bytes success!  中24是怎么得出来的？
2。为何strcpy(s->;saddr,saddrip);   //出现故障？？？
     我该如何做，才能把saddrip=210。76。120。6信息存放到链表中？

abyss

1.typedef struct outnode
{
char saddr[16];
unsigned long length;
struct outnode * next;
}linkoutlist;
这个结构的大小是16+4+4=24,所以malloc了24个字节。
2.这个问题我不知道为什么...看起来并没有错啊.

zhanner

谢谢abyss.

我还想向你请教     16+4+4=24 中struct outnode * next为何有四个字节 ，能详细说明一下吗？

abyss

问题可能不在这两句
strcpy(s->;saddr,saddrip); //出现故障？？？
s->;length = length; //出现故障？？？？
而在
s = (linkoutlist *) emalloc(LEN);
你定义的局部变量s没有初始化，第一次执行这个函数可能没有什么问题，但是当第二次进入这个函数体的时候，就可能产生segmentation fault了，试着把s初始化为NULL……

abyss

struct outnode * next
该结构成员为一个指针变量，在32位操作系统里面，所有指针都是32位的，也就是4个字节。

zhanner

谢谢abyss，我先试试，有问题在向你请教

abyss

好，没问题。

zhanner

我在strcpy(s->;saddr,saddrip); //出现故障？？？ 前先初始化s ，s=NULL  可是还是不行，提示的错误和以前的一样。

这问题到底出在哪里呢？

abyss

可不可以把完整的程序发出来一起研究？

li2002

讨论得好热烈，呵呵，不过问题不是这里！
看看：

1. 
   
2. void *emalloc(unsigned int n)
   
3. {
   
4. void *p;
   
5. p = malloc(n);
   
6. if(p=NULL)
   
7. printf("malloc of %u bytes failed:\n",n);
   
8. else
   
9. printf("malloc of %u bytes success!\n",n);
   
10. return p;
    
11. 
    
12. }
    
13. 
    

复制代码

里面的一句if(p=NULL)
发现问题了吗？？
应为if(p==NULL)
这个问题c陷阱那本书提及了，要仔细阿。