为什么重复调用 FileOpenDialog 会导致内存泄漏？答案

【问题标题】：Why do repeated calls to FileOpenDialog cause a memory leak?为什么重复调用 FileOpenDialog 会导致内存泄漏？
【发布时间】：2020-08-08 10:36:44
【问题描述】：

我尝试使用 COM 在我的程序中实现一个简单的 FileOpenDialog。但我注意到，当使用 FileOpenDialog 多次成功调用选择文件时，发生了内存泄漏。我决定从 msdn 复制 this 的确切示例，但是如果我在此示例中添加一个循环，即像这样：

#include <windows.h>
#include <shobjidl.h> 

int WINAPI wWinMain(HINSTANCE hInstance, HINSTANCE, PWSTR pCmdLine, int nCmdShow)
{
    HRESULT hr = CoInitializeEx(NULL, COINIT_APARTMENTTHREADED |
        COINIT_DISABLE_OLE1DDE);
    while(true)
    {
        IFileOpenDialog* pFileOpen;

        // Create the FileOpenDialog object.
        hr = CoCreateInstance(CLSID_FileOpenDialog, NULL, CLSCTX_ALL,
            IID_IFileOpenDialog, reinterpret_cast<void**>(&pFileOpen));

        if (SUCCEEDED(hr))
        {
            // Show the Open dialog box.
            hr = pFileOpen->Show(NULL);
            // Get the file name from the dialog box.
            if (SUCCEEDED(hr))
            {
                IShellItem* pItem;
                hr = pFileOpen->GetResult(&pItem);
                if (SUCCEEDED(hr))
                {
                    PWSTR pszFilePath;
                    hr = pItem->GetDisplayName(SIGDN_FILESYSPATH, &pszFilePath);

                    // Display the file name to the user.
                    if (SUCCEEDED(hr))
                    {
                        MessageBox(NULL, pszFilePath, L"File Path", MB_OK);
                        CoTaskMemFree(pszFilePath);
                    }
                    pItem->Release();
                }
            }
            pFileOpen->Release();
        }
    }
    CoUninitialize();
    return 0;
}

每次我选择一个文件时，进程内存仍然会增加。我不明白为什么重复调用此代码会导致内存泄漏。

【问题讨论】：

Visual Studio 具有出色的leak detection 功能。使用它们而不是猜测。
您确定有泄漏吗？在我的电脑上，如果我在对话框中一直按取消，内存会浮动在 7 MB 左右，如果我每次选择一个文件，它会增长到 14MB，然后在 14MB 和 15MB 之间浮动。
@GSerg 感谢您的评论！我从 vs 诊断工具中添加了我的进程内存的屏幕截图。每次我打开一个文件它都会增加，我是否使用了 VS 诊断工具错误？我不明白为什么这似乎与其他人的发现不同。
@botch 您显示的代码中没有“泄漏”。所有用户代码分配都是平衡和适当的释放。因此，进程内存使用量的任何增加都必须发生在操作系统内部。例如，可能在对话框本身内部使用了故障的 Shell 扩展。

标签： c++ memory-leaks com fileopendialog

【解决方案1】：

Deleaker 分析的代码，比较了快照，确实有很多漏洞（见下图）。

同时，代码本身是干净的（尽管避免显式调用 Release() 总是一个好主意：smart pointers 是我们的好朋友）。

作为一种解决方法，我将使用单个 IFileOpenDialog。

【讨论】：