C言語：メモリ解放に関して

Question

設定

/* 自己参照構造体 */
typedef struct list{
  char name[256];
  struct list *next;
} LIST;

を用意して、mallocでメモリを動的確保した際、プログラム終了時のfreeに関して質問です。

質問１

main関数の最後に、

/* メモリの解放 */
  p = head;
  while(p != NULL){
    q = p->next; // 次のノードアドレスを保存
    free(p->name);
    free(p->next);
    free(p);
    p = q;
  }

とすることで、リストに残っているノードのメモリを解放できているでしょうか？

質問２

自作関数で、単方向リストの特定のノードの削除を行う場合、ノードのつなぎ替えのみではメモリ解放は当然できていないと思いますが、削除する対象のノードはfreeした方が良いでしょうか？

実際に以下のプログラムでfreeするとエラーが出ます（コンパイルは通ります）。
※以下のプログラムを含む関数を通った際にエラーが出ます。

// 削除対象のノードをpとする
free(p->name);
free(p->next);
free(p);

問題点などを教えてください。

####追記：

free(p);

Cの場合にはエラーは出ませんでしたが、

free(p->name);
free(p);

だと、

free(): double free detected in tcache 2
Aborted (core dumped)

とエラーが出ます。

Comments

[fana]

いかにも間違っていそうなコードですが，それはそれとして，
malloc と free は対となる存在ですから
free部分だけじゃなくて，malloc部分のコードも示すと良いのではないでしょうか．

[rubato6809]

タイトルだけミス。違和感すごい。
×メモリ解法
〇メモリ解放

[SUNMOON_14]

ご指摘ありがとうございます。

Answer 1

質問１

ループ内でやるのは、free(p)だけにしてください。

• free(p->next);とすると、ループが次へ進んだ時にはすでにpが解放されている、という事態に陥ります。
• p->nameは構造体の一部なので、ここだけ別に解放するのは不要で不適当です。

質問２

これも同じで、free(p)だけにしましょう。

Answer 2

(1)
freeの引数へは，「malloc（や，その類似品の方々）が返してきた値」を指定します．

free(p->name);

これは妥当でしょうか？
p->name の値が，mallocが返してきた値でないならば，間違いです．

(2)

  p = head;
  while(p != NULL){
    q = p->next;
    free(p->name);  //←これは上述
    free(p->next);　//おおっと…？
    free(p);
    p = q;
  }

この free(p->next); の引数の値は q と同一ですよね．
つまり，free(q) と同じ．
その後で，p=q としても，pの指す先はfree済み．
それって，まずくないですかね．