httpd-cvs mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From rpl...@apache.org
Subject svn commit: r443503 - /httpd/httpd/trunk/modules/proxy/proxy_util.c
Date Thu, 14 Sep 2006 22:34:35 GMT
Author: rpluem
Date: Thu Sep 14 15:34:34 2006
New Revision: 443503

URL: http://svn.apache.org/viewvc?view=rev&rev=443503
Log:
* PROXY_WORKER_IS_INITIALIZED checks if PROXY_WORKER_INITIALIZED is set in
  worker->s->status, but we need to check here if it is set in worker->status.

  As PROXY_WORKER_INITIALIZED is set in worker->s->status in
  ap_proxy_initialize_worker_share, which is called just before
  ap_proxy_initialize_worker we never initialize the worker. This is very bad
  as we create no reslist for the pool in this case.

Modified:
    httpd/httpd/trunk/modules/proxy/proxy_util.c

Modified: httpd/httpd/trunk/modules/proxy/proxy_util.c
URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/modules/proxy/proxy_util.c?view=diff&rev=443503&r1=443502&r2=443503
==============================================================================
--- httpd/httpd/trunk/modules/proxy/proxy_util.c (original)
+++ httpd/httpd/trunk/modules/proxy/proxy_util.c Thu Sep 14 15:34:34 2006
@@ -1774,7 +1774,7 @@
     int mpm_threads;
 #endif
 
-    if (PROXY_WORKER_IS_INITIALIZED(worker)) {
+    if (worker->status & PROXY_WORKER_INITIALIZED) {
         /* The worker is already initialized */
         return APR_SUCCESS;
     }



Mime
View raw message