summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDionna Glaze <dionnaglaze@google.com>2023-02-14 23:07:19 +0000
committermergify[bot] <37929162+mergify[bot]@users.noreply.github.com>2023-02-15 00:08:00 +0000
commit1b5420e8071b4f9ba13136f19365542dfe66bf04 (patch)
tree61ab4ff3e2792a0d3802310327dbf72e7918e879
parentf67ec877041b18656fb0504ce86bd05af232747e (diff)
downloadedk2-1b5420e8071b4f9ba13136f19365542dfe66bf04.tar.gz
edk2-1b5420e8071b4f9ba13136f19365542dfe66bf04.tar.bz2
edk2-1b5420e8071b4f9ba13136f19365542dfe66bf04.zip
OvmfPkg/AmdSevDxe: Close mAcceptAllMemoryEvent
This event should only trigger once. It should be idempotent, but the allocation of the memory map itself is observable and can cause ExitBootServices to fail with a modified map key. Cc: Thomas Lendacky <Thomas.Lendacky@amd.com> Cc: Erdem Aktas <erdemaktas@google.com> Cc: James Bottomley <jejb@linux.ibm.com> Cc: Jiewen Yao <jiewen.yao@intel.com> Cc: Min Xu <min.m.xu@intel.com> Cc: Michael Roth <michael.roth@amd.com> Signed-off-by: Dionna Glaze <dionnaglaze@google.com> Reviewed-by: Ard Biesheuvel <ardb@kernel.org>
-rw-r--r--OvmfPkg/AmdSevDxe/AmdSevDxe.c1
1 files changed, 1 insertions, 0 deletions
diff --git a/OvmfPkg/AmdSevDxe/AmdSevDxe.c b/OvmfPkg/AmdSevDxe/AmdSevDxe.c
index 9c4e3bb406..a726498e27 100644
--- a/OvmfPkg/AmdSevDxe/AmdSevDxe.c
+++ b/OvmfPkg/AmdSevDxe/AmdSevDxe.c
@@ -130,6 +130,7 @@ AcceptAllMemory (
}
gBS->FreePool (AllDescMap);
+ gBS->CloseEvent (mAcceptAllMemoryEvent);
return Status;
}