Hi Mike, All I did was copy the existing error handling in the scenario where we are unable to set up the buffer for any reason:

Hi Dionna, My mistake. I misread the UEFI 2.9 Spec content and thought they were both for exit boot services. Your patch series with only Before Exit Boot Services is appropriate for your

From: Michael Kubacki <michael.kubacki@...> As recommended by CodeQL this change replaces cpp/potential-buffer-overflow with cpp/overrunning-write-with-float and

From: Michael Kubacki <michael.kubacki@...> The previous commits fixed issues with these queries across various packages. Now that those are resolved, enable the queries in the edk2 query

From: Michael Kubacki <michael.kubacki@...> Fixes CodeQL alerts for CWE-457: https://cwe.mitre.org/data/definitions/457.html Cc: Erich McMillan <emcmillan@...> Cc: Michael

From: Michael Kubacki <michael.kubacki@...> Fixes CodeQL alerts for CWE-457: https://cwe.mitre.org/data/definitions/457.html Cc: Michael D Kinney <michael.d.kinney@...> Cc: Michael

From: Michael Kubacki <michael.kubacki@...> Fixes CodeQL alerts for CWE-457: https://cwe.mitre.org/data/definitions/457.html Cc: Michael D Kinney <michael.d.kinney@...> Cc: Michael

From: Michael Kubacki <michael.kubacki@...> Fixes CodeQL alerts for CWE-457: https://cwe.mitre.org/data/definitions/457.html Cc: Jiaxin Wu <jiaxin.wu@...> Cc: Michael D Kinney

From: Michael Kubacki <michael.kubacki@...> Fixes CodeQL alerts for CWE-457: https://cwe.mitre.org/data/definitions/457.html Cc: Liming Gao <gaoliming@...> Cc: Michael D Kinney

From: Michael Kubacki <michael.kubacki@...> Fixes CodeQL alerts for CWE-457: https://cwe.mitre.org/data/definitions/457.html Cc: Eric Dong <eric.dong@...> Cc: Guomin Jiang

From: Michael Kubacki <michael.kubacki@...> Fixes CodeQL alerts for CWE-457: https://cwe.mitre.org/data/definitions/457.html Checks the return value from `ASN1_get_object()` to verify

From: Michael Kubacki <michael.kubacki@...> While more portable methods exist to handle these cases, this change does not attempt to do more than fix the immediate problem and follow the

From: Michael Kubacki <michael.kubacki@...> Purdue Compiler Construction Tool Set (PCCTS) source code was copied/ pasted into BaseTools/Source/C/VfrCompile/Pccts/. The code contains tab

From: Erich McMillan <emcmillan@...> Details for these CodeQL alerts can be found here: - Pointer overflow check (cpp/pointer-overflow-check): -

From: Michael Kubacki <michael.kubacki@...> Adds queries for the following: 1. cpp/conditionallyuninitializedvariable 2. cpp/pointer-overflow-check 3. cpp/overrunning-write 4.

Dionna, I can create a new patch that contains, the two Event definitions and the code that signals both events. I can use my patches that I already have and then add the code from your submission

I assume you are talking about x86? This is more complex than just instruction prefix. You need the CPU to be setup in big real mode via the GDT, so you basically have a 32-bit environment setup via

I saw no such event in the specification I thought that's the event you meant, since its mantis number is 2042, whereas before exit boot services is 2043. That fits the "same time" you mentioned in

Hi Dionna, I gave feedback about After Exit Boot Services event. Why is an After Ready To Boot signal now part of this series? Mike

Will you please comment on the patches in question where the error is? I don't follow what you're saying. Sounds good. -- -Dionna Glaze, PhD (she/her)