httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Graham Leggett <minf...@sharp.fm>
Subject Re: svn commit: r910017 - in /httpd/httpd/trunk: CHANGES docs/manual/filter.xml docs/manual/mod/allmodules.xml docs/manual/mod/mod_reflector.xml modules/filters/config.m4 modules/filters/mod_reflector.c
Date Mon, 15 Feb 2010 00:47:05 GMT
On 14 Feb 2010, at 6:55 PM, Ruediger Pluem wrote:

> +                /*
>>
>> +                 * We MUST read because in case we have an unknown- 
>> length
>> +                 * bucket or one that morphs, we want to exhaust it.
>> +                 */
>> +                status = apr_bucket_read(bucket, &data, &len,  
>> APR_BLOCK_READ);
>
> Can't we check first if this bucket is of unknown length and only  
> read if it is?
> Otherwise we could move it just over to the output brigade, correct?

I tried that and it didn't work for me. Without the apr_bucket_read(),  
no EOS bucket is ever generated, and the loop spins.

Regards,
Graham
--


Mime
View raw message