Re: Memory deallocation does not work.

From:

Andre Kostur <nntpspam@kostur.net>

Newsgroups:

comp.lang.c++

Date:

Mon, 12 Feb 2007 23:20:08 +0000 (UTC)

Message-ID:

<Xns98D59CF7AE0CDnntpspamkosutrnet@209.135.99.21>

"Grizlyk" <grizlyk1@yandex.ru> wrote in news:eqqrpd$or9$1@aioe.org:

christophe.chazeau@gmail.com wrote:

#include <stdio.h>
#include <stdlib.h>
#include <unistd.h>
#include <vector>
#include <iterator>

using namespace std;

class Test{
public :
unsigned char *data ;

Test(){

}
void Go(int taille)
{

   data = (unsigned char*) malloc(taille);
   printf("malloc : %p\n",data);
}

~Test()
{
   printf("delete %p\n",data);
   free(data);
}

};

vector<Test *> v;

It is dangerous - possible memory leak because vector<Test *> does not
own the object pointer point to. Use any RAII memory wrapper (see RAII
here: http://www.hackcraft.net/raii ) instead of Test *

Uh... not all RAII memory wrappers are usable for this purpose.

vector<auto_ptr<Test> > v;

Specifically auto_ptr is not!

or give each address (stored in vector<Test *>) to other object,
which will keep object.

{
auto_ptr<Test> owner(new Test);
vector<Test *> v;
v.push_back(*owner);
}

Only if you know these other objects are going to have a longer lifetime
than your vector...