wmadec: Verify bitstream size makes sense before calling init_get_bits.
authorAlex Converse <alex.converse@gmail.com>
Fri, 27 Jan 2012 22:24:07 +0000 (14:24 -0800)
committerReinhard Tartler <siretart@tauware.de>
Sun, 4 Mar 2012 11:26:06 +0000 (12:26 +0100)
Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
(cherry picked from commit 48f1e5212c90b511c90fa0449655abb06a9edda2)

Signed-off-by: Reinhard Tartler <siretart@tauware.de>
libavcodec/wmadec.c

index afc0658..b9fc21f 100644 (file)
@@ -877,6 +877,8 @@ static int wma_decode_superframe(AVCodecContext *avctx, void *data,
 
         /* read each frame starting from bit_offset */
         pos = bit_offset + 4 + 4 + s->byte_offset_bits + 3;
+        if (pos >= MAX_CODED_SUPERFRAME_SIZE * 8)
+            return AVERROR_INVALIDDATA;
         init_get_bits(&s->gb, buf + (pos >> 3), (MAX_CODED_SUPERFRAME_SIZE - (pos >> 3))*8);
         len = pos & 7;
         if (len > 0)