[FFmpeg-devel] [PATCH] Document why we always send a Range HTTP header.

Reimar Döffinger Reimar.Doeffinger at gmx.de
Sun Sep 23 15:27:20 CEST 2012


Signed-off-by: Reimar Döffinger <Reimar.Doeffinger at gmx.de>
---
 libavformat/http.c |    3 +++
 1 file changed, 3 insertions(+)

diff --git a/libavformat/http.c b/libavformat/http.c
index 376ff9e..e3bfbc4 100644
--- a/libavformat/http.c
+++ b/libavformat/http.c
@@ -411,6 +411,9 @@ static int http_connect(URLContext *h, const char *path, const char *local_path,
     if (!has_header(s->headers, "\r\nAccept: "))
         len += av_strlcpy(headers + len, "Accept: */*\r\n",
                           sizeof(headers) - len);
+    // Note: we send this on purpose even when s->off is 0,
+    // since it allows us to detect more reliably if a (non-conforming)
+    // server supports seeking by analysing the reply headers.
     if (!has_header(s->headers, "\r\nRange: ") && !post)
         len += av_strlcatf(headers + len, sizeof(headers) - len,
                            "Range: bytes=%"PRId64"-\r\n", s->off);
-- 
1.7.10.4



More information about the ffmpeg-devel mailing list