[PATCH] Fix some memory leaks

Dirk Hohndel dirk at hohndel.org
Mon Mar 23 08:47:13 PDT 2015


On Mon, Mar 23, 2015 at 05:37:32PM +0200, Claudiu Olteanu wrote:
> Yes, you are right. There is a simple white space fix in the
> context of the memory leaks. I thought that since it is regarded
> with the memory allocation/deallocation it wouldn't be a problem and
> a second patch will be unnecessary. Sorry, my mistake.
> I will create a new one when I will get home.
> 
> I also looked over the patch sent by Salvador and I believe
> that there will be no problem if we will merge this two patches.
> He doesn't use the freed variables and the ones implied are deallocated
> before a return statement.

Well, the patches conflict.

I'm OK with a single whitespace fix in the middle.
But I'd like to see new clean patches

/D


More information about the subsurface mailing list