编辑:问题不是我想的那样,在阅读我的(长期)问题之前检查答案会花费你一些时间。
我已经将我的小C项目用于学校(基本上是语言检测器),但根据valgrind的说法,我仍有一些问题。我认为问题是经典的,它可以恢复:当我(我想)别无选择时,我应该如何在另一个函数中释放一个变量?
以下是导致这个问题的两个功能的相关部分。基本上,get_modeles_names
从我的"集合"中收集包含正确子字符串的文件名。文件;然后det_langue
正在处理它们。
这是get_modeles_names:
void get_modeles_names(int mode, char ** modeles)
{
FILE * collection = get_collection_file();
int i = 0;
char * line = NULL;
size_t size = 0;
while (getline(&line, &size, collection) != -1) {
if (strstr(line, entete)) {
modeles[i] = (char*) malloc(strlen(line) * sizeof (char));
modeles[i] = line;
modeles[i][strlen(modeles[i]) - 1] = '\0';
//replaces the final '\n' by '\0'
i++;
line = NULL;
}
}
if (line)
free(line);
fclose(collection);
}
这是det_langue:
void det_langue(char * w)
{
int nb_modele = nb_modeles();
char ** modeles = (char **) malloc(nb_modele * sizeof (char *));
get_modeles_names(mode, modeles);
double * resultats = (double *) malloc(nb_modele * sizeof (double));
int i;
for (i = 0; i < nb_modele; i++) {
resultats[i] = calculate_result (w, modeles[i]);
}
}
for (i = 0; i < nb_modele; i++) {
free(modeles[i]);
}
free(modeles);
free(resultats);
}
正如你所看到的,我将我的char **模型在det_langue中正确尺寸(我确信nb_modeles函数工作正常),然后我在get_modeles_names中malloc每个元素元素;但是因为我需要稍后处理它们,我无法释放它们(或者不知道怎样)我在哪里使用它们,我会在det_langue结束时释放它们。我认为这很好,但这里是valgrind --leak-check=full ./projet -d pomme -m 1
的输出
==30547== Command: ./projet -d pomme -m 1
==30547==
pomme -> french
==30547==
==30547== HEAP SUMMARY:
==30547== in use at exit: 113 bytes in 4 blocks
==30547== total heap usage: 40 allocs, 36 frees, 30,906 bytes allocated
==30547==
==30547== 113 bytes in 4 blocks are definitely lost in loss record 1 of 1
==30547== at 0x4C2DB8F: malloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==30547== by 0x40163E: get_modeles_names (gestion_collection_modeles.c:120)
==30547== by 0x401919: det_langue (det_langue.c:12)
==30547== by 0x40189E: main (Main.c:76)
==30547==
==30547== LEAK SUMMARY:
==30547== definitely lost: 113 bytes in 4 blocks
==30547== indirectly lost: 0 bytes in 0 blocks
==30547== possibly lost: 0 bytes in 0 blocks
==30547== still reachable: 0 bytes in 0 blocks
==30547== suppressed: 0 bytes in 0 blocks
==30547==
==30547== For counts of detected and suppressed errors, rerun with: -v
==30547== ERROR SUMMARY: 1 errors from 1 contexts (suppressed: 0 from 0)
第120行在get_modeles_names中,它是:modeles[i] = (char*) malloc(strlen(line) * sizeof (char));
此外,模型的大小为4,这解释了40个分配但只有36个释放。
此外,如果我在get_modeles_names
函数中进行疯狂和简单的免费模式,我的程序显然是seg_faults,因为我应该处理的数据不再存在。
我已经用Google搜索过了,但是找不到任何已经问到的问题哪个会回答我的问题....任何想法?
编辑:mem_leak是解决方案中描述的那个,但是只要它解决了就会有另一个:线路泄漏,通过移动来解决
if(line)
free(line);
在while循环中。
答案 0 :(得分:2)
我看到以下问题:
if (strstr(line, entete)) {
modeles[i] = (char*) malloc(strlen(line) * sizeof (char));
modeles[i] = line;
在这里,您首先使用malloc
分配内存并在modeles[i]
中保存指向它的指针,然后用line
覆盖指针。如果您要复制line
的内容,请使用strcpy
。
请注意,您没有为strcpy
的{{1}}分配足够的内存:您忘记了终止空字符。它应该是:
line
或:
modeles[i] = malloc(strlen(line) + 1);
..并且不需要 int len= strlen(line);
line[len-2] = '\0';
modeles[i] = malloc(len);
strcpy(modeles[i], line);
,因为sizeof(char)
始终为1。
答案 1 :(得分:1)
问题是
strcpy(models[i], line);
只是将指针替换为malloc&#d; d 你想要一个strcpy():
{{1}}