From: Jim Jagielski <jim@apache.org>
Date: Mon, 29 Aug 2011 15:59:39 +0000 (+0000)
Subject: Remove dead, wrong code
X-Git-Tag: 2.2.20~3
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=cb6eecafb56a4d875adcab03ae6a52df22905495;p=thirdparty%2Fapache%2Fhttpd.git

Remove dead, wrong code


git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/branches/2.2.x@1162885 13f79535-47bb-0310-9956-ffa450edef68
---

diff --git a/modules/http/byterange_filter.c b/modules/http/byterange_filter.c
index cb63149fc43..6843df1bf89 100644
--- a/modules/http/byterange_filter.c
+++ b/modules/http/byterange_filter.c
@@ -137,7 +137,6 @@ static apr_status_t copy_brigade_range(apr_bucket_brigade *bb,
             if (off_first != start64) {
                 rv = apr_bucket_split(copy, (apr_size_t)(start64 - off_first));
                 if (rv == APR_ENOTIMPL) {
-                    int i;
                     rv = apr_bucket_read(copy, &s, &len, APR_BLOCK_READ);
                     if (rv != APR_SUCCESS) {
                         apr_brigade_cleanup(bbout);
@@ -148,14 +147,8 @@ static apr_status_t copy_brigade_range(apr_bucket_brigade *bb,
                      * of shorter length. So read and delete until we reached
                      * the correct bucket for splitting.
                      */
-                    i = 0;
                     while (start64 - off_first > (apr_uint64_t)copy->length) {
-                        apr_bucket *tmp;
-                        /* don't allow inf. spin */
-                        if (i++ >= 99999)
-                            return APR_EINVAL;
-
-                        tmp = APR_BUCKET_NEXT(copy);
+                        apr_bucket *tmp = APR_BUCKET_NEXT(copy);
                         off_first += (apr_uint64_t)copy->length;
                         APR_BUCKET_REMOVE(copy);
                         apr_bucket_destroy(copy);