[FFmpeg-cvslog] http: Support relative URL redirection

Zhang Rui git at videolan.org
Fri Nov 22 17:47:47 CET 2013


ffmpeg | branch: master | Zhang Rui <bbcallen at gmail.com> | Thu Jul  4 17:18:52 2013 +0800| [393d80cc278e2b2f5c9e7805db176e6f28335a33] | committer: Martin Storsjö

http: Support relative URL redirection

In RFC 2616, this was explicitly said to be an absolute URL,
while in an upcoming draft [1] it is allowed to be relative as well.

[1] http://tools.ietf.org/html/draft-ietf-httpbis-p2-semantics-25#section-7.1.2

Signed-off-by: Martin Storsjö <martin at martin.st>

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=393d80cc278e2b2f5c9e7805db176e6f28335a33
---

 libavformat/http.c |    5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/libavformat/http.c b/libavformat/http.c
index a26ec2a..eb08dfe 100644
--- a/libavformat/http.c
+++ b/libavformat/http.c
@@ -335,7 +335,10 @@ static int process_line(URLContext *h, char *line, int line_count,
         while (av_isspace(*p))
             p++;
         if (!av_strcasecmp(tag, "Location")) {
-            av_strlcpy(s->location, p, sizeof(s->location));
+            char redirected_location[MAX_URL_SIZE];
+            ff_make_absolute_url(redirected_location, sizeof(redirected_location),
+                                 s->location, p);
+            av_strlcpy(s->location, redirected_location, sizeof(s->location));
             *new_location = 1;
         } else if (!av_strcasecmp (tag, "Content-Length") && s->filesize == -1) {
             s->filesize = strtoll(p, NULL, 10);



More information about the ffmpeg-cvslog mailing list