summaryrefslogtreecommitdiffstats
path: root/MdeModulePkg/Universal
diff options
context:
space:
mode:
authorJiewen Yao <jiewen.yao@intel.com>2019-12-31 09:02:52 +0800
committermergify[bot] <37929162+mergify[bot]@users.noreply.github.com>2020-01-06 06:23:19 +0000
commitf27390516198b7a39bf1d306327a6b4334788390 (patch)
tree3e9a345755b7302a067e8c484bf08c49a0a3556d /MdeModulePkg/Universal
parent4fa25853cdccc1e6bcb613180d730d010f6d6a6b (diff)
downloadedk2-f27390516198b7a39bf1d306327a6b4334788390.tar.gz
edk2-f27390516198b7a39bf1d306327a6b4334788390.tar.bz2
edk2-f27390516198b7a39bf1d306327a6b4334788390.zip
MdeModulePkg/Smbios: Done measure Smbios multiple times.
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2439 In current implementation, the SMBIOS table is measured multiple time in every readytoboot event. This causes Smbios Table record appears multiple time in the TCG event log and confuses people. This issue makes it hard to implement 800-155 reference measurement. This patch closes the event to make sure Smbios is measured only once. Cc: Jian J Wang <jian.j.wang@intel.com> Cc: Hao A Wu <hao.a.wu@intel.com> Cc: Dandan Bi <dandan.bi@intel.com> Cc: Star Zeng <star.zeng@intel.com> Signed-off-by: Jiewen Yao <jiewen.yao@intel.com> Reviewed-by: Star Zeng <star.zeng@intel.com>
Diffstat (limited to 'MdeModulePkg/Universal')
-rw-r--r--MdeModulePkg/Universal/SmbiosMeasurementDxe/SmbiosMeasurementDxe.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/MdeModulePkg/Universal/SmbiosMeasurementDxe/SmbiosMeasurementDxe.c b/MdeModulePkg/Universal/SmbiosMeasurementDxe/SmbiosMeasurementDxe.c
index 7b5d473146..5ec2aca095 100644
--- a/MdeModulePkg/Universal/SmbiosMeasurementDxe/SmbiosMeasurementDxe.c
+++ b/MdeModulePkg/Universal/SmbiosMeasurementDxe/SmbiosMeasurementDxe.c
@@ -577,8 +577,8 @@ MeasureSmbiosTable (
TableAddress, // HashData
TableLength // HashDataLen
);
- if (EFI_ERROR (Status)) {
- return ;
+ if (!EFI_ERROR (Status)) {
+ gBS->CloseEvent (Event) ;
}
}