C：構造体の初期化がseg faultする

Question

下記のコードをコンパイルし実行するとsegmentation faultになるのですが、構造体はどのように初期化すべきなのでしょうか？？

#include <string.h>

struct sample
{
        char *test;
};


int main()
{
        struct sample *s;
        memset(s, 0, sizeof(struct sample));

        return 0;
}

調べるとmemsetが良くないという記事をたくさん見かけるのですが、上記のように構造体変数をポインタで宣言したい場合はどうすれば良いのかがわかりません。

ご教示いただけますと幸いです。
よろしくお願いいたします。

Answer 1

struct sample *s;

これは「構造体 sample へのポインタ」の宣言でしかありません。
つまりこのポインタが指す先が未だ決まっていません。（未定義状態）

この状態で

memset(s, 0, sizeof(struct sample));

したら、どこへアクセスするか分かったものではありません。結果的にアクセスしてはいけないメモリアドレスへアクセスすることになり、segmentation fault が起きたものです。

ちゃんとポインタが示す先に領域を確保しましょう。

Comments

[Ykkykk]

ご回答いただきありがとうございます。
事前にmallocすれば良いのでしょうか。。。？

[DreamTheater]

横から失礼します。
この種の問題でmallocすればという解決を良く見かけますが、構造体のサイズがさほど大きくなければ、ポインタで宣言せず、実体で宣言するだけで解決しますよ。

変更前：struct sample *s;
変更後：struct sample s;

もちろん memsetは
memset(&s, 0, sizeof(struct sample));
または
memset(&s, 0, sizeof(s));
となります。

[cateye]

malloc()でなくても、
struct sample v;
struct sample *s = &v;
で、大丈夫ｄ＾＾

[Ykkykk]

DreamTheater様
ご教示いただきありがとうございます。
実際、構造体変数を実体で宣言するのとポインタで宣言することの違いは構造体全体の大きさによる、ということでしょうか？
初歩的なことで恐縮ですが、また本来の質問からはそれてしまいますがお教えくだされば幸いです。

[Ykkykk]

cateye様
コメントいただきありがとうございます。
構造体のポインタに構造体の実体を代入するということですね。
ありがとうございます。

[cateye]

“構造体の実体を代入”ではなく、『構造体のアドレスを代入』する。・・・です。
また、ポインタで保持したほうがいいのか？実体だけでいいのか？・・・は、使い方次第です。

[tacsheaven]

確かに構造体を実体で宣言して使うのは楽は楽なのですが、C/C++ においてポインタは避けては通れないので、きちんと使い方を学ぶ意味でも、ここはちゃんと malloc とか free とか含めて勉強しましょう。

[Ykkykk]

tacsheaven
再度コメントくださりありがとうございます。
ポインタ周りとmalloc等について改めて学習しなおしたいと思います。
ありがとうございます！