diff options
| author | Michael Niedermayer <michael@niedermayer.cc> | 2018-02-11 03:38:54 +0100 |
|---|---|---|
| committer | Michael Niedermayer <michael@niedermayer.cc> | 2018-02-11 17:20:17 +0100 |
| commit | f2318aee8ca8df1c84092f7d6691a2d0df02c474 (patch) | |
| tree | e8f427740f409c567a485169939d9f143625de98 /libavcodec/vp3.c | |
| parent | 63a4bdbf3b732504e54cc2b9ec0886e6242a90bc (diff) | |
| download | ffmpeg-f2318aee8ca8df1c84092f7d6691a2d0df02c474.tar.gz | |
avcodec/vp3: Error out on invalid num_coeffs in unpack_vlcs()
This fixes a hypothetical integer overflow
Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
Diffstat (limited to 'libavcodec/vp3.c')
| -rw-r--r-- | libavcodec/vp3.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/libavcodec/vp3.c b/libavcodec/vp3.c index 33be345c14..1d83753314 100644 --- a/libavcodec/vp3.c +++ b/libavcodec/vp3.c @@ -951,9 +951,11 @@ static int unpack_vlcs(Vp3DecodeContext *s, GetBitContext *gb, Vp3Fragment *all_fragments = s->all_fragments; VLC_TYPE(*vlc_table)[2] = table->table; - if (num_coeffs < 0) + if (num_coeffs < 0) { av_log(s->avctx, AV_LOG_ERROR, "Invalid number of coefficients at level %d\n", coeff_index); + return AVERROR_INVALIDDATA; + } if (eob_run > num_coeffs) { coeff_i = |