httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Yann Ylavic <ylavic....@gmail.com>
Subject Re: svn commit: r1601291 - in /httpd/httpd/trunk: CHANGES docs/log-message-tags/next-number modules/proxy/proxy_util.c
Date Tue, 10 Jun 2014 13:06:02 GMT
On Tue, Jun 10, 2014 at 3:02 PM, Plüm, Rüdiger, Vodafone Group
<ruediger.pluem@vodafone.com> wrote:
>
>
>> -----Original Message-----
>> From: Takashi Sato [mailto:takashi@tks.st]
>> Sent: Dienstag, 10. Juni 2014 14:47
>> To: dev@httpd.apache.org
>> 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.

Mime
View raw message