Re: [tpop3d-discuss] [patch] auth_ldap.c

[Chris Lightfoot]

On Fri, Jul 18, 2003 at 01:08:44AM +0100, Paul Makepeace wrote:

 [ ... ]

thanks for these.

> @@ -240,4 +240,4 @@
>   * Try to connect to the LDAP server and bind. */
> -static int try_ldap_connect_bind(char *who, char *passwd) {
> -    int ret, i;
> +static int try_ldap_connect_bind(const char *who, const char *passwd) {
> +    int ret = !LDAP_SUCCESS, i; /* XXX */

-- oops. LDAP_OTHER seems the best choice here, but it's
pretty ugly.

> (ldap_open() is deprecated, btw.)

Oh dear. `Will be deprecated in a future release' in my
openldap, but times move on I suppose.

-- 
Dilbert: Do you think you might be abusing your power?
  Wally: What would be the other reasons to have power?
 (Scott Adams)