From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: intel.com, ip: 134.134.136.24, mailfrom: liming.gao@intel.com) Received: from mga09.intel.com (mga09.intel.com [134.134.136.24]) by groups.io with SMTP; Wed, 28 Aug 2019 00:59:04 -0700 X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga005.jf.intel.com ([10.7.209.41]) by orsmga102.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 28 Aug 2019 00:59:03 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.64,440,1559545200"; d="scan'208,217";a="356028684" Received: from fmsmsx104.amr.corp.intel.com ([10.18.124.202]) by orsmga005.jf.intel.com with ESMTP; 28 Aug 2019 00:59:03 -0700 Received: from fmsmsx606.amr.corp.intel.com (10.18.126.86) by fmsmsx104.amr.corp.intel.com (10.18.124.202) with Microsoft SMTP Server (TLS) id 14.3.439.0; Wed, 28 Aug 2019 00:59:03 -0700 Received: from fmsmsx606.amr.corp.intel.com (10.18.126.86) by fmsmsx606.amr.corp.intel.com (10.18.126.86) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.1713.5; Wed, 28 Aug 2019 00:59:02 -0700 Received: from shsmsx103.ccr.corp.intel.com (10.239.4.69) by fmsmsx606.amr.corp.intel.com (10.18.126.86) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256) id 15.1.1713.5 via Frontend Transport; Wed, 28 Aug 2019 00:59:02 -0700 Received: from shsmsx104.ccr.corp.intel.com ([169.254.5.112]) by SHSMSX103.ccr.corp.intel.com ([169.254.4.139]) with mapi id 14.03.0439.000; Wed, 28 Aug 2019 15:59:01 +0800 From: "Liming Gao" To: "Jayanth.Raghuram@dell.com" , "devel@edk2.groups.io" CC: "Wei.G.Liu@dell.com" Subject: Re: [PATCH] MdePkg/DxeHstiLib: Added checks to improve error handling. Thread-Topic: [PATCH] MdePkg/DxeHstiLib: Added checks to improve error handling. Thread-Index: AdVcR+OPrxJmOcouRhWzA1r7t1m4TQAMoO6AACKbwDAAHEv/sA== Date: Wed, 28 Aug 2019 07:59:00 +0000 Message-ID: <4A89E2EF3DFEDB4C8BFDE51014F606A14E4DD7DA@SHSMSX104.ccr.corp.intel.com> References: <31e5cee5c49d4381bfb26a0968c11749@ausx13mps324.AMER.DELL.COM> <4A89E2EF3DFEDB4C8BFDE51014F606A14E4DCA9B@SHSMSX104.ccr.corp.intel.com> In-Reply-To: Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.239.127.40] MIME-Version: 1.0 Return-Path: liming.gao@intel.com Content-Language: en-US Content-Type: multipart/alternative; boundary="_000_4A89E2EF3DFEDB4C8BFDE51014F606A14E4DD7DASHSMSX104ccrcor_" --_000_4A89E2EF3DFEDB4C8BFDE51014F606A14E4DD7DASHSMSX104ccrcor_ Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable OK. So, you can't use git send-email to send this patch. Another way is to = fork edk2 and create the branch to include this change. Then, send the mail to let people review this patch in your branch. And, for this patch, can you submit BZ https://bugzilla.tianocore.org/ fir= st? Then, update its commit message format based on this wiki. https://github.com/tianocore/tianocore.github.io/wiki/Commit-Message-Forma= t Thanks Liming From: Jayanth.Raghuram@dell.com [mailto:Jayanth.Raghuram@dell.com] Sent: Wednesday, August 28, 2019 2:30 AM To: Gao, Liming ; devel@edk2.groups.io Cc: Wei.G.Liu@dell.com Subject: RE: [PATCH] MdePkg/DxeHstiLib: Added checks to improve error hand= ling. Hi Liming, I sent the review request based on the description in the links that you m= entioned below. I sent it in an Dell Email since I cannot use GIT SMTP to send email out f= rom 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 > 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 hand= ling. [EXTERNAL EMAIL] Can you follow this process to send this patch again? https://github.com/tianocore/tianocore.github.io/wiki/EDK-II-Development-P= rocess And, the commit message format is https://github.com/tianocore/tianocore.github.io/wiki/Commit-Message-Forma= t 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 e= rror handling. Subject: [PATCH] MdePkg/DxeHstiLib: Added checks to improve error handling= . Added checks for return parameters of LocateHandleBuffer & GetSupportedTyp= es 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 =3D NULL, InfoTypesBufferCount =3D 0 and c= aused an FreePool assert. Signed-off-by: Jayanth Raghuram > Cc: Wei G Liu > Attached: 0001-MdePkg-DxeHstiLib-Added-checks-to-improve-error-hand.patch Regards Jayanth Raghuram DellEMC | Server Platform BIOS office + 1 512 723 1470 --_000_4A89E2EF3DFEDB4C8BFDE51014F606A14E4DD7DASHSMSX104ccrcor_ Content-Type: text/html; charset="us-ascii" Content-Transfer-Encoding: quoted-printable

OK. So, you can’= ;t use git send-email to send this patch. Another way is to fork edk2 and c= reate the branch to include this change.

Then, send the mail t= o let people review this patch in your branch.

 

And, for this patch, = can you submit BZ https://bugzilla.tianocore.org= / first?

Then, update its comm= it message format based on this wiki.

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

 

Thanks

Liming

From: Jayanth.Raghuram@dell.com [mailto:Jaya= nth.Raghuram@dell.com]
Sent: Wednesday, August 28, 2019 2:30 AM
To: Gao, Liming <liming.gao@intel.com>; devel@edk2.groups.io<= br> Cc: Wei.G.Liu@dell.com
Subject: RE: [PATCH] MdePkg/DxeHstiLib: Added checks to improve err= or handling.

 

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 S= MTP to send email out from our servers.

Please let me know what is wrong and I can help rec= tify that.

 

Regards

Jayanth Raghuram=

DellEMC | Server Platform BIOS

office + 1 512 723 1470

 

 

[EXTERNAL EMAIL]

Can you follow this p= rocess to send this patch again?

htt= ps://github.com/tianocore/tianocore.github.io/wiki/EDK-II-Development-Proce= ss

 

And, the commit messa= ge 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.R= aghuram@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 im= prove error handling.

 

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

Added checks for return parameters of LocateHandleB= uffer & GetSupportedTypes

function calls in InternalHstiFindAip to improve er= ror handling. An issue was

observed on Dell Poweredge R740, where the Dell PER= C H740P controller UEFI

driver returned InfoTypesBuffer =3D NULL, InfoTypes= BufferCount =3D 0 and caused

an FreePool assert.

 

Signed-off-by: Jayanth Raghuram <Jayanth.Raghuram@Dell.com><= /p>

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

 

Attached: 0001-MdePkg-DxeHstiLib-Added-checks-to-im= prove-error-hand.patch

 

Regards

Jayanth Raghuram=

DellEMC | Server Platform BIOS

office + 1 512 723 1470

 

--_000_4A89E2EF3DFEDB4C8BFDE51014F606A14E4DD7DASHSMSX104ccrcor_--