[SCM] Samba Shared Repository - branch master updated - tevent-0-9-8-196-gf74e8c2

[Matthias Dieter Wallnöfer]

Hi Andrew,

I inspected the changelog - and since I'm now a bit more familiar with 
async requests I strongly wanted to review this code fragment. I had in 
mind: to delete a "general" "talloc_free" and move it only in a "if" 
branch  - that can't be enough :-) .

Matthias

Andrew Kroeger schrieb:
> Matthias Dieter Wallnöfer wrote:
>> commit f74e8c266f43d0c640ab8068d28a1b197164d548
>> Author: Matthias Dieter Wallnöfer <mwallnoefer at yahoo.de>
>> Date:   Fri Sep 11 13:35:55 2009 +0200
>>
>>     s4:ldb_map_outbound - fix memory leak
>>         Patch from Andrew Kroeger wasn't fully correct - we need a 
>> "talloc_free" after
>>     the "if (ac->r_current == NULL)" statement.
>
> Matthias:
>
> Thank you!
>
> I got too focused on solving the problem I found and lost sight of the 
> overall issue.
>
> May I ask if you found this issue via inspection (just looking at the 
> code) or if you are using some tool that flags such issues?
>
> Sincerely,
> Andrew Kroeger
>
>