nsswitch: Fix wbcListUsers test
authorChristof Schmitt <cs@samba.org>
Fri, 30 Mar 2018 21:28:46 +0000 (14:28 -0700)
committerKarolin Seeger <kseeger@samba.org>
Wed, 11 Apr 2018 09:52:25 +0000 (11:52 +0200)
With an AD DC, wbcListUsers returns the users in the DOMAIN SEPARATOR
USERNAME format.  The test then calls wbcLookupName with the domain name
and the previous string (including domain and separator) as username.
Fix this by passing the correct username and adding some additional
checks.

BUG: https://bugzilla.samba.org/show_bug.cgi?id=13312

Signed-off-by: Christof Schmitt <cs@samba.org>
Reviewed-by: Andreas Schneider <asn@samba.org>
(cherry picked from commit 3c146be404affc894c0c702bbfbfcc4fb9ed902b)

nsswitch/libwbclient/tests/wbclient.c

index e80afc4bd78249278c4d6313fce0903cc350b871..8c532bbaa754e3d70fefc285f6845c38958bfccf 100644 (file)
@@ -296,6 +296,7 @@ static bool test_wbc_users(struct torture_context *tctx)
        char *name = NULL;
        char *sid_string = NULL;
        wbcErr ret = false;
+       char separator;
 
        torture_assert_wbc_ok(tctx, wbcInterfaceDetails(&details),
                "%s", "wbcInterfaceDetails failed");
@@ -306,6 +307,7 @@ static bool test_wbc_users(struct torture_context *tctx)
                            ret,
                            fail,
                            "Failed to allocate domain_name");
+       separator = details->winbind_separator;
        wbcFreeMemory(details);
        details = NULL;
 
@@ -323,9 +325,38 @@ static bool test_wbc_users(struct torture_context *tctx)
                struct wbcDomainSid sid;
                enum wbcSidType name_type;
                uint32_t num_sids;
+               const char *user;
+               char *c;
+
+               c = strchr(users[i], separator);
+
+               if (c == NULL) {
+                       /*
+                        * NT4 DC
+                        * user name does not contain DOMAIN SEPARATOR prefix.
+                        */
+
+                       user = users[i];
+               } else {
+                       /*
+                        * AD DC
+                        * user name starts with DOMAIN SEPARATOR prefix.
+                        */
+                       const char *dom;
+
+                       *c = '\0';
+                       dom = users[i];
+                       user = c + 1;
+
+                       torture_assert_str_equal_goto(tctx, dom, domain_name,
+                                                     ret, fail, "Domain part "
+                                                     "of user name does not "
+                                                     "match domain name.\n");
+               }
 
                torture_assert_wbc_ok_goto_fail(tctx,
-                                               wbcLookupName(domain_name, users[i], &sid, &name_type),
+                                               wbcLookupName(domain_name, user,
+                                                             &sid, &name_type),
                                                "wbcLookupName of %s failed",
                                                users[i]);
                torture_assert_int_equal_goto(tctx,