[libvirt] [PATCH 1/2] qemu: Fix leak in qemuProcessInitCpuAffinity()

Andrea Bolognani abologna at redhat.com
Thu Jun 6 13:46:48 UTC 2019


On Thu, 2019-06-06 at 06:44 -0400, John Ferlan wrote:
> On 6/4/19 8:46 AM, Andrea Bolognani wrote:
> >      } else if (vm->def->cputune.emulatorpin) {
> > -        cpumapToSet = vm->def->cputune.emulatorpin;
> > +        if (virBitmapCopy(cpumapToSet, vm->def->cputune.emulatorpin) < 0)
> 
> Now Coverity is unhappy for another reason.  What happens to the NULL
> cpumapToSet when calling virBitmapCopy?
> 
> Should have been virBitmapNewCopy

Geez, so embarrassing :/

I just posted a fix:

  https://www.redhat.com/archives/libvir-list/2019-June/msg00171.html

> (sorry didn't get a chance to look at the patch when first posted)

No need to apologize, but if you really feel like you need to make
up for it reviewing the above will do ;)

-- 
Andrea Bolognani / Red Hat / Virtualization




More information about the libvir-list mailing list