0

我有一个简单的保存文件对话框，我希望将其用作返回文件路径、名称和扩展名的工具。此函数产生运行时错误，表示文件名周围的堆栈已损坏。我希望像这样使用它：

wchar_t filename[] = L"";
newGradebookDialog( hwnd, filename );

这是我的功能。它会按照我的预期修改文件名，但运行时堆栈错误是我没有得到的。

void newGradebookDialog( HWND hwnd, wchar_t file[] )
{
 OPENFILENAME ofn;

 wchar_t saveFileName[MAX_PATH] = L"";

 ZeroMemory( &ofn, sizeof( ofn ) );

 ofn.lStructSize = sizeof(OPENFILENAME);
 ofn.hwndOwner = hwnd;
 ofn.lpstrFilter = L"Database File (*.db)\0*.db\0";
 ofn.lpstrFile = saveFileName;
 ofn.nMaxFile = MAX_PATH;
 ofn.lpstrDefExt = L"db";
 ofn.Flags  = OFN_EXPLORER | OFN_PATHMUSTEXIST | OFN_HIDEREADONLY | OFN_OVERWRITEPROMPT;
 ofn.lpstrTitle = L"Save New Database";

 if(GetSaveFileName(&ofn))
  wcscpy(file,saveFileName);
}
4

3 回答 3

9 
 wchar_t filename[] = L"";

那是一个包含一个元素的数组。您正在将一个更大的字符串复制到其中，这会破坏堆栈帧。使固定：

 wchar_t filename[MAX_PATH] = {0};
于 2011-01-09T16:09:38.867 回答
1

我有一种感觉，您通过 wcscpy 破坏了内存——您分配了空的“文件名”并将非空值（来自 saveFileName）复制到它，从而破坏了内存。

此外，最好通过分配 MAXPATH+1 元素而不是 MAXPATH 来为尾随 \0 保留空间。虽然在这个特定的地方可能不需要，但多保留一个字符有时可以为您节省数小时的内存损坏错误跟踪。

于 2011-01-09T16:11:10.467 回答
0

尝试

wchar_t filename[MAX_PATH]
于 2011-01-09T16:10:16.957 回答