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

John Ferlan jferlan at redhat.com
Tue Apr 17 16:22:11 UTC 2018


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




More information about the libvir-list mailing list