Commit 889b97c4 authored by multrus's avatar multrus
Browse files

only read initialized values from Cy_buf_e[k] in... 

only read initialized values from Cy_buf_e[k] in ivas_param_mc_dequantize_cov_fx() during renormalization
parent c0947d4a

lib_com/options.h

+1 −0
Original line number Diff line number Diff line
@@ -102,6 +102,7 @@ 
#define FIX_2319_CLDFB_INIT_FLAGS                       /* FhG: basop issue 2319: init hs->flags during openCldfb(), to avoid reading of uninitialized memory */

#define FIX_BASOP_2357_OBJECT_EDIT_MSAN                 /* Nokia: basop issue 2357: initialize missing exponents in object edit */

#define FIX_BASOP_2354_MSAN_DIFF_Q_VALUE                /* Nokia: Initialize Q value of diffuseness in rendering to Q30 */

#define FIX_2384_INIT_DEQUANT_COV                       /* FhG: basop issue 2384: only read initialized values from Cy_buf_e[k] in ivas_param_mc_dequantize_cov_fx() during renormalization */



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

lib_dec/ivas_mc_param_dec_fx.c

+19 −0
Original line number Diff line number Diff line
@@ -3142,6 +3142,9 @@ static void ivas_param_mc_dequantize_cov_fx( 
    const PARAM_MC_ILD_MAPPING *h_ild_mapping;

    Word32 Cy_state_int_fx[MAX_OUTPUT_CHANNELS * MAX_OUTPUT_CHANNELS];

    Word16 Cy_state_int_e[MAX_OUTPUT_CHANNELS * MAX_OUTPUT_CHANNELS];

#ifdef FIX_2384_INIT_DEQUANT_COV

    Word16 nY_norm;

#endif



    set16_fx( Cp_buf_e, Cproto_e, MAX_OUTPUT_CHANNELS * MAX_OUTPUT_CHANNELS );

    set32_fx( Nrqq_fx, 0, MAX_OUTPUT_CHANNELS );
@@ -3395,22 +3398,38 @@ static void ivas_param_mc_dequantize_cov_fx( 
                Cyp_e += nY_cov;

            }

        }

#ifdef FIX_2384_INIT_DEQUANT_COV

        nY_norm = nY_cov;

        move16();

#endif

    }

    ELSE

    {

        Copy32( Cy_state_int_fx, Cy_state_fx, imult1616( nY_int, nY_int ) );

        Copy( Cy_state_int_e, Cy_buf_e, imult1616( nY_int, nY_int ) );

#ifdef FIX_2384_INIT_DEQUANT_COV

        nY_norm = nY_int;

        move16();

#endif

    }



    /*normalize output matrix to a common exponent*/

    tmp = 0;

#ifdef FIX_2384_INIT_DEQUANT_COV

    FOR( k = 0; k < nY_norm * nY_norm; k++ )

#else

    FOR( k = 0; k < nY_int * nY_int; k++ )

#endif

    {

        Cy_state_fx[k] = BASOP_Util_Add_Mant32Exp( Cy_state_fx[k], Cy_buf_e[k], 0, 0, &Cy_buf_e[k] );

        move32();

        tmp = s_max( tmp, Cy_buf_e[k] );

    }

#ifdef FIX_2384_INIT_DEQUANT_COV

    FOR( k = 0; k < nY_norm * nY_norm; k++ )

#else

    FOR( k = 0; k < nY_int * nY_int; k++ )

#endif

    {

        L_tmp = L_shr( Cy_state_fx[k], sub( tmp, Cy_buf_e[k] ) );

        Cy_state_fx[k] = L_tmp;