メモリ割り当ての解放

Question

現在、ノードを削除するとリンクリストを作成してメモリ割り当てを解放しようとしています。しかし、これをやろうと数時間を経た後、私はきれいなvalgrind出力を得ることができません。

void * pop(struct List *list) 
{ 
    if(list->head == 0){ 
     return 0; 
    } 

    struct Node * tempNode = list->head->next;                            
    free(list->head); 
    list->head = tempNode; 
    ... 
} 

私が言ってスペースを割り当てています：

addNode(struct List *list, void *element){ 
    struct Node *node; 
    node = (struct Node *)malloc(sizeof(node)); 
    .... 
} 

を基本的に私は、リストの先頭を取り出し、頭の次のノードに新しいヘッドを作りたいポップ機能に。私は頭に与えられた記憶を解放したい。任意のヘルプ

Answer 1

すごい迫力ため

おかげで、あなたのmallocが正しくありません。あなたが持っている：

(struct Node *)malloc(sizeof(node)); 

あなたが必要なものは次のとおりです。あなたのオリジナルのコードで

(struct Node *)malloc(sizeof(struct Node)); 

、あなただけのポインタのために十分に配分されています。しかし、Nodeオブジェクトを割り当てようとしています。

Answer 2

正しく表示されます。または、他の場所にいくつかのリークがありますか？または、渡されたパラメータとして要素自体を解放しますか？

Answer 3

node = malloc(sizeof(*node)); 

nodeが指すもののスペースを割り当てます。

mallocの戻り値をキャストしないでください。そうすることで、失敗を#include <stdlib.h>にマスクすることができます。