Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

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?

like image 584
Ruggero Turra Avatar asked Oct 13 '11 13:10

Ruggero Turra


1 Answers

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.

like image 146
Nawaz Avatar answered Oct 22 '22 12:10

Nawaz