Нарушение прав доступа на CompStr (BSTR, BSTR) после получения строки?

Question

Я получаю нарушение доступа, которое я не могу понять при попытке сравнить с строками BSTR в функции C++. По какой-то причине код работает обычно большую часть времени. Но иногда, казалось бы, случайным образом, это исключает. Это то, что я делаю:

BSTR myString; // also tried this with a CComBSTR, same result, but less often it seemed 
pSomeObject->GetString(&myString); 
if (!CompStr(someOtherString, myString)) 
{ 
    //do stuff 
} 

SomeObject :: GetString является:

STDMETHODIMP SomeObject::GetString(BSTR* outStr) 
{ 
    if (!outStr) return E_POINTER; 

    *outStr = ::SysAllocString(m_memberString); 

    return S_OK; 
} 

Я получаю нарушение прав доступа в CompStr, который:

inline bool CompStr(BSTR str1, BSTR str2) 
{ 
    UINT len1 = ::SysStringLen(str1); 
    UINT len2 = ::SysStringLen(str2); // this is where I'm getting the access violation 

    return CompStr(str1, len1, str2, len2); 
} 

Любые идеи о том, что я делаю неправильно или почему это неудачно?

Answer 1

Вы не инициализируете myString и не проверяете результат GetString().

BSTR myString = NULL; // CComBSTR initializes for you 
if (SUCCEEDED(pSomeObject->GetString(&myString))) 
{ 
    if (!CompStr(someOtherString, myString)) 
    { 
     //do stuff 
    } 
} 

STDMETHODIMP SomeObject::GetString(BSTR* outStr) 
{ 
    if (!outStr) return E_POINTER; 

    *outStr = ::SysAllocString(m_memberString); 
    if (!*outStr) return E_OUTOFMEMORY; 

    return S_OK; 
} 

Кроме того, убедитесь, что someOtherString выделяется как BSTR, иначе SysStringLen() не будет работать на нем. Я бы предложил изменить CompStr() вместо WCHAR* и lstrlenW(). Вы можете передать BSTR к WCHAR*:

inline bool CompStr(WCHAR* str1, WCHAR* str2) 
{ 
    UINT len1 = ::lstrlenW(str1); 
    UINT len2 = ::lstrlenW(str2); 

    return CompStr(str1, len1, str2, len2); 
}