You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@httpd.apache.org by ni...@apache.org on 2008/09/09 02:52:04 UTC

svn commit: r693328 - in /httpd/httpd/trunk: CHANGES modules/aaa/mod_authnz_ldap.c

Author: niq
Date: Mon Sep  8 17:52:03 2008
New Revision: 693328

URL: http://svn.apache.org/viewvc?rev=693328&view=rev
Log:
mod_authnz_ldap: avoid returning NULL env vars
PR 39045

Modified:
    httpd/httpd/trunk/CHANGES
    httpd/httpd/trunk/modules/aaa/mod_authnz_ldap.c

Modified: httpd/httpd/trunk/CHANGES
URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/CHANGES?rev=693328&r1=693327&r2=693328&view=diff
==============================================================================
--- httpd/httpd/trunk/CHANGES [utf-8] (original)
+++ httpd/httpd/trunk/CHANGES [utf-8] Mon Sep  8 17:52:03 2008
@@ -2,6 +2,9 @@
 Changes with Apache 2.3.0
 [ When backported to 2.2.x, remove entry from this file ]
 
+  *) mod_authnz_ldap: don't return NULL-valued environment variables to
+     other modules.  PR 39045 [Francois Pesce <francois.pesce gmail.com>]
+
   *) Don't adjust case in pathname components that are not of interest
      to mod_mime.  Fixes mod_negotiation's use of such components.
      PR 43250 [Basant Kumar Kukreja <basant.kukreja sun.com>]

Modified: httpd/httpd/trunk/modules/aaa/mod_authnz_ldap.c
URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/modules/aaa/mod_authnz_ldap.c?rev=693328&r1=693327&r2=693328&view=diff
==============================================================================
--- httpd/httpd/trunk/modules/aaa/mod_authnz_ldap.c (original)
+++ httpd/httpd/trunk/modules/aaa/mod_authnz_ldap.c Mon Sep  8 17:52:03 2008
@@ -446,7 +446,7 @@
                 str[j] = apr_toupper(str[j]);
                 j++;
             }
-            apr_table_setn(e, str, vals[i]);
+            apr_table_setn(e, str, vals[i] ? vals[i] : "");
 
             /* handle remote_user_attribute, if set */
             if (sec->remote_user_attribute &&