Added a saturation overflow checks on PCM data (#44)

* Added saturation overflow check on PCM data [x] Tested for conformance. --------- Co-authored-by: Saketh Sathuvalli <100603@ittiam.com>
ittiam-systems · Feb 1, 2023 · 3b23f65 · 3b23f65
1 parent 0b8499a
commit 3b23f65
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 0 deletions.
diff --git a/decoder/ia_core_coder_constants.h b/decoder/ia_core_coder_constants.h
@@ -85,6 +85,9 @@
 #define MAX_16 (WORD16)0x7fff
 #define MIN_16 (WORD16)0x8000
 
+#define MAX_24 (WORD32)0x007fffffL
+#define MIN_24 (WORD32)0xFF800000L
+
 #define NULLPTR ((VOID *)0)
 
 #define IT_NULL ((VOID *)0)

diff --git a/decoder/ia_core_coder_decode_main.c b/decoder/ia_core_coder_decode_main.c
@@ -2083,6 +2083,8 @@ IA_ERRORCODE ia_core_coder_dec_ext_ele_proc(VOID *temp_handle, WORD32 *num_chann
       {
         pcm_sample =
             (WORD32)(pstr_dec_data->str_usac_data.time_sample_vector[channel][s] * 256.0f);
+        pcm_sample = ia_max_int(ia_min_int(pcm_sample, MAX_24), MIN_24);
+
         *ptr_ext_ren_pcm++ = (WORD32)pcm_sample & 0xff;
         *ptr_ext_ren_pcm++ = ((WORD32)pcm_sample >> 8) & 0xff;
         *ptr_ext_ren_pcm++ = ((WORD32)pcm_sample >> 16) & 0xff;