[FFmpeg-cvslog] r13050 - trunk/libavcodec/alac.c

michael subversion
Sat May 3 22:56:57 CEST 2008


Author: michael
Date: Sat May  3 22:56:57 2008
New Revision: 13050

Log:
Fix memset(0) based buffer overflow.


Modified:
   trunk/libavcodec/alac.c

Modified: trunk/libavcodec/alac.c
==============================================================================
--- trunk/libavcodec/alac.c	(original)
+++ trunk/libavcodec/alac.c	Sat May  3 22:56:57 2008
@@ -199,7 +199,8 @@ static void bastardized_rice_decompress(
 
         /* special case: there may be compressed blocks of 0 */
         if ((history < 128) && (output_count+1 < output_size)) {
-            int block_size, k;
+            int k;
+            unsigned int block_size;
 
             sign_modifier = 1;
 
@@ -208,6 +209,10 @@ static void bastardized_rice_decompress(
             block_size= decode_scalar(&alac->gb, k, rice_kmodifier, 16);
 
             if (block_size > 0) {
+                if(block_size >= output_size - output_count){
+                    av_log(alac->avctx, AV_LOG_ERROR, "invalid zero block size of %d %d %d\n", block_size, output_size, output_count);
+                    block_size= output_size - output_count - 1;
+                }
                 memset(&output_buffer[output_count+1], 0, block_size * 4);
                 output_count += block_size;
             }




More information about the ffmpeg-cvslog mailing list