Some Coverity patches

[libvirt] [PATCH 0/6] Some Coverity patches

John Ferlan posted 6 patches 5 years, 10 months ago

Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/20180608171019.32167-1-jferlan@redhat.com

Test syntax-check passed

src/conf/domain_conf.c         |  3 ++-
src/qemu/qemu_monitor.h        |  2 +-
src/vbox/vbox_common.c         | 28 +++++++++++++++-------------
tests/libxlxml2domconfigtest.c |  4 +++-
tests/qemumonitorjsontest.c    |  2 +-
5 files changed, 22 insertions(+), 17 deletions(-)

Expand all Fold all

[libvirt] [PATCH 0/6] Some Coverity patches

Posted by John Ferlan 5 years, 10 months ago

Resolve a few things that have built up previous and a couple
that are more recent.

John Ferlan (6):
  vbox: Fix resource leak
  vbox: Fix resource leak
  qemu: Fix Coverity build for qemu_monitor
  test: Fix resource leak in qemumonitorjsontest
  test: Check return status for libxlxml2domconfigtest
  conf: Check error from virXMLFormatElement call

 src/conf/domain_conf.c         |  3 ++-
 src/qemu/qemu_monitor.h        |  2 +-
 src/vbox/vbox_common.c         | 28 +++++++++++++++-------------
 tests/libxlxml2domconfigtest.c |  4 +++-
 tests/qemumonitorjsontest.c    |  2 +-
 5 files changed, 22 insertions(+), 17 deletions(-)

-- 
2.14.4

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

Re: [libvirt] [PATCH 0/6] Some Coverity patches

Posted by Katerina Koukiou 5 years, 10 months ago

On Fri, Jun 08, 2018 at 01:10:13PM -0400, John Ferlan wrote:
> Resolve a few things that have built up previous and a couple
> that are more recent.
> 
> John Ferlan (6):
>   vbox: Fix resource leak
>   vbox: Fix resource leak
>   qemu: Fix Coverity build for qemu_monitor
>   test: Fix resource leak in qemumonitorjsontest
>   test: Check return status for libxlxml2domconfigtest
>   conf: Check error from virXMLFormatElement call
> 
>  src/conf/domain_conf.c         |  3 ++-
>  src/qemu/qemu_monitor.h        |  2 +-
>  src/vbox/vbox_common.c         | 28 +++++++++++++++-------------
>  tests/libxlxml2domconfigtest.c |  4 +++-
>  tests/qemumonitorjsontest.c    |  2 +-
>  5 files changed, 22 insertions(+), 17 deletions(-)

Reviewed-by: Katerina Koukiou <kkoukiou@redhat.com>
--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list