Push @17f8c9e97d770c74f84194576bcd97322fbed21e From: [email protected] [mailto:[email protected]] On Behalf Of Liming Gao Sent: Monday, September 2, 2019 12:15 PM To: [email protected]; [email protected] Subject: Re: [edk2-devel] Subject: [PATCH v2] MdePkg/DxeHstiLib: Added checks to improve error handling.
This change is good to me. Reviewed-by: Liming Gao <[email protected]<mailto:[email protected]>> From: [email protected]<mailto:[email protected]> [mailto:[email protected]] On Behalf Of [email protected]<mailto:[email protected]> Sent: Friday, August 30, 2019 12:32 AM To: [email protected]<mailto:[email protected]> Subject: [edk2-devel] Subject: [PATCH v2] MdePkg/DxeHstiLib: Added checks to improve error handling. Hi Liming, per your recommendation, Please review the below [PATCH v2] and the commit can be found in: [email protected]:JayRaghuram/edk2.git<mailto:[email protected]:JayRaghuram/edk2.git> (https://github.com/JayRaghuram/edk2) REF:https://bugzilla.tianocore.org/show_bug.cgi?id=2138 ________________________________ >From 95c48454d6d14cb7537dec98d58f7dc01efa644e Mon Sep 17 00:00:00 2001 Message-Id: <95c48454d6d14cb7537dec98d58f7dc01efa644e.1567095734.git.jayanth_raghu...@dell.com<mailto:95c48454d6d14cb7537dec98d58f7dc01efa644e.1567095734.git.jayanth_raghu...@dell.com>> From: Jayanth Raghuram <[email protected]<mailto:[email protected]>> Date: Thu, 29 Aug 2019 11:17:09 -0500 Subject: [PATCH v2] MdePkg/DxeHstiLib: Added checks to improve error handling. REF:https://bugzilla.tianocore.org/show_bug.cgi?id=2138 Added checks for return parameters of GetSupportedTypes function calls in InternalHstiFindAip to improve error handling. An issue was observed on Dell Poweredge R740, where the Dell PERC H740P controller UEFI driver returned InfoTypesBuffer = NULL, InfoTypesBufferCount = 0 and caused an FreePool assert. Signed-off-by: Jayanth Raghuram <[email protected]<mailto:[email protected]>> Cc: Wei G Liu <[email protected]<mailto:[email protected]>> --- MdePkg/Library/DxeHstiLib/HstiDxe.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/MdePkg/Library/DxeHstiLib/HstiDxe.c b/MdePkg/Library/DxeHstiLib/HstiDxe.c index 9466e3dcb3..4e1c67616b 100644 --- a/MdePkg/Library/DxeHstiLib/HstiDxe.c +++ b/MdePkg/Library/DxeHstiLib/HstiDxe.c @@ -77,7 +77,7 @@ InternalHstiFindAip ( &InfoTypesBuffer, &InfoTypesBufferCount ); - if (EFI_ERROR (Status)) { + if (EFI_ERROR (Status) || (InfoTypesBuffer == NULL) || (InfoTypesBufferCount == 0)) { continue; } -- 2.19.1.windows.1 ________________________________ Regards Jayanth Raghuram DellEMC | Server Platform BIOS office + 1 512 723 1470 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#46680): https://edk2.groups.io/g/devel/message/46680 Mute This Topic: https://groups.io/mt/33071679/21656 Group Owner: [email protected] Unsubscribe: https://edk2.groups.io/g/devel/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
