[PSoC 1] VC6迁移到VC2008SP1

[复制链接]
187|34
 楼主 | 2018-2-13 11:44 | 显示全部楼层 |阅读模式
一个由VC6迁移到VC2008SP1的工程。
使用了Cypress的USB库:CyUSB.lib(静态链接,无dll文件)

症状:
在主线程中我new了一个字符串,然后作为参数,传递给线程,线程中使用该字符串,然后delete[],结果引发异常。
 楼主 | 2018-2-13 11:47 | 显示全部楼层
工程设置:Debug multithread DLL(/MDd)

自我诊断:
改用GlobalAlloc/GlobalFree依然无济于事。我认为CyUSB.lib可能静态链接到了CRT库,而我的版本是使用动态链接。不过,我改变了工程设置为“静态使用MFC、使用CRT库的多线程非DLL版本”,依然会报错。
在VC6中一切正常。
| 2018-2-13 11:49 | 显示全部楼层
不妨提供一下代码
 楼主 | 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;
}
更多操作
| 2018-2-13 11:57 | 显示全部楼层

是否存在SendCommand中没有运行case CONFIG_EMULATOR:里面的语句就到后面创建函数的代码啦的情况
| 2018-2-13 12:00 | 显示全部楼层

估计与Unicode字符集有关,你把项目改成多字节字符集验证一下。
| 2018-2-13 12:03 | 显示全部楼层

对,另外再调试一下,开一个内存窗口,监视从lpszPathName-0x20开始的内存,看看执行过程中是否发生变化。
| 2018-2-13 12:07 | 显示全部楼层

你要delete的指针是在SendCommand中由CCyUSBDevice作为参数传给你的,我说的没错吧。
那么你知道这个指针在CCyUSBDevice中是如何创建的?
| 2018-2-13 12:09 | 显示全部楼层
是,至少你怎么知道要用delete []而不是delete?甚至可能你根本不用自己释放,CCyUSBDevice会自己释放。
| 2018-2-13 12:14 | 显示全部楼层

一个很重要很容易出你这种问题的做法是在dll中分配内存在exe中释放。这是非常不推荐的
 楼主 | 2018-2-13 12:17 | 显示全部楼层
那应该?
| 2018-2-13 12:22 | 显示全部楼层
比如
void* CCyUSBDevice::Alloc()
{
return new char[20];
}
void CCyUSBDevice::Release(void *p)
{
delete [] p;
}
| 2018-2-13 12:25 | 显示全部楼层


如果你到网上搜索容易搜索到这样做的原因,我就不罗嗦了
 楼主 | 2018-2-13 12:28 | 显示全部楼层


感谢楼上的回复。字符串是在EXE的主线程里分配的,在EXE的辅助线程里释放的。没有用到任何自己的DLL和第三方的DLL
| 2018-2-13 12:32 | 显示全部楼层

哦,Cypress提供的是一个静态链接的lib,无DLL文件
 楼主 | 2018-2-13 12:36 | 显示全部楼层
改成multi byte就没问题了。
 楼主 | 2018-2-13 12:38 | 显示全部楼层
刚刚分配完内存之后,_CrtMemBlockHeader::pBlockHeaderPrev == NULL,继续执行到准备释放时,这个值被改变了
| 2018-2-13 12:41 | 显示全部楼层
其余的值都没有变吧?
 楼主 | 2018-2-13 12:44 | 显示全部楼层
没变~~~
| 2018-2-13 12:46 | 显示全部楼层
改用Multi-Byte测一下
扫描二维码,随时随地手机跟帖
*滑动验证:
您需要登录后才可以回帖 登录 | 注册 手机登录

本版积分规则

快速回复

您需要登录后才可以回帖
登录 | 注册 手机登录
高级模式
我要创建版块

论坛热帖

分享 快速回复 返回顶部 返回列表