summaryrefslogtreecommitdiff
path: root/source3/winbindd
diff options
context:
space:
mode:
authorMichael Adam <obnox@samba.org>2008-10-01 08:53:15 +0200
committerMichael Adam <obnox@samba.org>2008-12-25 12:14:03 +0100
commit45db33e73262d8e195a46fb96405dfb3dc43d6bc (patch)
tree307e1a7e621cc7da0b12849e8b9a1a2590f1e317 /source3/winbindd
parent9a9b64dbdfce4414ada22d4f882c8c757b5813e1 (diff)
downloadsamba-45db33e73262d8e195a46fb96405dfb3dc43d6bc.tar.gz
samba-45db33e73262d8e195a46fb96405dfb3dc43d6bc.tar.bz2
samba-45db33e73262d8e195a46fb96405dfb3dc43d6bc.zip
s3:idmap: Remove passd check from idmap_backends_unixid_to_sid().
As noted by Metzy, it makes no sense here to check id->sid. What is worse, this might even be passed in uninitialized. This still fixes the bug for me (of course), but we might need to check, if another special handling of passdb is needed (possibly changing from constant return code NT_STATUS_OK...) Michael Signed-off-by: Michael Adam <obnox@samba.org>
Diffstat (limited to 'source3/winbindd')
-rw-r--r--source3/winbindd/idmap.c9
1 files changed, 0 insertions, 9 deletions
diff --git a/source3/winbindd/idmap.c b/source3/winbindd/idmap.c
index 38cc0d0c0a..6be55ef03c 100644
--- a/source3/winbindd/idmap.c
+++ b/source3/winbindd/idmap.c
@@ -735,15 +735,6 @@ NTSTATUS idmap_backends_unixid_to_sid(const char *domname, struct id_map *id)
maps[0] = id;
maps[1] = NULL;
- if (sid_check_is_in_builtin(id->sid)
- || (sid_check_is_in_our_domain(id->sid))) {
- dom = idmap_init_passdb_domain(NULL);
- if (dom == NULL) {
- return NT_STATUS_NONE_MAPPED;
- }
- return dom->methods->unixids_to_sids(dom, maps);
- }
-
dom = idmap_find_domain(domname);
if (dom == NULL) {
return NT_STATUS_NONE_MAPPED;