[Mesa-dev] [PATCH] Fixed memory leak.

Siavash siavashserver at gmail.com
Tue Nov 26 22:18:26 PST 2013

I'll fix this issue in v2 series of patches, thanks again!

> Hi, I'm an outsider (for everyone who might have not noticed), I
> started following the list because I'm interested in the newbie
> projects, too.
> I think this free should be called regardless of new_data being null,
> as this doesn't imply there wasn't any in the original object.
> So, I believe it should be:
> if( bufObj->Data )
>    _mesa_align_free( bufObj->Data );
> if( new_data ) {
>    bufObj->Data = new_data;
> ...

More information about the mesa-dev mailing list