]> git.proxmox.com Git - mirror_edk2.git/commit - MdeModulePkg/Core/Pei/FwVol/FwVol.c
MdeModulePkg PeiCore: Fix issue AuthenticationStatus is not propagated correctly
authorEugene Cohen <eugene@hp.com>
Tue, 10 Nov 2015 10:02:24 +0000 (10:02 +0000)
committerlzeng14 <lzeng14@Edk2>
Tue, 10 Nov 2015 10:02:24 +0000 (10:02 +0000)
commit9ddd7d7aec0eb25e078c9ca95329df06b4410169
tree213d08af81bf3a03440a3ae224209bd63fefb7cb
parent52a99493cce88a9d4ec8a02d7f1bd1a1001ce60d
MdeModulePkg PeiCore: Fix issue AuthenticationStatus is not propagated correctly

This patch fixes an issue in PEI with encapsulated FV images where the
AuthenticationStatus is not correctly propagated down to child FV
handles.  The PEI core registers for callbacks for both FvInfo and
FvInfo2 PPIs.  These callbacks process the FVs which will recurse as
necessary to find more encapsulated FVs.  (FvInfo2 is an updated PPI
that includes an AuthenticationStatus field - the original FvInfo did
not include this.)

When encapsulated FV processing occurs the PEI core installs both
FvInfo and FvInfo2 PPIs.  The original implementation installs FvInfo
first and FvInfo2 second.  As soon as the FvInfo PPI is installed the
notification callback handler immediately fires causing recursive FV
processing to occur.  Since there is no AuthenticationStatus provided
for the original FvInfo the callback assumes AuthenticationStatus is
zero (unsigned / unverified) even though the parent FV may have been
verified.

This changes the order of FvInfo and FvInfo2 installs to ensure that
the notification callback occurs for FvInfo2 first and appropriate
AuthenticationStatus data can be propagated from parent FV to child
FV.

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Eugene Cohen <eugene@hp.com>
Reviewed-by: Star Zeng <star.zeng@intel.com>
git-svn-id: https://svn.code.sf.net/p/edk2/code/trunk/edk2@18764 6f19259b-4bc3-4df7-8a09-765794883524
MdeModulePkg/Core/Pei/FwVol/FwVol.c