]> granicus.if.org Git - apache/commitdiff
comment
authorStefan Fritsch <sf@apache.org>
Thu, 4 Jul 2013 13:32:43 +0000 (13:32 +0000)
committerStefan Fritsch <sf@apache.org>
Thu, 4 Jul 2013 13:32:43 +0000 (13:32 +0000)
git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/branches/2.4.x@1499749 13f79535-47bb-0310-9956-ffa450edef68

STATUS

diff --git a/STATUS b/STATUS
index 60b229893f258f17bf3262002b9085977d617e18..93f8d35aee0157bbd39df87fcf5e9ed79f4f0135 100644 (file)
--- a/STATUS
+++ b/STATUS
@@ -147,6 +147,13 @@ PATCHES PROPOSED TO BACKPORT FROM TRUNK:
                  http://svn.apache.org/viewvc?view=revision&revision=1484343
     2.4.x patch: trunk works, modulo CHANGES
     +1: jim, minfrin
+    sf says:
+    - ap_proxy_sync_balancer() seems to use conf->pool in several
+      places, but only one place is protected by conf->mutex.
+      I don't think this is correct.
+      Also, I can't find any place where conf->mutex is initialized.
+    - ap_proxy_initialize_worker() is called from proxy_handler() with
+      conf->pool. I guess this should also be mutex protected?
 
   * mod_proxy_http: Make the proxy-interim-response environment variable
     effective by formally overriding origin server behaviour.