内存分配与释放问题请教

[码大虾]

string AesEncryptor::EncryptString(string strInfor) {
int nLength = strInfor.length();
int spaceLength = 16 - (nLength % 16);
unsigned char* pBuffer = new unsigned char[nLength + spaceLength];
memset(pBuffer, '\0', nLength + spaceLength);
memcpy_s(pBuffer, nLength + spaceLength, strInfor.c_str(), nLength);
m_pEncryptor->Cipher(pBuffer, nLength + spaceLength);

// 这里需要把得到的字符数组转换成十六进制字符串
char* pOut = new char[2 * (nLength + spaceLength)];
memset(pOut, '\0', 2 * (nLength + spaceLength));
Byte2Hex(pBuffer, nLength + spaceLength, pOut);

string retValue(pOut);
delete[] pBuffer;
delete[] pOut;
return retValue;
}

在delete[] pOut时报错，弹出对话框:Windows 已在 ***.exe 中触发一个断点。其原因可能是堆被损坏

[码大虾]

没有专家来指导下吗？

[schlafenhamster]

int nLength = strInfor.length(); 如果 长度不大 ，不要 new delete char Out【4096】；

[棉猴]

个人觉得与string的“写时拷贝”特性有关

string retValue(pOut);

根据“写时拷贝”特性，retValue和pOut指向同一块内存，在程序结束时，string类的析构函数已经释放了这块内存，然后再调用

delete[] pOut;

释放这块已经被释放的内存，就会报错！

[smwhotjay]

要么用c方法。调用时传递buff和buffsize,如果保存的数据超过bufsize,就return 一个负的错误码，如果数据足够容纳，那么return 1 要么string retValue(pOut); 改成 string retValue; retValue=pOut; 这就是内部浅拷贝和深拷贝那些事，不熟则避免

[zgl7903]

Byte2Hex 是怎么写的？ 估计有越界问题

[零隐]

你的转换已经超出pOut所能承受的长度。 你把pOut设置长点。或者谁写的函数问下这个长度该定义多长合适。 记得给分。

[lonelyhacker]

Byte2Hex(pBuffer, nLength + spaceLength, pOut); //这句可能转换的时候越界了。 你可以尝试把这条注释掉，然后看看还有没有问题，如果有问题，就是这句出现的错误。

[这不是鸭头]

贴代码的时候，记得选一下C/C++的格式。