Re: [libvirt] [PATCH 0/8] Coverity related patches

2019-11-04 Thread Daniel P . Berrangé 
On Sun, Nov 03, 2019 at 08:53:32AM -0500, John Ferlan wrote:
> Some related to "new-ish" changes that have caused Coverity to
> discover new issues and some from older changes from a pile of
> 50 or so that are not essentially "false positives".
> 
> John Ferlan (8):
>   vbox: Fix possible NULL deref
>   conf: Remove ATTRIBUTE_NONNULL for virDomainQemuMonitorEventNew
>   tests: Fix memory leak in mymain
>   lxc: Remove unnecessary comment
>   tests: Remove _NULLABLE in virNetDevExists mock
>   qemu: Fix possible NULL deref in qemuDomainSaveImageStartVM
>   tests: Add return value check in checkUserInfo
>   tests: Fix logic to not have possible NULL deref

Reviewed-by: Daniel P. Berrangé 


Regards,
Daniel
-- 
|: https://berrange.com  -o-https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o-https://fstop138.berrange.com :|
|: https://entangle-photo.org-o-https://www.instagram.com/dberrange :|

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

[libvirt] [PATCH 0/8] Coverity related patches

2019-11-03 Thread John Ferlan 
Some related to "new-ish" changes that have caused Coverity to
discover new issues and some from older changes from a pile of
50 or so that are not essentially "false positives".

John Ferlan (8):
  vbox: Fix possible NULL deref
  conf: Remove ATTRIBUTE_NONNULL for virDomainQemuMonitorEventNew
  tests: Fix memory leak in mymain
  lxc: Remove unnecessary comment
  tests: Remove _NULLABLE in virNetDevExists mock
  qemu: Fix possible NULL deref in qemuDomainSaveImageStartVM
  tests: Add return value check in checkUserInfo
  tests: Fix logic to not have possible NULL deref

 src/conf/domain_event.c  | 6 +-
 src/conf/domain_event.h  | 2 +-
 src/lxc/lxc_container.c  | 5 -
 src/qemu/qemu_driver.c   | 2 +-
 src/vbox/vbox_common.c   | 4 ++--
 tests/qemuagenttest.c| 4 +++-
 tests/qemuhotplugtest.c  | 3 +--
 tests/qemuxml2argvmock.c | 2 +-
 tests/virbitmaptest.c| 6 --
 9 files changed, 18 insertions(+), 16 deletions(-)

-- 
2.20.1

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