| Message ID | 20230926201532.221152-1-vsementsov@yandex-team.ru (mailing list archive) |
|---|---|
| Headers | show |
| Series | coverity fixes | expand |
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>
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(-)