Problem beim Freigeben von Speicher mithilfe von free()

Question

Ich habe Probleme beim Freigeben von Speicher, den ich mithilfe von malloc zugewiesen habe. Das Programm läuft gut bis es der Teil ist, in dem es freigegeben werden soll. Hier friert das Programm ein. Also habe ich mich gefragt, was das Problem sein könnte, da ich gerade C lerne. Syntaktisch scheint der Code korrekt zu sein, also könnte es sein, dass ich alle Sachen an diesem Ort löschen muss, bevor ich Speicher von diesem Ort oder etwas anderem freigebe?

Hier ist der Code.

// 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; 
} 

Answer 1

Sie mißbrauchen strlen() und dies führt zu Pufferüberlauf:

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

sollte

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

auch free() nicht die an sie übergebenen Zeiger ändern. So dass

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

und so in Ihrem Code befreien Speicher sollte nur sein

//Freeing up allocated memory 
for(i=0; i<NOOFSTRINGS; i++) 
{ 
    free(arrayOfStrngs[i]); 
} 

Answer 2

arrayOfStrngs[i]=(char*)malloc(strlen(gets(buffer)+1));//calculates string length and allocates appropriate memory 

Sollte das nicht

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