Issue regarding memory allocation in C

Question

For some reason, I cannot manage to correctly free the structure I created, which goes as follows:

struct dictionary{

char word[50]; // Word to be described
char definition[150]; // Definition of said word
struct dictionary* next; // Linked list structure

}*dic;
typedef struct dictionary Dic;

Dic* newWord(char word[], char definition[]){
    Dic* dc;
    dc = malloc(sizeof(*dic));

    strcpy(dc->word,word);
    strcpy(dc->definition,definition);
    dc->next = NULL;

return dc;
}

Now, if I do something like:

dc = newWord("Word","definition");
free(dc);
printWord(dc);

where:

void printWord(Dic* dic){
    if(dic){
    printf("<%s>\n",dic->word);
    printf(".%s\n",dic->definition);
    }
}

will result in no word being printed, yet the definition still being printed, which I can only assume means I'm not properly freeing this structure's memory. How do I go by properly doing it?

Possible duplicate of [Why do I get different results when I dereference a pointer after freeing it?](http://stackoverflow.com/questions/1078677/why-do-i-get-different-results-when-i-dereference-a-pointer-after-freeing-it) — Petr Skocik, Oct 22 '16 at 15:43

score 2 · Accepted Answer · edited May 23 '17 at 12:33

2

You are using a pointer after freeing it. This is undefined behavior, which means the C compiler can do whatever it wants. It can crash, it can appear to behave properly, or it can do anything in between. Calling free on a pointer doesn't set it to NULL, it releases your claim on the memory, but you have to set it to NULL yourself.

edited May 23 '17 at 12:33

Community

1
1

answered Oct 22 '16 at 15:42

Erik Godard

5,930
6
30
33

Issue regarding memory allocation in C

1 Answers1