Bug 16140 - invalid byte-range-set MUST be ignored
Summary: invalid byte-range-set MUST be ignored
Status: RESOLVED FIXED
Alias: None
Product: Apache httpd-2
Classification: Unclassified
Component: mod_cache (show other bugs)
Version: 2.0-HEAD
Hardware: All All
: P3 normal (vote)
Target Milestone: ---
Assignee: Apache HTTPD Bugs Mailing List
URL: http://coad.measurement-factory.com/c...
Keywords: PatchAvailable
Depends on:
Blocks:
 
Reported: 2003-01-15 21:04 UTC by Co-Advisor
Modified: 2009-06-16 07:53 UTC (History)
1 user (show)



Attachments
test case trace (12.91 KB, text/html)
2003-01-15 21:05 UTC, Co-Advisor
Details
Fix in ap_set_byterange() in modules\http\http_protocol.c (1.42 KB, patch)
2003-07-17 10:47 UTC, Nicel KM
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Co-Advisor 2003-01-15 21:04:39 UTC
Looks like a possible RFC 2616 MUST violation.
Note that Apache origin server does not seem 
to have this problem.

See attached trace(s) for details and ways to reproduce
the violation mentioned above.

Test case IDs in the trace link to human-oriented test case
description and RFC quotes, if available.
Comment 1 Co-Advisor 2003-01-15 21:05:20 UTC
Created attachment 4450 [details]
test case trace
Comment 2 Nicel KM 2003-07-17 10:47:21 UTC
Created attachment 7346 [details]
Fix in ap_set_byterange() in modules\http\http_protocol.c
Comment 3 Nicel KM 2003-07-17 10:54:35 UTC
Attached patch validates the range values and ignores the range header if 
syntactically invalid. Range header is ignored if last-byte-pos is less than 
first-byte-pos in a byte-range-spec.
Fix in ap_set_byterange() in modules\http\Http_protocol.c
Comment 4 Co-Advisor 2003-07-21 20:28:46 UTC
With the patch in attachment 7346 [details], Apache passes
all test cases in this test clause.
Comment 5 Jeff Trawick 2003-11-21 17:18:00 UTC
I'm going through the bug db to make sure patches are findable.  Please see 
http://httpd.apache.org/dev/patches.html
Comment 6 Nicel KM 2004-02-18 10:04:46 UTC
Can some one look at this patch and commit if found ok. This would help us close
one of the open PRs on RFC 2616 violations.
Comment 7 Dan Poirier 2009-06-16 07:53:33 UTC
Has never been fixed in 2.0.x, but the byterange_filter, new in 2.2.x, does appear to implement these checks.  Is it worth making the fix back in 2.0.x?  If so, please re-open; otherwise calling this one fixed.