httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jess Holle <je...@ptc.com>
Subject Re: cvs commit: httpd-2.0/modules/ldap util_ldap_cache_mgr.c
Date Mon, 20 Sep 2004 13:15:27 GMT
See my util_ldap.c patch.

Whether or not this patch is absolutely necessary, leaving code paths 
that leave function return results undefined is never good!

Also, see the bigger patch to util_ldap_cache_mgr.c that I passed along 
yesterday.

I've attached both here for convenience.

--
Jess Holle

Joe Orton wrote:

>On Sun, Sep 19, 2004 at 11:00:25PM -0000, minfrin@apache.org wrote:
>  
>
>>  --- util_ldap_cache_mgr.c	13 Sep 2004 11:11:32 -0000	1.8
>>  +++ util_ldap_cache_mgr.c	19 Sep 2004 23:00:25 -0000	1.9
>>    
>>
>...
>  
>
>>  @@ -252,6 +254,8 @@
>>           newcurl = util_ald_cache_insert(st->util_ldap_cache, &curl);
>>   
>>       }
>>  +    else
>>  +      newcurl = NULL;
>>   
>>       return newcurl;
>>   }
>>    
>>
>
>This is not necessary after the r1.7 fix last week, so there's no point
>in asking for r1.7 to be back-ported as well as this.
>
>Also it looks as if the _child_init hook in util_ldap.c will still
>segfault with a NULL _cache_lock?
>  
>

Mime
View raw message