[FFmpeg-cvslog] prores: Reject negative run and level values

Luca Barbato git at videolan.org
Fri Oct 11 16:55:37 CEST 2013


ffmpeg | branch: master | Luca Barbato <lu_zero at gentoo.org> | Thu Oct 10 21:02:10 2013 +0200| [c0de9a23c7080e2fac8f879b9d9a0ce2b64ea953] | committer: Luca Barbato

prores: Reject negative run and level values

Sample-Id: 00000611-google

Reported-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
CC: libav-stable at libav.org

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

 libavcodec/proresdec.c |    4 ++++
 1 file changed, 4 insertions(+)

diff --git a/libavcodec/proresdec.c b/libavcodec/proresdec.c
index c65d729..144fa26 100644
--- a/libavcodec/proresdec.c
+++ b/libavcodec/proresdec.c
@@ -393,12 +393,16 @@ static inline int decode_ac_coeffs(GetBitContext *gb, int16_t *out,
             return 0;
 
         run = decode_vlc_codeword(gb, ff_prores_ac_codebook[run_cb_index]);
+        if (run < 0)
+            return AVERROR_INVALIDDATA;
 
         bits_left = get_bits_left(gb);
         if (bits_left <= 0 || (bits_left <= 8 && !show_bits(gb, bits_left)))
             return AVERROR_INVALIDDATA;
 
         level = decode_vlc_codeword(gb, ff_prores_ac_codebook[lev_cb_index]) + 1;
+        if (level < 0)
+            return AVERROR_INVALIDDATA;
 
         pos += run + 1;
         if (pos >= max_coeffs)



More information about the ffmpeg-cvslog mailing list