使用 ::SysFreeString() 释放 BSTR。更多平台依赖性？

我正在编写一个 COM 服务器，它有很多接口和方法。大多数方法都将 BSTR 作为参数并作为用于返回的本地参数。一个片段看起来像

更新5：

真正的代码。它根据数据库的特定条件从一堆数据中获取数据以填充对象数组。

STDMETHODIMP CApplication::GetAllAddressByName(BSTR bstrParamName, VARIANT *vAdddresses)
{
    AFX_MANAGE_STATE(AfxGetStaticModuleState())

//check the Database server connection

COleSafeArray saAddress;
HRESULT hr;

// Prepare the SQL Strings dan Query the DB

long lRecCount = table.GetRecordCount();

 if (lRecCount > 0)
 {
    //create one dimension safe array for putting  details
    saAddress.CreateOneDim(VT_DISPATCH,lRecCount);

    IAddress *pIAddress = NULL; 
    //retrieve details 
    for(long iRet = table.MoveFirst(),iCount=0; !iRet; iRet = table.MoveNext(),iCount++)
    {
        CComObject<CAddress> *pAddress;
        hr = CComObject<CAddress>::CreateInstance(&pAddress);
        if (SUCCEEDED(hr))
        {   
            BSTR bstrStreet = ::SysAllocString(table.m_pRecordData->Street);
            pAddress->put_StreetName(bstrStreet);

            BSTR bstrCity = ::SysAllocString(table.m_pRecordData->City);
            pAddress->put_CityName(bstrCity);
        }
        hr = pAddress->QueryInterface(IID_IAddress, (void**)&pIAddress);
        if(SUCCEEDED(hr)) 
        {
            saAddress.PutElement(&iCount,pIAddress); 
        }
    }
    *vAdddresses=saAddress.Detach(); 
}
table.Close(); 
return S_OK;
}


STDMETHODIMP CAddress::put_CityName(BSTR bstrCityName)
{
    AFX_MANAGE_STATE(AfxGetStaticModuleState())
    // m_sCityName is of CComBSTR Type
    m_sCityName.Empty();//free the old string 
    m_sCityName = ::SysAllocString(bstrCityName);//create the memory for the new string
    return S_OK;
}

问题出在内存释放部分。该代码在任何 Win XP 机器上都运行得很好，但是当涉及到 WIN2K8 R2 和 WIN7 时，代码崩溃并指出 ::SysFreeString() 是罪魁祸首。 MSDN 不足以提供解决方案。

任何人都可以帮助找到正确的解决方案吗？

预先非常感谢:)

更新1：

我已尝试按照建议使用 CComBSTR 代替原始 BSTR，使用直接 CString 进行初始化并排除 SysFreeString()。但对于我来说，在超出范围时，系统正在调用 SysFreeString()，这再次导致崩溃:(

更新2：使用相同的 CComBSTR 我尝试使用 SysAllocString() 进行分配，问题仍然相同:(

更新3：我厌倦了所有的选择，平静地我脑子里只有一个问题

是否有必要通过 SysFreeString() 释放 BSTR 使用 SysAllocString()/string.AllocSysString() 分配？

更新4：我错过了提供有关崩溃的信息。当我尝试调试 COM 服务器崩溃并显示错误时

“可能的堆损坏”

。请帮助我离开这里..:(

// Now All Things are packed in to the Object
obj.Name = bstrName;
obj.Name2 = bstrname2;

我不太明白你所说的东西被打包是什么意思，因为你只是复制指向字符串的指针，而当你调用 SysFreeString obj.Name 和 obj.Name2 时，它将指向无效的内存块。尽管此代码不安全，但看起来问题的根源是 CFoo 类。您应该向我们展示您的代码的更多详细信息

我建议您使用 CComBSTR 类，它将负责释放内存。

UPDATE

#include <atlbase.h>
using namespace ATL;
...
{
    CComBSTR bstrname(_T("Some Name")); 
    CComBSTR bstrname2(_T("Another Name"));
    // Here one may work with these variables if needed
    ...
    // Copy the local values to the Obj's member Variable 
    bstrname.Copy(&obj.Name); 
    bstrname2.Copy(&obj.Name2);
}

UPDATE2首先，应该在该块中使用 SysFreeString 释放 bstrCity 和 bstrStreetName 或使用 CComBSTR 代替：

if (SUCCEEDED(hr))
{   
    BSTR bstrStreet = ::SysAllocString(table.m_pRecordData->Street);
    pAddress->put_StreetName(bstrStreet);

    BSTR bstrCity = ::SysAllocString(table.m_pRecordData->City);
    pAddress->put_CityName(bstrCity);

    // SysFreeString(bstrStreet)
    // SysFreeString(bstrCity)
} 

考虑放大循环条件 !iRet，其中 iCount

for(...; !iRet /* && (iCount < lRecCount) */; ...)

也在这里：

m_sCityName = ::SysAllocString(bstrCityName);

您分配内存但从未释放它，因为 CComBSTR& 运算符 = (OLESTR ..) 本身会分配新的存储空间。应该重写如下：

m_sCityName = bstrCityName;

其他一切对我来说看起来都不错

UPDATE3嗯，堆损坏通常是在分配的内存块之外写入某些值的结果。假设您分配一个长度为 5 的数组并将一些值放入第 6 个位置