使用_bstr_t进行内存泄漏，无法修复

Question

在我正在研究的这个程序中存在内存泄漏，就提交而言，它已经存在很长一段时间了。根据这两个解释explanation 1和explanation 2每当使用=赋值运算符_bstr_t会导致内存泄漏。

上下文 - 有一个数据库对象，通常用于执行数据库的快速sql查询。每种方法最终使用以下方法

NvStatus DbUtils::ReadFromDatabase(IUnknown * poNvData, 
            const std::wstring & oConnectString, 
            const std::wstring & oSQLStatement) 
{ 
    //some checks 
    _bstr_t tbtSQLStr = oSQLStatement.c_str();//memory leak 
    _bstr_t tbtConnStr = oConnectString.c_str();//memory leak 

    //pass the _bstr_t to another method and get data from DB 
    return status; 
} 

根据文章这个方法会在每次它被称为查询，因为_bstr_t的数据库以及如何创建时间的泄露数据。我的问题是我能做些什么来防止程序炸毁并强制对_bstr_t对象进行垃圾回收？

Microsoft指出我使用它清理内存是我的责任，所以如何在不破坏数据传递给我的情况下做到这一点？我尝试做一个字符串的深层副本，但失败了...任何建议将不胜感激！

经过进一步调查我的内存泄漏的两个热点就是我第一次公布这一个，希望这有助于

static bool GetValueFromVariant(VARIANT & tvInputValue, 
    std::wstring & roOutputValue) 
{ 
    _bstr_t tTemp = tvInputValue.bstrVal; 
    if(tTemp.length()>0) 
    { 
     roOutputValue = (wchar_t*) tTemp; 
    } 
    return true; 
} 

意见建议，这些_bstr_t应自动清洗自己了......然而，当调试我的Windows服务的堆大小，堆大小不断增加，调试器继续指向所有使用这些_bstr_t对象的函数。显然这些_bstr_t没有被清理。更多的上下文，这个内存泄漏的大部分源于反复创建一个COM对象，但是当我完成它并且我检查Release（）函数调用返回的引用计数时我释放该对象，它返回0.因此，我知道我没有建立COM对象...

当把一个wstring指向_bstr_t的地址时会出现问题吗？

Answer 1

您的示例的第一行不泄漏内存，因为您将一个C样式的字符串分配给_bstr_t包装器。如果您将之前分配的BSTR分配给_bstr_t，情况会有所不同。这是你的第二个解释中描述的问题。

考虑以下情况：

void foo() 
{  
    BSTR s1 = SysAllocString(L"String1"); 
    _bstr_t s2 = s1; 
} 

在这里，本机BSTR使用SysAllocString分配并置于S1。下一行从s1构建新的BSTR，它位于s2中。当s2超出范围时，其析构函数调用SysFreeString，从而取消分配副本。然而，原始的s1变量保持完整并且会泄漏。

为了解决这个问题，你需要让S2采取S1的所有权：

void foo() 
{ 
    BSTR s1 = SysAllocString(L"String1"); 
    _bstr_t s2(s1, false); 
} 

或

void foo() 
{ 
    BSTR s1 = SysAllocString(L"String1");   
    _bstr_t s2; 
    s2.Attach(s1); 
} 

正如我的意见指出，_bstr_t的析构函数会调用SysFreeString，从而负责释放资源。

由于BSTR通常被缓存，所以您不可以立即目击内存释放。通过setting an environment variable，可以禁用此行为用于调试目的。

Answer 2

在第一种情况下，根本不要拨打SysFreeString。没有内存泄漏。

构造用于：

_bstr_t tbtSQLStr = oSQLStatement.c_str(); 

创建源字符串的一个副本，这个副本是由析构函数调用时tbtSQLStr超出范围释放。使用类类型包装的重点在于，您不需要手动调用SysFreeString。

请注意，根据// ....块中代码的确切性质，您可能甚至不需要创建该字符串的副本。

---

在第二种情况下（这是真的到了第一种情况下一个单独的问题，您应该已经张贴2个不同的问题），也没有内存泄漏。

tTemp分配副本; roOutputValue复制副本，然后tTemp的析构函数释放第一个副本。

但是您可以通过创建和销毁tTemp浪费的时间，你可以只写：

if (SysStringLen(tvInputValue.bstrVal) > 0) 
    roOutputValue = tvInputValue.bstrVal; 

在那里我假设了“真正的代码”其实检查变种保持在这一点上BSTR。