winbindd: explicit check for well-known SIDs in wb_lookupsids_bulk()

Those are implicitly already catched by the

  if (sid->num_auths != 5)

check, but I'd like to make the desired behaviour more obvious.

Bug: https://bugzilla.samba.org/show_bug.cgi?id=12727

Signed-off-by: Ralph Boehme <slow@samba.org>
Reviewed-by: Stefan Metzmacher <metze@samba.org>

diff --git a/source3/winbindd/wb_lookupsids.c b/source3/winbindd/wb_lookupsids.c
index 3f48ad748b32dbb4628f6e900d6ee0df6f4fbb8f..858616be182f38e4ae7880ecb6d4628b30a7297f 100644 (file)
--- a/source3/winbindd/wb_lookupsids.c
+++ b/source3/winbindd/wb_lookupsids.c
@@ -296,7 +296,10 @@ static bool wb_lookupsids_bulk(const struct dom_sid *sid)
            sid_check_is_in_unix_users(sid) ||
            sid_check_is_unix_users(sid) ||
            sid_check_is_in_builtin(sid) ||
-           sid_check_is_builtin(sid)) {
+           sid_check_is_builtin(sid) ||
+           sid_check_is_wellknown_domain(sid, NULL) ||
+           sid_check_is_in_wellknown_domain(sid))
+       {
                /*
                 * These are locally done piece by piece anyway, no
                 * need for bulk optimizations.