On Dec 13, 2013, at 4:33 AM, Ruediger Pluem <[email protected]> wrote:

> 
> 
> Yann Ylavic wrote:
>> Hi devs,
>> 
>> This was pointed out by Joe Orton's comment at 
>> https://issues.apache.org/bugzilla/show_bug.cgi?id=50335#c40.
>> 
>> Here is a proposal (patch against ap_proxy_http_process_response) to address 
>> the double lifetime transformation of the
>> buckets from the backend when its connection is released early (on EOS, 
>> before the last buckets are forwarded to the
>> client).
> 
> In this case the buckets that would be sent to the output filters would have 
> been allocated from the wrong allocator
> which can lead to crashes. IMHO the code is fine as is. It might be possible 
> to optimize here, but the patch below just
> sents down buckets with the wrong lifetime down the chain.
> 

+1.

Reply via email to