Wei, Thanks for fixing the 3 issues. Can you kindly separate the one patch to at least 2 patches? One patch is to fix minor issues. The other is to add support of nested uncompressed FV. Thanks, Ray ________________________________ From: Xu, Wei6 Sent: Friday, October 27, 2023 8:59 AM To: devel@edk2.groups.io Cc: Xu, Wei6 ; Laszlo Ersek ; Ard Biesheuvel ; Sami Mujawar ; Ni, Ray Subject: [PATCH v2 0/1] StandaloneMmCore finds drivers in uncompressed inner fv. V1: This patch is to fix the issue that StandaloneMmCore fails to detect uncompressed inner FV. PR: https://github.com/tianocore/edk2/pull/4943 V2: Based on V1, fix some other issues 1. Add Missing object size checks before casting pointers to header types a. InnerFvHeader = (EFI_FIRMWARE_VOLUME_HEADER *)SectionData; This is introduced in V1, add the size check on SectionDataSize against EFI_FIRMWARE_VOLUME_HEADER b. Section = (EFI_COMMON_SECTION_HEADER *)(FileHeader + 1); Use FfsFindSection instead of FfsFindSectionData to avoid pointer casting. 2. Fix potential memory leak issue that ScratchBuffer is not freed when page allocation for DstBuffer fails. PR: https://github.com/tianocore/edk2/pull/4965 Cc: Laszlo Ersek Cc: Ard Biesheuvel Cc: Sami Mujawar Cc: Ray Ni Wei6 Xu (1): StandaloneMmPkg: Fix some issues in function MmCoreFfsFindMmDriver. StandaloneMmPkg/Core/FwVol.c | 34 ++++++++++++++++++++++++++-------- 1 file changed, 26 insertions(+), 8 deletions(-) -- 2.29.2.windows.2 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#110189): https://edk2.groups.io/g/devel/message/110189 Mute This Topic: https://groups.io/mt/102212657/7686176 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/leave/12367111/7686176/1913456212/xyzzy [rebecca@openfw.io] -=-=-=-=-=-=-=-=-=-=-=-