Почему повторные вызовы FileOpenDialog вызывают утечку памяти? - PullRequest
2 голосов
/ 24 апреля 2020

Я пытался внедрить простой FileOpenDialog в мою программу, используя COM. Но я заметил, что когда было сделано несколько успешных вызовов для выбора файлов с помощью FileOpenDialog, произошла утечка памяти. Я решил скопировать этот точный пример из msdn, но если я добавлю к этому примеру al oop, например, так:

#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;
}

Память процесса все еще увеличивается каждый раз, когда я выбираю файл. Я не понимаю, почему повторные вызовы этого кода вызывают утечку памяти.

![enter image description here

1 Ответ

1 голос
/ 25 апреля 2020

Профилированный код Deleaker, сравнил снимки и действительно получил многочисленные утечки (см. Рисунок ниже).

В то же время сам код чист (хотя всегда стоит избегать явного Release()) звонки: умные указатели - наши хорошие друзья).

В качестве обходного пути я бы использовал один IFileOpenDialog.

Leaks

...