DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG 
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=31898>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND 
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=31898

Potential crash in util_ldap_cache_checkuserid





------- Additional Comments From [EMAIL PROTECTED]  2004-10-31 02:29 -------
   The attached patch combines the patches from both bugs 28253 and 31898 to 
provide the util_ldap_cache_getuserdn() API so that LDAP authorization can be 
completely split apart from LDAP authentication.  The main difference between 
this patch and the patches submitted with the bugs is the logic in 
util_ldap_cache_checkuserid() that determines when to insert, remove or update 
a cached entry.  The original patch caused the entries to be removed and 
reinserted needlessly.  After playing with this for a while, I feel a lot more 
comfortable using the same user search cache for both the 
util_ldap_cache_getuserdn() and util_ldap_cache_checkuserid() API's.  If this 
looks good, I will get it checked into CVS.  To get it into the 2.0 branch,  a 
similar patch will have to be created for that codebase and proposed for 
backport.

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to