一段C程序老是出現 Segmentation fault?

gangjh

請各位幫忙分析一下,問題在哪?
謝謝

#include <stdio.h> #include <stdlib.h> #include <string.h> #include "socket/socketutl.h" #define BSIZE 1024 struct sockfile { &nbsp;&nbsp;&nbsp;&nbsp;char buf[BSIZE +1]; &nbsp;&nbsp;&nbsp;&nbsp;char *tok , *bot; &nbsp;&nbsp;&nbsp;&nbsp;char *lim; &nbsp;&nbsp;&nbsp;&nbsp;int sock ; &nbsp;&nbsp;&nbsp;&nbsp;int state ; //socket state }; int sk_open(struct sockfile *sf, int sock) { &nbsp;&nbsp;&nbsp;&nbsp;sf->tok = sf->buf; &nbsp;&nbsp;&nbsp;&nbsp;sf->lim = sf->buf ; &nbsp;&nbsp;&nbsp;&nbsp;sf->bot = sf->buf; &nbsp;&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp;&nbsp;sf->sock = sock ; &nbsp;&nbsp;&nbsp;&nbsp;sf->state = 0;   &nbsp;&nbsp;&nbsp;&nbsp;return 0; } int sk_fill(struct sockfile *sf) { &nbsp;&nbsp;&nbsp;&nbsp;if  (sf->state <0) { &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;return  -1 ; &nbsp;&nbsp;&nbsp;&nbsp;} &nbsp;&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp;&nbsp;int cnt = sf->lim - sf->tok; &nbsp;&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp;&nbsp;if(cnt>0){ &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;memcpy(sf->buf, sf->tok, cnt); &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;sf->tok = sf->buf ; &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;sf->bot = sf->buf ; &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;sf->lim = sf->buf + cnt ; &nbsp;&nbsp;&nbsp;&nbsp;} &nbsp;&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp;&nbsp;int rcnt  = BSIZE - cnt ; &nbsp;&nbsp;&nbsp;&nbsp;if((cnt = recv(sf->sock, sf->lim, rcnt, 0)) >0){ &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;sf->lim[cnt] = '\0'; &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;sf->lim += cnt ; &nbsp;&nbsp;&nbsp;&nbsp;} else { &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;sf->state=-1 ; &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;return -1 ; &nbsp;&nbsp;&nbsp;&nbsp;} &nbsp;&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp;&nbsp;return  0; } int sk_readline(struct sockfile *sf, char **line, int *len) { &nbsp;&nbsp;&nbsp;&nbsp;char *ptr ; start: &nbsp;&nbsp;&nbsp;&nbsp;sf->bot = sf->tok ; &nbsp;&nbsp;&nbsp;&nbsp;ptr = sf->bot ; &nbsp;&nbsp;&nbsp;&nbsp;while (ptr < sf->lim) { &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;if (*ptr== '\n') { &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;if (ptr[-1] =='\r') { &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;ptr[-1] = '\0' ; &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;} &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;*ptr='\0'; &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;*len = ptr - sf->bot -1; &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;sf->tok = ++ptr ; &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;*line =sf->bot ; &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;return 0; &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;} &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;ptr ++; &nbsp;&nbsp;&nbsp;&nbsp;} &nbsp;&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp;&nbsp;if (sf->state <0){ &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;return -1; &nbsp;&nbsp;&nbsp;&nbsp;} &nbsp;&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp;&nbsp;sk_fill(sf) ; &nbsp;&nbsp;&nbsp;&nbsp;goto start ; } int main() { &nbsp;&nbsp;&nbsp;&nbsp;int sock; &nbsp;&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp;&nbsp;char *host = "172.17.2.43"; &nbsp;&nbsp;&nbsp;&nbsp;if ((sock= tcp_connect(host, 80)) < 0) { &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;printf("connect  error\n") ; &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;return -1 ; &nbsp;&nbsp;&nbsp;&nbsp;} &nbsp;&nbsp;&nbsp;&nbsp;char header[1024] ; &nbsp;&nbsp;&nbsp;&nbsp;sprintf(header,"%s%s%s%s", &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;"Pragma:no-cache\r\n", &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;"Accept-Encoding:GZIP\r\n", &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;"Connection:close\r\n", &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;"User-Agent: Mozilla/4.0 (compatible; MSIE 5.5; Windows NT5)\r\n"); &nbsp;&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp;&nbsp;tcp_send(sock, "GET %s HTTP/1.0\r\n%s   \r\n\r\n", "/auto_update/y3_cost/md5sum.txt", header); &nbsp;&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp;&nbsp;char *line =NULL; &nbsp;&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp;&nbsp;struct sockfile sk ; &nbsp;&nbsp;&nbsp;&nbsp;sk_open(&sk, sock); &nbsp;&nbsp;&nbsp;&nbsp;int cnt ; &nbsp;&nbsp;&nbsp;&nbsp;int len=0; &nbsp;&nbsp;&nbsp;&nbsp;while (1) { &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;cnt =sk_readline(&sk, &line, &len); &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;if (cnt <0) break ; &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;printf("size:%3d [%s]\n\n", len, line); &nbsp;&nbsp;&nbsp;&nbsp;} &nbsp;&nbsp;&nbsp;&nbsp;close(sock); &nbsp;&nbsp;&nbsp;&nbsp;exit(1); }

Godbach

那就是可以编译成可执行文件了吧。
这样建议lZ用GDB调试一下，
编译时加上-g选项，
gcc -g file.c -o file
然后程序到哪一行Fault了，GDB就结束了，这样就可以找出来了

ruchong

或者设置下
ulimit -c unlimited

然后运行程序...
core的时候出现core文件
gdb test core调试下....

gangjh

謝謝大家,用gdb調試了一下,
是這行 " sf->lim[cnt] = '\0';' 訪問出錯,
改成這樣就好了.

1. 
   
2.     if(cnt>0){
   
3.         memcpy(sf->buf, sf->tok, cnt);
   
4.         sf->tok = sf->buf ;
   
5.         sf->bot = sf->buf ;
   
6.         sf->lim = sf->buf + cnt ;
   
7.     }
   
8. 改成這樣
   
9.     if(cnt>0){
   
10.                 memcpy(sf->buf, sf->tok, cnt);
    
11.         }
    
12.         sf->tok = sf->buf ;
    
13.         sf->bot = sf->buf ;
    
14.         sf->lim = sf->buf + cnt ;
    
15. 
    
16. 
    

复制代码

Godbach

呵呵，好，自己解决了问题

DraculaW

看到段错误 马上查当时使用的内存 一般都会找到的

gangjh

呵
謝謝大家了

Linux下的socket 用fdopen 打開, 在用gnu getline有多方便.
也不用繞這個大彎子了, 函數還有些問題,我再調整調整.

fangbing007

指针问题!