被delete难倒了

话说我有一个结构体如下：

struct stReplayData

{

int nDelay; // 该数据在上一条消息之后的延迟，仿真（目前自定义1秒钟）

char* pData; // 网络数据包的内容

int nLen; // 长度

stReplayData()

{

nDelay = 0;

nLen = 0;

pData = NULL;

}

stReplayData(int nLength)

{

nDelay = 0;

nLen = nLength;

pData = NULL;

if (nLen > 0)

{

pData = new char[nLen];

}

stReplayData(const stReplayData& src)

{

if (this == &src)

{

return;

}

*this = src;

}

stReplayData& operator = (const stReplayData& src)

{

if (this == &src)

{

return *this;

}

nDelay = src.nDelay;

if (pData != NULL)

{

delete[] pData;

pData = NULL;

}

nLen = src.nLen;

if (nLen > 0)

{

pData = new char[nLen];

memcpy(pData, src.pData, nLen);

}

return *this;

}

~stReplayData()

{

nDelay = 0;

nLen = 0;

if (pData != NULL)

{

delete[] pData;

pData = NULL;

}

};

我定义了一个vector<stReplayData*> m_vecReplay，然后new了一些stReplayData ，push_back这些指针进去，最后程序释放资源的时候，居然报调用释放指针出错了，报的错就是平时见的很多的Heap上指针无效的错误，基本上是说stReplayData的析构函数有问题，我了个擦，我怎么没看出哪里有问题呢？

=====================================更多的代码如下===============================================
我封装了一个dll作为一个公共模块，自然数据都会在这个公共模块中存储，其中内存数据的管理也会在这个dll中去做，也就是说，new和delete都会由这个dll自己去管理，
使用者只要去调接口，然后把需要存储的数据地址传给dll,让dll自己去拷贝即可。模块其实非常简单：

class CReplayManager : public IReplayManager

{

typedef vector<stReplayData*> VECREPLAY;

public:

CReplayManager();

virtual ~CReplayManager();

virtual bool PushData(stReplayData* pData);

virtual stReplayData* PopData();

virtual stReplayData* GetTailData();

virtual void ClearData();

virtual void Release();

private:

VECREPLAY m_vecReplay;

};

这里还有个模版函数，让主程序朝dll写数据，

template<class M, class T>

void WriteReplay(M* m, const T& t)

{

CWrite cw;

cw.Write(t);

stReplayData* pReplay = new stReplayData(cw.GetLen());

if (pReplay != NULL)

{

pReplay->nLen = cw.GetLen();

memcpy(pReplay->pData, cw.GetData(), cw.GetLen());

m->PushData(pReplay);

}

另：在主程序里单独的操作stReplayData是没什么问题的，我也试验过。

=================================总结=================================
re: 被delete难倒了 2011-03-31 11:49 dizhu

问题就是SAFE_DELETE((*it)); 这个是在exe中new的，不能在dll中delete。

深入：
如果一个EXE调用一个DLL时，用new和delete分配和释放内存为什么应该放在同一个背景下的原因。得出的结论是，如果EXE和DLL有一个不是用动态链接CRT库(C runtime library)的方式使用CRT的话（Multi-threaded Debug DLL (/MDd)），或者是EXE和DLL动态链接的CRT库的版本不同时，EXE和DLL将会各自拥有各自的堆空间，所以在DLL中new的东西务必在DLL中delete。

posted on 2011-03-30 17:29 zuhd 阅读(2172) 评论(29) 编辑收藏引用所属分类: c/c++

# re: 被delete难倒了 2011-03-30 17:36 namelij

你去看看vector里面的对象是怎么释放的，就明白了回复更多评论

# re: 被delete难倒了 2011-03-30 17:46 dizhu

贴下完整的代码回复更多评论

# re: 被delete难倒了 2011-03-30 17:50 Kevin Lynx

@dizhu
问题确实可能出在其他地方。回复更多评论

# re: 被delete难倒了 2011-03-30 17:55 千暮(zblc)

我按你说的做了一遍没有报错 - -bnr 你咋不把调用的代码发下回复更多评论

# re: 被delete难倒了[未登录] 2011-03-30 20:58 vincent

应该还是也指着啊回复更多评论

# re: 被delete难倒了[未登录] 2011-03-30 20:59 vincent

野指针…… 回复更多评论

# re: 被delete难倒了 2011-03-31 00:50 Mensch88

1. 就这段代码本身来说，有一个错误：拷贝构造函数 stReplayData(const stReplayData& src) 里的指针pData没有初始化！

2.stReplayData的析构函数本身没有问题。程序报错应该是调用了两次析构函数造成的。
比如说，很有可能在使用时会犯这样的错误：
stReplayData data(otherdata);
vec.push_back(&data);
由于data本身会调用析构函数delete pData，而delete vec里面的数据时也会调用data的析构函数，于是挂了。

回复更多评论

# re: 被delete难倒了 2011-03-31 01:13 Mensch88

不好意思，第二点我说得不大对，因为楼主在析构函数里判断了 if (pData != NULL)，并且之后会把pData=NULL，所以调用两次析构在单线程环境里不会问题。当然多线程环境下就另说了。

但我所陈述的问题依然是存在的，只是这不是因为两次析构造成，而是因为data是栈上的数据，不允许delete，所以挂了。回复更多评论

# re: 被delete难倒了 2011-03-31 09:14 zuhd

@Mensch88
1. 就这段代码本身来说，有一个错误：拷贝构造函数 stReplayData(const stReplayData& src) 里的指针pData没有初始化！

拷贝构造函数是调用operator =来着回复更多评论

# re: 被delete难倒了 2011-03-31 09:36 dizhu

@Mensch88
vector里面保存的是指针，不会调用delete 的，不会存在你说的两次析构函数调用回复更多评论

# re: 被delete难倒了 2011-03-31 09:39 dizhu

@dizhu
如果楼主
stReplayData data(otherdata);
vec.push_back(&data);
然后还去delete vec里面的数据，那就是楼主代码写的有问题，所以说还是贴下完整的代码，才能知道问题。但就这个stReplayData，还真看不出为什么会挂回复更多评论

# re: 被delete难倒了 2011-03-31 10:13 zuhd

我更新了帖子贴了更多的代码想尝试的朋友可以自己简单修改下即可回复更多评论

# re: 被delete难倒了 2011-03-31 11:13 zuhd

问题我找到了，是我以前遇到的老问题
virtual bool PushData(stReplayData* pData);
这个接口设计有问题，dll的接口应该用标准的c++类型，我只知道其然，不知道所以然，了解详情的说下回复更多评论

# re: 被delete难倒了 2011-03-31 11:17 dizhu

@zuhd
重点把delete stReplayData 的代码和 CReplayManager 贴出来看下回复更多评论

# re: 被delete难倒了 2011-03-31 11:22 zuhd

@dizhu
看了头文件基本就能猜到代码了吧中规中矩的容器操作代码而已

另：我在gcc中的头文件大量的使用了自定义的类，貌似没发现过什么问题，怎么用vc上来就碰到这个，是巧合还是必然？肿么办？有没有，有没有？回复更多评论

# re: 被delete难倒了 2011-03-31 11:24 dizhu

@zuhd
我比较关心CReplayManager 的Release 以及 ClearData 的实现。是不是在这里面delete stReplayData 了？？回复更多评论

# re: 被delete难倒了 2011-03-31 11:41 zuhd

void CReplayManager::ClearData()
{
VECREPLAY::iterator it = m_vecReplay.begin();
for (; it != m_vecReplay.end(); it++)
{
SAFE_DELETE((*it));
}
m_vecReplay.clear();
} 回复更多评论

# re: 被delete难倒了 2011-03-31 11:49 dizhu

@zuhd
void CReplayManager::ClearData()
{
VECREPLAY::iterator it = m_vecReplay.begin();
for (; it != m_vecReplay.end(); it++)
{
SAFE_DELETE((*it));
}
m_vecReplay.clear();
}
问题就是SAFE_DELETE((*it)); 这个是在exe中new的，不能在dll中delete。
原因：http://blog.csdn.net/blz_wowar/archive/2008/03/13/2176536.aspx 回复更多评论

# re: 被delete难倒了 2011-03-31 11:56 zuhd

@dizhu
在exe中new，不能在dll中delete的？
exe和dll用的是同一个堆栈空间的，回复更多评论

# re: 被delete难倒了 2011-03-31 17:48 Mensch88

zuhd@zuhd
--拷贝构造函数是调用operator =来着

拷贝构造函数是调用了operator=，但是operator=里面会判断pData是否为NULL：
if (pData != NULL)
{
delete[] pData;
pData = NULL;
}
必须注意的是，这时pData还没有被初始化。这样就会执行 delete[] pData;
从而出错。回复更多评论

# re: 被delete难倒了 2011-03-31 17:55 Mensch88

@dizhu
--然后还去delete vec里面的数据，那就是楼主代码写的有问题，所以说还是贴下完整的代码，才能知道问题。但就这个stReplayData，还真看不出为什么会挂

因为楼主说了vec里面是一些new出来的指针，而之后程序释放资源时报错，于是我估计楼主所说的程序释放资源就是指将vec里面的指针delete。
楼主后来的代码也证实了我的猜测。

回复更多评论

# re: 被delete难倒了 2011-03-31 18:31 Mensch88

楼主的问题算是CRT的bug么？
Linux下测试，无论是静态还是动态链接都没发现这种问题。回复更多评论

# re: 被delete难倒了 2011-03-31 23:54 flyinghearts

建议还是用 vector<char> 或 string 代替 char*

stReplayData 设计得有点问题，
一般都是在拷贝构造函数中进行分配新内存处理，然后在赋值函数中调用拷贝构造函数，构造一个临时类对像，与原类对象进行交换。

你的实现刚好相反，拷贝构造函数调用赋值函数，但赋值函数中用到的 nlen 和 pdata两个值都未初始化，UB行为。

实际上，这两个判断都是可以去掉的，new char[0] 是有意义的，没必要对 nlen进行判断
delete[] p 当p是空指针时，没有任何效果，因此没必要对pdata进行判断

拷贝构造函数中对指针的判断也是多余的。
另外，要先分配新内存，再释放旧内存，保证异常安全。

回复更多评论

# re: 被delete难倒了 2011-04-01 09:44 zuhd

@flyinghearts
那个拷贝构造函数确实有点问题，以前拷贝构造函数调用=写顺手了，没发现有内存操作的这么写有这个陷阱，改了一下：
stReplayData(const stReplayData& src)
{
if (this == &src)
{
return;
}
nDelay = src.nDelay;
nLen = src.nLen;
pData = new char[nLen];
if (pData != NULL)
{
memcpy(pData, src.pData, nLen);
}
}

stReplayData& operator = (const stReplayData& src)
{
if (this == &src)
{
return *this;
}
nDelay = src.nDelay;
nLen = src.nLen;
if (pData != NULL)
{
delete[] pData;
pData = NULL;
}
pData = new char[nLen];
if (pData != NULL)
{
memcpy(pData, src.pData, nLen);
}
return *this;
}

至于你说的：
另外，要先分配新内存，再释放旧内存，保证异常安全。
好像我一直都是先delete 再new ，可能一直懒得用个临时的指针来保存pData吧，不过你这么说的道理是？？回复更多评论

# re: 被delete难倒了[未登录] 2011-04-01 11:32 hdqqq

如果楼主使用 /mt 编译，dll启动时使用自己的堆，在主程序中new 出来的对象，再通过指针传给dll，然后在dll中释放，会产生错误，主程序和dll使用不同的堆。回复更多评论

# re: 被delete难倒了[未登录] 2011-04-01 14:39 杨粼波

把struct改为class，然后把数据private，测试一次看看。排除客户代码应用上的错误。

当然了，也有可能在调用的时候，被其他地方的错误牵连导致这个struct的数据被损坏。

如果牵扯到DLL的话，那有可能是exe和dll两者之间的版本不一致所导致，DLL的导出地址是用的一个，却不想exe用了另外一个，所以发生错误，这是有可能发生的。用VS的编译器，特别是2003，你需要依照以下步骤重编译：清理全部(包括dll和exe)->重编。回复更多评论

# re: 被delete难倒了[未登录] 2011-04-01 15:40 杨粼波

http://blog.csdn.net/ssli/archive/2009/06/16/4272484.aspx 回复更多评论

# re: 被delete难倒了 2011-04-01 23:30 flyinghearts

class stReplayData
{
  int nDelay;     // 该数据在上一条消息之后的延迟，仿真（目前自定义1秒钟）
  size_t nLen;  // 长度
  char* pData;    // 网络数据包的内容

public:
  stReplayData(size_t nLength = 0): nDelay(0),
      nLen(nLength), pData(new char[nLength]()) {}

  ~stReplayData() { delete[] pData; }

  stReplayData(const stReplayData& src):  nDelay(src.nDelay),
      nLen(src.nLen), pData(new char[src.nLen])
  {
    memcpy(pData, src.pData, nLen);
  }

  stReplayData& operator=(stReplayData tmp) //构建一个临时对像
  {
    nDelay = tmp.nDelay;
    nLen = tmp.nLen;
    std::swap(pData, tmp.pData);
    return *this;
  }
};

可参考以前写的这篇文章：

http://blog.csdn.net/flyinghearts/archive/2010/06/16/5673089.aspx

回复更多评论

# re: 被delete难倒了 2011-04-02 11:42 tingya

涉及到跨模块内存使用的问题，应该遵守谁使用，谁释放回复更多评论

刷新评论列表

只有注册用户登录后才能发表评论。
【推荐】100%开源！大型工业跨平台软件C++源码提供，建模，组态！

相关文章: 不错记一个小bug 1长度数组使用备忘证明题： x & 3 == x % 4 成为符合ANSI和Unicode的应用程序 memcpy这个不定时炸弹被delete难倒了 C++的函数返回引用 union 晦涩的代码

网站导航: 博客园 IT新闻 BlogJava 博问 Chat2DB 管理

focus on linux, c/c++, lua

导航

统计

常用链接

留言簿(4)

随笔分类

随笔档案

搜索

最新评论

阅读排行榜

评论排行榜