diff options
author | Rajat Kumar <rajat.kumar@ittiam.com> | 2019-05-08 15:21:19 +0530 |
---|---|---|
committer | Ray Essick <essick@google.com> | 2020-05-10 22:13:03 -0700 |
commit | d198c6af562ddecea09c249c33acf12453f7d9a6 (patch) | |
tree | 8720ed6bcd4296721f40fd7db5f66c0e1f53f27b /decoder/ixheaacd_sbr_dec.c | |
parent | d78a5849d5d158233c61a8d5974da117b9639473 (diff) | |
download | libxaac-d198c6af562ddecea09c249c33acf12453f7d9a6.tar.gz |
Fix to handle missing checks on error returned
Added missing return checks and converted
functions to void which doesn't return anything
other than zero.
Test: Review
Test: atest android.media.cts.DecoderTest
Bug:131183694
Change-Id: Iea7bb9959fffd7a0e4f345f86581a1328e04a38e
Diffstat (limited to 'decoder/ixheaacd_sbr_dec.c')
-rw-r--r-- | decoder/ixheaacd_sbr_dec.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/decoder/ixheaacd_sbr_dec.c b/decoder/ixheaacd_sbr_dec.c index a8ac0b6..671f4d6 100644 --- a/decoder/ixheaacd_sbr_dec.c +++ b/decoder/ixheaacd_sbr_dec.c @@ -752,10 +752,13 @@ WORD32 ixheaacd_sbr_dec(ia_sbr_dec_struct *ptr_sbr_dec, WORD16 *ptr_time_data, ptr_pvc_data->pvc_rate = ptr_header_data->upsamp_fac; if (sbr_mode == PVC_SBR) { - ixheaacd_pvc_process( + err_code = ixheaacd_pvc_process( ptr_pvc_data, ptr_header_data->pstr_freq_band_data->sub_band_start, ptr_frame_data->str_pvc_frame_info.border_vec[0], &pvc_qmf_enrg_arr[0], &pvc_dec_out_buf[0]); + + if (err_code) return err_code; + ptr_pvc_data->prev_pvc_flg = 1; } else { memset(pvc_dec_out_buf, 0, 1024 * sizeof(FLOAT32)); |