使用ATL和IUnknownPtr时，正确的方法是什么？

在修改现有ATL COM对象的过程中，我遇到了一篇名为“人们弄乱了IUnknown :: QueryInterface的方式”的“The Old New Thing”博客文章，并且在评论部分有一个讨论，的受访者（诺曼钻石）指出，在文章的一个例子中，抛弃**是错误的。使用ATL和IUnknownPtr时，正确的方法是什么？

但是，当我尝试纠正我的代码来正确执行投射时，我最终发生内存泄漏。

的例子如下：

IShellFolder *psf = some object; 
IUnknown *punk = NULL; 
psf->QueryInterface(IID_IUnknown, (void**)&punk);

诺曼说

朋克不是一个void *。朋克是一个IUnknown *。

void **不是通用指针类型。 void *是一个通用的指针类型，并且char *和亲属是以类似的方式上传的，但void **不是。

如果你想服从调用约定并避免可怕的死亡，你必须这样做： IUnknown * punk; void * punkvoid; psf-> QueryInterface（IID_IUnknown，& punkvoid）; punk =（IUnknown *）punkvoid;

许多其他MSDN贡献者犯了同样的错误......有些人可能会说，它在所有VC++实现中都可以工作，但这并没有使它正确的代码，它仍然违反了调用约定。

鉴于此我去改变我的旧代码 - 这是如下：

#include <comdef.h> 

... 

HRESULT FinalConstruct() 
{ 
    if (m_dwROTCookie != 0) 
     return E_FAIL; 

    //Check whether there already is an instance of the Object 
    IUnknownPtr pUnk = NULL; 
    if (GetActiveObject(CLSID_Object, NULL, &pUnk) == S_OK) 
    { 
     TRACE_WARNING("An instance of Object already exists in the current context"); 
     return S_OK; 
    } 
    HRESULT hr = QueryInterface(IID_IUnknown, reinterpret_cast<void **>(&pUnk)); 

    hr = RegisterActiveObject(pUnk, CLSID_Object, ACTIVEOBJECT_WEAK, m_dwROTCookie);   
    if (FAILED(hr)) 
     return hr; 

    hr = CoLockObjectExternal(pUnk, TRUE, TRUE); 
    pUnk = NULL; 
    ATLASSERT(m_dwRef == 2); 
    return hr; 
}

我再变，如下所示：

HRESULT FinalConstruct() 
{ 
    if (m_dwROTCookie != 0) 
     return E_FAIL; 

    //Check whether there already is an instance of the Object 
    IUnknownPtr pUnk = NULL; 
    if (GetActiveObject(CLSID_Object, NULL, &pUnk) == S_OK) 
    { 
     TRACE_WARNING("An instance of Object already exists in the current context"); 
     return S_OK; 
    } 
    void* pUnkVoid = NULL; 
    HRESULT hr = QueryInterface(IID_IUnknown, &pUnkVoid); 

    if (SUCCEEDED(hr) 
    { 
     pUnk = reinterpret_cast<IUnknown*>(pUnkVoid); 
     hr = RegisterActiveObject(pUnk, CLSID_Object, ACTIVEOBJECT_WEAK, m_dwROTCookie);   
     if (FAILED(hr)) 
      return hr; 

     hr = CoLockObjectExternal(pUnk, TRUE, TRUE); 
     pUnk = NULL; 
    } 
    ATLASSERT(m_dwRef == 2); 

    return hr;

但是现在我的应用程序有一个内存泄漏从这个COM对象

来源

2010-03-08 Peter Nimmo

你可能有内存泄漏，因为你叫GetActiveObject()和QueryInterface()其在成功递增对象的引用计数，但不叫Release()以后递减引用计数。

来源

2010-03-09 07:41:04 sharptooth

嗯，我认为，而不是分配void * pUnk我应该使用：

pUnk.Attach(reinterpret_cast<IUnknown*>(pUnkVoid));

来源

2010-03-08 15:51:10

使用ATL和IUnknownPtr时，正确的方法是什么？

回答

相关问题