Правильное освобождение пространства памяти, размещенного в другой функции

Question

Редактировать: проблема не в том, что я так и думал, проверьте ответы перед тем, как прочитать мой (длинный) вопрос, пощадит вас некоторое время.

Я был знаком с моим маленьким проектом C для школы (в основном, детектором языка), но есть набор указателей. У меня все еще есть проблемы с согласно valgrind. Я думаю, что проблема классическая, ее можно было бы возобновить в: Как мне освободить переменную i malloced в другой функции, когда у меня (думаю, у меня нет выбора)?

Вот некоторые из двух функций, которые вызывают у меня эту проблему. В принципе, 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); 
} 

Как вы можете видеть, я таНос мой символ ** Modeles в det_langue до нужного размера (я уверен, что nb_modeles функция работает fine), затем i malloc каждый элемент моделей в get_modeles_names; но так как мне нужно обработать их позже, я не могу их освободить (или не знаю, как), где я их malloced, я освобожу их позже, в конце det_langue. Я думаю, что это Allright, но вот выход 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) 

Line 120 в get_modeles_names, это: modeles[i] = (char*) malloc(strlen(line) * sizeof (char)); Кроме того, размер Modeles составляет 4, которая объясняет для 40 между выделенными, но только 36 освобождает. Кроме того, если я схожу с ума и просто свободные модели в функции get_modeles_names, моя программа, очевидно, seg_faults, так как данные, которые я должен обрабатывать, больше не существуют.

Я искал googled вокруг, но не мог найти какой-либо вопрос, который уже задал вопрос, который ответит мне .... любая идея?

Изменить: mem_leak был одним из описанных в растворе, но есть другой, как только она решена: утечки линии, которая решается путем перемещения

if(line) 
    free(line); 

в пределах цикла.

Answer 1

Я вижу следующую задачу:

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

Здесь вы сначала выделяете память malloc и сохраняете t он указатель на него в modeles[i], но затем вы переписываете указатель line. Если вы хотите скопировать содержимое line, используйте strcpy.

Обратите внимание, что вы не выделяете достаточное количество памяти для strcpy из 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) не требуется, потому что sizeof(char) всегда 1.

Answer 2

Проблема заключается в том

modeles[i] = line; 

, который просто заменить указатель на память вы malloc'd Вы хотите STRCPY() там:

strcpy(models[i], line);