一个段简单程序怎么出的错,大家帮看看

cmh_2003

不多说了直接看代码.

1. //
   
2. //统计line<550的个数
   
3. //
   
4. 
   
5. #include<stdio.h>
   
6. #include<stdlib.h>
   
7. #include<string.h>
   
8. #include<unistd.h>
   
9. int main()
   
10. {
    
11.         int i = 0,j,num;
    
12.         FILE *fd;
    
13.         char fline[200],*a,*b;
    
14.        
    
15.         if(  ( fd = fopen("/home/changminghua/code/qyld.log.2007-03-01.11","r") ) == NULL  )
    
16.         printf("open log error");
    
17.        
    
18.         a = malloc(200);
    
19.         b = malloc(10);
    
20.        
    
21. 
    
22.         while(!feof(fd))
    
23.         {
    
24.                 memset(a,0x00,200);
    
25.                 memset(b,0x00,10);
    
26.                 memset(fline,0x00,200);
    
27.                 fgets(fline,200,fd);
    
28.        
    
29.                 if( (a = strstr(fline,"line=")) != NULL  )
    
30.                 {
    
31.                         for(j = 0 ; *(a+5+j) != ',';j++)        //1。取line=后面的整数
    
32.                         *(b+j) = *(a+5+j);        //
    
33.                         if ( (num=atoi(b))<550 )        //2。统计line小于550的个数       
    
34.                         i = i+1;
    
35.                 };
    
36.         }
    
37.        
    
38.        
    
39.         fclose(fd);
    
40.         free(a);
    
41.         free(b);
    
42.         printf("line <550 的个数为 %d\n",i);
    
43.         return 0;
    
44. }
    

复制代码

gdb第二次跟到fgets这里就报段错

27                      fgets(fline,200,fd);
(gdb)

Program received signal SIGSEGV, Segmentation fault.
0x40080454 in fgets () from /lib/libc.so.6

lenovo

这个程序错误很多。
首先不应该使用while(!feof(fd))这种方式
来判断文件的结束。原因本版已有很多描述，
自己搜吧。
在if( (a = strstr(fline,"line=")) != NULL  )
中，a先前所指的空间发生内存泄露。a指向fline数组的某个位置，
而在第二次的循环中，memset(a,0x00,200);
又对a所的内容清零，已经发生了数组访问越界。

自己好好想一想你的逻辑有没有问题。

cmh_2003

谢谢,我知道是哪里的错了.但是发生错误之后为什么第二次fd为0了?

1. 4              while(!feof(fd))
   
2. (gdb)
   
3. 26                      memset(a,0x00,200);
   
4. (gdb) p fd
   
5. $2 = (FILE *) 0x8049a38
   
6. (gdb) n
   
7. 27                      memset(b,0x00,10);
   
8. (gdb) n
   
9. 28                      memset(fline,0x00,200);
   
10. (gdb) p fd
    
11. $3 = (FILE *) 0x0

复制代码

lenovo

原帖由 cmh_2003 于 2007-3-2 09:59 发表
谢谢,我知道是哪里的错了.但是发生错误之后为什么第二次fd为0了?
[code]4              while(!feof(fd))
(gdb)
26                      memset(a,0x00,200);
(gdb) p fd
$2 = (FILE *) 0x8049a38
(gdb)  ...

你数组访问越界，
在memset时，可能把fd所在的内存空间
也给清零了。
遇到问题还是多想想吧。