为什么这个 COM 代码会泄露?

Ben*_*enj 1 c++ windows com atl windows-shell

我正在维护一个使用 Windows 资源管理器覆盖图标的应用程序。有时,某些操作需要我强制刷新特定文件夹的资源管理器视图。我使用以下使用 COM 的函数来执行此操作:

void RefreshExplorerView(CString strPath)
{
    CComPtr<IShellWindows> pShellWindows;

    CoInitialize(NULL);

    if(SUCCEEDED(pShellWindows.CoCreateInstance(CLSID_ShellWindows)))
    {
        IDispatch* pFolder=NULL;
        VARIANT variant;
        V_VT(&variant) = VT_I4;

        for(V_I4(&variant) = 0; pShellWindows->Item(variant, &pFolder) == S_OK; V_I4(&variant)++)
        {
            CComPtr<IWebBrowserApp> pWebBrowserApp;
            if(SUCCEEDED(pFolder->QueryInterface(IID_PPV_ARGS(&pWebBrowserApp))))
            {
                BSTR LocationURL = NULL;
                pWebBrowserApp->get_LocationURL(&LocationURL);

                if(LocationURL != NULL && strPath.CompareNoCase(LocationURL) == 0)
                {
                    CComPtr<IServiceProvider> pServiceProvider;
                    if(SUCCEEDED(pWebBrowserApp->QueryInterface(IID_PPV_ARGS(&pServiceProvider))))
                    {
                        CComPtr<IShellBrowser> pShellBrowser;
                        if(SUCCEEDED(pServiceProvider->QueryInterface(IID_PPV_ARGS(&pShellBrowser))))
                        {
                            IShellView* pShellView;
                            if(SUCCEEDED(pShellBrowser->QueryActiveShellView(&pShellView)))
                            {
                                pShellView->Refresh();
                                pShellView->Release();
                            }
                        }
                    }
                }

                SysFreeString(LocationURL);
            }
            pFolder->Release();
            pFolder = NULL;
        }
    }

    CoUninitialize();
}
Run Code Online (Sandbox Code Playgroud)

我注意到,当我的程序定期进行此刷新时,它的大小会慢慢增长,并且 UMDH 向我表明,每次运行时,我似乎都在泄漏pFolder和实例。pShellWindow我无法弄清楚为什么会发生这种情况,因为据我所知,这些都已正确发布。谁能看到我缺少什么吗?

Rom*_* R. 5

pShellWindows您在 后释放CoUninitialize,这是不正确的。

其余接口似乎发布得很好。请注意,通过使用代替,并且根本不使用原始指针(,)并用智能自动释放包装器替换它们,可以极大地提高干净性和可读性。CComQIPtrQueryInterfaceBSTRIFoo*

pFolder如果Item调用成功但返回的代码不是S_OK. 同样,使用CComPtr<IFolder>代替IFolder*会立即解决这个问题,甚至不会引起任何注意。