-1

我正在使用以下函数打开一个对话框，使用 OPENFILENAME。问题是，打开对话框并关闭它后，一切正常，但是当我退出我的应用程序时，Windows 说它崩溃了。

bool openDialog()
{
  OPENFILENAME ofn;       // common dialog box structure
  char szFile[260];       // buffer for file name
  HWND hwnd = NULL;       // owner window
  HANDLE hf;              // file handle

  // Initialize OPENFILENAME
  ZeroMemory(&ofn, sizeof(ofn));
  ofn.lStructSize = sizeof(ofn);
  ofn.hwndOwner = hwnd;
  ofn.lpstrFile = szFile;
  //
  // Set lpstrFile[0] to '\0' so that GetOpenFileName does not 
  // use the contents of szFile to initialize itself.
  //

  ofn.lpstrFile[0] = '\0';
  ofn.nMaxFile = sizeof(szFile);
  ofn.lpstrTitle = "Open File";
  ofn.lpstrFilter = "Custom File\0*.Cus\0";

  ofn.nFilterIndex = 1;
  ofn.lpstrFileTitle = NULL;
  ofn.nMaxFileTitle = 0;
  ofn.lpstrInitialDir = NULL;
  ofn.Flags = OFN_PATHMUSTEXIST | OFN_FILEMUSTEXIST;

  // Display the Open dialog box. 

  if (GetOpenFileName(&ofn) == true) 
  {
    path = ofn.lpstrFile;
    return true;
  }
  else
  {
    return false;
  }
}

我注意到它是对话框，因为只有在运行时使用它时才会发生崩溃，这意味着当我关闭应用程序而不在某个时候打开对话框时，它会成功退出。

无论如何，据我研究，这可能是导致我的“堆损坏”或类似的东西，我不太了解，所以当我的应用程序试图关闭时，它不会释放应该的内存（？ ）。如果有人能找到解决方案，将不胜感激。

4

1 回答 1

1

最有可能的问题与path这是否只是一个const char *或char *在函数外部声明的有关。

当openDialog函数返回时，ofn类型OPENFILENAME和字符数组的结构szFile都超出范围（以及所有其他本地的、堆栈分配的变量）并且不再有效。结果，ofn.lpstrFile因此path指向的字符数组不再有效。

strcpy您应该在函数外部为文件名分配空间，并strncpy在从它返回之前使用 或等效项执行复制。另一种方法是将szFile字符数组移出函数并作为成员变量移入包含类。无论哪种方式，包含返回文件名的字符数组的范围都需要超出openDialog函数执行的生命周期。

于 2013-08-08T17:31:34.590 回答