1

Подскажите, этот код имеет право на существование или нет?

HRESULT CMyClass::Assign(IUnknown * src) {
    if (!src)
        return E_INVALIDARG;
    HRESULT res;
    ATL::CComPtr<IAddress> addr;
    res = src->QueryInterface<IAddress>(&addr);
    if (SUCCEEDED(res))
        return addr->getStreet(*this->street);
    else
        return res;
}

Что меня смущает. В конструкторе класса CBasePtr, который является родительским классом для класса CComPtr, я вижу вызов AddRef(). Но QueryInterface, по идее уже сам вызывает AddRef(), т.е. получится утечка памяти. Или т.к. в QueryInterface передается указатель, то дополнительный AddRef() вызываться не будет?

15
  • 1
    В приведенном коде CBasePtr не видать, и вообще, он не собирается. 19 мая 2021 в 12:10
  • @user7860670 CBasePtr родитель CComPtr. И в каком месте он не собирается? 22 мая 2021 в 17:40
  • 1
    Да в самом начале - символ STDMETHODIMP не определен. Как вы умудрились заработать 30к репутации, но не быть в курсе, что в вопросе требуется приводить минимальный воспроизводимый пример? 23 мая 2021 в 6:42
  • @user7860670 STDMETHODIMP - это стандартный макрос, определенный в winnt.h. Убрал его из кода 28 мая 2021 в 13:12
  • @user7860670 минимальный пример не означает, что нужно копировать в пример все из стандартных заголовков, особенно если пример не про отладку, а про поведение стандартной библиотеки при определенном порядке вызовов.
    – user177221
    31 мая 2021 в 14:08

1 ответ 1

1

Базовым классом для CComPtr у меня в VS2017 является CComPtrBase. Вот его конструкторы (из atlcomcli.h):

CComPtrBase() throw()
{
    p = NULL;
}
CComPtrBase(_Inout_opt_ T* lp) throw()
{
    p = lp;
    if (p != NULL)
        p->AddRef();
}

Как видно, здесь есть вызов AddRef, но только в конструкторе с параметром. Этот конструктор вызывался бы в коде вида ATL::CComPtr<IAddress> addr(somePointer);. В вашем случае же вызывается конструктор без параметра, а затем в пустой экземпляр CComPtr записывается адрес из QueryInterface. AddRef будет вызван только один раз (в QueryInterface).

Сказать, есть ли утечка памяти, по приведенному неполному коду нельзя. Если в метод передается правильная реализация IUnknown, то ее не должно быть, в том смысле, что в этом коде нечему вызвать утечку памяти. Есть ли утечка памяти в вашем реальном коде можно узнать с помощью отладчика, поставив точку останова на метод Release в COM-объекте. Если все нормально, при последнем вызове Release счетчик ссылок должен быть равен нулю.

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими пользовательским соглашением, политикой конфиденциальности и политикой о куки

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.