[edk2-devel] [PATCH v2 0/5] patches for some warnings raised by "RH covscan"

Laszlo Ersek posted 5 patches 5 years ago
Failed in applying to current master (apply log)
BaseTools/Source/C/Include/Common/PiFirmwareFile.h | 11 +++++++--
MdePkg/Include/Pi/PiFirmwareFile.h                 | 26 +++++++++++++++-----
OvmfPkg/Sec/SecMain.c                              |  6 ++---
3 files changed, 32 insertions(+), 11 deletions(-)
[edk2-devel] [PATCH v2 0/5] patches for some warnings raised by "RH covscan"
Posted by Laszlo Ersek 5 years ago
Bugzilla: https://bugzilla.tianocore.org/show_bug.cgi?id=1710
Repo:     https://github.com/lersek/edk2.git
Branch:   covscan_bz_1710_v2

Patch-level updates relative to v1 have been noted on the patches
themselves. And earlier I described the changes to the series's
structure at <https://edk2.groups.io/g/devel/message/39301>.

Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Cc: Bob Feng <bob.c.feng@intel.com>
Cc: Jordan Justen <jordan.l.justen@intel.com>
Cc: Liming Gao <liming.gao@intel.com>
Cc: Michael D Kinney <michael.d.kinney@intel.com>
Cc: Yonghong Zhu <yonghong.zhu@intel.com>

Thanks
Laszlo

Laszlo Ersek (5):
  MdePkg/PiFirmwareFile: express IS_SECTION2 in terms of SECTION_SIZE
  MdePkg/PiFirmwareFile: fix undefined behavior in SECTION_SIZE
  BaseTools/PiFirmwareFile: fix undefined behavior in SECTION_SIZE
  MdePkg/PiFirmwareFile: fix undefined behavior in FFS_FILE_SIZE
  OvmfPkg/Sec: fix out-of-bounds reads

 BaseTools/Source/C/Include/Common/PiFirmwareFile.h | 11 +++++++--
 MdePkg/Include/Pi/PiFirmwareFile.h                 | 26 +++++++++++++++-----
 OvmfPkg/Sec/SecMain.c                              |  6 ++---
 3 files changed, 32 insertions(+), 11 deletions(-)

-- 
2.19.1.3.g30247aa5d201


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.

View/Reply Online (#39306): https://edk2.groups.io/g/devel/message/39306
Mute This Topic: https://groups.io/mt/31233845/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-

Re: [edk2-devel] [PATCH v2 0/5] patches for some warnings raised by "RH covscan"
Posted by Michael D Kinney 5 years ago
Series Reviewed-by: Michael D Kinney <michael.d.kinney@intel.com>

> -----Original Message-----
> From: Laszlo Ersek [mailto:lersek@redhat.com]
> Sent: Thursday, April 18, 2019 10:47 AM
> To: edk2-devel-groups-io <devel@edk2.groups.io>
> Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>; Feng,
> Bob C <bob.c.feng@intel.com>; Justen, Jordan L
> <jordan.l.justen@intel.com>; Gao, Liming
> <liming.gao@intel.com>; Kinney, Michael D
> <michael.d.kinney@intel.com>; Zhu, Yonghong
> <yonghong.zhu@intel.com>
> Subject: [PATCH v2 0/5] patches for some warnings
> raised by "RH covscan"
> 
> Bugzilla:
> https://bugzilla.tianocore.org/show_bug.cgi?id=1710
> Repo:     https://github.com/lersek/edk2.git
> Branch:   covscan_bz_1710_v2
> 
> Patch-level updates relative to v1 have been noted on
> the patches
> themselves. And earlier I described the changes to the
> series's
> structure at
> <https://edk2.groups.io/g/devel/message/39301>.
> 
> Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
> Cc: Bob Feng <bob.c.feng@intel.com>
> Cc: Jordan Justen <jordan.l.justen@intel.com>
> Cc: Liming Gao <liming.gao@intel.com>
> Cc: Michael D Kinney <michael.d.kinney@intel.com>
> Cc: Yonghong Zhu <yonghong.zhu@intel.com>
> 
> Thanks
> Laszlo
> 
> Laszlo Ersek (5):
>   MdePkg/PiFirmwareFile: express IS_SECTION2 in terms
> of SECTION_SIZE
>   MdePkg/PiFirmwareFile: fix undefined behavior in
> SECTION_SIZE
>   BaseTools/PiFirmwareFile: fix undefined behavior in
> SECTION_SIZE
>   MdePkg/PiFirmwareFile: fix undefined behavior in
> FFS_FILE_SIZE
>   OvmfPkg/Sec: fix out-of-bounds reads
> 
>  BaseTools/Source/C/Include/Common/PiFirmwareFile.h |
> 11 +++++++--
>  MdePkg/Include/Pi/PiFirmwareFile.h                 |
> 26 +++++++++++++++-----
>  OvmfPkg/Sec/SecMain.c                              |
> 6 ++---
>  3 files changed, 32 insertions(+), 11 deletions(-)
> 
> --
> 2.19.1.3.g30247aa5d201


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.

View/Reply Online (#39314): https://edk2.groups.io/g/devel/message/39314
Mute This Topic: https://groups.io/mt/31233845/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-

Re: [edk2-devel] [PATCH v2 0/5] patches for some warnings raised by "RH covscan"
Posted by Jordan Justen 5 years ago
Series Reviewed-by: Jordan Justen <jordan.l.justen@intel.com>

On 2019-04-18 10:47:05, Laszlo Ersek wrote:
> Bugzilla: https://bugzilla.tianocore.org/show_bug.cgi?id=1710
> Repo:     https://github.com/lersek/edk2.git
> Branch:   covscan_bz_1710_v2
> 
> Patch-level updates relative to v1 have been noted on the patches
> themselves. And earlier I described the changes to the series's
> structure at <https://edk2.groups.io/g/devel/message/39301>.
> 
> Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
> Cc: Bob Feng <bob.c.feng@intel.com>
> Cc: Jordan Justen <jordan.l.justen@intel.com>
> Cc: Liming Gao <liming.gao@intel.com>
> Cc: Michael D Kinney <michael.d.kinney@intel.com>
> Cc: Yonghong Zhu <yonghong.zhu@intel.com>
> 
> Thanks
> Laszlo
> 
> Laszlo Ersek (5):
>   MdePkg/PiFirmwareFile: express IS_SECTION2 in terms of SECTION_SIZE
>   MdePkg/PiFirmwareFile: fix undefined behavior in SECTION_SIZE
>   BaseTools/PiFirmwareFile: fix undefined behavior in SECTION_SIZE
>   MdePkg/PiFirmwareFile: fix undefined behavior in FFS_FILE_SIZE
>   OvmfPkg/Sec: fix out-of-bounds reads
> 
>  BaseTools/Source/C/Include/Common/PiFirmwareFile.h | 11 +++++++--
>  MdePkg/Include/Pi/PiFirmwareFile.h                 | 26 +++++++++++++++-----
>  OvmfPkg/Sec/SecMain.c                              |  6 ++---
>  3 files changed, 32 insertions(+), 11 deletions(-)
> 
> -- 
> 2.19.1.3.g30247aa5d201
> 

-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.

View/Reply Online (#39315): https://edk2.groups.io/g/devel/message/39315
Mute This Topic: https://groups.io/mt/31233845/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-

Re: [edk2-devel] [PATCH v2 0/5] patches for some warnings raised by "RH covscan"
Posted by Laszlo Ersek 5 years ago
On 04/18/19 19:47, Laszlo Ersek wrote:
> Bugzilla: https://bugzilla.tianocore.org/show_bug.cgi?id=1710
> Repo:     https://github.com/lersek/edk2.git
> Branch:   covscan_bz_1710_v2
> 
> Patch-level updates relative to v1 have been noted on the patches
> themselves. And earlier I described the changes to the series's
> structure at <https://edk2.groups.io/g/devel/message/39301>.
> 
> Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
> Cc: Bob Feng <bob.c.feng@intel.com>
> Cc: Jordan Justen <jordan.l.justen@intel.com>
> Cc: Liming Gao <liming.gao@intel.com>
> Cc: Michael D Kinney <michael.d.kinney@intel.com>
> Cc: Yonghong Zhu <yonghong.zhu@intel.com>
> 
> Thanks
> Laszlo
> 
> Laszlo Ersek (5):
>   MdePkg/PiFirmwareFile: express IS_SECTION2 in terms of SECTION_SIZE
>   MdePkg/PiFirmwareFile: fix undefined behavior in SECTION_SIZE
>   BaseTools/PiFirmwareFile: fix undefined behavior in SECTION_SIZE
>   MdePkg/PiFirmwareFile: fix undefined behavior in FFS_FILE_SIZE
>   OvmfPkg/Sec: fix out-of-bounds reads
> 
>  BaseTools/Source/C/Include/Common/PiFirmwareFile.h | 11 +++++++--
>  MdePkg/Include/Pi/PiFirmwareFile.h                 | 26 +++++++++++++++-----
>  OvmfPkg/Sec/SecMain.c                              |  6 ++---
>  3 files changed, 32 insertions(+), 11 deletions(-)
> 

Series pushed as commit range 1a734ed85fda..b9d4847ec258.

Thank you all,
Laszlo

-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.

View/Reply Online (#39509): https://edk2.groups.io/g/devel/message/39509
Mute This Topic: https://groups.io/mt/31233845/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-