Hi Liming,

I sent the review request based on the description in the links that you mentioned below.

I sent it in an Dell Email since I cannot use GIT SMTP to send email out from our servers.

Please let me know what is wrong and I can help rectify that.

Regards

Jayanth Raghuram

DellEMC | Server Platform BIOS

office + 1 512 723 1470

From: Gao, Liming <liming.gao@intel.com>
Sent: Monday, August 26, 2019 8:57 PM
To: devel@edk2.groups.io; Raghuram, Jayanth
Cc: Liu, Wei G
Subject: RE: [PATCH] MdePkg/DxeHstiLib: Added checks to improve error handling.

[EXTERNAL EMAIL]

Can you follow this process to send this patch again?

https://github.com/tianocore/tianocore.github.io/wiki/EDK-II-Development-Process

And, the commit message format is

https://github.com/tianocore/tianocore.github.io/wiki/Commit-Message-Format

Thanks

Liming

From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of Jayanth.Raghuram@dell.com
Sent: Tuesday, August 27, 2019 3:55 AM
To: devel@edk2.groups.io
Cc: Wei.G.Liu@dell.com
Subject: [edk2-devel] [PATCH] MdePkg/DxeHstiLib: Added checks to improve error handling.

Subject: [PATCH] MdePkg/DxeHstiLib: Added checks to improve error handling.

Added checks for return parameters of LocateHandleBuffer & 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 <Jayanth.Raghuram@Dell.com>

Cc: Wei G Liu <Wei_G_Liu@Dell.com>

Attached: 0001-MdePkg-DxeHstiLib-Added-checks-to-improve-error-hand.patch

Regards

Jayanth Raghuram

DellEMC | Server Platform BIOS

office + 1 512 723 1470