On Sun, Sep 01, 2019 at 11:10:25PM +0200, Michael Niedermayer wrote: > Fixes: Timeout (82sec -> 1sec) > Fixes: > 16411/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_VP3_fuzzer-5166958151991296 > > Found-by: continuous fuzzing process > https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg > Signed-off-by: Michael Niedermayer <mich...@niedermayer.cc> > --- > libavcodec/vp3.c | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/libavcodec/vp3.c b/libavcodec/vp3.c > index 28ed0461c7..a2bd2ef07d 100644 > --- a/libavcodec/vp3.c > +++ b/libavcodec/vp3.c > @@ -710,6 +710,8 @@ static int vp4_unpack_macroblocks(Vp3DecodeContext *s, > GetBitContext *gb) > has_partial = 0; > bit = get_bits1(gb); > for (i = 0; i < s->yuv_macroblock_count; i += current_run) { > + if (get_bits_left(gb) <= 0) > + return AVERROR_INVALIDDATA; > current_run = vp4_get_mb_count(s, gb); > if (current_run > s->yuv_macroblock_count - i) > return -1; > @@ -719,6 +721,8 @@ static int vp4_unpack_macroblocks(Vp3DecodeContext *s, > GetBitContext *gb) > } > > if (has_partial) { > + if (get_bits_left(gb) <= 0) > + return AVERROR_INVALIDDATA; > bit = get_bits1(gb); > current_run = vp4_get_mb_count(s, gb); > for (i = 0; i < s->yuv_macroblock_count; i++) { > -- > 2.23.0
approve. -- Peter (A907 E02F A6E5 0CD2 34CD 20D2 6760 79C5 AC40 DD6B)
signature.asc
Description: PGP signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
