The new changes in SecureBootVariableLib brought in a new dependency of PlatformPKProtectionLib. This change added the new library instance from SecurityPkg to resolve AmpereAltraPkg platforms

The new changes in SecureBootVariableLib brought in a new dependency of PlatformPKProtectionLib. This change added the new library instance from SecurityPkg to resolve U5SeriesPkg platforms

The new changes in SecureBootVariableLib brought in a new dependency of PlatformPKProtectionLib. This change added the new library instance from SecurityPkg to resolve RaspberryPi platforms

This v5 series is a follow up of previously submitted patches: https://edk2.groups.io/g/devel/message/92128 The v5 patches include a new `Reviewed-by` tag collected during the v4 round and skipped 2

Thank you for the resend. @Liming: these are all bug fixes that affect SCT results, so unless there are any objections, I intend to merge these tomorrow (Tuesday). Thanks, Ard.

Some functions across OVMF don't check pointer arguments for validity, which causes null pointer dereferences and crashes in the SCT test suite. This series adds checks to return

The current implementation does not check if Progress or Results pointers in ExtractConfig are NULL, or if Progress pointer in RouteConfig is NULL. This causes the SCT test suite to crash. Add a

The current implementation does not check if Language or DriverName are NULL. This causes the SCT test suite to crash. Add a check to return EFI_INVALID_PARAMETER if any of these pointers are

The current implementation does not check if Info or SizeInfo pointers are NULL. This causes the SCT test suite to crash. Add a check to return EFI_INVALID_PARAMETER if any of these pointers are

Hi Ray, How do you envision this library working with SEV and/or TDX which use the one of the bits in PageTableBaseAddress to indicate whether the page is encrypted or shared? For SEV, the encryption

Hello Dimitrije, These patches look corrupted again. Can you please double check your setup, and maybe compare notes with colleagues how to send patches to the list in a way that doesn't corrupt

The current implementation does not check if Progress or Results pointers in ExtractConfig are NULL, or if Progress pointer in RouteConfig is NULL. This causes the SCT test suite to crash. Add a

The current implementation does not check if Info or SizeInfo pointers are NULL. This causes the SCT test suite to crash. Add a check to return EFI_INVALID_PARAMETER if any of these pointers are

Acked-by: Gerd Hoffmann <kraxel@...>

The current implementation does not check if Language or DriverName are NULL. This causes the SCT test suite to crash. Add a check to return EFI_INVALID_PARAMETER if any of these pointers are

Acked-by: Gerd Hoffmann <kraxel@...> take care, Gerd

<jbrasen@...> wrote: What I was wondering is whether such allocations are themselves multiples of 64k. This is perhaps orthogonal to the issue this patch addresses, as we'' still need

Thanks for the fix Reviewed-by: Ard Biesheuvel <ardb@...> Pushed as 01e3a8877180..9675047d7245

Thanks for the fix Reviewed-by: Ard Biesheuvel <ardb@...> Pushed as 01e3a8877180..9675047d7245

The edk2 patch at: "6eb407947592 SecurityPkg: SecureBootVariableLib: Added newly supported interfaces" introduced a new dependency of PlatformPKProtectionLib to SecureBootVariableLib. This causes the