c++ - IUnknown.Release 标准实现竞争条件？

Question

这是实现 IUnknown COM 的Release 方法的标准(不是推荐)方法界面(直接取自MSDN):

ULONG CMyMAPIObject::Release()
{
    // Decrement the object's internal counter.
    ULONG ulRefCount = InterlockedDecrement(m_cRef);
    if (0 == m_cRef)
    {
        delete this;
    }
    return ulRefCount;
}

我想知道如果公寓模型不是 STA 是否会出现竞争条件:

• 说还剩一个引用
• 线程 1 通过调用 Release 释放它
• 它在 delete this 之前运行并停止
• 线程 2 被调度并获取对象的新引用，例如通过调用 QueryInterface 或 AddRef
• 线程1继续执行并运行delete this
• 线程 2 留下了一个无效的对象

对我来说，确保一致性的唯一方法是创建一个标志，比如 deleted，锁定整个关键部分，即除了返回之外的所有 Release 方法，并将标志设置为 true。

并且在 AddRef 和 QueryInterface 方法中检查这个标志，如果它被设置则拒绝新引用的请求。

我错过了什么？

提前致谢。

Answer 1

thread 2 is scheduled and obtain a new reference to the object, e.g. by calling QueryInterface or AddRef

只有当它已经引用了 IUnknown 或该对象实现的其他接口(interface)之一时，它才能执行此操作。之前为其调用了 AddRef()。因此，引用计数永远不会被另一个线程的 Release 调用减少到小于 1 的值。

请正确编写代码，您必须将 ulRefCount 与 0 进行比较，而不是 m_cRef。