Re: [PATCH 0/1] MR3583: wldap32: Map LDAP_OPT_AUTO_RECONNECT to LDAP_OPT_RESTART and add tests.

[Hans Leidekker (＠hans)]

Hans Leidekker (@hans) commented about dlls/wldap32/option.c:
> return WLDAP32_LDAP_SUCCESS;
>      }
> +    case WLDAP32_LDAP_OPT_AUTO_RECONNECT:
> +    {
> +        BOOL *on = value;
> +        ret = ldap_get_option( CTX(ld), LDAP_OPT_RESTART, value );
> +        *on = !!*on;
> +        return ret;

We probably shouldn't touch 'value' on failure.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/3583#note_42599