Commit 1f209a2c authored by Jan Kiene's avatar Jan Kiene
Browse files

Merge branch 'main' into... 

Merge branch 'main' into float-1486-clang18-msan-use-of-uninitialized-value-in-lib_com-ivas_tools-c-134-23
parents e6d8e757 c81254b0

apps/renderer.c

+16 −0
Original line number Diff line number Diff line
@@ -172,7 +172,9 @@ typedef struct 
    int16_t numInMetadataFiles;

    char outMetadataFilePath[RENDERER_MAX_CLI_ARG_LENGTH];

    char headRotationFilePath[RENDERER_MAX_CLI_ARG_LENGTH];

#ifndef FIX_1494_SET_SPLITBFI_UNUSED

    char splitRendBFIFilePath[RENDERER_MAX_CLI_ARG_LENGTH];

#endif

    char referenceVectorFilePath[RENDERER_MAX_CLI_ARG_LENGTH];

    char referenceRotationFilePath[RENDERER_MAX_CLI_ARG_LENGTH];

    char externalOrientationFilePath[RENDERER_MAX_CLI_ARG_LENGTH];
@@ -223,7 +225,9 @@ typedef enum 
    CmdLnOptionId_listFormats,

    CmdLnOptionId_inputGain,

    CmdLnOptionId_outputMetadata,

#ifndef FIX_1494_SET_SPLITBFI_UNUSED

    CmdLnOptionId_SplitRendBFIFile,

#endif

    CmdLnOptionId_referenceVectorFile,

    CmdLnOptionId_exteriorOrientationFile,

    CmdLnOptionId_framing,
@@ -282,12 +286,14 @@ static const CmdLnParser_Option cliOptions[] = { 
        .matchShort = "om",

        .description = "coded metadata file for BINAURAL_SPLIT_PCM output mode",

    },

#ifndef FIX_1494_SET_SPLITBFI_UNUSED

    {

        .id = CmdLnOptionId_SplitRendBFIFile,

        .match = "post_rend_bfi_file",

        .matchShort = "prbfi",

        .description = "Split rendering option: bfi file",

    },

#endif

    {

        .id = CmdLnOptionId_refRotFile,

        .match = "reference_rotation_file",
@@ -763,7 +769,9 @@ int main( 
    IVAS_CLDFB_FILTER_BANK_HANDLE cldfbAna[RENDERER_MAX_INPUT_CHANNELS];

    IVAS_CLDFB_FILTER_BANK_HANDLE cldfbSyn[RENDERER_MAX_INPUT_CHANNELS];

    int16_t cldfb_in_flag, CLDFBframeSize_smpls;

#ifndef FIX_1494_SET_SPLITBFI_UNUSED

    SplitRendBFIFileReader *splitRendBFIReader = NULL;

#endif

    Vector3PairFileReader *referenceVectorReader = NULL;

    hrtfFileReader *hrtfFileReader = NULL;

    IVAS_DEC_HRTF_CREND_HANDLE *hHrtfCrend = NULL;
@@ -893,11 +901,13 @@ int main( 
        }

    }



#ifndef FIX_1494_SET_SPLITBFI_UNUSED

    if ( !isEmptyString( args.splitRendBFIFilePath ) )

    {

        convert_backslash( args.splitRendBFIFilePath );

        SplitRendBFIFileReader_open( args.splitRendBFIFilePath, &splitRendBFIReader );

    }

#endif

    if ( !isEmptyString( args.externalOrientationFilePath ) )

    {

        if ( RotationFileReader_open( args.externalOrientationFilePath, &externalOrientationFileReader ) != IVAS_ERR_OK )
@@ -2183,7 +2193,9 @@ cleanup: 
    }



    split_rend_reader_writer_close( &hSplitRendFileReadWrite );

#ifndef FIX_1494_SET_SPLITBFI_UNUSED

    SplitRendBFIFileReader_close( &splitRendBFIReader );

#endif



    for ( i = 0; i < RENDERER_MAX_MC_INPUTS; ++i )

    {
@@ -2785,7 +2797,9 @@ static CmdlnArgs defaultArgs( 


    clearString( args.headRotationFilePath );

    clearString( args.outMetadataFilePath );

#ifndef FIX_1494_SET_SPLITBFI_UNUSED

    clearString( args.splitRendBFIFilePath );

#endif

    clearString( args.referenceVectorFilePath );

    clearString( args.referenceRotationFilePath );

    clearString( args.customHrtfFilePath );
@@ -2892,10 +2906,12 @@ static void parseOption( 
            assert( numOptionValues == 1 );

            strncpy( args->outMetadataFilePath, optionValues[0], RENDERER_MAX_CLI_ARG_LENGTH - 1 );

            break;

#ifndef FIX_1494_SET_SPLITBFI_UNUSED

        case CmdLnOptionId_SplitRendBFIFile:

            assert( numOptionValues == 1 );

            strncpy( args->splitRendBFIFilePath, optionValues[0], RENDERER_MAX_CLI_ARG_LENGTH - 1 );

            break;

#endif

        case CmdLnOptionId_referenceVectorFile:

            assert( numOptionValues == 1 );

            strncpy( args->referenceVectorFilePath, optionValues[0], RENDERER_MAX_CLI_ARG_LENGTH - 1 );

lib_com/options.h

+2 −0
Original line number Diff line number Diff line
@@ -117,6 +117,8 @@ 
#define FIX_2290_COPY_OF_UNINIT_DATA                    /* VA: prevent the copy of un-initialized data */

#define CLEANUP_ACELP_ENC                               /* VA: basop issue 2304: Remove duplicated code in excitation encoding in the ACELP core */

#define CLEANUP_VBR_CAM_ENC                             /* VA: basop issue 2299: Remove unused core-encoder VBR and CAM code */

#define FIX_1494_SET_SPLITBFI_UNUSED                    /* Dolby: Fix issue 1494, remove unused function setting BFI flag in ISAR renderer */

#define FIX_1479_MSAN_SPAR_UNINITIALIZED_VALUE          /* Dolby: Fix for issue 1479, MSAN error due to uninitialized value in SPAR */



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

lib_dec/ivas_spar_md_dec_fx.c

+7 −0
Original line number Diff line number Diff line
@@ -631,6 +631,13 @@ ivas_error ivas_spar_md_dec_init( 
    hMdDec->first_valid_frame = 1;

    move16();



#ifdef FIX_1479_MSAN_SPAR_UNINITIALIZED_VALUE

    hMdDec->spar_md.res_ind = 0;

    hMdDec->spar_md.prior_dyn_active_w_flag = 0;

    move16();

    move16();

#endif



    return IVAS_ERR_OK;

}

lib_rend/lib_rend.h

+2 −0
Original line number Diff line number Diff line
@@ -344,9 +344,11 @@ ivas_error IVAS_REND_SetReferenceVector( 
    const IVAS_VECTOR3 refPos                       /* i  : Reference position                                  */

);



#ifndef FIX_1494_SET_SPLITBFI_UNUSED

ivas_error IVAS_REND_SetSplitRendBFI(

    IVAS_REND_HANDLE hIvasRend,

    const Word16 bfi);

#endif



ivas_error IVAS_REND_SetExternalOrientation(

    IVAS_REND_HANDLE hIvasRend,                     /* i/o: Renderer handle                                                 */

lib_rend/lib_rend_fx.c

+9 −0
Original line number Diff line number Diff line
@@ -94,7 +94,9 @@ typedef struct 
    const EFAP_WRAPPER *pEfapOutWrapper;

    IVAS_REND_HeadRotData *pHeadRotData; // for now removing the const qualifier TODO: will modify later

    const RENDER_CONFIG_HANDLE *hhRendererConfig;

#ifndef FIX_1494_SET_SPLITBFI_UNUSED

    const Word16 *pSplitRendBFI;

#endif

    const SPLIT_REND_WRAPPER *pSplitRendWrapper;

    const COMBINED_ORIENTATION_HANDLE *pCombinedOrientationData;

    const IVAS_DefaultReverbSize *pSelectedRoomReverbSize;
@@ -216,7 +218,9 @@ struct IVAS_REND 
    EFAP_WRAPPER efapOutWrapper;

    IVAS_LSSETUP_CUSTOM_STRUCT customLsOut;



#ifndef FIX_1494_SET_SPLITBFI_UNUSED

    Word16 splitRendBFI;

#endif

    SPLIT_REND_WRAPPER *splitRendWrapper;

    IVAS_REND_AudioBuffer splitRendEncBuffer;

    IVAS_REND_HeadRotData headRotData;
@@ -1365,7 +1369,9 @@ static rendering_context getRendCtx( 
    ctx.pEfapOutWrapper = &hIvasRend->efapOutWrapper;

    ctx.pHeadRotData = &hIvasRend->headRotData;

    ctx.hhRendererConfig = &hIvasRend->hRendererConfig;

#ifndef FIX_1494_SET_SPLITBFI_UNUSED

    ctx.pSplitRendBFI = &hIvasRend->splitRendBFI;

#endif

    ctx.pSplitRendWrapper = hIvasRend->splitRendWrapper;

    ctx.pCombinedOrientationData = &hIvasRend->hCombinedOrientationData;

    ctx.pSelectedRoomReverbSize = &hIvasRend->selectedRoomReverbSize;
@@ -5758,6 +5764,7 @@ ivas_error IVAS_REND_DisableHeadRotation( 
    return IVAS_ERR_OK;

}



#ifndef FIX_1494_SET_SPLITBFI_UNUSED



/*-------------------------------------------------------------------*

 * IVAS_REND_SetSplitRendBFI()
@@ -5775,6 +5782,8 @@ ivas_error IVAS_REND_SetSplitRendBFI( 
    return IVAS_ERR_OK;

}



#endif





/*-------------------------------------------------------------------*

 * IVAS_REND_SetOrientationTrackingMode()