mpjpeg: Cope with multipart lacking the initial CRLF
authorLuca Barbato <lu_zero@gentoo.org>
Thu, 22 Oct 2015 03:07:05 +0000 (05:07 +0200)
committerLuca Barbato <lu_zero@gentoo.org>
Fri, 23 Oct 2015 09:56:38 +0000 (11:56 +0200)
Some server in the wild do not put the boundary at a newline
as rfc1347 7.2.1 states.
Cope with that by reading a line and if it is not empty reading
a second one.

Reported-By: bitingsock
libavformat/mpjpegdec.c

index fda4c38..2888fb5 100644 (file)
@@ -153,10 +153,20 @@ static int parse_multipart_header(AVFormatContext *s)
     int found_content_type = 0;
     int ret, size = -1;
 
+    // get the CRLF as empty string
     ret = get_line(s->pb, line, sizeof(line));
     if (ret < 0)
         return ret;
 
+    /* some implementation do not provide the required
+     * initial CRLF (see rfc1341 7.2.1)
+     */
+    if (!line[0]) {
+        ret = get_line(s->pb, line, sizeof(line));
+        if (ret < 0)
+            return ret;
+    }
+
     if (strncmp(line, "--", 2))
         return AVERROR_INVALIDDATA;
 
@@ -208,9 +218,6 @@ static int mpjpeg_read_packet(AVFormatContext *s, AVPacket *pkt)
     if (ret < 0)
         return ret;
 
-    // trailing empty line
-    avio_skip(s->pb, 2);
-
     return 0;
 }