On Tue, 16 Aug 2022 at 13:39, Sunny Wang <sunny.w...@arm.com> wrote: > > 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 >
I tried to push these but they failed in CI. Could you please have a look and respin with the reported issues addressed? Thanks. https://github.com/tianocore/edk2/pull/3211 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#92482): https://edk2.groups.io/g/devel/message/92482 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] -=-=-=-=-=-=-=-=-=-=-=-