[FFmpeg-cvslog] avcodec/mpeg4videodec: Check also for negative versions in the validity check

Michael Niedermayer git at videolan.org
Thu Feb 1 01:51:58 EET 2018


ffmpeg | branch: release/2.8 | Michael Niedermayer <michael at niedermayer.cc> | Tue Nov 21 03:15:53 2017 +0100| [ea24e70a6a416036d191bb51ca5469679a7c174c] | committer: Michael Niedermayer

avcodec/mpeg4videodec: Check also for negative versions in the validity check

Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>
(cherry picked from commit 0e7865ce4152f8b04cda6a698bbee4fd4a94009d)
Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>

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

 libavcodec/mpeg4videodec.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libavcodec/mpeg4videodec.c b/libavcodec/mpeg4videodec.c
index dc556a45b5..7185844922 100644
--- a/libavcodec/mpeg4videodec.c
+++ b/libavcodec/mpeg4videodec.c
@@ -2139,7 +2139,7 @@ static int decode_user_data(Mpeg4DecContext *ctx, GetBitContext *gb)
     if (e != 4) {
         e = sscanf(buf, "Lavc%d.%d.%d", &ver, &ver2, &ver3) + 1;
         if (e > 1) {
-            if (ver > 0xFF || ver2 > 0xFF || ver3 > 0xFF) {
+            if (ver > 0xFFU || ver2 > 0xFFU || ver3 > 0xFFU) {
                 av_log(s->avctx, AV_LOG_WARNING,
                      "Unknown Lavc version string encountered, %d.%d.%d; "
                      "clamping sub-version values to 8-bits.\n",



More information about the ffmpeg-cvslog mailing list