fclose的时候出现段错误

sagarfan

gateway_file.h

1. #ifndef _GATEWAY_FILE_H_
   
2. #define _GATEWAY_FILE_H_
   
3. #include <stdio.h>
   
4. #include <stdlib.h>
   
5. #include <string.h>
   
6. 
   
7. typedef struct DeviceInfo
   
8. {
   
9.         char type[10];
   
10.         int id;
    
11. }DeviceInfo;
    
12. FILE *Fopen(char *path);
    
13. int ReadFile(FILE *fp);
    
14. void Fprint(DeviceInfo *di,int len);
    
15. void Fclose(FILE *fp);
    
16. 
    
17. #endif
    

复制代码

gateway_file.c

1. #include "gateway_file.h"
   
2. 
   
3. #define USE_DEBUG
   
4. #ifdef USE_DEBUG
   
5. #define DEBUG_ERR(fmt, args...) printf("\033[46;31m[line:%d]\033[0m "#fmt"\r\n",__LINE__, ##args)
   
6. #define DEBUG_INFO(fmt, args...) printf("\033[33m[line:%d]\033[0m "#fmt"\r\n",__LINE__, ##args)
   
7. #else
   
8. #define DEBUG_ERR(fmt, ...)
   
9. #define DEBUG_INFO(fmt, ...)
   
10. #endif
    
11. 
    
12. DeviceInfo *record;
    
13. FILE *Fopen(char *path)
    
14. {
    
15.         FILE *fp = fopen(path,"r");
    
16.         if(fp == NULL)
    
17.                 DEBUG_ERR(open file %s failed,path);
    
18.         return fp;
    
19. }
    
20. 
    
21. int ReadFile(FILE *fp)
    
22. {
    
23.         long len;
    
24.         int i = 0;
    
25.         char *type=NULL;
    
26.         char *id=NULL;
    
27.         char buffer[128] = "";
    
28.         fseek(fp,0,SEEK_END);
    
29.         len = ftell(fp);
    
30.         printf("length for ftell in Read = %ld\n",len);
    
31.         fseek(fp,0,SEEK_SET);
    
32.         record = (DeviceInfo*)malloc(len);
    
33.         //memset(record,0,len+40);
    
34.         while(fgets(buffer,sizeof(buffer),fp) != NULL)
    
35.         {
    
36.                 type = strtok(buffer," ");
    
37.                 id = strtok(NULL," ");
    
38.                 if(type != NULL && id != NULL)
    
39.                 {
    
40.                         printf("sizeof type = %d\n",strlen(type));
    
41.                         printf("sizeof id = %d\n",strlen(id));
    
42.                         strcpy(record[i].type,type);
    
43.                         record[i].id = atoi(id);
    
44.                         i++;
    
45.                 }
    
46.         }
    
47.         return i;
    
48. }
    
49. 
    
50. void Fprint(DeviceInfo *di,int len)
    
51. {
    
52.         int i;
    
53.         printf("length=%d\n",len);
    
54.         for(i=0; i<len; i++)
    
55.         {
    
56.                 printf("record[%d].type=%s\t",i,di[i].type);
    
57.                 printf("record[%d].id=%d\n",i,di[i].id);
    
58.         }
    
59. }
    
60. void Fclose(FILE *fp)
    
61. {
    
62.         printf("%x\n",fp);
    
63.         if(fp != NULL)
    
64.         {
    
65.                 fprintf(stderr,"bug5...\n");
    
66.                 fclose(fp);
    
67.                 fprintf(stderr,"bug6...\n");
    
68.         }
    
69. }
    

复制代码

test_read_record.c

1. #include "gateway_file.h"
   
2. 
   
3. extern DeviceInfo *record;
   
4. int main()
   
5. {
   
6.         int length;
   
7. 
   
8.         FILE *fp = Fopen("record.txt");
   
9.         fprintf(stderr,"bug1...\n");
   
10.         length = ReadFile(fp);
    
11.         printf("length in test = %d\n",length);
    
12.         fprintf(stderr,"bug2...\n");
    
13.         Fprint(record,length);
    
14.         fprintf(stderr,"bug3...\n");
    
15.         free(record);
    
16.         fprintf(stderr,"bug4...\n");
    
17.         length = ftell(fp);
    
18.         printf("length for ftell = %d\n",length);
    
19.         Fclose(fp);
    
20. 
    
21.         return 0;
    
22. }
    

复制代码

record.txt

PMAC901 12
PMAC901 12
PMAC901 12
PMAC901 22
PMAC901 32
gateway_file.c封装了读取文件，以及打印数据的一些函数
test_read_record.c为测试文件
gcc test_read_record.c gateway_file.c -o test_read_record
运行./test_read_record，在调用Fclose时，出现段错误

hellioncu

record = (DeviceInfo*)malloc(len);

这里len应改为记录数＊记录大小，跟文件大小不是一回事

sagarfan

你的意思是每一行的大小*行数吗？但是文件内容的大小len不也是这个意思吗？回复 2# hellioncu


   

hellioncu

回复 3# sagarfan

sizeof(DeviceInfo)＊行数

   

heguangwu

gdb用bt打出堆栈信息来看看，感觉不是fclose的错误

coolmoon_133319

这是写越界了，错误原因在于前面朋友所说的申请内存小了。
record = (DeviceInfo*)malloc(len);，

sagarfan

嗯，我是按照char*类型来申请内存的，忘记了结构式是字节对齐的回复 4# hellioncu