在 C 中传递结构时我做错了什么？

Question

所以我正在使用 C 语言开发一个项目，该项目要求我将指向结构的指针传递给函数。项目结构如下：

struct structName {
    unsigned short thing2;
    char thing1[];
};

void function_1(struct structName *s) {
    strcpy(s->thing1, "Hello");
    printf("Function 1\n%s\n\n", s->thing1); // prints correctly
}

void function_2(struct structName *s) {
    // can read thing2's value correctly
    // thing1 comes out as a series of arbitrary characters
    // I'm guessing it's an address being cast to a string or something?

    printf("Function 2\n%s\n\n", s->thing1); // prints arbitrary characters ('É·/¨')
}

int main() {
    struct structName s;
    function_1(&s);
    printf("Main\n%s\n\n", s.thing1);
    function_2(&s);
    printf("Main 2\n%s\n\n", s.thing1);
}

此代码输出以下内容：

Function 1
Hello

Main
Hello

Function 2
É·/¨

Main 2
É·/¨

显然，该程序不仅仅包含我在这里写的内容；这只是一个简化版本；所以如果有什么我应该检查可能导致这种情况的原因，请告诉我。老实说，我认为这可能只是我在某个地方犯的一个愚蠢的新手错误。

[编辑：似乎s.thing1 在对function_2() 的调用中以某种方式发生了变异，因为在main() 中复制了奇数值——我应该指出，在我的程序中printf()s 是位于函数调用之前和函数的第一行，所以它不可能被我正在做的任何事情写入。我已经更新了上面的示例代码以显示这一点。]

提前致谢！

Answer 1

该结构在其末尾包含一个灵活的成员，如果您声明一个具有该类型的静态对象，该成员的长度将为零，因此strcpy(s->thing1, "Hello"); 将具有未定义的行为。

您应该为此类结构的实例分配足够的额外空间来处理您希望存储到灵活数组中的任何数据。

这是一个例子：

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

typedef struct pstring {
    size_t length;
    char data[];
} pstring;

pstring *allocate_pstring(const char *s) {
    size_t length = strlen(s);
    pstring *p = malloc(sizeof(*p) + length + 1);
    if (p != NULL) {
        p->length = length;
        strcpy(p->data, s);
    }
    return p;
}

void free_pstring(pstring *p) {
    free(p);
}

int main() {
    pstring *p = allocate_pstring("Hello");
    printf("Main\n%.*s\n\n", (int)p->length, p->data);
    free_pstring(p);
    return 0;
}