Утечка памяти в C при добавлении в строку - PullRequest
Новая
       33

Утечка памяти в C при добавлении в строку

0 голосов
/ 17 марта 2019

Вальгринд говорит, что произошла утечка памяти, но я не могу понять, что это такое.Я попытался добавить free в функцию добавления, освобождая строку в конце main, и она все еще жалуется. 

#include <stdio.h>
#include <stdlib.h>
#include <string.h>

void append(char* string, char c)
{
    int stringlength = strlen(string);
    char* tmp = realloc(string, sizeof(char) * (stringlength + sizeof(char) + sizeof(char)));
    if(tmp == NULL)
    {
        free(tmp);
        free(string);
        printf("Error appending character\n");
        exit(1);
    }

    string = tmp;
    string[stringlength] = c;
    string[stringlength+1] = '\0';
}

int main()
{
    char* string = malloc(sizeof(char));
    string[0] = '\0';

    printf("string before appending: %s\n", string);
    append(string, 'c');
    printf("string after appending: %s\n", string);
    free(string);
    return 0;
}

Вот вывод из Valgrind: https://pastebin.com/dtXFm5YC (Это в Pastebin, так что StackПереполнение фактически позволило бы мне отправить этот вопрос

1 Ответ

1 голос
/ 17 марта 2019

Ваши основные проблемы не в утечке памяти, а в неправильном доступе: 

Invalid read of size 1
    ...
Address 0x49d0028 is 0 bytes inside a block of size 1 free'd

и недействительно бесплатно: 

Invalid free() / delete / delete[] / realloc()
...
Address 0x49d0028 is 0 bytes inside a block of size 1 free'd

потому что у них неопределённое поведение

  • когда вы делаете char* tmp = realloc(string, sizeof(char) * (stringlength + sizeof(char) + sizeof(char)));, который освободил строку , исходящую от main , поэтому, когда вы printf это в main Вы получаете доступ к освобожденному блоку. Обратите внимание, что факт realloc освобождает блок не является обязательным

  • потому что вы не освобождаете tmp , недавно выделенный realloc в append вы создаете утечку памяти

  • затем вы снова освобождаетесь строка уже освобождена realloc

Также sizeof(char) * (stringlength + sizeof(char) + sizeof(char)) странно, у вас есть изменения sizeof(char) по определению 1, может быть просто stringlength + 2

Вы хотите это: 

#include <stdio.h>
#include <stdlib.h>
#include <string.h>

void append(char ** string, char c)
{
   size_t stringlength = strlen(*string);

  *string = realloc(*string, stringlength + 2);

  if (*string == NULL)
  {
     fprintf(stderr, "cannot allocate memory\n");
     exit(1);
  }

  (*string)[stringlength] = c;
  (*string)[stringlength+1] = '\0';
}

int main()
{
    char* string = malloc(1);

    string[0] = '\0';

    printf("string before appending: '%s'\n", string);
    append(&string, 'c');
    printf("string after appending: '%s'\n", string);
    free(string);
    return 0;
}

Компиляция и исполнение: 

pi@raspberrypi:/tmp $ gcc -g -pedantic -Wextra m.c
pi@raspberrypi:/tmp $ valgrind ./a.out
==17097== Memcheck, a memory error detector
==17097== Copyright (C) 2002-2017, and GNU GPL'd, by Julian Seward et al.
==17097== Using Valgrind-3.13.0 and LibVEX; rerun with -h for copyright info
==17097== Command: ./a.out
==17097== 
string before appending: ''
string after appending: 'c'
==17097== 
==17097== HEAP SUMMARY:
==17097==     in use at exit: 0 bytes in 0 blocks
==17097==   total heap usage: 3 allocs, 3 frees, 1,027 bytes allocated
==17097== 
==17097== All heap blocks were freed -- no leaks are possible
==17097== 
==17097== For counts of detected and suppressed errors, rerun with: -v
==17097== ERROR SUMMARY: 0 errors from 0 contexts (suppressed: 6 from 3)
...