diff options
| author | Michael Adam <obnox@samba.org> | 2008-10-01 08:53:15 +0200 |
|---|---|---|
| committer | Michael Adam <obnox@samba.org> | 2008-12-25 12:14:03 +0100 |
| commit | 45db33e73262d8e195a46fb96405dfb3dc43d6bc (patch) | |
| tree | 307e1a7e621cc7da0b12849e8b9a1a2590f1e317 /source3/winbindd/idmap.c | |
| parent | 9a9b64dbdfce4414ada22d4f882c8c757b5813e1 (diff) | |
| download | samba-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/idmap.c')
| -rw-r--r-- | source3/winbindd/idmap.c | 9 |
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; |