Search code examples
c++polymorphismsegmentation-faultnew-operatorvirtual-destructor

virtual destructor fail using array


I found this code on a web site

#include <iostream>

using namespace std;

struct Base
{
    Base() { cout << "Base" << " "; }
    virtual ~Base() { cout << "~Base" << endl; }

    int i;
};
struct Der : public Base
{
    Der() { cout << "Der" << endl; }
    virtual ~Der() { cout << "~Der" << " "; }

    int it[10]; // sizeof(Base) != sizeof(Der)
};

int main()
{
    Base *bp = new Der;
    Base *bq = new Der[5];

    delete    bp;
    delete [] bq;   // this causes runtime error
}

why does it crash?


Solution

  • Base *bq = new Der[5];
    delete [] bq;   // this causes runtime error
    

    The reason is arrays are not treated polymorphically. Therefore, in the above code, the delete statement invokes undefined behaviour.

    §5.3.5/3 C++03 says

    In the first alternative (delete object), if the static type of the operand is different from its dynamic type, the static type shall be a base class of the operand’s dynamic type and the static type shall have a virtual destructor or the behavior is undefined. In the second alternative (delete array) if the dynamic type of the object to be deleted differs from its static type, the behavior is undefined.

    You're lucky that it gives runtime-error, and you got the opportunity to know a serious bug in your code, as soon as possible.