Re: Is there a memory leak in this code ?
On Dec 28, 2:15 pm, "Victor Bazarov" <v.Abaza...@comAcast.net> wrote:
Diwa wrote:
// -----------------------------------
class Column
{
public:
string name;
vector<int> values;
};
// -----------------------------------
void loadValues()
{
Column *p = new Column();
p->values.push_back(55); // <--- Line 1
p->values.push_back(66); // <--- Line 2
delete p; // <--- Line 3
}
// -----------------------------------
Are the values inserted (Line 1 and 2) on
the stack or on the heap ?
Values that the container
'p->values' stores are _always_ in the free store, the vector
allocates all its values there, unless you provide some kind
of custom allocator, which you didn't.
Is there a memory leak for the two inserted
values inspite of the "delete p" at line 3 ?
No.
I suspect there is a memory leak. Here is my reasoning.
When "Column *p = new Column( )" is done it allocates,
lets say, 20 bytes. Just before the memory address
returned by "new", maybe it stores the num of bytes.
The "push_back()" done later at line 1 and line 2 may
results in some more "new" but still it will not
change the value (num of bytes) just before addr "p"
So at line 3, when "delete p" executes, it sees the
velue "20" just before p and then deletes only 20
bytes.
Am I missing something ?