From: Ryan Bloom Date: Wed, 8 Aug 2001 22:24:04 +0000 (+0000) Subject: Remove the block argument from apr_brigade_partition. This looked good X-Git-Tag: 2.0.24~140 X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=6513c027a789d096875c1c4f77412b1952f1d90a;p=apache Remove the block argument from apr_brigade_partition. This looked good in theory, but in reality, we should just rely on the timeouts to handle this. git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@90043 13f79535-47bb-0310-9956-ffa450edef68 --- diff --git a/modules/http/http_protocol.c b/modules/http/http_protocol.c index 7fe48a22be..2f9fbd394e 100644 --- a/modules/http/http_protocol.c +++ b/modules/http/http_protocol.c @@ -656,7 +656,7 @@ apr_status_t ap_http_filter(ap_filter_t *f, apr_bucket_brigade *b, ap_input_mode ### READBYTES bytes, and we wouldn't have to do any work. */ - apr_brigade_partition(ctx->b, *readbytes, &e, APR_NONBLOCK_READ); + apr_brigade_partition(ctx->b, *readbytes, &e); APR_BRIGADE_CONCAT(b, ctx->b); ctx->b = apr_brigade_split(b, e); apr_brigade_length(b, 1, &total); @@ -2400,12 +2400,12 @@ AP_CORE_DECLARE_NONSTD(apr_status_t) ap_byterange_filter( /* these calls to apr_brigade_partition() should theoretically * never fail because of the above call to apr_brigade_length(), * but what the heck, we'll check for an error anyway */ - if ((rv = apr_brigade_partition(bb, range_start, &ec, APR_BLOCK_READ)) != APR_SUCCESS) { + if ((rv = apr_brigade_partition(bb, range_start, &ec)) != APR_SUCCESS) { ap_log_rerror(APLOG_MARK, APLOG_ERR, rv, r, PARTITION_ERR_FMT, range_start, clength); continue; } - if ((rv = apr_brigade_partition(bb, range_end+1, &e2, APR_BLOCK_READ)) != APR_SUCCESS) { + if ((rv = apr_brigade_partition(bb, range_end+1, &e2)) != APR_SUCCESS) { ap_log_rerror(APLOG_MARK, APLOG_ERR, rv, r, PARTITION_ERR_FMT, range_end+1, clength); continue;