[FFmpeg-cvslog] mjpeg_decode_com: remove redundant len checks

Michael Niedermayer git at videolan.org
Sun May 19 18:12:20 CEST 2013


ffmpeg | branch: master | Michael Niedermayer <michaelni at gmx.at> | Sun May 19 18:03:53 2013 +0200| [73b9f31c74ccec359a917a9efc314a0cba1c29c7] | committer: Michael Niedermayer

mjpeg_decode_com: remove redundant len checks

Signed-off-by: Michael Niedermayer <michaelni at gmx.at>

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

 libavcodec/mjpegdec.c |    6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/libavcodec/mjpegdec.c b/libavcodec/mjpegdec.c
index b933a71..b2802ff 100644
--- a/libavcodec/mjpegdec.c
+++ b/libavcodec/mjpegdec.c
@@ -1501,14 +1501,14 @@ static int mjpeg_decode_com(MJpegDecodeContext *s)
                 av_log(s->avctx, AV_LOG_INFO, "comment: '%s'\n", cbuf);
 
             /* buggy avid, it puts EOI only at every 10th frame */
-            if (len > 3 && (!strncmp(cbuf, "AVID", 4))) {
+            if (!strncmp(cbuf, "AVID", 4)) {
                 s->buggy_avid = 1;
                 if (len > 14 && cbuf[12] == 1) /* 1 - NTSC, 2 - PAL */
                     s->interlace_polarity = 1;
             } else if (!strcmp(cbuf, "CS=ITU601"))
                 s->cs_itu601 = 1;
-            else if ((len > 31 && !strncmp(cbuf, "Intel(R) JPEG Library, version 1", 32)) ||
-                     (len > 19 && !strncmp(cbuf, "Metasoft MJPEG Codec", 20)))
+            else if ((!strncmp(cbuf, "Intel(R) JPEG Library, version 1", 32)) ||
+                     (!strncmp(cbuf, "Metasoft MJPEG Codec", 20)))
                 s->flipped = 1;
 
             av_free(cbuf);



More information about the ffmpeg-cvslog mailing list