On Tue, Jun 10, 2014 at 3:02 PM, Plüm, Rüdiger, Vodafone Group <[email protected]> wrote: > > >> -----Original Message----- >> From: Takashi Sato [mailto:[email protected]] >> Sent: Dienstag, 10. Juni 2014 14:47 >> To: [email protected] >> Subject: Re: svn commit: r1601291 - in /httpd/httpd/trunk: CHANGES >> docs/log-message-tags/next-number modules/proxy/proxy_util.c >> >> r1601291 causes SEGV. >> >> # Failed test 2 in t/ssl/proxy.t at line 56 >> # Failed test 115 in t/ssl/proxy.t at line 56 fail #3 >> [ error] oh gosh, server dumped core >> [ error] for stacktrace, run: gdb /home/st/c99fix/apache/bin/httpd >> -core /home/st/c99fix/httpd-test/t/core.3649 >> [ error] oh nuts, server dumped core again >> [ error] for stacktrace, run: gdb /home/st/c99fix/apache/bin/httpd >> -core /home/st/c99fix/httpd-test/t/core.3647 >> t/ssl/proxy.t ....................... >> Failed 2/172 subtests >> >> The new function connection_shutdown is a pool cleanup func. >> connection_shutdown calls ap_shutdown_conn, >> but ap_shutdown_conn finally calls apr_pool_clear! >> (see attached strack trace) > > Hm, very bad. Actually we are trying to clean a subpool of the connection > pool in setaside_remaining_output which > has been already destroyed at this point. So I guess we need to have to use > apr_pool_pre_cleanup_register instead of > apr_pool_cleanup_register for the connection_shutdown.
I'm testing this, will commit when sure it works. Regards, Yann.
