You've already forked FFmpeg
mirror of
https://github.com/FFmpeg/FFmpeg.git
synced 2025-08-15 14:13:16 +02:00
avcodec/mpeg4videodec: Check mb_num also against 0
The spec implies that 0 is invalid in addition to the existing checks Found-by: <kierank> Reviewed-by: Kieran Kunhya <kieran618@googlemail.com> Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
This commit is contained in:
@@ -465,7 +465,7 @@ int ff_mpeg4_decode_video_packet_header(Mpeg4DecContext *ctx)
|
|||||||
}
|
}
|
||||||
|
|
||||||
mb_num = get_bits(&s->gb, mb_num_bits);
|
mb_num = get_bits(&s->gb, mb_num_bits);
|
||||||
if (mb_num >= s->mb_num) {
|
if (mb_num >= s->mb_num || !mb_num) {
|
||||||
av_log(s->avctx, AV_LOG_ERROR,
|
av_log(s->avctx, AV_LOG_ERROR,
|
||||||
"illegal mb_num in video packet (%d %d) \n", mb_num, s->mb_num);
|
"illegal mb_num in video packet (%d %d) \n", mb_num, s->mb_num);
|
||||||
return -1;
|
return -1;
|
||||||
|
Reference in New Issue
Block a user