Segmentation faultについて

Question

前提・実現したいこと

温度を入力し、平均値以上の温度の場合出力を行うというプログラムを作成しているのですがSegmentation fault (core dumped)と表示され動作しません。Segmentation faultとは何なのでしょうか

発生している問題・エラーメッセージ

Segmentation fault (core dumped)

該当のソースコード

#include <stdio.h>
#include <stdlib.h>
#include <string.h>

#define LIMIT_LOW 0
#define LIMIT_HIGH 25

/* 構造体の定義 */
typedef struct tag {
  int temp;   /* 温度 */
  struct tag *next; /* 自分自身の型へのポインタ変数 */
} tempData;       /* 温度データ */


/* 新データ作成関数 */
tempData* makeNewNode(int t) {
  tempData* pNewNode;
  /*** person 型のメモリ領域確保 ***/
  pNewNode = (tempData*)malloc(sizeof(tempData));
  if (pNewNode != NULL) {
    /*** データ設定 ***/
    pNewNode->temp= t;
    pNewNode->next = NULL;
  }
  return pNewNode ;
}


int main(void) {
  int temp; /* 温度入力用変数 */
  tempData *pTop;  /* 温度データリストのトップ */
  tempData *pNow;  /* 温度データリスト内の現在位置 */
  tempData *pNew;  /* 温度データの新規データ */
  /* 必要であれば，ここに変数を追加 */
  int i = 1; /*データ数の計測用*/
  int ave = temp; /* 平均値の計算用 */

  /* 最初のデータは，必ず範囲内のデータであるとする */
  printf("温度を入力\n");
  scanf("%d", &temp);
  pTop = makeNewNode(temp);
  pNow = pTop;
  /* 次のデータを入力 */
  scanf("%d", &temp);
  
  while ( (LIMIT_LOW <= temp) && (temp <= LIMIT_HIGH)) {
    /* ここにリスト作成処理を記述 */
    pNew = makeNewNode(temp);
    pNow->next = pNew;
    pNow = pNew;
    
    ave = ave + temp;
    i = i + 1; /*データの数を計測*/
   
    /* 次のデータを入力 */
    scanf("%d", &temp);
  }


  /* ここに平均より大きいデータだけ表示する処理 */
  ave = ave / i ; /*平均値の導出*/
  

  tempData *pMae = NULL; /*温度データリスト内の現在位置の一つ前*/

  /*リストから平均値以下を削除*/
  pNow = pTop;
  while(pNow != NULL){
    if(pNow->temp < ave){
      pMae->next = pNow->next;
    
      
    }else{
      pMae = pNow;
    }
    pNow = pNow->next;
   }
  
  /*データ出力*/
  printf("--------\n");
  printf("実行結果\n");
  
  
  pNow = pTop;
  while (pNow != NULL){
    printf("%d\n", pNow->temp);
    pNow = pNow->next;
  }
  
  return 0 ;
}

試したこと

ここに問題に対して試したことを記載してください。

補足情報（FW/ツールのバージョンなど）

AWS cloud9

Answer 1

問題が発生しているのは、下記の「ここが問題」とコメントを記した行と思われます。pMaeの値がNULLのままこの行に到達すると、アクセスできない領域への書き込みを行おうとすることになりますので、Segmentation faultが発生します。

  tempData *pMae = NULL; /*温度データリスト内の現在位置の一つ前*/

  /*リストから平均値以下を削除*/
  pNow = pTop;
  while(pNow != NULL){
    if(pNow->temp < ave){
      pMae->next = pNow->next;  // ここが問題


    }else{
      pMae = pNow;
    }
    pNow = pNow->next;
   }

この行にはいきなり到達しないつもりで作ったつもりならば、それも問題でしょう。どうやらaveの計算処理に誤りがあるようです。

  int temp; /* 温度入力用変数 */
  （中略）
  int ave = temp; /* 平均値の計算用 */

のように宣言されていますが、これではaveは不定値になってしまうでしょう。int ave = 0;の誤りなのでは？

Answer 2

Segmentation faultとは、アクセス違反が起こったときに、運がよかったら表示されるメッセージです
運が悪かったら、なにも言わずに暴走するか、
もっと悪ければ何食わぬ顔して動き続けます。

＃そして、数ヶ月して問題が出る

Answer 3

原因かどうかは別にして・・・

pNew = makeNewNode(temp);

これって、NULLが帰ってきたらどうなります？

Comments

[cateye]

scanf()やmalloc()のライブラリが、常に成功すると思わないことです。必ずエラー判定をしましょう。

Answer 4

Segmentation faultとは何なのでしょうか

メモリアクセスの失敗と言うことで、原因としては、
・ポインタ処理のバグ
・配列の添え字範囲外をアクセスしようとした
がほとんどでしょう。

Comments

[otn]

とりあえず、tempData *pMae = NULL; がSegmentation faultの直接原因だと思われるので、それを tempData *pMae = pTop; に変えればSegmentation faultにはなりませんが、先頭ノードの処理を間違えているので、このままでは駄目ですね。

あと、int ave = 0; です。

[otn]

もし、 /* 最初のデータは，必ず範囲内のデータであるとする */
というのが、 /* 最初のデータは，必ず範囲内で、平均以上のデータであるとする */
ならば、先頭ノードの処理は、このままでいいかな。