]> granicus.if.org Git - libvpx/commitdiff
ANS: Remove extra buffer size checks causing a false decode error.
authorAlex Converse <aconverse@google.com>
Tue, 12 Apr 2016 22:10:19 +0000 (15:10 -0700)
committerAlex Converse <aconverse@google.com>
Tue, 12 Apr 2016 22:19:30 +0000 (15:19 -0700)
The minimal ans partition size is now one byte. This is checked in
ans_read_init().

The read_is_valid() condition is handled by setup_token_decoder().

Change-Id: I7b202b896630bc4285532208bf7cf84567afe158

vp10/decoder/decodeframe.c

index af57ac6bdaabb47b0079b0b71c5a39791bc16dbe..17e7401629f21931a2f2257fc61e86d67e4f745f 100644 (file)
@@ -3173,9 +3173,6 @@ static const uint8_t *decode_tiles(VP10Decoder *pbi,
                          &td->bit_reader, pbi->decrypt_cb,
                          pbi->decrypt_state);
 #else
-      if (buf->size < 3 || !read_is_valid(buf->data, buf->size, data_end))
-        vpx_internal_error(&cm->error, VPX_CODEC_CORRUPT_FRAME,
-                           "Truncated packet or corrupt tile length");
       setup_token_decoder(buf->data, data_end, buf->size, &cm->error,
                           &td->bit_reader, pbi->decrypt_cb,
                           pbi->decrypt_state);