[v1] coverity fixes

[PATCH 00/12] coverity fixes

Vladimir Sementsov-Ogievskiy posted 12 patches 7 months, 1 week ago

Diff against v2 v3
Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20230925194040.68592-1-vsementsov@yandex-team.ru

Maintainers: Paolo Bonzini <pbonzini@redhat.com>, Stefan Hajnoczi <stefanha@redhat.com>, Fam Zheng <fam@euphon.net>, "Philippe Mathieu-Daudé" <philmd@linaro.org>, Kevin Wolf <kwolf@redhat.com>, Hanna Reitz <hreitz@redhat.com>, "Michael S. Tsirkin" <mst@redhat.com>, Peter Xu <peterx@redhat.com>, Jason Wang <jasowang@redhat.com>, Marcel Apfelbaum <marcel.apfelbaum@gmail.com>, Richard Henderson <richard.henderson@linaro.org>, Eduardo Habkost <eduardo@habkost.net>, "Daniel P. Berrangé" <berrange@redhat.com>, Alistair Francis <alistair.francis@wdc.com>, David Gibson <david@gibson.dropbear.id.au>

There is a newer version of this series

accel/kvm/kvm-all.c                       | 15 +++++++++++
block/nvme.c                              |  6 ++---
hw/core/loader.c                          | 32 +++++++++++++++++++----
hw/i386/intel_iommu.c                     | 11 +++++---
hw/pci/pcie_sriov.c                       |  9 +++----
hw/rtc/mc146818rtc.c                      |  2 +-
include/hw/loader.h                       |  2 --
io/channel-socket.c                       |  5 ++++
softmmu/device_tree.c                     |  2 +-
subprojects/libvhost-user/libvhost-user.c |  1 +
util/filemonitor-inotify.c                | 21 +++++++++------
11 files changed, 77 insertions(+), 29 deletions(-)

Expand all Fold all

[PATCH 00/12] coverity fixes

Posted by Vladimir Sementsov-Ogievskiy 7 months, 1 week ago

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

Vladimir Sementsov-Ogievskiy (12):
  hw/core/loader: load_at(): check size
  hw/i386/intel_iommu: vtd_slpte_nonzero_rsvd(): reduce magic numbers
  util/filemonitor-inotify: qemu_file_monitor_watch(): avoid overflow
  libvhost-user.c: add assertion to vu_message_read_default
  device_tree: qmp_dumpdtb(): stronger assertion
  mc146818rtc: rtc_set_time(): initialize tm to zeroes
  pcie_sriov: unregister_vfs(): fix error path
  block/nvme: nvme_process_completion() fix bound for cid
  kvm-all: introduce limits for name_size and num_desc
  hw/core/loader: gunzip(): initialize z_stream
  hw/core/loader: read_targphys(): add upper bound
  io/channel-socket: qio_channel_socket_flush(): improve msg validation

 accel/kvm/kvm-all.c                       | 15 +++++++++++
 block/nvme.c                              |  6 ++---
 hw/core/loader.c                          | 32 +++++++++++++++++++----
 hw/i386/intel_iommu.c                     | 11 +++++---
 hw/pci/pcie_sriov.c                       |  9 +++----
 hw/rtc/mc146818rtc.c                      |  2 +-
 include/hw/loader.h                       |  2 --
 io/channel-socket.c                       |  5 ++++
 softmmu/device_tree.c                     |  2 +-
 subprojects/libvhost-user/libvhost-user.c |  1 +
 util/filemonitor-inotify.c                | 21 +++++++++------
 11 files changed, 77 insertions(+), 29 deletions(-)

-- 
2.34.1