[edk2-devel] [PATCH] [PATCH]MdeModulePkg/Ufs: Coverity scan flags multiple issues in edk2-stable202205 Signed-off-by: sivaparvathic@ami.com

sivaparvathi C via groups.io posted 1 patch 1 year, 8 months ago
Failed in applying to current master (apply log)
MdeModulePkg/Bus/Ufs/UfsBlockIoPei/DmaMem.c   | 18 ++++++++++++------
MdeModulePkg/Bus/Ufs/UfsBlockIoPei/UfsHcMem.c |  3 +++
2 files changed, 15 insertions(+), 6 deletions(-)
[edk2-devel] [PATCH] [PATCH]MdeModulePkg/Ufs: Coverity scan flags multiple issues in edk2-stable202205 Signed-off-by: sivaparvathic@ami.com
Posted by sivaparvathi C via groups.io 1 year, 8 months ago
---
 MdeModulePkg/Bus/Ufs/UfsBlockIoPei/DmaMem.c   | 18 ++++++++++++------
 MdeModulePkg/Bus/Ufs/UfsBlockIoPei/UfsHcMem.c |  3 +++
 2 files changed, 15 insertions(+), 6 deletions(-)

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

   )

 {

-  PeiServicesLocatePpi (

-    &gEdkiiIoMmuPpiGuid,

-    0,

-    NULL,

-    (VOID **)&mIoMmu

-    );

+    EFI_STATUS                Status;

+

+    Status =   PeiServicesLocatePpi (

+                    &gEdkiiIoMmuPpiGuid,

+                    0,

+                    NULL,

+                    (VOID **)&mIoMmu

+                    );

+

+    if (EFI_ERROR(Status)) {

+        DEBUG ((DEBUG_INFO, "Locate mIoMmu Ppi is failed!!!\n"));

+    }

 }

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 (
   //

   ASSERT (Block != NULL);



+  if (Block == NULL) {

+      return;

+  }

   //

   // Release the current memory block if it is empty and not the head

   //

--
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, you 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.


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#92040): https://edk2.groups.io/g/devel/message/92040
Mute This Topic: https://groups.io/mt/92765057/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-
回复: [edk2-devel] [PATCH] [PATCH]MdeModulePkg/Ufs: Coverity scan flags multiple issues in edk2-stable202205 Signed-off-by: sivaparvathic@ami.com
Posted by gaoliming via groups.io 1 year, 6 months ago
Reviewed-by: Liming Gao <gaoliming@byosoft.com.cn>

> -----邮件原件-----
> 发件人: devel@edk2.groups.io <devel@edk2.groups.io> 代表 sivaparvathi
> C via groups.io
> 发送时间: 2022年8月2日 12:02
> 收件人: devel@edk2.groups.io; Sivaparvathi Chellaiah
> <sivaparvathic@ami.com>
> 抄送: Vasudevan Sambandan <vasudevans@ami.com>; Sundaresan S
> <sundaresans@ami.com>
> 主题: [edk2-devel] [PATCH] [PATCH]MdeModulePkg/Ufs: Coverity scan flags
> multiple issues in edk2-stable202205 Signed-off-by: sivaparvathic@ami.com
> 
> ---
>  MdeModulePkg/Bus/Ufs/UfsBlockIoPei/DmaMem.c   | 18
> ++++++++++++------
>  MdeModulePkg/Bus/Ufs/UfsBlockIoPei/UfsHcMem.c |  3 +++
>  2 files changed, 15 insertions(+), 6 deletions(-)
> 
> 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
> 
>    )
> 
>  {
> 
> -  PeiServicesLocatePpi (
> 
> -    &gEdkiiIoMmuPpiGuid,
> 
> -    0,
> 
> -    NULL,
> 
> -    (VOID **)&mIoMmu
> 
> -    );
> 
> +    EFI_STATUS                Status;
> 
> +
> 
> +    Status =   PeiServicesLocatePpi (
> 
> +                    &gEdkiiIoMmuPpiGuid,
> 
> +                    0,
> 
> +                    NULL,
> 
> +                    (VOID **)&mIoMmu
> 
> +                    );
> 
> +
> 
> +    if (EFI_ERROR(Status)) {
> 
> +        DEBUG ((DEBUG_INFO, "Locate mIoMmu Ppi is failed!!!\n"));
> 
> +    }
> 
>  }
> 
> 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 (
>    //
> 
>    ASSERT (Block != NULL);
> 
> 
> 
> +  if (Block == NULL) {
> 
> +      return;
> 
> +  }
> 
>    //
> 
>    // Release the current memory block if it is empty and not the head
> 
>    //
> 
> --
> 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, you
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.
> 
> 
> 
> 





-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#93694): https://edk2.groups.io/g/devel/message/93694
Mute This Topic: https://groups.io/mt/93649766/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-
Re: [edk2-devel] 回复: [edk2-devel] [PATCH] [PATCH]MdeModulePkg/Ufs: Coverity scan flags multiple issues in edk2-stable202205 Signed-off-by: sivaparvathic@ami.com
Posted by sivaparvathi C via groups.io 1 year, 5 months ago
On Tue, Sep 13, 2022 at 10:44 AM, gaoliming wrote:

> 
> gaoliming

Hi GaoLiming,

Could you please review the changes?

Thanks,
Sivaparvathi


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#94782): https://edk2.groups.io/g/devel/message/94782
Mute This Topic: https://groups.io/mt/93649766/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-


回复: [edk2-devel] 回复: [edk2-devel] [PATCH] [PATCH]MdeModulePkg/Ufs: Coverity scan flags multiple issues in edk2-stable202205 Signed-off-by: sivaparvathic@ami.com
Posted by gaoliming via groups.io 1 year, 5 months ago
I have gave my reviewed-by https://edk2.groups.io/g/devel/message/93694. 

 

But, I can’t get this patch from the mail. Can you share this patch in Pull Request or your fork GitHub Edk2 repo branch?

 

Thanks

Liming

发件人: devel@edk2.groups.io <devel@edk2.groups.io> 代表 sivaparvathi C via groups.io
发送时间: 2022年10月6日 16:17
收件人: gaoliming <gaoliming@byosoft.com.cn>; devel@edk2.groups.io
主题: Re: [edk2-devel] 回复: [edk2-devel] [PATCH] [PATCH]MdeModulePkg/Ufs: Coverity scan flags multiple issues in edk2-stable202205 Signed-off-by: sivaparvathic@ami.com

 

On Tue, Sep 13, 2022 at 10:44 AM, gaoliming wrote:

gaoliming

Hi GaoLiming,

Could you please review the changes?

Thanks,
Sivaparvathi  





-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#94830): https://edk2.groups.io/g/devel/message/94830
Mute This Topic: https://groups.io/mt/94193359/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-


Re: [edk2-devel] 回复: [edk2-devel] 回复: [edk2-devel] [PATCH] [PATCH]MdeModulePkg/Ufs: Coverity scan flags multiple issues in edk2-stable202205 Signed-off-by: sivaparvathic@ami.com
Posted by sivaparvathi C via groups.io 1 year, 5 months ago
Hi mikuback@linux.microsoft.com , gaoliming@byosoft.com.cn , michael.d.kinney@intel.com

Created PULL request for edk2_stable202205 coverity issue changes for Ufs driver.
https://github.com/tianocore/edk2/pull/3472

Thanks,
Sivaparvathi C


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#95147): https://edk2.groups.io/g/devel/message/95147
Mute This Topic: https://groups.io/mt/94193359/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-


回复: [edk2-devel] 回复: [edk2-devel] 回复: [edk2-devel] [PATCH] [PATCH]MdeModulePkg/Ufs: Coverity scan flags multiple issues in edk2-stable202205 Signed-off-by: sivaparvathic@ami.com
Posted by gaoliming via groups.io 1 year, 5 months ago
Sivaparvathi:

 You can check this wiki https://github.com/tianocore/tianocore.github.io/wiki/EDK-II-Code-Formatting to know how to check edk2 CI failures. 

 

Thanks

Liming

发件人: sivaparvathic via groups.io <sivaparvathic=ami.com@groups.io> 
发送时间: 2022年10月13日 19:18
收件人: gaoliming <gaoliming@byosoft.com.cn>; devel@edk2.groups.io
主题: Re: [edk2-devel] 回复: [edk2-devel] 回复: [edk2-devel] [PATCH] [PATCH]MdeModulePkg/Ufs: Coverity scan flags multiple issues in edk2-stable202205 Signed-off-by: sivaparvathic@ami.com

 

Hi  <mailto:mikuback@linux.microsoft.com> mikuback@linux.microsoft.com,  <mailto:gaoliming@byosoft.com.cn> gaoliming@byosoft.com.cn, <mailto:michael.d.kinney@intel.com> michael.d.kinney@intel.com


Created PULL request for edk2_stable202205 coverity issue changes for Ufs driver.
https://github.com/tianocore/edk2/pull/3472

Thanks,
Sivaparvathi C 



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#95282): https://edk2.groups.io/g/devel/message/95282
Mute This Topic: https://groups.io/mt/94378547/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-


Re: [edk2-devel] 回复: [edk2-devel] 回复: [edk2-devel] 回复: [edk2-devel] [PATCH] [PATCH]MdeModulePkg/Ufs: Coverity scan flags multiple issues in edk2-stable202205 Signed-off-by: sivaparvathic@ami.com
Posted by sivaparvathi C via groups.io 1 year, 5 months ago
Hi mikuback@linux.microsoft.com, gaoliming@byosoft.com.cn,michael.d.kinney@intel.com, vasudevans@ami.com, sundaresans@ami.com

Created PULL request for Coverity Issue changes.

UFS PR: https://github.com/tianocore/edk2/pull/3552
BugZilla ID: https://bugzilla.tianocore.org/show_bug.cgi?id=3989 ( https://bugzilla.tianocore.org/show_bug.cgi?id=3994 )

All Checks are Passed.

Thanks,
Sivaparvathi C


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#95669): https://edk2.groups.io/g/devel/message/95669
Mute This Topic: https://groups.io/mt/94378547/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-


回复: [edk2-devel] 回复: [edk2-devel] 回复: [edk2-devel] 回复: [edk2-devel] [PATCH] [PATCH]MdeModulePkg/Ufs: Coverity scan flags multiple issues in edk2-stable202205 Signed-off-by: sivaparvathic@ami.com
Posted by gaoliming via groups.io 1 year, 4 months ago
Sivaparvathi:

 Thanks for your update. Reviewed-by: Liming Gao <gaoliming@byosoft.com.cn>

 I will merge this patch if no other comments. 

 

Thanks

Liming

发件人: devel@edk2.groups.io <devel@edk2.groups.io> 代表 sivaparvathi C via groups.io
发送时间: 2022年10月28日 16:04
收件人: gaoliming <gaoliming@byosoft.com.cn>; devel@edk2.groups.io
主题: Re: [edk2-devel] 回复: [edk2-devel] 回复: [edk2-devel] 回复: [edk2-devel] [PATCH] [PATCH]MdeModulePkg/Ufs: Coverity scan flags multiple issues in edk2-stable202205 Signed-off-by: sivaparvathic@ami.com

 

Hi mikuback@linux.microsoft.com <mailto:mikuback@linux.microsoft.com> , gaoliming@byosoft.com.cn,michael.d.kinney@intel.com, <mailto:gaoliming@byosoft.com.cn,michael.d.kinney@intel.com,>  vasudevans@ami.com, <mailto:vasudevans@ami.com,>  sundaresans@ami.com <mailto:sundaresans@ami.com> 

 

Created PULL request for Coverity Issue changes.

UFS PR: https://github.com/tianocore/edk2/pull/3552 

BugZilla ID: https://bugzilla.tianocore.org/show_bug.cgi?id=3989 <https://bugzilla.tianocore.org/show_bug.cgi?id=3994>  

All Checks are Passed.

Thanks,
Sivaparvathi C





-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#95790): https://edk2.groups.io/g/devel/message/95790
Mute This Topic: https://groups.io/mt/94701046/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-


Re: [edk2-devel] [PATCH] [PATCH]MdeModulePkg/Ufs: Coverity scan flags multiple issues in edk2-stable202205 Signed-off-by: sivaparvathic@ami.com
Posted by sivaparvathi C via groups.io 1 year, 6 months ago
Hi gaoliming@byosoft.com.cn ,

Could you please review the changes ?

Thanks,
Sivaparvathi C


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#93644): https://edk2.groups.io/g/devel/message/93644
Mute This Topic: https://groups.io/mt/92765057/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-