mbox series

[v3,0/6] coverity fixes

Message ID 20231017125941.810461-1-vsementsov@yandex-team.ru (mailing list archive)
Headers show
Series coverity fixes | expand

Message

Vladimir Sementsov-Ogievskiy Oct. 17, 2023, 12:59 p.m. UTC
v3: rebase on master (two patches dropped, as already merged)
    add r-b marks

v2:
01: add explanations, new assert and avoid extra assignment
    add CIDs [thx to Paolo]
02: add explanation, improve wording
04,07: s/{0}/{}
06,08: improve wording

Hi! Here are some improvements to handle issues found by Coverity (not
public Coverity site, so there are no CIDs).


Vladimir Sementsov-Ogievskiy (6):
  hw/i386/intel_iommu: vtd_slpte_nonzero_rsvd(): assert no overflow
  util/filemonitor-inotify: qemu_file_monitor_watch(): assert no
    overflow
  mc146818rtc: rtc_set_time(): initialize tm to zeroes
  block/nvme: nvme_process_completion() fix bound for cid
  hw/core/loader: gunzip(): initialize z_stream
  io/channel-socket: qio_channel_socket_flush(): improve msg validation

 block/nvme.c               |  7 ++++---
 hw/core/loader.c           |  2 +-
 hw/i386/intel_iommu.c      | 23 ++++++++++++++++++++---
 hw/rtc/mc146818rtc.c       |  2 +-
 io/channel-socket.c        |  5 +++++
 util/filemonitor-inotify.c | 25 +++++++++++++++++--------
 6 files changed, 48 insertions(+), 16 deletions(-)

Comments

Peter Maydell Nov. 6, 2023, 12:38 p.m. UTC | #1
On Tue, 17 Oct 2023 at 13:59, Vladimir Sementsov-Ogievskiy
<vsementsov@yandex-team.ru> wrote:
>
> v3: rebase on master (two patches dropped, as already merged)
>     add r-b marks
>
> v2:
> 01: add explanations, new assert and avoid extra assignment
>     add CIDs [thx to Paolo]
> 02: add explanation, improve wording
> 04,07: s/{0}/{}
> 06,08: improve wording
>
> Hi! Here are some improvements to handle issues found by Coverity (not
> public Coverity site, so there are no CIDs).
>

Looks like nobody picked this up, and they all got reviewed,
so I'm going to apply it via the target-arm.next tree.

thanks
-- PMM