On 17 May 2013, at 10:26, Ruediger Pluem <rpl...@apache.org> wrote:

> We should not see an EOS bucket before
> 
> inflate(&ctx->stream, Z_NO_FLUSH); in line 1061 returned Z_STREAM_END. This 
> would mean we received an incomplete
> compressed stream.
> And if we see Z_STREAM_END we leave the for loop and stop processing buckets. 
> Hence we cannot hit a possible EOS bucket
> in the brigade any longer.
> So the test possibly sents an incomplete compressed body.

I tried to get my head around the ap_http_filter, it contains way too many 'if 
special case do special stuff' statements, particularly around the sending of 
eos buckets, I suspect one of those might be tripping it up.

It looks like some simplifying could eliminate an apr_brigade_flatten and 
another 40-ish bytes per request, will take a look.

Regards,
Graham
--

Reply via email to