Fixes to LDB memory handling
Matthias Dieter Wallnöfer
mdw at samba.org
Fri Dec 17 02:08:54 MST 2010
It was more related to visual code analysis. I thought that if the
message sending would fail, then we wouldn't need the message anymore
and we could free it.
Thanks,
Matthias
Andrew Bartlett wrote:
> On Fri, 2010-12-17 at 09:08 +0100, Matthias Dieter Wallnöfer wrote:
>
>> Andrew,
>>
>> yes, this looks to be my fault. Feel free to revert this code piece!
>>
> Yes, I plan to. But I still would like to know what brought you to
> making this change in the first place. How did you determine this was a
> leak? I would like to ensure we resolve whatever issue you were
> originally dealing with.
>
> Or was this just a the result of visual code analysis? (i.e. reading
> the code and 'spotting' the leak). Did you do any validation other than
> 'make test' that I should repeat after reverting the patch?
>
> Thanks,
>
> Andrew Bartlett
>
>
More information about the samba-technical
mailing list