diff options
author | Fraunhofer IIS FDK <audio-fdk@iis.fraunhofer.de> | 2018-12-20 15:52:46 +0100 |
---|---|---|
committer | Jean-Michel Trivi <jmtrivi@google.com> | 2019-01-02 13:48:42 -0500 |
commit | 443d32ea7fffba4d95a25218ad0119c9696005c2 (patch) | |
tree | 7b40fb0eda9a0f44afcf0f1c3893ffeb32530a6f /libFDK/src/nlc_dec.cpp | |
parent | f440f28aa3cfa02d52abb89e7db24d16f6eb0172 (diff) | |
download | fdk-aac-443d32ea7fffba4d95a25218ad0119c9696005c2.tar.gz fdk-aac-443d32ea7fffba4d95a25218ad0119c9696005c2.tar.bz2 fdk-aac-443d32ea7fffba4d95a25218ad0119c9696005c2.zip |
Add sanity check in huff_decode()
Test: atest DecoderTestXheAac ; atest DecoderTestAacDrc
Change-Id: I33e99629665df9aa6262c90dd7ebdde4b4b9d773
Diffstat (limited to 'libFDK/src/nlc_dec.cpp')
-rw-r--r-- | libFDK/src/nlc_dec.cpp | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/libFDK/src/nlc_dec.cpp b/libFDK/src/nlc_dec.cpp index 8a8ccfd..6e98ce0 100644 --- a/libFDK/src/nlc_dec.cpp +++ b/libFDK/src/nlc_dec.cpp @@ -647,6 +647,10 @@ static ERROR_t huff_decode(HANDLE_FDK_BITSTREAM strm, SCHAR* out_data_1, } df_rest_flag_1 = num_val_1_int % 2; if (df_rest_flag_1) num_val_1_int -= 1; + if (num_val_1_int < 0) { + err = HUFFDEC_NOTOK; + goto bail; + } } if (out_data_2 != NULL) { if (diff_type_2 == DIFF_FREQ) { @@ -658,6 +662,10 @@ static ERROR_t huff_decode(HANDLE_FDK_BITSTREAM strm, SCHAR* out_data_1, } df_rest_flag_2 = num_val_2_int % 2; if (df_rest_flag_2) num_val_2_int -= 1; + if (num_val_2_int < 0) { + err = HUFFDEC_NOTOK; + goto bail; + } } if (out_data_1 != NULL) { |