【发布时间】:2016-05-08 01:52:58
【问题描述】:
在以下代码中释放 malloc 的字符串时出现错误:
long authenticate(char* user, long* uid, long* gid){
FILE* fp = fopen("/etc/passwd", "r");
char * line = NULL;
size_t len = 0;
long lineSize;
char* uname;
if (fp == NULL)
exit(EXIT_FAILURE);
while ((lineSize = getline(&line, &len, fp)) != -1) {
uname = (char*)malloc(lineSize*sizeof(char));
getInfo(line, lineSize, &uname, &uid, &gid);
if(strcmp(uname, user) == 0) { // Strings are equal
printf("we've found a user!\n");
printf("uname: %s uid: %ld gid: %ld\n", uname, *uid, *gid);
free(uname);
exit(1);
}
free(uname);
}
}
每当我找到一个有效的用户名,免费uname 然后退出什么都没有发生。但是,当我在循环结束时尝试释放 uname 时找不到有效的 uname(如果语句未得到评估),我会收到以下错误:
malloc: *** 对象 0x10d20ef2c 错误:未分配指针被释放
我不明白为什么 free() 没有注册 uname 以前是 malloc 的。鉴于错误的性质,我认为问题在于我不了解malloc/free。
【问题讨论】:
-
使用
gcc -Wall -Wextra -g编译您的代码,然后使用gdb和valgrind 对其进行调试。 BTWuname是一个 POSIX 函数,所以最好使用其他名称。 -
Off-by-one:
getline()返回缓冲区中的字符数不包括终止'\0'。您的 malloc'ed 缓冲区太小。其余的可能只是 UB 的结果。 -
感谢您对名称的提醒。当我使用 gdb 逐步完成它时,我在 strcmp 上得到一个段错误:(
-
我不确定
getInfo在内部做什么,但很可能,它可能正在调整uname的值(即更改通过uname++存储的地址,例如示例)。 -
@dhke getline() 实际上占了额外的字符
标签: c pointers memory-management malloc free