正确释放其他功能malloced

Question

编辑内存空间：这个问题是不是我还以为是，读我（长）问题将免去你前一段时间检查答案。

我一直valgrinding学校我的小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); 
} 

正如你所看到的，我的malloc我的字符** MODELES在det_langue到合适的大小（我相信在nb_modeles功能工作罚款），然后我malloc每个元素的get_modeles_names中的模型;但是因为我以后需要处理它们，所以我无法释放它们（或者不知道如何）在哪里放置它们，之后我会释放它们，直到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));此外，MODELES的大小为4，这也解释了40个allocs但只有36的FreeS。 另外，如果我在get_modeles_names函数中发疯，并且只是自由模式，我的程序显然是seg_faults，因为我应该处理的数据不再存在。

我GOOGLE了四周，却找不到已经问这会回答我的任何问题....任何想法？

编辑：mem_leak在溶液中所描述的，但只要它的解决还有另一种：线泄漏，这是由在while循环内移动

if(line) 
    free(line); 

解决。

Answer 1

我看到下面的问题：

 if (strstr(line, entete)) { 
      modeles[i] = (char*) malloc(strlen(line) * sizeof (char)); 
      modeles[i] = line; 

在这里，您首先分配内存与malloc并保存吨他在modeles[i]指向它，但是你用line覆盖指针。如果您想复制line的内容，请使用strcpy。

请注意，您没有为line的strcpy分配足够的内存：您忘记了终止空字符。它应该是：

  modeles[i] = malloc(strlen(line) + 1); 

或：

  int len= strlen(line); 
      line[len-2] = '\0'; 
      modeles[i] = malloc(len); 
      strcpy(modeles[i], line); 

..和sizeof(char)没有必要的，因为sizeof(char)始终为1

Answer 2

问题是

modeles[i] = line; 

这只是更换指针，你malloc分配 内存你想的strcpy（）有：

strcpy(models[i], line);