Reviewed-by: Liming Gao <liming....@intel.com> > -----Original Message----- > From: Feng, YunhuaX <yunhuax.f...@intel.com> > Sent: Friday, February 14, 2020 5:15 PM > To: devel@edk2.groups.io > Cc: Gao, Liming <liming....@intel.com>; Feng, Bob C <bob.c.f...@intel.com>; > Zhang, Hongbin1 <hongbin1.zh...@intel.com> > Subject: [edk2-staging/FceFmmt][PATCH] BaseTools/FCE: Fix multiple FVs with > SEC module issue > > REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2527 > > One BIOS image may have more than FVs with SEC module. > FCE tool should find the latest FV with SEC module as BFV image. > > Cc: Bob Feng <bob.c.f...@intel.com> > Cc: Liming Gao <liming....@intel.com> > Signed-off-by: Yunhua Feng<yunhuax.f...@intel.com> > --- > BaseTools/Source/C/BfmLib/BinFileManager.c | 11 +++++++++++ > 1 file changed, 11 insertions(+) > > diff --git a/BaseTools/Source/C/BfmLib/BinFileManager.c > b/BaseTools/Source/C/BfmLib/BinFileManager.c > index 9b4e0929f5..69521044b8 100644 > --- a/BaseTools/Source/C/BfmLib/BinFileManager.c > +++ b/BaseTools/Source/C/BfmLib/BinFileManager.c > @@ -251,10 +251,11 @@ BfmImageAdd ( > BOOLEAN FfsLevelFoundFlag; > CHAR8 *OutputFileName; > CHAR8 *FvId; > BOOLEAN FirstInFlag; > BOOLEAN FvGuidExisted; > + FV_INFORMATION *FvInFdTmp; > > NewFvLength = 0; > FvEncapLevel = 0; > NewAddedFfsLevel = 0; > > @@ -269,10 +270,11 @@ BfmImageAdd ( > TemDir = NULL; > LocalEncapDataTemp = NULL; > OutputFileName = NULL; > FvId = NULL; > FvGuidExisted = FALSE; > + FvInFdTmp = NULL; > > // > // Get the size of ffs file to be inserted. > // > NewFfsLength = GetFileSize(NewFile); > @@ -306,10 +308,19 @@ BfmImageAdd ( > // > FvInFd = FdData->Fv; > do { > if ((FvGuidExisted && mFvGuidIsSet && FvInFd->IsInputFvFlag) || > ((!FvGuidExisted || (!mFvGuidIsSet)) && FvInFd->IsBfvFlag)) { > > + if (FvInFd->IsBfvFlag) { > + FvInFdTmp = FdData->Fv; > + while (FvInFdTmp != NULL) { > + if (FvInFdTmp->IsBfvFlag) { > + FvInFd = FvInFdTmp; > + } > + FvInFdTmp = FvInFdTmp->FvNext; > + } > + } > Status = LibLocateBfv (FdData, &FvId, &FvInFd); > > if (EFI_ERROR (Status)) { > printf("Error while locate BFV from FD.\n"); > LibBfmFreeFd(FdData); > -- > 2.12.2.windows.2
-=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#54638): https://edk2.groups.io/g/devel/message/54638 Mute This Topic: https://groups.io/mt/71274996/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-