新建/删除导致访问冲突

Question

好吧，这让我很困惑...下面的代码位于 DLL 中，当我的控制台应用程序调用此代码时，它突然在 delete[] lpBuffer 行引发访问冲突。我一整天都在使用这段代码，根本没有改变它。到目前为止，它一直运行良好。

访问违规消息

rhcopy.exe 中 0x6948b1a5 处未处理的异常：0xC0000005：读取位置 0x4de1c37f 时发生访问冲突。

图书馆代码

#define MAX_PACKET_SIZE  0x3FFF

DWORD MyClass::GetFile( LPCSTR lpszRemoteFile, LPCSTR lpszLocalFile )
{
    LPBYTE lpBuffer = NULL;

    // ...

    lpBuffer = new BYTE[MAX_PACKET_SIZE];

    // ...

    if( NULL != lpBuffer )
        delete[] lpBuffer;

    // ...
}

我做错了什么吗？

附带说明：我一直在考虑将 lpBuffer 转换为向量。意见？

编辑

我要感谢你们的帮助！但显然……问题不是在这里。问题实际上是调用应用程序中的 printf() 语句，该语句在调用 GetFile(...) 之后立即发生。对于造成的混乱，我深表歉意。看起来微软的调试工具并没有指向导致错误的行，而是指向最后执行的行。我已投票结束该问题。

Answer 1

您在此处显示的代码没有任何问题。

我想到了两种可能性：

1. 错误实际上并不在 delete 中，而是在两侧的代码中。

2. 您通过使用无效指针覆盖某些随机内存或多次删除指针来破坏堆。

通过在显示的代码中切换到 RAII（即 vector），这些问题都不会被捕获，但如果您在代码的其余部分中一致地使用它们，则可能会有所改善。

Answer 2

鉴于发布的代码，我不太确定为什么这里会导致访问冲突。

由于缓冲区仅在本地引用，您可以将其设为堆栈变量而不是动态分配吗？

如果它必须是一个指针，也许你可以使用 Boost 智能指针来代替：

#include <boost/scoped_array.hpp>

#define MAX_PACKET_SIZE  0x3FFF

DWORD MyClass::GetFile( LPCSTR lpszRemoteFile, LPCSTR lpszLocalFile )
{

    boost::scoped_array<BYTE> bufferPtr;
    ...
    bufferPtr = boost::scoped_array<BYTE>(new BYTE[MAX_PACKET_SIZE]);
    ...
    //No delete needed

}

在向量点上，如果使用向量很容易，我会选择它或任何其他 STL 容器。