svn commit: r652513 - /httpd/httpd/branches/2.2.x/STATUS

[co...@apache.org]

Author: covener
Date: Thu May  1 06:23:14 2008
New Revision: 652513

URL: http://svn.apache.org/viewvc?rev=652513&view=rev
Log:
address a review comment

Modified:
    httpd/httpd/branches/2.2.x/STATUS

Modified: httpd/httpd/branches/2.2.x/STATUS
URL: http://svn.apache.org/viewvc/httpd/httpd/branches/2.2.x/STATUS?rev=652513&r1=652512&r2=652513&view=diff
==============================================================================
--- httpd/httpd/branches/2.2.x/STATUS (original)
+++ httpd/httpd/branches/2.2.x/STATUS Thu May  1 06:23:14 2008
@@ -214,6 +214,13 @@
      through the array.  So how does the caller know how to walk
      through the list? (It seems that we need need to allocate numvals+1,
      then store in (*retvals)[j++] when vals[i] != NULL.)
+   covener: The caller has asked for values corresponding to a passed list 
+     of attributes (attrs), so the returned list does not need to be null 
+     terminated (or otherwise convey a length)
+     mod_authnz_ldap iterates through it's own copy of the _attributes_ 
+     (null terminated, like the native LDAP API) and uses the 
+     correspondingly-indexed entry in the returned values 
+     (retvals in uldap_cache_getuserdn() signature)
 
  * mod_headers: Add merge option to avoid duplicate values within
    the same header.