summaryrefslogtreecommitdiffstats
path: root/StandaloneMmPkg
diff options
context:
space:
mode:
authorHongbin1 Zhang <hongbin1.zhang@intel.com>2025-01-10 21:25:15 +0800
committermergify[bot] <37929162+mergify[bot]@users.noreply.github.com>2025-01-13 06:59:28 +0000
commit8c180acf1db9404bef847d617e35788ed23b04de (patch)
treecb6f394be31f30e166c1bc14861df57777788d61 /StandaloneMmPkg
parent3c8f47bf2105374e9b0c3c0ea68cde6ceb0c06d1 (diff)
downloadedk2-8c180acf1db9404bef847d617e35788ed23b04de.tar.gz
StandaloneMmPkg/MmIpl: Correct FV HOB check founction
When at last hob, the FV HOB check function should exit from the loop Signed-off-by: Hongbin1 Zhang <hongbin1.zhang@intel.com> Cc: Jiewen Yao <jiewen.yao@intel.com> Cc: Ray Ni <ray.ni@intel.com> Cc: Star Zeng <star.zeng@intel.com> Cc: Jiaxin Wu <jiaxin.wu@intel.com> Cc: Wei6 Xu <wei6.xu@intel.com> Cc: Sami Mujawar <sami.mujawar@arm.com> Cc: Ard Biesheuvel <ardb+tianocore@kernel.org> Cc: Supreeth Venkatesh <supreeth.venkatesh@arm.com>
Diffstat (limited to 'StandaloneMmPkg')
-rw-r--r--StandaloneMmPkg/Drivers/StandaloneMmIplPei/MmFoundationHob.c9
1 files changed, 3 insertions, 6 deletions
diff --git a/StandaloneMmPkg/Drivers/StandaloneMmIplPei/MmFoundationHob.c b/StandaloneMmPkg/Drivers/StandaloneMmIplPei/MmFoundationHob.c
index 27103a4564..b4eb7bbcd5 100644
--- a/StandaloneMmPkg/Drivers/StandaloneMmIplPei/MmFoundationHob.c
+++ b/StandaloneMmPkg/Drivers/StandaloneMmIplPei/MmFoundationHob.c
@@ -823,24 +823,21 @@ IsFvHobExist (
)
{
EFI_PEI_HOB_POINTERS Hob;
- UINTN HobLength;
if ((HobList == NULL) || (HobSize == 0)) {
return FALSE;
}
- Hob.Raw = (UINT8 *)HobList;
- HobLength = GET_HOB_LENGTH (Hob);
//
// Parse the HOB list until end of list or matching type is found.
//
- while (HobLength <= HobSize) {
+ Hob.Raw = HobList;
+ while ((UINTN)(Hob.Raw - HobList) < HobSize) {
if (Hob.Header->HobType == EFI_HOB_TYPE_FV) {
return TRUE;
}
- Hob.Raw = GET_NEXT_HOB (Hob);
- HobLength += GET_HOB_LENGTH (Hob);
+ Hob.Raw = GET_NEXT_HOB (Hob);
}
return FALSE;