[Bug 26456] New: ldap_set_option with LDAP_OPT_OFF does not work
wine-bugs at winehq.org
wine-bugs at winehq.org
Thu Mar 17 09:43:30 CDT 2011
http://bugs.winehq.org/show_bug.cgi?id=26456
Summary: ldap_set_option with LDAP_OPT_OFF does not work
Product: Wine
Version: unspecified
Platform: x86
OS/Version: Linux
Status: UNCONFIRMED
Severity: normal
Priority: P2
Component: wldap32
AssignedTo: wine-bugs at winehq.org
ReportedBy: edwintorok at gmail.com
LDAP_OPT_OFF is defined as #define LDAP_OPT_OFF ((void*)0)
This code in dlls/wldap32/option.c returns an error for any
ldap_set_option(..,...,LDAP_OPT_OFF):
ULONG CDECL ldap_set_optionW( WLDAP32_LDAP *ld, int option, void *value )
{
ULONG ret = WLDAP32_LDAP_NOT_SUPPORTED;
#ifdef HAVE_LDAP
TRACE( "(%p, 0x%08x, %p)\n", ld, option, value );
if (!ld || !value) return WLDAP32_LDAP_PARAM_ERROR;
...
Similarly for ldap_set_optionA.
Which is not correct, you should let LDAP_OPT_OFF pass, otherwise
LDAP_OPT_REFERRALS can't be turned off, and anthing else that gets turned on
via LDAP_OPT_ON won't be turnable off with LDAP_OPT_OFF.
The fix would be to remove that !value from the if.
--
Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email
Do not reply to this email, post in Bugzilla using the
above URL to reply.
------- You are receiving this mail because: -------
You are watching all bug changes.
More information about the wine-bugs
mailing list