httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Stefan Eissing <stefan.eiss...@greenbytes.de>
Subject Re: svn commit: r1779528 - /httpd/httpd/trunk/modules/http2/h2_mplx.c
Date Thu, 19 Jan 2017 21:41:33 GMT

> Am 19.01.2017 um 22:38 schrieb Yann Ylavic <ylavic.dev@gmail.com>:
> 
> On Thu, Jan 19, 2017 at 10:09 PM,  <icing@apache.org> wrote:
>> Author: icing
>> Date: Thu Jan 19 21:09:18 2017
>> New Revision: 1779528
>> 
>> URL: http://svn.apache.org/viewvc?rev=1779528&view=rev
>> Log:
>> On the trunk:
>> 
>> reverting change 1779525.
> 
> We really need to not destroy m->pool in its parent's cleanup.

Hmm, why is that bad?

> How about something like the attached instead, worth (other-)Stefan a try no?
> That's temporary, we probably could avoid that with a flag (re)set in
> a cleanup of mplx->pool itself...
> <h2_mplx_destroy.patch>

Would work. But then we could just never destroy this directly, because it will always disappear
via the parent h2_session pool?

Stefan Eissing

<green/>bytes GmbH
Hafenstrasse 16
48155 M√ľnster
www.greenbytes.de


Mime
View raw message