mjpegenc: Fix JFIF header byte ordering
authorShiina Hideaki <shiina@yndrd.com>
Thu, 7 May 2015 00:46:55 +0000 (01:46 +0100)
committerVittorio Giovara <vittorio.giovara@gmail.com>
Tue, 19 May 2015 11:15:00 +0000 (12:15 +0100)
The header had a wrong version description.

Bug-Id: 808
Signed-off-by: Shiina Hideaki <shiina@yndrd.com>
Signed-off-by: Vittorio Giovara <vittorio.giovara@gmail.com>
libavcodec/mjpegenc_common.c

index adb335e5e19045192995d5ecd25eecca1c7a61f5..9373e4a467f65db2858daa9528a73af17f53a228 100644 (file)
@@ -96,7 +96,10 @@ static void jpeg_put_comments(AVCodecContext *avctx, PutBitContext *p)
         put_marker(p, APP0);
         put_bits(p, 16, 16);
         avpriv_put_string(p, "JFIF", 1); /* this puts the trailing zero-byte too */
-        put_bits(p, 16, 0x0201);         /* v 1.02 */
+        /* The most significant byte is used for major revisions, the least
+         * significant byte for minor revisions. Version 1.02 is the current
+         * released revision. */
+        put_bits(p, 16, 0x0102);
         put_bits(p,  8, 0);              /* units type: 0 - aspect ratio */
         put_bits(p, 16, avctx->sample_aspect_ratio.num);
         put_bits(p, 16, avctx->sample_aspect_ratio.den);