今天在写一个堆的程序的时候,分别在Ubuntu12.04的gcc上面跑,使用gdb单步跟进的时候可以看得到正常申请内存以及正常释放。
但是在VS2013上面跑,出现以下的错误。
先不说,贴上关键代码
(1、需要用到的结构体 2、致命错误代码)
typedef struct { char * name; int age; }human;
下面这段代码是通过使用堆内存动态添加姓名以及年龄的小程序
char fname[2048] = { 0 }; int age = 0; printf("请输入姓名:"); scanf("%s", fname); printf("请输入年龄:"); scanf("%d", &age); if (strcmp(fname, "exit") == 0) break; human * tmp = (human *)malloc(sizeof(human)); tmp->name = (char *)calloc(sizeof(strlen(fname) + 1), sizeof(char)); strcpy(tmp->name, fname); tmp->age = age; p = (human**)realloc(p, sizeof(human**)* (index + 1)); *(p + index) = tmp; free(p[index]->name); ++index;
当直接执行到这一步的时候程序就崩了。。。。
初步诊断:应该在free的时候有地址错误的被free了
打印debug信息:
发现所有地址都已经正确获取,并没有出现free错的嫌疑
在上述这一步操作连续做了接近1个半小时,但是上述操作在Ubuntu12.04的gcc跑 的时候正常free掉。
再次诊断:估计是自家编译器出问题
解决方案:移植到其他电脑再次尝试。
结果是错误信息重复出现,证明BUG依旧存在
最后一次诊断:重新看写的代码
发现问题:
tmp->name = (char *)calloc(sizeof(strlen(fname) + 1), sizeof(char));
发现在申请name内存的时候手误多写了一个sizeof(),导致这片内存分配了错误的空间大小,而在后续的strcpy的时候往这片内存区域存放的信息大于分配的空间,因为修改了申请的内存以外的内存,从而导致释放指针指向的堆内存出现错误
最终解决方案:
tmp->name = (char *)calloc(strlen(fname) + 1, sizeof(char));
希望大家不要再犯这个错误了。