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 V3: 1. Separate patch per individual issue fix on patch V2. 2. Fix one more issue: Limit FwVol encapsulation section recursion in MmCoreFfsFindMmDriver(). PR: https://github.com/tianocore/edk2/pull/4975 Cc: Laszlo Ersek <ler...@redhat.com> Cc: Ard Biesheuvel <ardb+tianoc...@kernel.org> Cc: Sami Mujawar <sami.muja...@arm.com> Cc: Ray Ni <ray...@intel.com> Wei6 Xu (4): StandaloneMmPkg/Core: Limit FwVol encapsulation section recursion StandaloneMmPkg/Core: Fix potential memory leak issue StandaloneMmPkg/Core: Fix issue that section address might be wrong StandaloneMmPkg/Core: Fix the failure to find uncompressed inner FV StandaloneMmPkg/Core/FwVol.c | 50 ++++++++++++++++++----- StandaloneMmPkg/Core/StandaloneMmCore.c | 5 ++- StandaloneMmPkg/Core/StandaloneMmCore.inf | 3 ++ StandaloneMmPkg/StandaloneMmPkg.dec | 5 +++ 4 files changed, 51 insertions(+), 12 deletions(-) -- 2.29.2.windows.2 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#110296): https://edk2.groups.io/g/devel/message/110296 Mute This Topic: https://groups.io/mt/102270545/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-