From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from walk.intel-email.com (walk.intel-email.com [101.227.64.242]) by mx.groups.io with SMTP id smtpd.web10.1296.1663046083512019966 for ; Mon, 12 Sep 2022 22:14:44 -0700 Authentication-Results: mx.groups.io; dkim=pass header.i=@byosoft.com.cn header.s=cloud-union header.b=Y5viU98d; spf=pass (domain: byosoft.com.cn, ip: 101.227.64.242, mailfrom: gaoliming@byosoft.com.cn) Received: from walk.intel-email.com (localhost [127.0.0.1]) by walk.intel-email.com (Postfix) with ESMTP id 2917BCD1F7F5 for ; Tue, 13 Sep 2022 13:14:41 +0800 (CST) DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=byosoft.com.cn; s=cloud-union; t=1663046081; bh=QX+VbFZVNg2LkvmvvNXN6IqSZOfi80/Uhkj7Rsy16eY=; h=From:To:Cc:References:In-Reply-To:Subject:Date; b=Y5viU98d6o+2KD540NRQ6ikKtuz5XdGW9XhL7CNyPT+AjvONJdoZqU0enzG/bqP4v jCuPvoieru5DvkTlQe0WMq9IC1Q3EMMvAmLvqi3q6CSPU8jSJeGVgy87sAseCsc+/4 yehYRoaCd0kmGXVSiqbjgtUOXvRMZNYhgE8BBPUM= Received: from localhost (localhost [127.0.0.1]) by walk.intel-email.com (Postfix) with ESMTP id 248A3CD1F7F4 for ; Tue, 13 Sep 2022 13:14:41 +0800 (CST) X-Virus-Scanned: by SpamTitan at intel-email.com Received: from walk.intel-email.com (localhost [127.0.0.1]) by walk.intel-email.com (Postfix) with ESMTP id E15F8CD1F7ED for ; Tue, 13 Sep 2022 13:14:40 +0800 (CST) Authentication-Results: walk.intel-email.com; none Received: from mail.byosoft.com.cn (mail.byosoft.com.cn [58.240.74.242]) by walk.intel-email.com (Postfix) with SMTP id 75F81CD1F783 for ; Tue, 13 Sep 2022 13:14:37 +0800 (CST) Received: from DESKTOPS6D0PVI ([58.246.60.130]) (envelope-sender ) by 192.168.6.13 with ESMTP for ; Tue, 13 Sep 2022 13:14:31 +0800 X-WM-Sender: gaoliming@byosoft.com.cn X-Originating-IP: 58.246.60.130 X-WM-AuthFlag: YES X-WM-AuthUser: gaoliming@byosoft.com.cn From: "gaoliming" To: , Cc: "'Vasudevan Sambandan'" , "'Sundaresan S'" References: <20220802040020.631-1-sivaparvathic@ami.com> In-Reply-To: <20220802040020.631-1-sivaparvathic@ami.com> Subject: =?UTF-8?B?5Zue5aSNOiBbZWRrMi1kZXZlbF0gW1BBVENIXSBbUEFUQ0hdTWRlTW9kdWxlUGtnL1VmczogQ292ZXJpdHkgc2NhbiBmbGFncyBtdWx0aXBsZSBpc3N1ZXMgaW4gZWRrMi1zdGFibGUyMDIyMDUgU2lnbmVkLW9mZi1ieTogc2l2YXBhcnZhdGhpY0BhbWkuY29t?= Date: Tue, 13 Sep 2022 13:14:33 +0800 Message-ID: <033001d8c72f$b5b99b60$212cd220$@byosoft.com.cn> MIME-Version: 1.0 X-Mailer: Microsoft Outlook 16.0 Thread-Index: AQHSqGGl7SxmgiqXTWBDvThNSRPjSq3pDo1w Sender: "gaoliming" Content-Type: text/plain; charset="gb2312" Content-Transfer-Encoding: quoted-printable Content-Language: zh-cn Reviewed-by: Liming Gao > -----=D3=CA=BC=FE=D4=AD=BC=FE----- > =B7=A2=BC=FE=C8=CB: devel@edk2.groups.io =B4=FA=B1= =ED sivaparvathi > C via groups.io > =B7=A2=CB=CD=CA=B1=BC=E4: 2022=C4=EA8=D4=C22=C8=D5 12:02 > =CA=D5=BC=FE=C8=CB: devel@edk2.groups.io; Sivaparvathi Chellaiah > > =B3=AD=CB=CD: Vasudevan Sambandan ; Sundaresan S > > =D6=F7=CC=E2: [edk2-devel] [PATCH] [PATCH]MdeModulePkg/Ufs: Coverity scan= flags > multiple issues in edk2-stable202205 Signed-off-by: sivaparvathic@ami.com >=20 > --- > MdeModulePkg/Bus/Ufs/UfsBlockIoPei/DmaMem.c | 18 > ++++++++++++------ > MdeModulePkg/Bus/Ufs/UfsBlockIoPei/UfsHcMem.c | 3 +++ > 2 files changed, 15 insertions(+), 6 deletions(-) >=20 > diff --git a/MdeModulePkg/Bus/Ufs/UfsBlockIoPei/DmaMem.c > b/MdeModulePkg/Bus/Ufs/UfsBlockIoPei/DmaMem.c > index 62aeaf86c8..b4b0f07d5e 100644 > --- a/MdeModulePkg/Bus/Ufs/UfsBlockIoPei/DmaMem.c > +++ b/MdeModulePkg/Bus/Ufs/UfsBlockIoPei/DmaMem.c > @@ -240,10 +240,16 @@ IoMmuInit ( > VOID >=20 > ) >=20 > { >=20 > - PeiServicesLocatePpi ( >=20 > - &gEdkiiIoMmuPpiGuid, >=20 > - 0, >=20 > - NULL, >=20 > - (VOID **)&mIoMmu >=20 > - ); >=20 > + EFI_STATUS Status; >=20 > + >=20 > + Status =3D PeiServicesLocatePpi ( >=20 > + &gEdkiiIoMmuPpiGuid, >=20 > + 0, >=20 > + NULL, >=20 > + (VOID **)&mIoMmu >=20 > + ); >=20 > + >=20 > + if (EFI_ERROR(Status)) { >=20 > + DEBUG ((DEBUG_INFO, "Locate mIoMmu Ppi is failed!!!\n")); >=20 > + } >=20 > } >=20 > diff --git a/MdeModulePkg/Bus/Ufs/UfsBlockIoPei/UfsHcMem.c > b/MdeModulePkg/Bus/Ufs/UfsBlockIoPei/UfsHcMem.c > index b7dd9b6acd..b5f0324908 100644 > --- a/MdeModulePkg/Bus/Ufs/UfsBlockIoPei/UfsHcMem.c > +++ b/MdeModulePkg/Bus/Ufs/UfsBlockIoPei/UfsHcMem.c > @@ -414,6 +414,9 @@ UfsPeimFreeMem ( > // >=20 > ASSERT (Block !=3D NULL); >=20 >=20 >=20 > + if (Block =3D=3D NULL) { >=20 > + return; >=20 > + } >=20 > // >=20 > // Release the current memory block if it is empty and not the head >=20 > // >=20 > -- > 2.31.0.windows.1 > -The information contained in this message may be confidential and > proprietary to American Megatrends (AMI). This communication is intended to > be read only by the individual or entity to whom it is addressed or by their > designee. If the reader of this message is not the intended recipient, yo= u are > on notice that any distribution of this message, in any form, is strictly > prohibited. Please promptly notify the sender by reply e-mail or by telephone > at 770-246-8600, and then delete or destroy all copies of the transmission. >=20 >=20 >=20 >=20