Memory leaks patch

Zack Kirsch zack.kirsch at isilon.com
Wed Mar 7 22:48:08 GMT 2007


Good catch. Thanks for the update.

Zack 

> -----Original Message-----
> From: Jeremy Allison [mailto:jra at samba.org] 
> Sent: Wednesday, March 07, 2007 2:28 PM
> To: Zack Kirsch
> Cc: Jeremy Allison; samba-technical at samba.org
> Subject: Re: Memory leaks patch
> 
> On Wed, Mar 07, 2007 at 01:44:25PM -0800, Zack Kirsch wrote:
> > Hi Jeremy and samba-technical,
> > 
> > I've got another round of memory leak fixes for you guys. 
> As you might 
> > guess, we've run into a few memory leak problems. :)
> > 
> > Please let me know if the patch is committed as-is or if it 
> needs to 
> > be altered at all so that I can change my local code here.
> 
> Here's my version. You missed initializing os1,os2,os3 to 
> zero blobs so you could potentially have been freeing 
> uninitialized memory, other than that it's the same I think.
> 
> Jeremy.
> 


More information about the samba-technical mailing list