httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Graham Leggett <minf...@sharp.fm>
Subject Re: svn commit: r160707 - in httpd/httpd/branches/2.0.x: CHANGES STATUS docs/manual/mod/mod_ldap.xml include/util_ldap.h modules/experimental/util_ldap.c
Date Wed, 27 Apr 2005 18:05:56 GMT
Paul Querna wrote:

> It looks like this change is causing crashes with PHP:
> http://issues.apache.org/bugzilla/show_bug.cgi?id=34618
> http://issues.apache.org/bugzilla/show_bug.cgi?id=34620
> 
> I think the call to ldap_set_option is the cause.
> 
> Quoting from http://docs.sun.com/source/816-5616-10/function.htm#24534
> 
> """If NULL, you are setting the default options that will apply to any
> new LDAP connection handles that are subsequently created.  """
> 
> Doesn't it seem bad to set this globally?  We are trampling on other
> citizens inside the process.
> 
> Why isn't this call to ldap_set_option done per-LDAP context?

I think it probably should be done per context when each connection is 
created, rather than set globally.

+1.

Regards,
Graham
--

Mime
View raw message