我想在下面的函数中返回指向链表头部结构的指针,但由于我的fread不能读取struct的字符串,我必须使用malloc一些变量并指向newheader-> name string它。在以下函数中如果我释放我的临时变量然后返回新标题只打印空值,如果我不知道然后我得到内存泄漏但函数工作正常。我希望能够返回完整链表的头部而不是空值。
struct head *read_strings(char *file) {
FILE *my_file;
struct head * newheader = NULL;
char *str = NULL;
newheader = buildhead();
int len, lenStr, lenTotal = 0;
printf("\n\n");
my_file = fopen(filename, "rb");
if (my_file == NULL) {
printf("error\n");
}
fread(&len,sizeof(int),1, my_file);
newheader->name = malloc(len);
fread(newheader->name,sizeof(char),len, my_file);
fread(&newheader->length,sizeof(int),1, my_file);
if (newheader->length != 0) {
if (newheader->length == lenTotal) {
fread(&lenStr,sizeof(int),1, my_file);
str = malloc(lenStr); //char *str = malloc(lenStr);
fread(str,sizeof(char),lenStr, my_file);
create_string(newheader, str);
lenTotal += lenStr;
str = NULL; //free(str);
}
else {
while (newheader->length != lenTotal) {
fread(&lenStr,sizeof(int),1, my_file);
str = malloc(lenStr); //char *str = malloc(lenStr);
fread(str,sizeof(char),lenStr, my_file);
create_string(newheader, str);
lenTotal += lenStr;
str = NULL; //free(str);
}
}
}
printString(newheader);
fclose(my_file);
free(str);
//if i free newheader->name here then I get no memory leaks
//free(newheader->name);
freeStructure(newheader);
return newheader;
}
我需要从二进制文件中读取字符串并将它们存储到struct中。但我不能将值直接存储到struct的字符串变量,除非我malloc一个新的字符串并指向它。您可以在上面的代码中看到我能够从fread而不是newheader->名称读取newheader->长度。我试图将fread结果放在一个字符串数组中,但是出现了分段错误。
这是我的二进制文件的样子。它在每个字符串和int之后都有null终止符。
0000000 021 \0 \0 \0 i t i s a g o o d
0000020 d a y \0 R \0 \0 \0 # \0 \0 \0 I t
0000040 w a s t h e b e s t o f
0000060 o y e h o y e t i m e s . \0
0000100 034 \0 \0 \0 I t w a s t h e b
0000120 l u r s t o f t i m e s . \0
0000140 \a \0 \0 \0 m o n k e y \0 006 \0 \0 \0 p
0000160 a n d a \0 006 \0 \0 \0 s h i f u \0
0000177
如果我在my freestruct()函数中释放newheader->名称,则会出现以下错误
*** Error in `./test': munmap_chunk(): invalid pointer:
0x000000000040133e *** Aborted.
如果不释放它,那么我在1个区块中丢失了17个字节。
==24169== HEAP SUMMARY:
==24169== in use at exit: 0 bytes in 0 blocks
==24169== total heap usage: 5 allocs, 6 frees, 1,201 bytes allocated
==24169==
==24169== All heap blocks were freed -- no leaks are possible
==24169==
==24169== ERROR SUMMARY: 1 errors from 1 contexts (suppressed: 0 from0)
==24169==
==24169== 1 errors in context 1 of 1:
==24169== Invalid free() / delete / delete[] / realloc()
==24169== at 0x4C29E90: free (in
/usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==24169== by 0x401298: freestruct (A1.c:267)
==24169== by 0x400F8A: write_strings (A1.c:189)
==24169== by 0x400840: main (test.c:25)
==24169== Address 0x40133e is not stack'd, malloc'd or (recently)
free'd
==24169==
==24169== ERROR SUMMARY: 1 errors from 1 contexts (suppressed: 0 from0)
这是我的freestruct()函数的一小段代码
if (header->next == NULL) {
header->length = 0;
free(header->name);
free(header);
}
将字符串写入二进制文件后,我释放了整个结构,然后在read_strings函数中再次创建
编辑:如果我在readString函数中释放newheader->名称,那么我没有内存泄漏,但我希望将newheader返回给main,然后调用printString和freestruct。
我为我糟糕的语法道歉。
答案 0 :(得分:0)
如果我正确理解您的setName
和getName
功能,那么这三行就是问题所在:
...
setName(newheader, name);
newheader->name = getName(newheader);
...
free(name);
首先,两条第一行做同样的事情,但这不是问题,只是多余。这是导致问题的最后一行。
您使newheader->name
指向name
指向的相同内存。赋值后,两个指针都指向同一个内存。然后你释放记忆。这使得两个指针无效。
您需要重复(例如,使用非标准但常见的strdup
功能)。或者不释放记忆。
让我们看看我是否可以让它更清楚......
您有一个变量name
,您可以为其分配内存。它看起来像这样:
+------+ +---------------------+ | name | --> | memory for the name | +------+ +---------------------+
作业
newheader->name = name;
实际上只是复制指针,而不是它指向的内存。所以在完成任务后你会有这样的事情:
+------+ | name | ------------\ +------+ \ +---------------------+ >-> | memory for the name | +-----------------+ / +---------------------+ | newheader->name | -/ +-----------------+
现在有两个指向同一内存的变量。
然后你做
free(name);
这是免费的内存,但请记住,我们有两个变量指向同一个内存。现在看起来像
+------+ | name | -------------> ? +------+ +-----------------+ | newheader->name | --> ? +-----------------+
因此,每当您尝试访问newheader->name
时,您将尝试访问现在免费的内存,并且您将拥有未定义的行为。