0

我遇到了“*** glibc detected *** /home/ubuntu[....] : double free or corruption (fasttop) : 0x09851168 ***”问题。

调试我发现了似乎是问题根源的行:

data[i]->~Class();

指的是

 class Class {
public:
    Class();
    Class(char *name, double value);
    virtual ~Class();
    char *name;
    double value;
private:

};

然后

    Class::Class() {
}

Class::Class(char* name, double value){
    this->name = new char[std::strlen(name)];
    std::strcpy(this->name, name);
    this->value = value;
}

Class::~Class() {
    delete name;
}

现在,显然我在那里做错了什么。谁能告诉我什么?

4

3 回答 3

7

您做错的事情是显式调用析构函数:

data[i]->~Class();

除非你绝对知道你应该明确地调用它(这是不寻常的,许多大型程序从来不需要这样做),你可能应该改用delete

delete data[i];
于 2012-04-11T02:25:20.653 回答
1
data[i]->~Class();

你永远不应该这样调用析构函数。delete如果它是用 分配的,或者new什么都不做,当对象超出范围时将调用析构函数。

于 2012-04-11T02:25:51.280 回答
1

我只知道您在几种情况下使用:data[i]->~Class();. 一种是您正在使用placement new 在该位置创建一个对象。另一个是您通过销毁对象来重新初始化一个对象,然后立即使用placement new 在那里创建一个新对象。这两种技术都是相当先进的技术,坦率地说,相当不寻常,除非您正在创建自己的集合类。

我的猜测是 1)你不需要那个,2)你没有真正向我们展示重要的代码(这很可能是围绕显式 dtor 调用的东西)。

您确实还有另一个小问题,即使用 分配namenew[]删除它delete name;- 两者应该匹配,所以您应该使用delete [] name;. 然而,鉴于这是一个数组char,这主要是一个技术问题——在这种情况下,它几乎肯定不会造成问题。如果它是一组具有重要析构函数的对象,典型的症状是一些(大多数)对象没有被正确销毁(即它们的析构函数不会被调用)。从理论上讲,这只是未定义的行为,所以任何事情都可能发生,但它成为问题根源的真正机会非常渺茫(特别是,正如我所说,在数组的情况下char)。

当然,你真正应该做的是做name一个std::string并完全跳过所有废话。

于 2012-04-11T02:29:33.740 回答