summaryrefslogtreecommitdiff
path: root/source3/winbindd
diff options
context:
space:
mode:
authorStefan Metzmacher <metze@samba.org>2014-02-20 16:50:32 +0100
committerStefan Metzmacher <metze@samba.org>2014-02-25 09:17:07 +0100
commit8fec421543ff25c99655acca801d363c416fd61d (patch)
tree0ba6fda481d057d6c7e1c799a3f059ca2a711176 /source3/winbindd
parentfcf846a795085d24468548165d92762a628ef54d (diff)
downloadsamba-8fec421543ff25c99655acca801d363c416fd61d.tar.gz
samr: don't block the sam sid or the builtin domain sid in sid_to_name
Previously only members of these domains were handled. But we also need to handle the domain itself. Bug: https://bugzilla.samba.org/show_bug.cgi?id=10463 Change-Id: I44f85267eda243d586fffd24a799e153de0ff982 Pair-Programmed-With: Gregor Beck <gbeck@sernet.de> Signed-off-by: Gregor Beck <gbeck@sernet.de> Signed-off-by: Stefan Metzmacher <metze@sernet.de> Reviewed-by: Michael Adam <obnox@samba.org>
Diffstat (limited to 'source3/winbindd')
-rw-r--r--source3/winbindd/winbindd_samr.c2
1 files changed, 2 insertions, 0 deletions
diff --git a/source3/winbindd/winbindd_samr.c b/source3/winbindd/winbindd_samr.c
index 9f45126d5c5..e5504349423 100644
--- a/source3/winbindd/winbindd_samr.c
+++ b/source3/winbindd/winbindd_samr.c
@@ -684,7 +684,9 @@ static NTSTATUS sam_sid_to_name(struct winbindd_domain *domain,
/* Paranoia check */
if (!sid_check_is_in_builtin(sid) &&
+ !sid_check_is_builtin(sid) &&
!sid_check_is_in_our_sam(sid) &&
+ !sid_check_is_our_sam(sid) &&
!sid_check_is_in_unix_users(sid) &&
!sid_check_is_unix_users(sid) &&
!sid_check_is_in_unix_groups(sid) &&