Re: [libvirt] [PATCH 0/2] Couple of Coverity fixes

2019-08-07 Thread Michal Privoznik 

On 8/7/19 4:45 PM, John Ferlan wrote:

Both from recent changes, it was just as easy for me to post the patches
as write the annoying email that someone else needs to generate them ;-).

John Ferlan (2):
   qemu: Fix possible NULL deref in qemuDomainGetResctrlMonData
   tests: Fix memory leak in mymain

  src/qemu/qemu_driver.c | 3 ++-
  tests/virhostdevtest.c | 4 +---
  2 files changed, 3 insertions(+), 4 deletions(-)



Reviewed-by: Michal Privoznik 

Michal

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

[libvirt] [PATCH 0/2] Couple of Coverity fixes

2019-08-07 Thread John Ferlan 
Both from recent changes, it was just as easy for me to post the patches
as write the annoying email that someone else needs to generate them ;-).

John Ferlan (2):
  qemu: Fix possible NULL deref in qemuDomainGetResctrlMonData
  tests: Fix memory leak in mymain

 src/qemu/qemu_driver.c | 3 ++-
 tests/virhostdevtest.c | 4 +---
 2 files changed, 3 insertions(+), 4 deletions(-)

-- 
2.20.1

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

[libvirt] [PATCH 0/2] Couple of Coverity fixes

2016-03-31 Thread Michal Privoznik 
*** SOME BLURB HERE ***

Michal Privoznik (2):
  virPerfReadEvent: Prefer saferead over read
  qemuProcessVerifyGuestCPU: Avoid coverity false positive

 src/qemu/qemu_process.c | 2 ++
 src/util/virperf.c  | 2 +-
 2 files changed, 3 insertions(+), 1 deletion(-)

-- 
2.7.3

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

[libvirt] [PATCH 0/2] Couple of coverity fixes

2016-02-10 Thread Michal Privoznik 
It was brought to my attention that coverity spotted couple of defects. Here
are the fixes.

Michal Privoznik (2):
  xenconfig: Properly check retval of virDomainGraphicsListenSetAddress
  virDomainFormatSchedDef: Check @schedMap bounds

 src/conf/domain_conf.c   | 21 +++--
 src/xenconfig/xen_sxpr.c |  4 ++--
 2 files changed, 13 insertions(+), 12 deletions(-)

-- 
2.4.10

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

Re: [libvirt] [PATCH 0/2] Couple of coverity fixes

2016-01-09 Thread John Ferlan 


On 01/07/2016 05:58 AM, Michal Privoznik wrote:
> I went through coverity report and found these.
> 
> Michal Privoznik (2):
>   virDomainMigrateUnmanagedParams: Don't blindly dereference @dconnuri
>   virt-host-validate-common: Print warning on missing IOMMU
> 
>  src/libvirt-domain.c  | 2 +-
>  src/qemu/qemu_migration.c | 2 +-
>  tools/virt-host-validate-common.c | 3 ---
>  3 files changed, 2 insertions(+), 5 deletions(-)
> 

ACK to both

John

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