Run-Time Check Failure #2：变量'msg'栈损坏问题求助

安全阿岳

2026-6-15

问题分析与解决

核心问题：栈损坏的直接原因

你代码里的char msg[](bufLen, 0);是非法的C++语法，而且即使语法修正，如果bufLen的大小小于你要读取的lenMsg（也就是strlen(cstr)，这里是13），fread会写入超出数组容量的内容，直接破坏栈结构，触发Run-Time Check Failure #2。

具体修正步骤

修复数组定义语法
先定义足够大的缓冲区常量，再正确初始化数组：

const int bufLen = 100; // 定义足够容纳读取内容的缓冲区
char msg[bufLen] = {0}; // 合法初始化栈数组

或者用std::string替代C风格数组，完全避免手动管理缓冲区的麻烦：

std::string msg(lenMsg, '\0');
fread(&msg[0], 1, lenMsg, fp);

修正fseek偏移量
你当前用-(lenMsg+1)作为偏移量是错误的：刚写完内容后，文件指针在写入内容的末尾，要回到写入内容的开头，只需要回退lenMsg个字节。修正后：
```
fseek(fp, -(long)lenMsg, SEEK_CUR); // 强制转long避免偏移量溢出
```

完整修正后的代码示例

FILE* fp = fopen(fname, "a+");
if (!fp) { // 必须检查文件是否打开成功
    perror("Failed to open file");
    return 1;
}

std::string response{ "Hello World!\n" };
const char* cstr = response.c_str();
const int lenMsg = response.size(); // 直接用string的size，比strlen高效

int loop{ 0 };
const int bufLen = 100;

while (loop < 10)
{
    fwrite(cstr, 1, lenMsg, fp);
    
    fseek(fp, -(long)lenMsg, SEEK_CUR);
    char msg[bufLen] = {0};
    const size_t ret_code = fread(msg, 1, lenMsg, fp);
    fseek(fp, 0, SEEK_END);

    std::cout << msg; // 直接输出数组，无需循环
    
    loop++;
}

fclose(fp);