memory leak in assignment operator (using vectors)

[piranhaofeden]

Hey all - my set class uses two vectors, data<int> and child<set*>. I keep getting a heap leak inside the assignment operator. I've been back and forth over this multiple times. I can get more info if needed, but I'm stuck! Thanks so much.

void set::operator =(const set& source)
	{
		
		if (this == &source) return; //Checks for self-assignment
		
		clear();
		
		data = source.data;
		for (vector<set*>::size_type i = 0; i < source.child.size(); i++)
		{
			child.push_back(new set(*(source.child[i])));
		}
	}

[helios]

Does set::clear() delete set::child[i]?

[piranhaofeden]

Yes -

 void set::clear( )
	{
		data.clear();
		child.clear();
	} 

My destructor just calls clear as well.

[helios]

That doesn't delete each element.

for (size_t a=0;a<this->child.size();a++)
    delete this->child[a];

[piranhaofeden]

aha! worked like a charm. thanks muchly.