Question

我正在使用以下函数打开一个对话框，使用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;
  }
}

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

无论如何，就我研究而言，它可能导致我的'堆腐败'或类似的东西，我不太了解，所以当我的应用程序试图关闭时，它不释放内存，因为它应该（？）。如果有人能找到解决方案，我们将不胜感激。

Answer 1

如果这只是在函数外声明的path或const char *，则问题很可能与char *有关。

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

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

退出后C ++程序崩溃

1 个答案: