[PATCH] tricky bug in idmap_sid_to_uid().

[boyang]

Hi, everyone:
     Please have a look at idmap_sid_to_uid(),
idmap_backend_sid_to_unixid() can returns status other than
NT_STATUS_NONE_MAPPED or NT_STATUS_OK. eg, NT_STATUS_NO_MEMORY in
idmap_tdb_to_id(). This is very faint, SID to UID mapping might exists,
but failed only because of lack of memory. Then, idmap_new_mapping()
will allocate a new unix id and update SID <--> UID mapping....... which
causes inconsistent SID <---> UID mapping. Please correct me if I am wrong.
    Patch for master in the attachment, please review it.
    Thanks!
Best
        Regards
BoYang
-------------- next part --------------
A non-text attachment was scrubbed...
Name: bug-idmap_sid_to_ugid-master.diff
Type: text/x-patch
Size: 2924 bytes
Desc: not available
Url : http://lists.samba.org/archive/samba-technical/attachments/20090226/78fc97f2/bug-idmap_sid_to_ugid-master.bin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: boyang.vcf
Type: text/x-vcard
Size: 187 bytes
Desc: not available
Url : http://lists.samba.org/archive/samba-technical/attachments/20090226/78fc97f2/boyang.vcf