VC6迁移到VC2008SP1

1万 · 2018-2-13 11:44

一个由VC6迁移到VC2008SP1的工程。
使用了Cypress的USB库：CyUSB.lib（静态链接，无dll文件）

症状：
在主线程中我new了一个字符串，然后作为参数，传递给线程，线程中使用该字符串，然后delete[]，结果引发异常。

1万 · 2018-2-13 11:47

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

自我诊断：
改用GlobalAlloc/GlobalFree依然无济于事。我认为CyUSB.lib可能静态链接到了CRT库，而我的版本是使用动态链接。不过，我改变了工程设置为“静态使用MFC、使用CRT库的多线程非DLL版本”，依然会报错。
在VC6中一切正常。

1万 · 2018-2-13 11:49

不妨提供一下代码

1万 · 2018-2-13 11:52

代码：
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;
}
更多操作

1万 · 2018-2-13 11:57

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

2万 · 2018-2-13 12:00

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

2万 · 2018-2-13 12:03

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

2万 · 2018-2-13 12:07

你要delete的指针是在SendCommand中由CCyUSBDevice作为参数传给你的，我说的没错吧。
那么你知道这个指针在CCyUSBDevice中是如何创建的？

2万 · 2018-2-13 12:09

是，至少你怎么知道要用delete []而不是delete?甚至可能你根本不用自己释放，CCyUSBDevice会自己释放。

2万 · 2018-2-13 12:14

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

1万 · 2018-2-13 12:17

那应该？

2万 · 2018-2-13 12:22

比如
void* CCyUSBDevice::Alloc()
{
return new char[20];
}
void CCyUSBDevice::Release(void *p)
{
delete [] p;
}

1万 · 2018-2-13 12:25

如果你到网上搜索容易搜索到这样做的原因，我就不罗嗦了

1万 · 2018-2-13 12:28

感谢楼上的回复。字符串是在EXE的主线程里分配的，在EXE的辅助线程里释放的。没有用到任何自己的DLL和第三方的DLL

1万 · 2018-2-13 12:32

哦，Cypress提供的是一个静态链接的lib，无DLL文件

1万 · 2018-2-13 12:36

改成multi byte就没问题了。

1万 · 2018-2-13 12:38

刚刚分配完内存之后，_CrtMemBlockHeader::pBlockHeaderPrev == NULL，继续执行到准备释放时，这个值被改变了

1万 · 2018-2-13 12:41

其余的值都没有变吧？

1万 · 2018-2-13 12:44

没变~~~

1万 · 2018-2-13 12:46

改用Multi-Byte测一下