A stupid, obvious user memory leak

#ifndef __GOODCLASS_H__

#define __GOODCLASS_H__

class good

{

public:

good();

virtual ~good();

protected:

int * pointer;

};

#endif /* __GOODCLASS_H__ */

#include "goodclass.h"

good::good()

{

pointer = new int[1024]; // 4kb memory

}

good::~good()

{

// delete the allocated memory

delete [] pointer;

}

#include "goodclass.h"

#include <iostream.h>

int main()

{

good *g;

int i = 0;

// we leak 1000 * 4kb memory.

while ( i++ < 1000)

{

cout << i << endl;

g = new good();

}

}

With each new object of class “good” created and not deleted, we leak at least 4kb of memory. Each time we overwrite the pointer with the pointer to the new object, we leave the previous object behind. There is no way to get back at the old one.

Pretty obvious, but you see it from time to time.

Previous slide

Next slide

Back to first slide

View graphic version