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]
-=-=-=-=-=-=-=-=-=-=-=-


Reply via email to