[FFmpeg-cvslog] avcodec/vorbisdec: Check vlc for floor0 dec vector offset
Michael Niedermayer
git at videolan.org
Sun Jul 21 15:06:23 EEST 2019
ffmpeg | branch: master | Michael Niedermayer <michael at niedermayer.cc> | Sun Jul 7 23:23:53 2019 +0200| [99f95f39c6978f0d91e42b3bced126a98173dbef] | committer: Michael Niedermayer
avcodec/vorbisdec: Check vlc for floor0 dec vector offset
Fixes: out of array access
Fixes: 15649/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_VORBIS_fuzzer-5729191309344768
Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg
Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>
> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=99f95f39c6978f0d91e42b3bced126a98173dbef
---
libavcodec/vorbisdec.c | 6 ++++--
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/libavcodec/vorbisdec.c b/libavcodec/vorbisdec.c
index 7f2d27d1a2..a86d4c4c36 100644
--- a/libavcodec/vorbisdec.c
+++ b/libavcodec/vorbisdec.c
@@ -1128,8 +1128,10 @@ static int vorbis_floor0_decode(vorbis_context *vc,
ff_dlog(NULL, "floor0 dec: maximum depth: %d\n", codebook.maxdepth);
/* read temp vector */
vec_off = get_vlc2(&vc->gb, codebook.vlc.table,
- codebook.nb_bits, codebook.maxdepth)
- * codebook.dimensions;
+ codebook.nb_bits, codebook.maxdepth);
+ if (vec_off < 0)
+ return AVERROR_INVALIDDATA;
+ vec_off *= codebook.dimensions;
ff_dlog(NULL, "floor0 dec: vector offset: %d\n", vec_off);
/* copy each vector component and add last to it */
for (idx = 0; idx < codebook.dimensions; ++idx)
More information about the ffmpeg-cvslog
mailing list