From: Alex Converse <aconverse@google.com>
Date: Tue, 12 Apr 2016 22:10:19 +0000 (-0700)
Subject: ANS: Remove extra buffer size checks causing a false decode error.
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=493a5852733bf1b58fe7f6b4fb43d54f36078910;p=libvpx

ANS: Remove extra buffer size checks causing a false decode error.

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
---

diff --git a/vp10/decoder/decodeframe.c b/vp10/decoder/decodeframe.c
index af57ac6bd..17e740162 100644
--- a/vp10/decoder/decodeframe.c
+++ b/vp10/decoder/decodeframe.c
@@ -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);