Commit dccae17d authored by norvell's avatar norvell
Browse files

Merge branch... 

Merge branch '1296-decoder-memorysanitizer-use-of-uninitialized-value-in-limiter_process-scheduled-pipeline-c' into 'main'

[Non-BE][Renderer Non-BE][Split Non-BE] Resolve potential out-of-bounds access in TD renderer ITD synthesis.

See merge request !2056
parents eebddcf6 2a40a505

lib_com/options.h

+1 −0
Original line number Diff line number Diff line
@@ -174,6 +174,7 @@ 
#define NONBE_1244_FIX_SWB_BWE_MEMORY                   /* VA: issue 1244: fix to SWB BWE memory in case of switching from FB coding - pending a review by Huawei */ 

#define NONBE_FIX_1180_HQMDCT_PHECU_LT_MUTING           /* Ericsson: issue 1180, corrected long term mute loop attnuation after 200ms  in PhECU-PLC  */

#define NONBE_1289_STEREO_SW_TO_MONO                    /* VA: issue 1289: Fix glitch when stereo signal is decoded to mono n TD->DFT switching */

#define NONBE_1296_TDREND_ITD_OUT_OF_BOUNDS_ACCESS      /* Eri: issue 1296: ITD resampling can occasionally read out of bounds, especially when the requested subframes are short (1.25 ms). Seen for headtracking+JBM. */



/* ##################### End NON-BE switches ########################### */

lib_rend/ivas_objectRenderer_sfx.c

+2 −0
Original line number Diff line number Diff line
@@ -216,7 +216,9 @@ static void sincResample( 
            p_forward++;

            p_backward--;

        }

#ifndef NONBE_1296_TDREND_ITD_OUT_OF_BOUNDS_ACCESS

        tmp += ( *p_forward ) * ( *p_sinc_forward ); /* Integer index always rounded down --> 4 steps backward, 5 steps forward */

#endif



        output[i] = tmp;