回答編集履歴

  p はcharのアドレス（ポインタ）になってますが、

  > p -> next = malloc(sizeof(struct list));

  char にはnextというメンバはありません