Commit 06155ee0 authored by multrus's avatar multrus
Browse files

Merge branch '1387-msan-clang-18-use-of-uninitialized-value-in-ivas_mdct_core_invq' into 'main' 

Resolve "[MSAN Clang 18] use-of-uninitialized value in ivas_mdct_core_invQ"

See merge request !2266
parents 2a6893c4 a219f2aa

lib_com/options.h

+1 −0
Original line number Diff line number Diff line
@@ -171,6 +171,7 @@ 
#define FIX_1388_MSAN_ivas_init_decoder                 /* VA: issue 1388: fix use-of-uninitialized value in ivas_init_decoder() */

#define FIX_1383_HEAD_TRACK_SANITIZER                   /* Nok: issue 1383: Fix head tracking struc values reading in renderer */

#define FIX_1385_INIT_IGF_STOP_FREQ                     /* FhG: Initialize infoIGFStopFreq in init_igf_dec() */

#define FIX_1387_INIT_PRM_SQQ                           /* FhG: initialize pointer prm_sqQ, which might be uninitialized in case of bfi == 1 */





/* #################### End BE switches ################################## */

lib_dec/ivas_mdct_core_dec.c

+3 −0
Original line number Diff line number Diff line
@@ -532,6 +532,9 @@ void ivas_mdct_core_invQ( 
    set_s( total_nbbits, 0, CPE_CHANNELS );

    set_s( bitsRead, 0, CPE_CHANNELS );

    tmp_concealment_method = 0;

#ifdef FIX_1387_INIT_PRM_SQQ

    prm_sqQ = NULL; /* set prm_sqQ to NULL - in case of bfi == 1 it's not set or needed, but it triggers sanitizers */

#endif



    for ( ch = 0; ch < CPE_CHANNELS; ch++ )

    {