commit eae606af591e5ba24510f18b85a54cbfe8d61e7d
author Joseph Pirozzo <pirozzoj@google.com> Mon May 25 10:36:26 2020 -0700
committer Anis Assi <anisassi@google.com> Tue Jun 30 16:11:10 2020 -0700
tree 838af0652d730bbed259a84694732ea093f878f2
parent 73c22e6cdbfc5d099985c70edf9670686cc331a9

Enable bitpool sanity checks

Enable bitpool sanity checks to run all the time, not just in debug
mode.

Tag: #security
Test: sbcdecoder_fuzzer
Bug: 146398979
Change-Id: Iff58305cd18de35e37290f0c09fba01ee14e787a
(cherry picked from commit 59c234a8fddda37147bb3fe1dd3b3a668828bcab)

embdrv/sbc/decoder/srce/decoder-sbc.c

diff --git a/embdrv/sbc/decoder/srce/decoder-sbc.c b/embdrv/sbc/decoder/srce/decoder-sbc.c
index 20c5b67..674645e 100644
--- a/embdrv/sbc/decoder/srce/decoder-sbc.c
+++ b/embdrv/sbc/decoder/srce/decoder-sbc.c
@@ -311,7 +311,6 @@
     return OI_CODEC_SBC_CHECKSUM_MISMATCH;
   }
 
-#ifdef OI_DEBUG
   /*
    * Make sure the bitpool values are sane.
    */
@@ -328,7 +327,6 @@
            OI_SBC_MaxBitpool(&context->common.frameInfo)));
     return OI_STATUS_INVALID_PARAMETERS;
   }
-#endif
 
   /*
    * Now decode the SBC data. Partial decode is not yet implemented for an SBC