svn commit: samba r16813 - in trunk/source: include nsswitch sam utils

jra at samba.org jra at samba.org
Wed Jul 5 03:40:41 GMT 2006


Author: jra
Date: 2006-07-05 03:40:40 +0000 (Wed, 05 Jul 2006)
New Revision: 16813

WebSVN: http://websvn.samba.org/cgi-bin/viewcvs.cgi?view=rev&root=samba&rev=16813

Log:
Rewrote much of the idmap code. This design is *shit*,
and needs fixing. We must refactor out the cache code
and separate it from the idmap_tdb so they are two
separate pointer tables. They may call down into the
same underlying functions but the MUST NOT BE ALLOWED
to continue to mix up allocation and cachine. I'll know
the design is correct when the flags field dissapears
from the backend interface. Having said that this code
is now a lot cleaner and easier to understand. Expect
more hard words (and work) in this area.

Anyone wanting to work on this code please see me
first ! ("You behind the bike sheds, stand still
laddie !" :-)

Jeremy.

Modified:
   trunk/source/include/idmap.h
   trunk/source/nsswitch/winbindd_async.c
   trunk/source/nsswitch/winbindd_sid.c
   trunk/source/nsswitch/winbindd_user.c
   trunk/source/sam/idmap.c
   trunk/source/sam/idmap_ad.c
   trunk/source/sam/idmap_ldap.c
   trunk/source/sam/idmap_rid.c
   trunk/source/sam/idmap_smbldap.c
   trunk/source/sam/idmap_tdb.c
   trunk/source/sam/idmap_util.c
   trunk/source/utils/net_idmap.c


Changeset:
Sorry, the patch is too large (1296 lines) to include; please use WebSVN to see it!
WebSVN: http://websvn.samba.org/cgi-bin/viewcvs.cgi?view=rev&root=samba&rev=16813


More information about the samba-cvs mailing list