Hacker News new | ask | show | jobs
by bhangi 4209 days ago
Sorry if this is an ignorant question, but can you elaborate on this? It's not clear to me why this would lead to memory leaks.
3 comments

for example: http://stackoverflow.com/questions/461203/when-to-use-virtua...

class B : public A { ... } void foo() { A* someA = new B(...); ... delete someA; }

if A destructor is not virtual, then B's resources leak.

If the destructor isn't implemented in derived classes, they will use the parent's destructor. Not a guaranteed memory leak..... but something you need to be aware of.
It's worse than that - if you ever have a a variable with the static type of the base class but the dynamic type of a derived class, memory will leak, even if you've implemented the destructor in derived classes. For example, this code:

  class Base {
    public:
      Base() {}
      ~Base() {}
  };

  class Derived : public Base {
    private:
      Foo* _foo;
    public:
      Derived() : _foo(new Foo) {}
      ~Derived() { delete _foo; }
  };

  int main() {
    std::unique_ptr<Base>(new Derived);
  }
The problem here is that if the destructor is not marked virtual, then when unique_ptr<Base> goes to invoke the destructor, it calls ~Base instead of ~Derived. That has no knowledge of Derived's member variables, so any memory that Derived allocated leaks. (Technically, it's undefined behavior and can do whatever it wants, but most implementations in practice will call ~Base.)