C 语言新手请教一个段错误问题

[Uu6p94cFbGXx7duC]

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

struct Child {
};

struct Parent {
    struct Child *child;
};

int main(void)
{
    struct Child *child = (struct Child *)malloc(sizeof(struct Child));
    struct Parent *parent;

    parent->child = child;

    return 0;
}

报错为：bash: line 1: 30009 Segmentation fault: 11 "~/Code/hello_c/main"，但是如果把这两行交换，就不会报错，如下：

    struct Parent *parent;
    struct Child *child = (struct Child *)malloc(sizeof(struct Child));

想不通是为啥，特来请教。

[justou]

你的 parent 指针没分配内存

[newmlp]

这。。。也太新手了

[YouKnowIt]

哈哈哈，果然是新手

[araraloren]

这。。建议学习 rust 去吧

[Uu6p94cFbGXx7duC]

各位，我才学了一天，大可不必冷嘲热讽。原以为 v 站风气会好一点，看来也不过如此。顺便请教一下，如何注销账号。

[rbut]

脾气好像有点大。

[lance6716]

自己能说自己是新手，别人说不行🤔

[JamesMackerel]

我有点好奇，为啥交换就不会错了……

[killeder]

newbie

[sunnywqf]

@JamesMackerel 同好奇，一会儿用 Windows 试试

[sunnywqf]

@JamesMackerel #8 估计楼主说错了吧, 用 https://gcc.godbolt.org/ 看了下生成的 arm 汇编, 交换这两行,没有任何的变化 .

[sunnywqf]

多执行几遍呢, 是不是会有出错的时候.

[zjsxwc]

换不换顺序都会报错

[zjsxwc@manjaro]$ git diff q.c q2.c
diff --git a/q.c b/q2.c
index 032ef4a..517a6db 100644
--- a/q.c
+++ b/q2.c
@@ -10,8 +10,8 @@ struct Parent {

int main(void)
{
- struct Child *child = (struct Child *)malloc(sizeof(struct Child));
struct Parent *parent;
+ struct Child *child = (struct Child *)malloc(sizeof(struct Child));

parent->child = child;

[zjsxwc@manjaro]$ gcc -o q.out q.c
[zjsxwc@manjaro]$ gcc -o q2.out q2.c
[zjsxwc@manjaro]$ ./q.out
Segmentation fault (core dumped)
[zjsxwc@manjaro]$ ./q2.out
Segmentation fault (core dumped)

[yolee599]

这个行为很危险啊，函数内定义的变量 parent 没初始化的时候是不确定的，也就是说你给一个不确定的地址赋值，百分百造成内存溢出。

[csfreshman]

正确的写法是：
int main(void)
{
struct Child *child = (struct Child *)malloc(sizeof(struct Child));
struct Parent parent;

parent.child = child;
printf("hello,world");
return 0;
}

交换顺序也不会正确运行，最简单的写法是声明栈变量 parent，当然也可以分配堆变量(malloc),你的写法没有分配内存，直接对一个未初始化的指针操作是未定义行为。

[BrettD]

新手脾气还这么大，牛气上天了

[ch2]

交换后没报错只是你运气好，这种写法是完全错误的

[crazytec]

兄弟，我觉得这种问题直接上 stackoverflow 或者在各种群里问就行了（反正 rust 有中国 tg 非官方群，c 不知道有没有）。我个人的见解是 V2EX 和知乎的定位比较类似，都是偏向生活（比如讨论一下新闻或者是自己的人生经验）而非严肃的编程话题（比如说代码哪里错了之类的）。v2ex 的主页、热门机制都是为了更多人看到一个帖子并且讨论，因此很多情况下点进帖子的人根本没想解答你的疑惑（甚至连 c 语言怎么写都不知道），单纯就是想发表一下自己的意见。在这种偏技术的帖子里，他们就只好喷一下“新手”这个词。不用太放在心上就行了🙂

[Kasumi20]

struct Parent *parent;

这个应该是为定义行为吧？有的编译器会设为 0，所以段错误
有的是个不处理的野指针，所以可能不报错。

[raysonx]

@Kasumi20 全局变量的话会被初始化为 0，非全局变量的初始值是未定义的。