奇怪的Heap corrupt问题，_CrtIsValidHeapPointer失败

2万 · 2010-7-30 20:50

一个由VC6迁移到VC2008SP1的工程。
使用了Cypress的USB库：CyUSB.lib（静态链接，无dll文件）
在主线程中我new了一个字符串，然后作为参数，传递给线程，线程中使用该字符串，然后delete[]，结果引发异常。

2万 · 2010-7-30 21:01

你的设置？

2万 · 2010-7-30 21:01

详细点，可以的话上个代码吧

2万 · 2010-7-30 21:08

工程设置：Debug multithread DLL(/MDd)

2万 · 2010-7-30 21:08

改用GlobalAlloc/GlobalFree依然无济于事。

2万 · 2010-7-30 21:09

我认为CyUSB.lib可能静态链接到了CRT库，而我的版本是使用动态链接。

2万 · 2010-7-30 21:25

改下工程设置

2万 · 2010-7-30 21:27

设置为“静态使用MFC、使用CRT库的多线程非DLL版本”，依然会报错。

2万 · 2010-7-30 21:27

在VC6中一切正常。

2万 · 2010-7-30 21:31

代码：
C/C++ code
//CEmuDevice继承自CCyUSBDevice，该类由CyUSB.lib导出
LRESULT CEmuDevice::SendCommand(COMMAND nCommand, WPARAM wParam/* =0 */, LPARAM lParam/* =0 */)
{
ASSERT(IsOpen());

WaitWorkerThreadExit();

PWORKITEM pWorkItem = new WORKITEM;
pWorkItem->wParam = wParam;
pWorkItem->lParam = lParam;
pWorkItem->pDevice = this;
switch(nCommand)
{
      case CONFIG_EMULATOR:
      {
         LPCTSTR lpszConfigFile = (LPCTSTR)wParam;
         LPTSTR lpszPathName = new TCHAR[_tcslen(lpszConfigFile)+1];
         ZeroMemory(lpszPathName, (_tcslen(lpszConfigFile)+1)*sizeof(TCHAR));
         _tcscpy_s(lpszPathName, (_tcslen(lpszConfigFile)+1)*sizeof(TCHAR), lpszConfigFile);

         pWorkItem->wParam = (WPARAM)lpszPathName;
         pWorkItem->uCmd = CONFIG_EMULATOR;

         break;
      }

      ……
}

if(pWorkItem != NULL)
{
      if(m_hWorkerThread == NULL)
      {
         m_hWorkerThread = ::CreateThread(NULL, 0, CommandDispatchWorker, (LPVOID)pWorkItem, 0, &m_dwThreadID);//参数传入
         if(m_hWorkerThread == NULL)
         {
            TRACE0("Create command dispatch worker thread failed.\n");
            return 0;
         }
      }
}

//线程函数
DWORD WINAPI CEmuDevice::CommandDispatchWorker(LPVOID lpParameter)
{
ASSERT(lpParameter != NULL);

WORKITEM* pWorkItem = (WORKITEM*)lpParameter;
CEmuDevice* pDevice = pWorkItem->pDevice;
WPARAM wParam = pWorkItem->wParam;
LPARAM lParam = pWorkItem->lParam;

ASSERT(pDevice != NULL);
ASSERT(pDevice->m_hNeedStop != NULL);

if(!pDevice->IsOpen())
{
      return 1;
}

switch(pWorkItem->uCmd)
{
      /* Board level commands */
      case CONFIG_EMULATOR:
      {
         LPTSTR lpszConfigFile = (LPTSTR)wParam;
         pDevice->ConfigEmulator(lpszConfigFile);
         delete[] lpszConfigFile; //<------------异常

         break;
      }

      ……

}
delete[] pWorkItem;
return 0;
}

2万 · 2010-7-30 21:37

是否存在SendCommand中没有运行case CONFIG_EMULATOR:里面的语句就到后面创建函数的代码啦的情况

2万 · 2010-7-30 21:38

估计与Unicode字符集有关，你把项目改成多字节字符集验证一下。

2万 · 2010-7-30 21:39

另外再调试一下，开一个内存窗口，监视从lpszPathName-0x20开始的内存，看看执行过程中是否发生变化。

2万 · 2010-7-30 21:39

建议好好看一下CCyUSBDevice的说明文档。

2万 · 2010-7-30 21:40

这个……

2万 · 2010-7-30 21:41

你要delete的指针是在SendCommand中由CCyUSBDevice作为参数传给你的，我说的没错吧。

2万 · 2010-7-30 21:41

那么你知道这个指针在CCyUSBDevice中是如何创建的？

1万 · 2010-7-30 21:43

至少你怎么知道要用delete []而不是delete?

1万 · 2010-7-30 21:43

甚至可能你根本不用自己释放，CCyUSBDevice会自己释放。

2万 · 2010-7-30 21:49

一个很重要很容易出你这种问题的做法是在dll中分配内存在exe中释放，这是非常不推荐的