Re: Code duplication is bad :/ On Mon, May 28, 2007 at 08:35:38PM -0400, simo wrote:
> I just found that we open winbindd_idmap.tdb independently in
> tdbsam_new_rid() in pdb_tdb.c
> With the new idmap code which can have different providers for the
> allocation backend this is probably wrong.
> 
> I would like to change it to ask winbindd for a RID.
> Is there a reason why this was not done before and instead we were
> opening the file directly and fiddling with another subsystem data?

Why would winbind have to allocate a RID? This is a generic
task of the SAM, and if I'm right the SAM is inside smbd.
winbind is there to allocate UID/GIDs. I had gone through
some pain in the past to separate this cleanly.

Volker
