Проблема освобождения памяти с помощью free () - PullRequest
Новая
       26

Проблема освобождения памяти с помощью free ()

1 голос
/ 27 декабря 2010

У меня проблемы с освобождением памяти, которую я выделил с помощью malloc. Программа работает нормально до тех пор, пока не освободится память. Здесь программа зависает. Поэтому мне было интересно, в чем может быть проблема, так как я только изучаю C. Синтаксически код кажется правильным, так может ли быть, что мне нужно удалить все вещи в этом месте, прежде чем освободить память из этого места или что-то еще?

Вот код. 

// Program to accept and print out five strings
#include <stdio.h>
#include <stdlib.h>
#include <string.h>

#define NOOFSTRINGS 5
#define BUFFSIZE 255

int main()
{
    char buffer[BUFFSIZE];//buffer to temporarily store strings input by user
    char *arrayOfStrngs[NOOFSTRINGS];
    int i;

    for(i=0; i<NOOFSTRINGS; i++)
    {
        printf("Enter string %d:\n",(i+1));
        arrayOfStrngs[i]=(char*)malloc(strlen(gets(buffer)+1));//calculates string length and allocates appropriate memory
        if( arrayOfStrngs[i] != NULL)//checking if memory allocation was successful
        {
            strcpy(arrayOfStrngs[i], buffer);//copies input string srom buffer to a storage loacation
        }
        else//prints error message and exits
        {
            printf("Debug: Dynamic memory allocation failed");
            exit (EXIT_FAILURE);
        }
    }

    printf("\nHere are the strings you typed in:\n");
    //outputting all the strings input by the user
    for(i=0; i<NOOFSTRINGS; i++)
    {
        puts(arrayOfStrngs[i]);
        printf("\n");
    }

    //Freeing up allocated memory
    for(i=0; i<NOOFSTRINGS; i++)
    {
        free(arrayOfStrngs[i]);
        if(arrayOfStrngs[i] != NULL)
        {
            printf("Debug: Memory deallocation failed");
            exit(EXIT_FAILURE);
        }
    }

    return 0;
}

Ответы [ 2 ]

4 голосов
/ 27 декабря 2010

Вы неправильно используете strlen(), и это приводит к переполнению буфера: 

arrayOfStrngs[i]=(char*)malloc(strlen(gets(buffer)+1)); //pointer from gets() is incremented and passed to strlen()  - that's wrong

должно быть 

arrayOfStrngs[i]=(char*)malloc(strlen(gets(buffer))+1); //pointer from gets() is passed to strlen(), then returned value is incremented - correct

также free() не изменяет переданный ему указатель.Так что 

 char* originalValue = pointerToFree;
 free( pointerToFree ); 
 assert( pointerToFree == originalValue ); //condition will always hold true

и так в вашем коде освобождающей памяти должно быть просто 

//Freeing up allocated memory
for(i=0; i<NOOFSTRINGS; i++)
{
    free(arrayOfStrngs[i]);
}
2 голосов
/ 27 декабря 2010 
arrayOfStrngs[i]=(char*)malloc(strlen(gets(buffer)+1));//calculates string length and allocates appropriate memory

Разве это не должно быть 

arrayOfStrngs[i]=(char*)malloc(strlen(gets(buffer))+1);
...