wldap32: Fix server controls lookup logic.

This fixes 'supported == user_count' comparison if the server has duplicate control OIDs in the returned list. Signed-off-by: Dmitry Timoshkov <dmitry@baikal.ru> Signed-off-by: Alexandre Julliard <julliard@winehq.org>

wldap32: Fix server controls lookup logic.
6df25aaf · Dmitry Timoshkov · Alexandre Julliard · bd34ffbd · 6df25aaf
Commit 6df25aaf authored Apr 06, 2020 by Dmitry Timoshkov Committed by Alexandre Julliard Apr 06, 2020
Hide whitespace changes
Inline Side-by-side

Showing with 5 additions and 0 deletions

option.c dlls/wldap32/option.c +5 -0

No files found.
--- a/dlls/wldap32/option.c
+++ b/dlls/wldap32/option.c
@@ -439,10 +439,15 @@ static BOOL is_supported_server_ctrls( WLDAP32_LDAP *ld, LDAPControl **ctrls )

    for (n = 0; n < user_count; n++)
    {
+        TRACE("looking for %s\n", debugstr_a(ctrls[n]->ldctl_oid));
+
        for (i = 0; i < server_count; i++)
        {
            if (!strncmp( ctrls[n]->ldctl_oid, ld->ld_server_ctrls[i]->bv_val, ld->ld_server_ctrls[i]->bv_len))
+            {
                supported++;
+                break;
+            }
        }
    }