svn commit: r1166823 - in /httpd/httpd/trunk: CHANGES modules/ldap/util_ldap.c

[co...@apache.org]

Author: covener
Date: Thu Sep  8 17:37:46 2011
New Revision: 1166823

URL: http://svn.apache.org/viewvc?rev=1166823&view=rev
Log:
mod_ldap: Optional function uldap_ssl_supported(r) always returned false
if called from a virtual host with mod_ldap directives in it.  Did not
affect mod_authnz_ldap's usage of mod_ldap.  [Eric Covener]

Modified:
    httpd/httpd/trunk/CHANGES
    httpd/httpd/trunk/modules/ldap/util_ldap.c

Modified: httpd/httpd/trunk/CHANGES
URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/CHANGES?rev=1166823&r1=1166822&r2=1166823&view=diff
==============================================================================
--- httpd/httpd/trunk/CHANGES [utf-8] (original)
+++ httpd/httpd/trunk/CHANGES [utf-8] Thu Sep  8 17:37:46 2011
@@ -7,6 +7,10 @@ Changes with Apache 2.3.15
      the original file, ignore the ranges and send the complete file.
      PR 51714. [Stefan Fritsch, Jim Jagielski, Ruediger Pluem, Eric Covener]
 
+  *) mod_ldap: Optional function uldap_ssl_supported(r) always returned false
+     if called from a virtual host with mod_ldap directives in it.  Did not
+     affect mod_authnz_ldap's usage of mod_ldap.  [Eric Covener]
+
   *) mod_filter: Instead of dropping the Accept-Ranges header when a filter
      registered with AP_FILTER_PROTO_NO_BYTERANGE is present,
      set the header value to "none". [Eric Covener, Ruediger Pluem]

Modified: httpd/httpd/trunk/modules/ldap/util_ldap.c
URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/modules/ldap/util_ldap.c?rev=1166823&r1=1166822&r2=1166823&view=diff
==============================================================================
--- httpd/httpd/trunk/modules/ldap/util_ldap.c (original)
+++ httpd/httpd/trunk/modules/ldap/util_ldap.c Thu Sep  8 17:37:46 2011
@@ -2757,7 +2757,7 @@ static void *util_ldap_merge_config(apr_
     st->util_ldap_cache_lock = base->util_ldap_cache_lock; 
 
     st->connections = NULL;
-    st->ssl_supported = 0;
+    st->ssl_supported = 0; /* not known until post-config and re-merged */
     st->global_certs = apr_array_append(p, base->global_certs,
                                            overrides->global_certs);
     st->secure = (overrides->secure_set == 0) ? base->secure
@@ -2938,6 +2938,16 @@ static int util_ldap_post_config(apr_poo
                      result_err ? result_err->reason : "");
     }
 
+    /* ssl_supported is really a global setting */
+    s_vhost = s->next;
+    while (s_vhost) {
+        st_vhost = (util_ldap_state_t *)
+                   ap_get_module_config(s_vhost->module_config,
+                                        &ldap_module);
+
+        st_vhost->ssl_supported = st->ssl_supported;
+    }
+
     /* Initialize the rebind callback's cross reference list. */
     apr_ldap_rebind_init (p);