diff options
author | Jiewen Yao <jiewen.yao@intel.com> | 2019-12-31 09:02:52 +0800 |
---|---|---|
committer | mergify[bot] <37929162+mergify[bot]@users.noreply.github.com> | 2020-01-06 06:23:19 +0000 |
commit | f27390516198b7a39bf1d306327a6b4334788390 (patch) | |
tree | 3e9a345755b7302a067e8c484bf08c49a0a3556d /MdeModulePkg/Universal/SmbiosMeasurementDxe | |
parent | 4fa25853cdccc1e6bcb613180d730d010f6d6a6b (diff) | |
download | edk2-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/SmbiosMeasurementDxe')
-rw-r--r-- | MdeModulePkg/Universal/SmbiosMeasurementDxe/SmbiosMeasurementDxe.c | 4 |
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) ;
}
}
|