Как найти и устранить утечки памяти и ошибки в моей программе на C после проверки с помощью Valgrind - PullRequest
Новая
       63

Как найти и устранить утечки памяти и ошибки в моей программе на C после проверки с помощью Valgrind

4 голосов
/ 18 ноября 2011

После проверки моего кода с помощью Valgrind, он показывает много сообщений об ошибках и информации, и я не уверен, как найти и удалить ошибки в коде?

Мой исходный код выглядит следующим образом.

myheader.h 

 #include<stdlib.h>
 #define MAX 15
 typedef enum
 {
  METROPOLITAN_AREA,
  TOURIST_AREA,
 }area_type_t;

 typedef struct
 {
  int population;
  int area;
 }metropolitan_t;

 typedef struct
 {
  char *type_place;
  char *near_airport;
 }tourist_t;

 typedef struct
 {
  char *name;
  char *country;
  area_type_t area_t;
  union
  {
      metropolitan_t metro;
      tourist_t   tourist;
  }u;

 }*place_t;
 extern void get_input(place_t);

getinput.c 

#include<myheader.h>
void get_input(place_t place)
{
 int check=1;
 int num,i,ch;
 printf("\nEnter the no of records : \n");
 scanf("%d",&num);
 place=(place_t)malloc(sizeof(place_t)*num);
 if(NULL==place)
 {
  printf("\nMemory allocation failed\n");
  exit(1);
 }
 for(i=0;i<num;i++)
 {
  printf("\nEnter the place name : ");
  place->name=(char *)malloc(sizeof(char)*MAX);
  if(NULL==place->name)
  {
   free(place);
   printf("\nMemory allocation failed\n");
   exit(1);
  }
  scanf("%s",place->name);
  printf("\nEnter the coutry name : ");
  place->country=(char *)malloc(sizeof(char)*MAX);
  if(NULL==place->country)
  {
   free(place);
   free(place->name);
   printf("Memory allocation failed\n");
   exit(1);
  }
  scanf("%s",place->country);
   printf("\nPlease Enter '0' if the place is Metropolitan, '1' if the place Tourist place\n");
   scanf("%d",&ch);
  while(check)
  {
   switch(ch)
   {
    case 0:
     check=0;
     place->area_t=METROPOLITAN_AREA;
     break;
    case 1:
     check=0;
     place->area_t=TOURIST_AREA;
      break;
    default:
     printf("\nPlease enter valid choice \n");
   }
  }
  if(place->area_t==METROPOLITAN_AREA)
  {
    printf("\nEnter the population of the place : ");
    scanf("%d",&(place->u.metro.population));
    printf("\nEnter the area of place :"); 
    scanf("%d",&(place->u.metro.area));
  } else
  {
   printf("\nEnter the nearest airport name : ");
   place->u.tourist.near_airport=(char *)malloc(sizeof(char)*MAX);
   if(NULL==place->u.tourist.near_airport)
   {
    printf("Memory allocation failed\n");
    exit(1);
   }
   scanf("%s",(place->u.tourist.near_airport));
   printf("\nEnter the type of tourist spot : ");
   place->u.tourist.type_place=(char *)malloc(sizeof(char)*MAX);

   if(NULL==place->u.tourist.type_place)
   {
    printf("Memory allocation failed\n");
    exit(1);
   }
   scanf("%s",(place->u.tourist.type_place));
  }
  check=1;
  place++;
 }
}

main.c 

#include"myheader.h"
int main()
{
 place_t place;
 get_input(place);
 return 0;
}

Стенограмма командной строки 

 bash-3.00$ valgrind --tool=memcheck --leak-check=yes --show-reachable=yes a.out
 ==13514== Memcheck, a memory error detector.
 ==13514== Copyright (C) 2002-2005, and GNU GPL'd, by Julian Seward et al.
 ==13514== Using LibVEX rev 1575, a library for dynamic binary translation.
 ==13514== Copyright (C) 2004-2005, and GNU GPL'd, by OpenWorks LLP.
 ==13514== Using valgrind-3.1.1, a dynamic binary instrumentation framework.
 ==13514== Copyright (C) 2000-2005, and GNU GPL'd, by Julian Seward et al.
 ==13514== For more details, rerun with: -v
 ==13514==

 Enter the no of records :
 2

 Enter the place name : Banglore

 Enter the coutry name : India

 Please Enter '0' if the place is Metropolitan, '1' if the place Tourist place
 0
 ==13514== Invalid write of size 4
 ==13514==    at 0x400789: get_input (getinput.c:43)
 ==13514==    by 0x4005E8: main (main.c:5)
 ==13514==  Address 0x4A2D040 is 0 bytes after a block of size 16 alloc'd
 ==13514==    at 0x4904A06: malloc (vg_replace_malloc.c:149)
 ==13514==    by 0x400637: get_input (getinput.c:8)
 ==13514==    by 0x4005E8: main (main.c:5)
 ==13514==
 ==13514== Invalid read of size 4
 ==13514==    at 0x4007BB: get_input (getinput.c:53)
 ==13514==    by 0x4005E8: main (main.c:5)
 ==13514==  Address 0x4A2D040 is 0 bytes after a block of size 16 alloc'd
 ==13514==    at 0x4904A06: malloc (vg_replace_malloc.c:149)
 ==13514==    by 0x400637: get_input (getinput.c:8)
 ==13514==    by 0x4005E8: main (main.c:5)

 Enter the population of the place : 303030

 Enter the area of place :2500
 ==13514==
 ==13514== Invalid write of size 8
 ==13514==    at 0x40068C: get_input (getinput.c:17)
 ==13514==    by 0x4005E8: main (main.c:5)
 ==13514==  Address 0x4A2D058 is not stack'd, malloc'd or (recently) free'd
 ==13514==
 ==13514== Invalid read of size 8
 ==13514==    at 0x400693: get_input (getinput.c:18)
 ==13514==    by 0x4005E8: main (main.c:5)
 ==13514==  Address 0x4A2D058 is not stack'd, malloc'd or (recently) free'd
 ==13514==
 ==13514== Invalid read of size 8
 ==13514==    at 0x4006BF: get_input (getinput.c:24)
 ==13514==    by 0x4005E8: main (main.c:5)
 ==13514==  Address 0x4A2D058 is not stack'd, malloc'd or (recently) free'd

 Enter the place name : London

==13514==
 ==13514== Invalid write of size 8
 ==13514==    at 0x4006EE: get_input (getinput.c:26)
 ==13514==    by 0x4005E8: main (main.c:5)
 ==13514==  Address 0x4A2D060 is 16 bytes before a block of size 15 alloc'd
 ==13514==    at 0x4904A06: malloc (vg_replace_malloc.c:149)
 ==13514==    by 0x40068B: get_input (getinput.c:17)
 ==13514==    by 0x4005E8: main (main.c:5)
 ==13514==
 ==13514== Invalid read of size 8
 ==13514==    at 0x4006F6: get_input (getinput.c:27)
 ==13514==    by 0x4005E8: main (main.c:5)
 ==13514==
 ==13514== Invalid read of size 8
 ==13514==    at 0x4006F6: get_input (getinput.c:27)
 ==13514==    by 0x4005E8: main (main.c:5)
 ==13514==  Address 0x4A2D060 is 16 bytes before a block of size 15 alloc'd
 ==13514==    at 0x4904A06: malloc (vg_replace_malloc.c:149)
 ==13514==    by 0x40068B: get_input (getinput.c:17)
 ==13514==    by 0x4005E8: main (main.c:5)
 ==13514==
 ==13514== Invalid read of size 8
 ==13514==    at 0x40072F: get_input (getinput.c:34)
 ==13514==    by 0x4005E8: main (main.c:5)
 ==13514==  Address 0x4A2D060 is 16 bytes before a block of size 15 alloc'd
 ==13514==    at 0x4904A06: malloc (vg_replace_malloc.c:149)
 ==13514==    by 0x40068B: get_input (getinput.c:17)
 ==13514==    by 0x4005E8: main (main.c:5)

 Enter the coutry name : London

 Please Enter '0' if the place is Metropolitan, '1' if the place Tourist place
 1
 ==13514==
 ==13514== Invalid write of size 4
 ==13514==    at 0x40079D: get_input (getinput.c:47)
 ==13514==    by 0x4005E8: main (main.c:5)
 ==13514==  Address 0x4A2D068 is 8 bytes before a block of size 15 alloc'd
 ==13514==    at 0x4904A06: malloc (vg_replace_malloc.c:149)
 ==13514==    by 0x40068B: get_input (getinput.c:17)
 ==13514==    by 0x4005E8: main (main.c:5)

 ==13514==
 ==13514== Invalid write of size 8
 ==13514==    at 0x40082F: get_input (getinput.c:62)
 ==13514==    by 0x4005E8: main (main.c:5)
 ==13514==  Address 0x4A2D078 is 8 bytes inside a block of size 15 alloc'd
 ==13514==    at 0x4904A06: malloc (vg_replace_malloc.c:149)
 ==13514==    by 0x40068B: get_input (getinput.c:17)
 ==13514==    by 0x4005E8: main (main.c:5)
 ==13514==
 ==13514== Invalid read of size 8
 ==13514==    at 0x400837: get_input (getinput.c:63)
 ==13514==    by 0x4005E8: main (main.c:5)
 ==13514==  Address 0x4A2D078 is 8 bytes inside a block of size 15 alloc'd
 ==13514==    at 0x4904A06: malloc (vg_replace_malloc.c:149)
 ==13514==    by 0x40068B: get_input (getinput.c:17)
 ==13514==    by 0x4005E8: main (main.c:5)
 ==13514==
 ==13514== Invalid read of size 8
 ==13514==    at 0x40085B: get_input (getinput.c:68)
 ==13514==    by 0x4005E8: main (main.c:5)
 ==13514==  Address 0x4A2D078 is 8 bytes inside a block of size 15 alloc'd
 ==13514==    at 0x4904A06: malloc (vg_replace_malloc.c:149)
 ==13514==    by 0x40068B: get_input (getinput.c:17)
 ==13514==    by 0x4005E8: main (main.c:5)

 Enter the nearest airport name : London

 Enter the type of tourist spot : Entertainment

 ==13514==
 ==13514== ERROR SUMMARY: 13 errors from 12 contexts (suppressed: 4 from 1)
 ==13514== malloc/free: in use at exit: 106 bytes in 7 blocks.
 ==13514== malloc/free: 7 allocs, 0 frees, 106 bytes allocated.
 ==13514== For counts of detected errors, rerun with: -v
 ==13514== searching for pointers to 7 not-freed blocks.
 ==13514== checked 75,768 bytes.
 ==13514==
 ==13514==
 ==13514== 15 bytes in 1 blocks are indirectly lost in loss record 1 of 7
 ==13514==    at 0x4904A06: malloc (vg_replace_malloc.c:149)
 ==13514==    by 0x40088A: get_input (getinput.c:70)
 ==13514==    at 0x4904A06: malloc (vg_replace_malloc.c:149)
 ==13514==    by 0x40088A: get_input (getinput.c:70)
 ==13514==    by 0x4005E8: main (main.c:5)
 ==13514==
 ==13514==
 ==13514== 15 bytes in 1 blocks are definitely lost in loss record 2 of 7
 ==13514==    at 0x4904A06: malloc (vg_replace_malloc.c:149)
 ==13514==    by 0x40082E: get_input (getinput.c:62)
 ==13514==    by 0x4005E8: main (main.c:5)
 ==13514==
 ==13514==
 ==13514== 15 bytes in 1 blocks are definitely lost in loss record 3 of 7
 ==13514==    at 0x4904A06: malloc (vg_replace_malloc.c:149)
 ==13514==    by 0x4006ED: get_input (getinput.c:26)
 ==13514==    by 0x4005E8: main (main.c:5)
 ==13514==
 ==13514==
 ==13514== 15 bytes in 1 blocks are definitely lost in loss record 4 of 7
 ==13514==    at 0x4904A06: malloc (vg_replace_malloc.c:149)
 ==13514==    by 0x40068B: get_input (getinput.c:17)
 ==13514==    by 0x4005E8: main (main.c:5)
 ==13514==
 ==13514==
 ==13514== 15 bytes in 1 blocks are indirectly lost in loss record 5 of 7
 ==13514==    at 0x4904A06: malloc (vg_replace_malloc.c:149)
 ==13514==    by 0x4006ED: get_input (getinput.c:26)
 ==13514==    by 0x4005E8: main (main.c:5)
 ==13514==
 ==13514==
 ==13514== 15 bytes in 1 blocks are indirectly lost in loss record 6 of 7
 ==13514==    at 0x4904A06: malloc (vg_replace_malloc.c:149)
 ==13514==    by 0x40068B: get_input (getinput.c:17)
 ==13514==    by 0x4005E8: main (main.c:5)
 ==13514==
 ==13514==
 ==13514==
 ==13514== 61 (16 direct, 45 indirect) bytes in 1 blocks are definitely lost in loss record 7 of 7
 ==13514==    at 0x4904A06: malloc (vg_replace_malloc.c:149)
 ==13514==    by 0x400637: get_input (getinput.c:8)
 ==13514==    by 0x4005E8: main (main.c:5)
 ==13514==
 ==13514== LEAK SUMMARY:
 ==13514==    definitely lost: 61 bytes in 4 blocks.
 ==13514==    indirectly lost: 45 bytes in 3 blocks.
 ==13514==      possibly lost: 0 bytes in 0 blocks.
 ==13514==    still reachable: 0 bytes in 0 blocks.
 ==13514==         suppressed: 0 bytes in 0 blocks.

Как я могу решить эту проблему и сделать мою программу более эффективной с точки зрения памяти?Что-то не так в malloc() утверждениях, которые я использовал?

Ответы [ 2 ]

5 голосов
/ 18 ноября 2011

Первое, что нужно сделать, это избавиться от всех ошибок памяти (переполнения буфера).Первая ошибка возникает в get_input.c в строке 45, поэтому перейдите к этой строке и выясните, что идет не так.

Одна вещь, которая наверняка идет не так, это 

 place=(place_t)malloc(sizeof(place_t)*num);

.выделяя place_t, однако вы определили place_t следующим образом: 

typedef struct
{

 }*place_t;

т.е. place_t - это просто указатель.Это означает, что, например, malloc (sizeof (place_t)) просто выделяет пространство для указателя, а не для всего place_t.Не скрывайте имена структур как указатели с typedef, но при необходимости измените ваш оператор malloc на 

     place=(place_t)malloc(sizeof *place)*num);

Другая проблема заключается в том, что вы освобождаете структуру () и впоследствии пытаетесь получить доступ к члену внутри нее., что неверно. 

free(place);
free(place->name);

Вы должны сделать это в следующем порядке 

free(place->name);
free(place);

Вы также теряете память, например, здесь: for (i = 0; iname = (char *)) malloc (sizeof (char) * MAX);

Что происходит 2. время в этом цикле? Вы присваиваете place->name, теряя указатель на память, выделенную в первой итерации цикла.Возможно, вы намеревались использовать place [num] внутри цикла, поскольку вы пытаетесь выделить много place_t в place = (place_t) malloc (sizeof (place_t) * num); оператор.

4 голосов
/ 18 ноября 2011

Проблема не столько в выражениях malloc, сколько в порядке, в котором вы освобождаете память, как только закончите с ней.

, например 

free(place);
free(place->name);

должно быть 

free(place->name);
free(place);

в противном случае память, выделенная для name, не будет освобождена.

Точно так же звонит free(place); не будет свободной памяти, выделенной в place->name=(char *)malloc(sizeof(char)*MAX);

Вы должны вручную освободить память, выделенную для каждого оператора Malloc.

Всегда лучше проверить, являются ли указатели на ячейки памяти null, прежде чем использовать их снова с malloc.

Если они не null, тогда free их, а затем установите их на null. Это поможет вам избежать утечек памяти.

...