You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@httpd.apache.org by tr...@apache.org on 2006/04/11 01:37:44 UTC

svn commit: r393120 - /httpd/httpd/branches/2.2.x/modules/ldap/util_ldap.c

Author: trawick
Date: Mon Apr 10 16:37:43 2006
New Revision: 393120

URL: http://svn.apache.org/viewcvs?rev=393120&view=rev
Log:
Backport a comment from Greg Ames in trunk which helps
clarify part of this backport from 10 days ago:

  http://svn.apache.org/viewcvs.cgi?rev=390548&view=rev
  

Modified:
    httpd/httpd/branches/2.2.x/modules/ldap/util_ldap.c

Modified: httpd/httpd/branches/2.2.x/modules/ldap/util_ldap.c
URL: http://svn.apache.org/viewcvs/httpd/httpd/branches/2.2.x/modules/ldap/util_ldap.c?rev=393120&r1=393119&r2=393120&view=diff
==============================================================================
--- httpd/httpd/branches/2.2.x/modules/ldap/util_ldap.c (original)
+++ httpd/httpd/branches/2.2.x/modules/ldap/util_ldap.c Mon Apr 10 16:37:43 2006
@@ -1756,7 +1756,9 @@
     util_ldap_state_t *st =
         (util_ldap_state_t *)apr_pcalloc(p, sizeof(util_ldap_state_t));
 
-    /* Create a pool for mod_ldap to use */
+    /* Create a per vhost pool for mod_ldap to use, serialized with 
+     * st->mutex (also one per vhost) 
+     */
     apr_pool_create(&st->pool, p);
 #if APR_HAS_THREADS
     apr_thread_mutex_create(&st->mutex, APR_THREAD_MUTEX_DEFAULT, st->pool);