c内存释放出错

bjkillall

上面的截图是小弟的程序在运行结束时的提示，查找了好久结果也没有找到，知道本版有很多大牛，所以发上来跟大家咨询一下；
下面是我的内存分配和回收部分的代码；

【分配一】
222     char* seqing[685];
223     int a;
224     for(a=0;a<685;a++)
225     {
226         seqing[a]=(char*)malloc(sizeof(char)*1024);
227     }

【分配二】
252     char* item[5];
253     int m;
254     for(m=0;m<5;m++)
255     {
256         item[m]=(char*)malloc(sizeof(char)*204;
257     }


【回收】
317     for(n=0;n<685;n++)
318     {
319         free(seqing[n]);
320     }
321     for(n=0;n<5;n++)
322     {
323         free(item[n]);
324     }

大家看看有没有什么问题，谢谢了～



应大家要求我把main函数的代码发给大家；同时大家也可以给一下优化建议；

1. int main(int argc,char* argv[])
   
2. {
   
3.     if (argc!=3)
   
4.     {
   
5.         printf("[USAGE:] %s matchFile illegalFile.\n",argv[0]);
   
6.         return 0;
   
7.     }
   
8. 
   
9.     FILE* matchFile=fopen(argv[1],"r");
   
10.     assert(matchFile!=NULL);
    
11. 
    
12.     FILE* illegalFile=fopen(argv[2],"r");
    
13.     assert(illegalFile!=NULL);
    
14. 
    
15.     FILE* HEFA=fopen("./hefa.txt","a+");
    
16.     assert(HEFA!=NULL);
    
17. 
    
18.     FILE* FEIFA=fopen("./feifa.txt","a+");
    
19.     assert(FEIFA!=NULL);
    
20. 
    
21.     FILE* FEIFA_LINE=fopen("./feifa_line.txt","a+");
    
22.     assert(HEFA!=NULL);
    
23. 
    
24.     FILE* FEIFA_DETAIL=fopen("./feifa_detail.txt","a+");
    
25.     assert(HEFA!=NULL);
    
26. 
    
27.     char* seqing[685];
    
28.     int a;
    
29.     for(a=0;a<685;a++)
    
30.     {
    
31.         seqing[a]=(char*)malloc(sizeof(char)*1024);
    
32.     }
    
33.     //memset(seqing,0,sizeof(seqing));
    
34.     char buffer[1024];
    
35.     memset(buffer,0,sizeof(buffer));
    
36.     int i=0;
    
37.     while((fgets(buffer,1023,illegalFile)!=NULL)&&(i<685))
    
38.     {
    
39.         char* ptest=strtrim_ln(buffer);  //这一行的目的是将fgets读进来的'\n'转化成'\0'
    
40.         strcpy(seqing[i],ptest);
    
41.         i++;
    
42.     }
    
43. 
    
44. 
    
45. 
    
46.     int line[685];   
    
47.     for(i=0;i<685;i++)
    
48.     {
    
49.         line[i]=0;
    
50.     }
    
51.     int detail[685];
    
52.     for(i=0;i<685;i++)
    
53.     {
    
54.         detail[i]=0;
    
55.     }
    
56. 
    
57.     char* item[5];
    
58.     int m;
    
59.     for(m=0;m<5;m++)
    
60.     {
    
61.         item[m]=(char*)malloc(sizeof(char)*2048);
    
62.     }
    
63.     char sbuffer[2048];
    
64.     while(fgets(sbuffer,2047,matchFile)!=NULL)       //sbuffer的格式如下 ： 字符串'\t'数字 ,'\t'表示两者之间的分割符；
    
65.     {
    
66.         char middle[2047];
    
67.         memset(middle,0,sizeof(middle));
    
68.         strcpy(middle,sbuffer);
    
69.         char* pmiddle=strtrim_ln(middle);
    
70. 
    
71. 
    
72.         SplitStr(pmiddle,item,5,'\t');    //该行的目的是以'\t'为标记切割pmiddle，分别存在item[0]到item[4]中，最大分割成5个部分
    
73. 
    
74.         int i;
    
75.         int flag=0;
    
76.         for(i=0;i<685;i++)
    
77.         {
    
78.             if(compare(item[0],seqing[i])==0)  //该行的目的就是匹配item[0]和seqing[i]，成功返回0，并且将相关信息写入文件；
    
79.             {
    
80.                 flag=1;
    
81.                 line[i]=line[i]+1;
    
82.                 int mid=atoi(item[1]);
    
83.                 detail[i]=detail[i]+mid;
    
84.                 fwrite(item[0],strlen(item[0]),1,FEIFA);
    
85.                 fwrite("\t",1,1,FEIFA);
    
86.                 fwrite(item[1],strlen(item[1]),1,FEIFA);
    
87.                 fwrite("\n",1,1,FEIFA);
    
88.                 break;
    
89.             }
    
90.         }
    
91.         if(0 == flag)
    
92.         {
    
93.             fwrite(item[0],strlen(item[0]),1,HEFA);
    
94.             fwrite("\t",1,1,HEFA);
    
95.             fwrite(item[1],strlen(item[1]),1,HEFA);
    
96.             fwrite("\n",1,1,HEFA);
    
97.         }
    
98. 
    
99.     }
    
100. 
     
101.     int n=0;
     
102. 
     
103.     for(n=0;n<685;n++)
     
104.     {
     
105.         fwrite(seqing[n],strlen(seqing[n]),1,FEIFA_LINE);
     
106.         fwrite("\t",1,1,FEIFA_LINE);
     
107.         char str1[100];
     
108.         memset(str1,0,sizeof(str1));
     
109.         sprintf(str1,"%d",line[n]);
     
110.         fwrite(str1,strlen(str1),1,FEIFA_LINE);
     
111.         fwrite("\n",1,1,FEIFA_LINE);
     
112. 
     
113.         fwrite(seqing[n],strlen(seqing[n]),1,FEIFA_DETAIL);
     
114.         fwrite("\t",1,1,FEIFA_DETAIL);
     
115.         char str2[100];
     
116.         memset(str2,0,sizeof(str2));
     
117.         sprintf(str2,"%d",detail[n]);
     
118.         fwrite(str2,strlen(str2),1,FEIFA_DETAIL);
     
119.         fwrite("\n",1,1,FEIFA_DETAIL);
     
120.     }
     
121. 
     
122.     for(n=0;n<685;n++)
     
123.     {
     
124.         free(seqing[n]);
     
125.     }
     
126.     for(n=0;n<5;n++)
     
127.     {
     
128.         free(item[n]);
     
129.     }
     
130. 
     
131.     fclose(matchFile);
     
132.     fclose(illegalFile);
     
133.     fclose(HEFA);
     
134.     fclose(FEIFA);
     
135.     fclose(FEIFA_LINE);
     
136.     fclose(FEIFA_DETAIL);
     
137.     return 0;
     
138. }

复制代码

好了，有一些自己写的函数我已经在后面标记了，如果有什么地方有问题大家可以提出来，帮我分析下这个问题吧，我搞不定了～

donglongchao

感觉没啥问题。LZ贴完整测试代码吧。

lenky0401

请先试着检查一下malloc返回值 不是所有malloc都执行成功 如何返回NULL，那你后面free空指针可能会有问题

hellioncu

应该是中间把指针给破坏了，free NULL不会有问题的

mournjust

回复 1# bjkillall


感觉LZ对指针的概念还不是很熟悉
char* seqing[685];是一个指向数组的指针，而不是像LZ所想这个数组中的元素是指针。

ilwmin

回复  bjkillall


感觉LZ对指针的概念还不是很熟悉
char* seqing[685];是一个指向数组的指针，而不是 ...
mournjust 发表于 2010-09-28 15:33

是你错了

ilwmin

回复 5# mournjust


    是你弄错了.楼主那样定义没问题.

surpass_li

论坛ID

回复  bjkillall


感觉LZ对指针的概念还不是很熟悉
char* seqing[685];是一个指向数组的指针，而不是 ...
mournjust 发表于 2010-09-28 15:33

lz没有错..lz的seqing是色情的意思吗

bjkillall

回复 9# 论坛ID


    这是一个不能说的秘密～哈哈