cURL / Mailing Lists / curl-library / Single Mail

curl-library

Re: [PATCH] OpenLDAP support

From: Howard Chu <hyc_at_highlandsun.com>
Date: Mon, 24 May 2010 16:29:57 -0700

Daniel Stenberg wrote:
> On Thu, 20 May 2010, Howard Chu wrote:
>
>> Try this patch then. I dropped openldap.h since it wasn't really needed;
>> just added the #ifdefs to curl_ldap.h instead.
>
> Thanks!
>
> I had to edit it a bit to silence compiler warnings (try using configure
> --enable-debug): I added includes and I removed a bunch of unused variables. I
> don't have any particular LDAP server to test against so I've only tested my
> changes by compiling. I then pushed it.
>
> With OpenLDAP installed on my debian unstable system (the libldap2-dev
> package, version 2.4.21-1) I still get the following warnings:
>
> openldap.c: In function 'ldap_setup':
> openldap.c:141: warning: implicit declaration of function
> 'ldap_pvt_url_scheme2proto'
> openldap.c:141: warning: nested extern declaration of
> 'ldap_pvt_url_scheme2proto'

It's an OpenLDAP private function, advertised in <ldap_pvt.h> and not
<ldap.h>. Unfortunately <ldap_pvt.h> is not usually installed.

> openldap.c: In function 'ldap_connect':
> openldap.c:169: warning: implicit declaration of function 'ldap_init_fd'
> openldap.c:169: warning: nested extern declaration of 'ldap_init_fd'

This is also only declared in <ldap_pvt.h>. Maybe it belongs in <ldap.h>
instead, need to think about that.

For now, I think we can get by with explicit extern declarations in openldap.c.

> openldap.c: In function 'ldap_disconnect':
> openldap.c:296: warning: implicit declaration of function 'ldap_unbind'
> openldap.c:296: warning: nested extern declaration of 'ldap_unbind'
> openldap.c: In function 'ldap_done':
> openldap.c:359: warning: implicit declaration of function 'ldap_abandon'
> openldap.c:359: warning: nested extern declaration of 'ldap_abandon'

I guess I should have used ldap_unbind_ext() and ldap_abandon_ext() instead.

-- 
   -- Howard Chu
   CTO, Symas Corp.           http://www.symas.com
   Director, Highland Sun     http://highlandsun.com/hyc/
   Chief Architect, OpenLDAP  http://www.openldap.org/project/
-------------------------------------------------------------------
List admin: http://cool.haxx.se/list/listinfo/curl-library
Etiquette:  http://curl.haxx.se/mail/etiquette.html
Received on 2010-05-25