apr-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Graham Leggett <minf...@sharp.fm>
Subject Re: [Patch]: Add basic function to APR for LDAP rebind callback support.
Date Wed, 05 Dec 2007 22:40:20 GMT
Paul J. Reder wrote:

> Your changes look good to me (assuming the ldap struct does get passed 
> in to
> the cleanup). I made a few minor alterations to your patch and attached it
> here. I like the simplification of automatically registering the callback
> and better name space protection.

Can you try this version?

I have asked the _remove function to kill the cleanup that was 
registered, which should make this version safe, as well as guaranteeing 
that the rebind will never outlive the pool that created it.

Regards,
Graham
--

Mime
View raw message