[FFmpeg-cvslog] h261: check mtype.

Michael Niedermayer git at videolan.org
Sun May 6 01:43:50 CEST 2012


ffmpeg | branch: release/0.10 | Michael Niedermayer <michaelni at gmx.at> | Fri Mar  2 15:58:14 2012 +0100| [c785a7058aa6c4b89ea1fa02278eee88006dce45] | committer: Michael Niedermayer

h261: check mtype.

Fixes out of array read

Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
(cherry picked from commit ec3cd74f2dab8e3e8234ccb994132b23d3098585)

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

 libavcodec/h261dec.c |    4 ++++
 1 files changed, 4 insertions(+), 0 deletions(-)

diff --git a/libavcodec/h261dec.c b/libavcodec/h261dec.c
index 1dbd75d..3b26e8b 100644
--- a/libavcodec/h261dec.c
+++ b/libavcodec/h261dec.c
@@ -286,6 +286,10 @@ static int h261_decode_mb(H261Context *h){
 
     // Read mtype
     h->mtype = get_vlc2(&s->gb, h261_mtype_vlc.table, H261_MTYPE_VLC_BITS, 2);
+    if (h->mtype < 0) {
+        av_log(s->avctx, AV_LOG_ERROR, "illegal mtype %d\n", h->mtype);
+        return SLICE_ERROR;
+    }
     h->mtype = h261_mtype_map[h->mtype];
 
     // Read mquant



More information about the ffmpeg-cvslog mailing list