Graham Leggett wrote:

> +    if ((ct = apr_table_get(r->headers_in, "Content-Type")) &&
> +        (!strncasecmp(ct, "multipart/byteranges", 20) ||
> +         !strncasecmp(ct, "multipart/x-byteranges", 22))) {
> +       return 0;

Oo-er! This is only half the story - single byte ranges must also be
covered. Also, header_in should be headers_out...

Regards,
Graham
-- 
-----------------------------------------
[EMAIL PROTECTED]                "There's a moon
                                        over Bourbon Street
                                                tonight..."
--- http_protocol.c-orig        Fri Apr 13 05:43:29 2001
+++ http_protocol.c     Mon Apr 16 19:41:51 2001
@@ -2347,6 +2347,18 @@
     if (r->assbackwards)
         return 0;
 
+    /* is content already a single range? */
+    if (apr_table_get(r->headers_out, "Content-Range")) {
+       return 0;
+    }
+
+    /* is content already a multiple range? */
+    if ((ct = apr_table_get(r->headers_out, "Content-Type")) &&
+        (!strncasecmp(ct, "multipart/byteranges", 20) ||
+         !strncasecmp(ct, "multipart/x-byteranges", 22))) {
+       return 0;
+    }
+
     /* Check for Range request-header (HTTP/1.1) or Request-Range for
      * backwards-compatibility with second-draft Luotonen/Franks
      * byte-ranges (e.g. Netscape Navigator 2-3).

Reply via email to