Утечка памяти Вальгринд - PullRequest
Новая
       9

Утечка памяти Вальгринд

0 голосов
/ 26 октября 2018

Я написал программу для разбиения строки на пять.Это моя программа. 

struct list
{
    char *str;
    struct list* next;
};

struct list* head = NULL;

void insert(char *cont)
{
    struct list* temp = (struct list*)malloc(sizeof(struct list));

    size_t len = strlen(cont);
    char *heapString = (char*)malloc(len);
    strcpy(heapString,cont);

    temp->str = heapString;
    temp->next = NULL;

    if(head == NULL)
    {
        head = temp;
        return ;
    }

    temp->next = head;
    head = temp;
}
void print()
{
    struct list* temp = head;
    while(temp != NULL)
    {
        printf("%s\n",temp->str);
        temp = temp->next;
    }
}
void clearmem()
{
    struct list* temp = head;
    while(temp != NULL)
    {
        free(temp->str);
        free(temp);
        temp = temp->next;
    }
}
int main()
{
    char text[] = "abcdefghijklmno";
    size_t len = strlen(text);
    while(len !=0)
    {
        char *temp;
        temp = text ;
        temp = temp + len - 5;

        insert(temp);
        *(text+len-5) = '\0';
        len = strlen(text);
        free(temp);
    }
    print();
    clearmem();
}

Моя программа работает нормально.Но когда я пытаюсь запустить эту программу через Valgrind, я получаю следующие сообщения.Там написано 12 ошибок. 

==2055== Invalid write of size 1
==2055==    at 0x4C32E0D: strcpy (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==2055==    by 0x10888C: insert (in /home/infant/Documents/Sample_codes/a.out)
==2055==    by 0x1089BD: main (in /home/infant/Documents/Sample_codes/a.out)
==2055==  Address 0x522d095 is 0 bytes after a block of size 5 alloc'd
==2055==    at 0x4C2FB0F: malloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==2055==    by 0x108875: insert (in /home/infant/Documents/Sample_codes/a.out)
==2055==    by 0x1089BD: main (in /home/infant/Documents/Sample_codes/a.out)
==2055== 
==2055== Invalid free() / delete / delete[] / realloc()
==2055==    at 0x4C30D3B: free (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==2055==    by 0x1089EB: main (in /home/infant/Documents/Sample_codes/a.out)
==2055==  Address 0x1fff00030a is on thread 1's stack
==2055==  in frame #1, created by main (???:)
==2055== 
==2055== Invalid read of size 1
==2055==    at 0x4C32D44: __strlen_sse2 (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==2055==    by 0x4EBC9D1: puts (ioputs.c:35)
==2055==    by 0x1088FC: print (in /home/infant/Documents/Sample_codes/a.out)
==2055==    by 0x1089FC: main (in /home/infant/Documents/Sample_codes/a.out)
==2055==  Address 0x522d1d5 is 0 bytes after a block of size 5 alloc'd
==2055==    at 0x4C2FB0F: malloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==2055==    by 0x108875: insert (in /home/infant/Documents/Sample_codes/a.out)
==2055==    by 0x1089BD: main (in /home/infant/Documents/Sample_codes/a.out)
==2055== 
abcde
fghij
klmno
==2055== Invalid read of size 8
==2055==    at 0x108947: clearmem (in /home/infant/Documents/Sample_codes/a.out)
==2055==    by 0x108A06: main (in /home/infant/Documents/Sample_codes/a.out)
==2055==  Address 0x522d188 is 8 bytes inside a block of size 16 free'd
==2055==    at 0x4C30D3B: free (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==2055==    by 0x108942: clearmem (in /home/infant/Documents/Sample_codes/a.out)
==2055==    by 0x108A06: main (in /home/infant/Documents/Sample_codes/a.out)
==2055==  Block was alloc'd at
==2055==    at 0x4C2FB0F: malloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==2055==    by 0x108855: insert (in /home/infant/Documents/Sample_codes/a.out)
==2055==    by 0x1089BD: main (in /home/infant/Documents/Sample_codes/a.out)
==2055== 
==2055== 
==2055== HEAP SUMMARY:
==2055==     in use at exit: 0 bytes in 0 blocks
==2055==   total heap usage: 7 allocs, 10 frees, 1,087 bytes allocated
==2055== 
==2055== All heap blocks were freed -- no leaks are possible
==2055== 
==2055== For counts of detected and suppressed errors, rerun with: -v
==2055== ERROR SUMMARY: 12 errors from 4 contexts (suppressed: 0 from 0)

Несмотря на то, что я очистил всю память в куче, я получаю 12 ошибок в 4 контекстах.В чем здесь моя ошибка?

1 Ответ

0 голосов
/ 26 октября 2018

Шаг за шагом.

Invalid write of size 1

Ваш malloc() не выделяет место для ограничителя строки, но strcpy() пытается его записать. Используйте 

char *heapString = malloc(len + 1);

вместо этого. (Примечание: нет необходимости приводить void* к char*!). Для простоты вы также можете попробовать использовать (нестандартный) strdup(cont).

Invalid free() / delete / delete[] / realloc()

Ваш temp указывает на char в text. Нет смысла использовать free(), так как там ничего не выделено. Удалить этот вызов.

Invalid read of size 1

Это должно быть связано с первой ошибкой. Интересно отметить, как в print() во время компиляции printf("%s\n",temp->str) переводится в (быстрее) puts(temp->str). Вот почему Valgrind жалуется на звонок puts.

Invalid read of size 8

В 

free(temp);
temp = temp->next;

вы читаете temp после того, как оно было освобождено.

...