svn commit: samba r22020 - in branches/SAMBA_3_0/source: auth include lib smbd

Gerald (Jerry) Carter jerry at
Wed Apr 4 02:27:08 GMT 2007

Hash: SHA1

Andrew Bartlett wrote:

> I think the getpwnam cache is now cleaner 
> avoiding talloc_reference, and doing a tcopy_passwd instead.
> Having a magic cache, using talloc_reference() I think
> caused confusing behaviours.  Because callers
> are not expecting a cache, they would be entirely 
> within their rights to talloc_steal() parts of the the
> results, but this would break the cache.

It's the stealing members from a structure that concerns me.

> The bug I was actually trying to fix is that I suspected 
> that not everything was being cleaned up when a session
> terminated.  I was hoping to make this simpler, with talloc
> taking over from the manual cleanup system.

Has Jeremy chatted with you about this?  He was going to
follow up.  Like I said,  I'll review it again but the
talloc_steal() calls make it more confusing for me, and
not cleaner.  Maybe another read through will change my mind.

It will take me about a day to get to it though.

cheers, jerry
Samba                                    -------
Centeris                         -----------
"What man is a man who does not make the world better?"      --Balian
Version: GnuPG v1.4.3 (GNU/Linux)
Comment: Using GnuPG with Mozilla -


More information about the samba-technical mailing list