-3

这个错误让我很生气......

Valgrinding 程序表明delete[]in DataPackage::~DataPackage（第 40 行）产生了问题。但是如果我删除它，程序就会泄漏。那么，如何修复它，我做错了什么？

主文件

#include "main.h" // currently includes DataPackage.h only

DataPackage aTestFunction(){
  return DataPackage("hello",5);
}

int main(){
  DataPackage pack1 = aTestFunction(), pack2 = aTestFunction();
  pack1 = pack2;

  for(unsigned int i = 0; i < pack1.getLength(); i++){
    printf("0x%02x ", *(pack1.getData()+i)&0xff);
  }

  return 0;
}

数据包.cxx

#include "DataPackage.h" // defines only the class, and private members m_data (char*) and m_length (size_t), includes <cstdio> and <cstring>

DataPackage::DataPackage(){
  m_data = NULL;
  m_length = 0;
}

DataPackage::DataPackage(string data){
  m_data = NULL;
  setData(data.c_str(),data.length()+1);
}

DataPackage::DataPackage(const char *data, size_t length) {
  m_data = NULL;
  setData(data,length);
}

DataPackage::DataPackage(const DataPackage &pack) {
  m_data = NULL;
  setData(pack.m_data,pack.m_length);
}

const char* DataPackage::getData(){
  return m_data;
}

void DataPackage::setData(const char *newdata,size_t newlength){
  char* tmpdata = new char[newlength];
  m_length = newlength;
  memcpy(tmpdata,newdata,m_length);
  delete[] m_data;
  m_data = tmpdata;
}

size_t DataPackage::getLength(){
  return m_length;
}

DataPackage::~DataPackage() {
  delete[] m_data;
}
4

4 回答 4

3

您忘记了三法则并且没有提供复制分配运算符。因此，当从另一个分配一个时DataPackage，两者都以指向同一个缓冲区的指针结束，并且都试图删除它。

我会扔掉那个类并使用std::stringorstd::vector<char>代替。

于 2013-05-15T18:24:07.737 回答
1

错误是由'pack1 = pack2;'引起的 在 main.cxx

实施 DataPackage::operator= 来修复它。

于 2013-05-15T18:22:53.997 回答
0

您需要实现一个分配运算符来管理m_data. 自动生成的实现只会复制指针，而不是复制指向的数组。赋值运算符可能如下所示：

DataPackage& DataPackage::operator=(const DataPackage &other) {
   setData(other.m_data, other.m_length);
   return *this;
}
于 2013-05-15T18:22:53.983 回答
-1

尝试 ：

void DataPackage::setData(const char *newdata,size_t newlength){
  char* tmpdata = new char[newlength];
  m_length = newlength;
  memcpy(tmpdata,newdata,m_length);

  if (m_data)
  {
      delete[] m_data;
  }

  m_data = tmpdata;
}

从我看到你的复制构造函数将 m_data 设置为 NULL 并在之后调用 setData 。在 setData 你基本上是在做 delete[] NULL;

于 2013-05-15T18:22:11.847 回答