]> git.sesse.net Git - ffmpeg/commitdiff
h264_cabac: Break infinite loops
authorMichael Niedermayer <michaelni@gmx.at>
Thu, 31 Jan 2013 03:20:24 +0000 (04:20 +0100)
committerMartin Storsjö <martin@martin.st>
Thu, 15 Jan 2015 08:17:01 +0000 (10:17 +0200)
This fixes out of array reads and/or infinite loops.

30 is the maximum number of bits that can be read into
coeff_abs below.

CC: libav-stable@libav.org
Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
Signed-off-by: Martin Storsjö <martin@martin.st>
libavcodec/h264_cabac.c

index 1e9162621ca6f412cca3d0bbccb85e7ef8247f6d..0ad8ac0bdee24241244621d6752ade948af95e91 100644 (file)
@@ -1712,7 +1712,7 @@ decode_cabac_residual_internal(H264Context *h, int16_t *block,
 \
             if( coeff_abs >= 15 ) { \
                 int j = 0; \
-                while( get_cabac_bypass( CC ) ) { \
+                while (get_cabac_bypass(CC) && j < 30) { \
                     j++; \
                 } \
 \