最後のノードでリンクリストをクラッシュさせる

Question

これは私が昨日投稿した問題の続きですが、解決したと思っていましたが、ループとその終了条件を繰り返す方法に遭遇しました。新しい質問スレッドがより適切かもしれないと感じました。

次のfreeAllListMembers（）関数は、 "temp"を解放しようとしているのでループの最後の反復まで機能しているようですが、tempは既に解放されています。それはもう一度走っているから、私が試したことはすべて動作していないようです。いくつかの小さなヒューマンエラーを取り除くために与えられた例をクリーンアップ任意の洞察力

int main() { 

    struct node *head = NULL; 
    createList(&head); 

    //do stuff with list 

    freeAllListMembers(&head); 

    return 0; 
} 

int createList(struct node **head) { 
    struct node *newNode= NULL; 

    for(int I = 0; I < 100; I++) 
    { 
     struct node *node = (struct node*)malloc(sizeof(struct node)); 

     node->data = someData; 

     node->next = NULL; 

     //if we havent created an initial start node, create it 
     if (*head == NULL) 
     { 
      *head = node; 
     } 
     //otherwise, navigate to the end of the list to add a new node 
     else 
     { 
      newNode = *head; 
      while (newNode->next != NULL){ 
       newNode = newNode->next; 
      } 
      newNode->next = node; 
     } 
    } 
    return 0; 
} 

void freeAllListMembers(struct node **head){ 
    struct node *temp; 
    while (*head != NULL) { 
     temp = *head; 
     *head = (*head)->next; 
     free(temp); 
    } 
    return; 
} 

Answer 1

ため

おかげで、プログラムがOKコンパイルし、私はGDBにOKをステップ実行することができます。それがエラー

#include <malloc.h> 

#define someData 12345 

struct node 
{ 
     int data; 
     struct node* next; 
}; 

int createList(struct node **head) 
{ 
    struct node *newNode= NULL; 

    int i; 
    for(i = 0; i < 100; i++) 
    { 
     struct node *node = (struct node*)malloc(sizeof(struct node)); 

     node->data = someData; 

     node->next = NULL; 

     //if we havent created an initial start node, create it 
     if (*head == NULL) 
     { 
      *head = node; 
     } 
     //otherwise, navigate to the end of the list to add a new node 
     else 
     { 
      newNode = *head; 
      while (newNode->next != NULL){ 
       newNode = newNode->next; 
      } 
      newNode->next = node; 
     } 
    } 
    return 0; 
} 

void freeAllListMembers(struct node **head) 
{ 
    struct node *temp; 
    while (*head != NULL) { 
     temp = *head; 
     *head = (*head)->next; 
     free(temp); 
    } 
    return; 
} 


int main() 
{ 

    struct node *head = NULL; 
    createList(&head); 

    //do stuff with list 

    freeAllListMembers(&head); 

    return 0; 
} 

なしでコンパイルした後、それは基本的に私はあなたが私はあなたのループではなく、しばらく（ヘッド！= NULL）のようになるはずだと思うものは何でもあなたのバグは「抽象化」のプロセスに