Use after talloc_free...

simo idra at samba.org
Wed Nov 11 07:31:12 MST 2009


On Wed, 2009-11-11 at 12:59 +0100, Kai Blin wrote:
> On Wednesday 11 November 2009 12:45:45 Volker Lendecke wrote:
> > On Wed, Nov 11, 2009 at 12:37:07PM +0100, Volker Lendecke wrote:
> > > On Wed, Nov 11, 2009 at 12:33:29PM +0100, Kai Blin wrote:
> > > > TALLOC_FREE is used in the shared code already. Why come up with yet
> > > > another way to do it(tm)? Personally I think coding on Samba is
> > > > complicated enough already.
> > >
> > > Because Rusty hates SHOUTING? :-)
> >
> > BTW, if there is a better alternative with the same
> > semantics, a complete replace run over the source code
> > should not be too difficult and I would support that.
> 
> s/TALLOC_FREE/talloc_free_zero/ perhaps? that's a bit more obvious on 
> the "it'll zero out the context" side that just capitals, and it's lower 
> case.

/me votes for talloc_zfree() :-)

Simo.

-- 
Simo Sorce
Samba Team GPL Compliance Officer <simo at samba.org>
Principal Software Engineer at Red Hat, Inc. <simo at redhat.com>



More information about the samba-technical mailing list