On Aug 1, 2008, at 4:44 AM, Mladen Turk wrote:
Ruediger Pluem wrote:
Ok, this is caused by http://svn.apache.org/viewvc?
rev=677505&view=rev
This is the reslist pre_cleanup patch. I don't know why so far, but
as
I have a proxy configuration I suspect that it blocks on tearing down
the proxy connection pools.
Here is the fix for trunk.
Index: proxy_util.c
===================================================================
--- proxy_util.c (revision 681621)
+++ proxy_util.c (working copy)
@@ -1939,10 +1939,11 @@
worker->hmax, worker->ttl,
connection_constructor,
connection_destructor,
worker, worker->cp->pool);
-
+#if 0
apr_pool_cleanup_register(worker->cp->pool, (void *)worker,
conn_pool_cleanup,
apr_pool_cleanup_null);
+#endif
Note that because of using pre_cleanup in reslist we don't need
the extra registered cleanup (conn_pool_cleanup),
just to make sure the ordering is correct.
This was bogus anyhow, because we were destroying the reslist in
cleanup (that already has it's own cleanup), so the ordering of
cleanup callbacks was essential.
I wonder how many other just uses in other modules would be just
so affected?
So does this mean that trunk is now based on a "broken" or
incompatible version of apr? Do we need to now break off
trunk to 2.4 and baseline APR 1.3 to allow trunk to now work
with an incompatible APR rev?
