在 CLI 中处理 String^ ？

Question

我的代码中有一个关于 CLI/C++ 中字符串的小内存泄漏。 我尝试通过删除 unsigned char 数组来修复泄漏，但是当我这样做时，我遇到了内存访问冲突。

我认为这是因为 System::String 是引用类型，因此，内存与下面代码中的“testString”和“uch”相关联。 那是对的吗？ 如果是这样，我如何分离内存，以便释放内存并仍然返回 System::String？

myStatus get_testString(String^% testString)
{
    uchar* uch = 0;
    bool b = MgdStringToUChar(testString, uch);
    myStatus s = m_NativeMsg->get_testString(uch);
    testString = (reinterpret_cast<const char*>(uch));

    delete []uch;//this line causes an error       
    uch=0; 
    return s;
}

static bool MgdStringToUChar(System::String^ s, uchar*& uch)
{
    pin_ptr<const wchar_t> wch = PtrToStringChars( s ); 
    int len = (( s->Length+1) * 2); 
    size_t * st = 0; 
    char *ch = new char[ len ]; 
    bool result = wcstombs_s(st,ch, len, wch, len ) != -1; 
    if(!result) 
        throw (gcnew Exception("Could not parse string in :: MgdStringToUChar")); 
    uch = new uchar[len]; 
    int i=0; 
    while(i<len+1) 
    { 
        uch[i] = ch[i]; 
        i++; 
    } 
    delete st; 
    st=0; 
    delete [] ch; 
    ch=0; 
    return true; 
}; 

Answer 1

好的，首先您的 MgdStringToUChar 函数有一些问题。 wcstombs_s 的第一个参数应该是指向实际 size_t 的指针，而不是指向 null 的指针。 如果您想确保 ch 以 null 终止，那么您也应该使用 _TRUNCATE 或 len-1 作为计数参数。

从 ch 缓冲区到生成的 uchar 缓冲区的复制也可能超出了缓冲区的末尾。

static bool MgdStringToUChar(System::String^ s, uchar*& uch)
{
    pin_ptr<const wchar_t> wch = PtrToStringChars( s ); 
    int len = (( s->Length+1) * 2); 
    size_t st = 0;
    char *ch = new char[ len ]; 

    bool result = wcstombs_s(&st, ch, len, wch, _TRUNCATE) != -1; 
    if(!result) 
        throw (gcnew Exception("Could not parse string in :: MgdStringToUChar")); 

    uch = new uchar[st+1]; 
    uch[st] = NULL;

    for(int i = 0; i < st; i++) 
    { 
        uch[i] = ch[i]; 
    } 

    delete [] ch; 

    return true; 
}; 

鉴于您似乎需要基本的 ANSI 转换，您也可以像这样更简单地执行此操作：

static bool MgdStringToUChar(System::String^ s, uchar*& uch)
{
    char* ch = (char*)(void*)Marshal::StringToHGlobalAnsi(str);

    size_t st = strlen(ch);
    uch = new uchar[st + 1]; 
    uch[st] = NULL;

    for(int i = 0; i < st; i++) 
    { 
        uch[i] = ch[i]; 
    } 

    Marshal::FreeHGlobal(ch);

    return true; 
}; 

这应该会更好一些。 但是我仍然担心 get_testString 对 uch 中的数据做了什么。 如果没有看到完整的图片，就很难了解这里发生了什么。