The series looks good to me. Reviewed-by: Sunny Wang <sunny.w...@arm.com>
-----Original Message----- From: Dimitrije Pavlov <dimitrije.pav...@arm.com> Sent: 15 August 2022 17:31 To: devel@edk2.groups.io Cc: Ard Biesheuvel <ardb+tianoc...@kernel.org>; Jiewen Yao <jiewen....@intel.com>; Liming Gao <gaolim...@byosoft.com.cn>; Sunny Wang <sunny.w...@arm.com>; Jeff Booher-Kaeding <jeff.booher-kaed...@arm.com>; Samer El-Haj-Mahmoud <samer.el-haj-mahm...@arm.com> Subject: [PATCH v2 0/3] OvmfPkg: Check arguments for validity 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 EFI_INVALID_PARAMETER if a pointer argument is NULL. v2: - Add Liming Gao to Cc [Ard] - Turn individual patches into a series [Ard] - Fix issue with corrupted patches [Ard] Cc: Ard Biesheuvel <ardb+tianoc...@kernel.org> Cc: Jiewen Yao <jiewen....@intel.com> Cc: Liming Gao <gaolim...@byosoft.com.cn> Cc: Sunny Wang <sunny.w...@arm.com> Cc: Jeff Booher-Kaeding <jeff.booher-kaed...@arm.com> Cc: Samer El-Haj-Mahmoud <samer.el-haj-mahm...@arm.com> Dimitrije Pavlov (3): OvmfPkg/PlatformDxe: Check ExtractConfig and RouteConfig arguments OvmfPkg/VirtioGpuDxe: Check QueryMode arguments OvmfPkg/VirtioFsDxe: Check GetDriverName arguments OvmfPkg/PlatformDxe/Platform.c | 8 ++++++++ OvmfPkg/VirtioFsDxe/DriverBinding.c | 4 ++++ OvmfPkg/VirtioGpuDxe/Gop.c | 4 +++- 3 files changed, 15 insertions(+), 1 deletion(-) -- 2.37.2 IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you. -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#92475): https://edk2.groups.io/g/devel/message/92475 Mute This Topic: https://groups.io/mt/93040550/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
