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

Question

我尝试使用 COM 在我的程序中实现一个简单的 FileOpenDialog。但我注意到，当使用 FileOpenDialog 对 select 文件进行多次成功调用时，发生了内存泄漏。我决定从 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;
}

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

Answer 1

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

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

作为解决方法，我会使用单个 IFileOpenDialog.

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

Why do repeated calls to FileOpenDialog cause a memory leak?

c++

com

memory-leaks

fileopendialog