Code duplication is bad :/

[simo]

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?

Simo.

-- 
Simo Sorce
Samba Team GPL Compliance Officer
email: idra at samba.org
http://samba.org