将结构从 main() 传递给工作函数时的段错误

Question

我正在尝试用 C 编写一个简单的游戏，但我得到了一个 SEGFAULT，但不知道为什么！

这是程序的代码：

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

#define MAX_PLYS_PER_GAME (1024)
#define MAX_LEN (100)

typedef struct {
   char positionHistory[MAX_PLYS_PER_GAME][MAX_LEN];
} Game;

void getInitialGame(Game * game) {
    memset(game->positionHistory, 0, MAX_PLYS_PER_GAME*MAX_LEN*sizeof(char));
}

void printGame(Game game) {
    printf("Game -> %p (%d)\n", &game, sizeof(game));
    fflush(stdout);
}

int hasGameEnded(Game game) {
    printGame(game);
    return 0;
}

int main(int argc, char *argv[]) {
    Game game;
    getInitialGame(&game);

    if (hasGameEnded(game))
        return -1;

    return 0;
}

我尝试使用 gdb 进行调试，但结果并没有让我走得太远：

C:\Users\test>gdb test.exe
GNU gdb 5.1.1 (mingw experimental)
<snip>
This GDB was configured as "mingw32"...
(gdb) run
Starting program: C:\Users\test/test.exe

Program received signal SIGSEGV, Segmentation fault.
0x00401368 in main (argc=1, argv=0x341c88) at fast-chess-bug.c:29
29              if (hasGameEnded(game))
(gdb) bt
#0  0x00401368 in main (argc=1, argv=0x341c88) at fast-chess-bug.c:29

Answer 1

您可能会用完堆栈空间。

C 是按值传递。所以这段代码

int hasGameEnded(Game game)

创建整个struct {} Game 的副本，很可能在堆栈上。

如果以下代码有效，则说明您的堆栈空间不足：

...

void printGame(Game *game) {
    printf("Game -> %p (%zu)\n", game, sizeof(*game));
    fflush(stdout);
}

int hasGameEnded(Game *game) {
    printGame(game);
    return 0;
}

int main(int argc, char *argv[]) {
    Game game;
    getInitialGame(&game);

    if (hasGameEnded(&game))
        return -1;

    return 0;
}

仔细注意更改。而不是将整个结构传递给hasGameEnded，它现在只传递结构的地址。该更改沿调用堆栈向下流动，最终更改为printGame()。

还要注意proper format specifier for sizeof includes a z modifier. 我冒昧地将u 设为无符号，因为大小不能为负数。

Answer 2

这可能是堆栈溢出（真的！），虽然我不确定。

1. 您在main() 中声明Game game;。这意味着 game 的所有 102400 字节都在堆栈中。
2. printGame 和 hasGameEnded 都采用 Game game，而不是 Game * game。也就是说，他们得到的是Game 的副本，而不是指向现有Game 的指针。因此，无论何时调用任何一个，您都会在堆栈上转储另外 102400 个字节。

我猜测对printGame 的调用正在破坏堆栈，从而导致hasGameEnded 调用出现问题。

我知道的最简单的解决方法（不涉及动态内存分配，从长远来看可能更好）是：

1. 将Game game; 移到main() 之外，例如，移到int main(...) 正上方的行。这样它将在数据段中，而不是在堆栈中。

2. 将printGame和hasGameEnded改为Game *：

   void printGame(Game * game) {
       printf("Game -> %p (%d)\n", game, sizeof(Game));
       fflush(stdout);
   }
   
   int hasGameEnded(Game * game) {
       printGame(game);
       return 0;
   }
   

这应该会让你继续前进。