Проблема с доступом к SafeArray с использованием CComVariant - PullRequest
Новая
       22

Проблема с доступом к SafeArray с использованием CComVariant

2 голосов
/ 12 марта 2009

у меня следующий блок кода 

 /////////////////////////////////////
 CComVariant newVal;

 //pass the CComVariant and get the strings array!!!
 GetStrList(newVal);
 USES_CONVERSION;

 if (((newVal.vt & VT_ARRAY) == VT_ARRAY) && ((newVal.vt & VT_BSTR) == VT_BSTR))
 {
SAFEARRAY* paArray = newVal.parray;
BSTR * str = NULL;
SafeArrayAccessData(paArray, (void**)&str);

SafeArrayUnaccessData(paArray);

long lLBound = 0;
long lUBound = 0;
long nCount = 0;

if (FAILED(SafeArrayGetLBound(paArray, 1, &lLBound)) ||
    FAILED(SafeArrayGetUBound(paArray, 1, &lUBound)))
{
    ASSERT(false);
    return FALSE;
}


nCount = ( lUBound - lLBound + 1 );
for (int i = 0 ; i < nCount ; i++)
{           
    m_cstrList.AddString(W2T(str[i]));                  
}   
//SafeArrayDestroy(paArray); ---> is it required here???

 }

 /////////////////////////////////////

метод извлечения безопасных массивов 

 HRESULT GetStrList(VARIANT *pVal)
 {
USES_CONVERSION;

if (!pVal)
    return E_FAIL;

SAFEARRAYBOUND bound[1]; //single dimension array
bound[0].lLbound = 0;
bound[0].cElements = 10;

SAFEARRAY * A = SafeArrayCreate(VT_BSTR, 1, bound);

BSTR * str = NULL;
SafeArrayAccessData(A, (void**)&str);

//user wants the NT view OPC drivers list.
for (int i=0;i<10;i++)
{       
    str[i] = SysAllocString(T2W(mystrings[i]));     
}


VariantInit(pVal);
pVal->vt        = VT_ARRAY | VT_BSTR;
pVal->parray    = A;

SafeArrayUnaccessData(A);
A = NULL;

return S_OK;
 }

Я сомневаюсь, что выше первого блока кода есть утечки памяти? Сам ли CComVariant справляется с уборкой? или я тоже вручную SafeArrayDestroy(paArray);

Заранее спасибо!

1 Ответ

3 голосов
/ 12 марта 2009

Деструктор CComVariant вызывает VariantClear (), который освобождает все, что был инкапсулирован, включая массивы.

Одно предупреждение: массив не должен быть заблокирован во время вызова VariantClear (). Это означает, что если исключение выдается после SafeArrayAccessData (), но до SafeArrayUnaccessData (), последнее не будет вызвано и VariantClear () не освободит ресурсы.

Поэтому лучше написать класс скобок для сопряжения вызовов SafeArrayAccessData () и SafeArrayUnaccessData ().

...