You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@httpd.apache.org by Brad Nicholes <BN...@novell.com> on 2005/09/22 19:39:47 UTC

Re: [PATCH 36563] Backported patch invalid handling of NULL values in mod_ldap caching

    This patch is sitting in the STATUS file waiting for one more vote. 
It sure would be nice to get it into the 2.0.55 release.  Anyone willing
to give it a quick review?

Brad

>>> ondrej@sury.org 9/19/2005 >>>
Hello,

attached is backported patch from 2.1-BETA7 which fixes invalid
handling
of NULL values in mod_ldap caching code.

There is an open bug in bugzilla:
http://issues.apache.org/bugzilla/show_bug.cgi?id=36563 

And this is just updated patch against today SVN checkouted 2.0.x
(patch
from bugzilla applies ok with offset of 20 lines).

Hopefully this gets accepted before 2.0.55 is out, since this bug can
cause memory leaks in util_ldap_search_node_free (not all items are
freed only those before first item with NULL value).

Ondrej.
-- 
Ondrej Sury <on...@sury.org>

Re: [PATCH 36563] Backported patch invalid handling of NULL values in mod_ldap caching

Posted by Jim Jagielski <ji...@jaguNET.com>.
+1 by inspection... will have a test-vote in a few hours.
Will update STATUS at that point :)

On Sep 22, 2005, at 1:39 PM, Brad Nicholes wrote:

>     This patch is sitting in the STATUS file waiting for one more  
> vote.
> It sure would be nice to get it into the 2.0.55 release.  Anyone  
> willing
> to give it a quick review?
>
> Brad
>
>
>>>> ondrej@sury.org 9/19/2005 >>>
>>>>
> Hello,
>
> attached is backported patch from 2.1-BETA7 which fixes invalid
> handling
> of NULL values in mod_ldap caching code.
>
> There is an open bug in bugzilla:
> http://issues.apache.org/bugzilla/show_bug.cgi?id=36563
>
> And this is just updated patch against today SVN checkouted 2.0.x
> (patch
> from bugzilla applies ok with offset of 20 lines).
>
> Hopefully this gets accepted before 2.0.55 is out, since this bug can
> cause memory leaks in util_ldap_search_node_free (not all items are
> freed only those before first item with NULL value).
>
> Ondrej.
> -- 
> Ondrej Sury <on...@sury.org>
>
>