[libvirt] [PATCH 0/7] Addressing some Coverity complaints...

John Ferlan posted 7 patches 6 years ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/20180417162218.14378-1-jferlan@redhat.com
Test syntax-check passed
src/conf/snapshot_conf.c         | 16 ++++++++++------
src/qemu/qemu_migration_params.c |  1 +
src/qemu/qemu_monitor_json.c     |  7 ++++++-
src/remote/remote_driver.c       |  2 +-
tests/commandhelper.c            |  2 +-
tests/testutilsqemuschema.c      | 40 +++++++++++++++++++++++++++++-----------
tests/virnetserverclienttest.c   |  3 ++-
7 files changed, 50 insertions(+), 21 deletions(-)
[libvirt] [PATCH 0/7] Addressing some Coverity complaints...
Posted by John Ferlan 6 years ago
My current pile is 48 patches - most of those are known false
positives or have been posted before and either rejected or
ignored. In any case, the following is a list of more recent
adjustments from my coverity tree. Seems we've largely moved
away from resource leaks and into return value checking issues.
Some good, some not so good. 


John Ferlan (7):
  conf: Add error checking to virDomainSnapshotDiskDefFormat
  remote: Fix usage of ATTRIBUTE_FALLTHROUGH
  tests: Return failure if log not fopen'd
  tests: Don't call virNetServerClientClose without valid client
  tests: Add checks for possible errors
  qemu: Fix possible memory leak in migration param processing
  qemu: Check for missing 'return' in qemuMonitorJSONCheckReply

 src/conf/snapshot_conf.c         | 16 ++++++++++------
 src/qemu/qemu_migration_params.c |  1 +
 src/qemu/qemu_monitor_json.c     |  7 ++++++-
 src/remote/remote_driver.c       |  2 +-
 tests/commandhelper.c            |  2 +-
 tests/testutilsqemuschema.c      | 40 +++++++++++++++++++++++++++++-----------
 tests/virnetserverclienttest.c   |  3 ++-
 7 files changed, 50 insertions(+), 21 deletions(-)

-- 
2.13.6

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list