[SCM] Samba Shared Repository - branch master updated

Matthias Dieter Wallnöfer mdw at samba.org
Mon Apr 19 09:07:05 MDT 2010


Sorry Jelmer,

this wasn't clear (to prevent the use of "TALLOC_FREE"). But a real 
issue is the fact that we have already approximately 150-200 occourences 
of it under "source4" (checked with "cscope"). And we merge more and 
more with the s3 codebase so we get more and more occourences. Therefore 
I would simply propose to allow it for s4 as well.

Greets,
Matthias

Jelmer Vernooij wrote:
> Hi Matthias,
>
> On Sat, 2010-04-17 at 13:12 -0500, Matthias Dieter Wallnöfer wrote:
>    
>> - Log -----------------------------------------------------------------
>> commit c678563efec82e84c9c86309cbf9495950a02b8c
>> Author: Matthias Dieter Wallnöfer<mwallnoefer at yahoo.de>
>> Date:   Sat Apr 17 20:08:15 2010 +0200
>>
>>      s4:kdc/db-glue.c - use "TALLOC_FREE" insteal of "talloc_free" for the "priv" context
>>
>>      Also after a free "priv" could be != NULL and may be freed again.
>>      This should fix bug #7365.
>>      
> We tend to avoid TALLOC_FREE in Samba 4, opting for explicitly setting
> variable to NULL explicitly. This way valgrind will tell us about double
> frees.
>
> If you think we should consider using TALLOC_FREE, I think we should
> discuss that first - for now, please use talloc_free() and assignments
> to NULL.
>
> Cheers,
>
> Jelmer
>
>    



More information about the samba-technical mailing list