mbox series

[v2,0/8] coverity fixes

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

Message

Vladimir Sementsov-Ogievskiy Sept. 26, 2023, 8:15 p.m. UTC
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 (8):
  hw/i386/intel_iommu: vtd_slpte_nonzero_rsvd(): assert no overflow
  util/filemonitor-inotify: qemu_file_monitor_watch(): assert no
    overflow
  libvhost-user.c: add assertion to vu_message_read_default
  mc146818rtc: rtc_set_time(): initialize tm to zeroes
  pcie_sriov: unregister_vfs(): fix error path
  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/pci/pcie_sriov.c                       |  9 +++-----
 hw/rtc/mc146818rtc.c                      |  2 +-
 io/channel-socket.c                       |  5 +++++
 subprojects/libvhost-user/libvhost-user.c |  1 +
 util/filemonitor-inotify.c                | 25 +++++++++++++++--------
 8 files changed, 52 insertions(+), 22 deletions(-)

Comments

Maksim Davydov Sept. 27, 2023, 8:06 a.m. UTC | #1
On 9/26/23 23:15, Vladimir Sementsov-Ogievskiy wrote:
> 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 (8):
>    hw/i386/intel_iommu: vtd_slpte_nonzero_rsvd(): assert no overflow
>    util/filemonitor-inotify: qemu_file_monitor_watch(): assert no
>      overflow
>    libvhost-user.c: add assertion to vu_message_read_default
>    mc146818rtc: rtc_set_time(): initialize tm to zeroes
>    pcie_sriov: unregister_vfs(): fix error path
>    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/pci/pcie_sriov.c                       |  9 +++-----
>   hw/rtc/mc146818rtc.c                      |  2 +-
>   io/channel-socket.c                       |  5 +++++
>   subprojects/libvhost-user/libvhost-user.c |  1 +
>   util/filemonitor-inotify.c                | 25 +++++++++++++++--------
>   8 files changed, 52 insertions(+), 22 deletions(-)
>
all patches:
Reviewed-by: Maksim Davydov <davydov-max@yandex-team.ru>