C言語 構造体 Segmentation fault:11

Question

Segmentation fault:11が出ました

構造体のnumに0を代入して構造体を返すという関数funcを使って、構造体kのnumに0が代入されて、返ってきた構造体をsに代入するというコードです。
参照しては行けないメモリ領域に対して参照した時に出るエラーということはわかりましたが、なぜこのコードでエラーが発生するのかがわかりません。

かなり初歩的な質問で申し訳ございませんが、ご教授いただきたいです。よろしくお願いします。

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

struct _cell {
  union {
    int num;
  }data;
};
typedef struct _cell cell;
typedef cell *ptr;


ptr func(ptr p) {
  p->data.num = 0;
  return p;
}


int main(void) {
  ptr s, k;
  int count = 0;
  while(count < 10) {
    s = func(k);
    printf("%d\n", s->data.num);
    count++;
  }
  return 0;
}

Comments

[cateye]

kが初期化されていませんが？
・・・warning: variable 'k' is uninitialized when used here・・・

Answer 1

構造体のnumに0を代入して構造体を返すという関数funcを使って、構造体kのnumに0が代入されて、返ってきた構造体をsに代入するというコードです。

ポインターに関する理解が不完全なのでは。

ポインター変数kは、cell型のデータ(すなわち_cellタグを持つ構造体)のメモリ上の位置を保存するための変数であって、cell型の変数そのものではありません。

そして、C言語の自動変数(staticでないローカル変数)の初期値は不定で、どんな値が入っているかわかりません。

つまり、あなたの書いたコードは、「どこかわからない位置のメモリ領域に0を書き込む」ことを試みるので、その結果「Segmentation fault:11」が発生します。

まずは、cell型の変数を別に宣言してください。これで、メモリ上にcell型のデータを格納するメモリ領域が確保されます。それから、その変数のメモリ上の位置を&変数名で取得して、ポインター変数kに代入してください。ここまですれば、func(k)を呼び出しても「Segmentation fault:11」は出なくなります。

Answer 2

ローカル変数はデタラメな値が入ります
なので、必ず初期化するか何か値を入れて使いましょう

Answer 3

ｋの初期化漏れです。

int main(void)
{
    cell c; // 追加
    ptr s, k = &c; // d^^
    int count = 0;
    while(count < 10) {
        s = func(k);
        printf("%d\n", s->data.num);
        count++;
    }
    return 0;
}

結果

usr ~/Project/test % ./a.out 
0
0
0
0
0
0
0
0
0
0