returnとexitから起こるエラー

Question

課題で登録された名前、番地を要素としたリスト構造からなるリストを、名前で検索する関数SearchListを作成しています。もし一致するものがあれば番地である変数idを返し、それ以外は何も返さないという関数です。
ただ、この状態で関数を使用するとSegmentation　faultになってしまいます。exitの使い方があまり分かっていないため、このような使い方でよいのか教えていただきたいです。よろしくお願いいたします。

int SearchList(node *list, char *id) {
  node *p = list->next;

  while(p != NULL) {
	if((strcmp(p->id, id)) == 0) {
	  return (p->num);
	}
	p = p->next;
  }
  printf("ident not find.\n");
  exit(0);
}

/*node = リスト構造*/
typedef struct _node{
  char id[15];
  int num;
  struct _node *next;
}node;

Comments

[SHOMI]

関数を呼んでいる側のコードも貼って下さい

[Zuishin]

list か id が NULL か不正なアドレスを指しているんじゃないでしょうか。そうでなければ各ノードの初期化に失敗しているか。

[KoichiSugiyama]

SearchListの引数に何を渡しているか判らないと何とも言えません。呼び出し側の処理も開示してみてください。

[pepperleaf]

そもそもノードを初期化してない。そのため、
> while(p != NULL) {
のループを抜けることが無く、不正アクセスになるとか、、。

[thkana]

segmentation faultとexit()はどのように結びつくのでしょうか。

他の間違いの結果としてならともかく、returnやexit()がsegmentation faultの原因になることは非常に考えにくいです。

[episteme]

> int SearchList(node *list, char *id)
こいつに引き渡すlistがNULLだったらおかしなことが起こりそうですが...

[cateye]

コンパイラは何でしょう？ printf()でも入れてlistの値を確認しましょう。

Answer 1

return：上位に戻り値を返す
exit：即時終了(プログラムが落ちる)
となります。

「もし一致するものがあれば番地である変数idを返し、それ以外は何も返さないという関数」
で、「それ以外は何も返さない」の実現はC言語だと難しいかと。
(longjmpを使えば無理くり実現できますが、あまり美しくないと思います。C++ならthrowなどでハンドリングできるんですけどね。)

こういう場合、

exit(0);

ではなく、

return NULL_PTR;

で、NULL_PTRを返し、上位側では戻り値がNULL_PTRの時に
エラーハンドリングするような構成にした方が、素直かと思います