On Thu, Apr 07, 2016 at 12:52:19PM +0200, Peter Krempa wrote: > The new perf code didn't bother to clear a pointer in 'priv' causing a > double free or other memory corruption goodness if a VM failed to start. > > Clear the pointer after freeing the memory. > > Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1324757 > --- > src/qemu/qemu_process.c | 2 ++ > 1 file changed, 2 insertions(+) > ACK Jan > diff --git a/src/qemu/qemu_process.c b/src/qemu/qemu_process.c > index 2b600c1..6c870f5 100644 > --- a/src/qemu/qemu_process.c > +++ b/src/qemu/qemu_process.c > @@ -3444,6 +3444,7 @@ qemuDomainPerfRestart(virDomainObjPtr vm) > > cleanup: > virPerfFree(priv->perf); > + priv->perf = NULL; > return -1; > } > > @@ -5970,6 +5971,7 @@ void qemuProcessStop(virQEMUDriverPtr driver, > virCgroupFree(&priv->cgroup); > > virPerfFree(priv->perf); > + priv->perf = NULL; > > qemuProcessRemoveDomainStatus(driver, vm); > > -- > 2.8.0 > > -- > libvir-list mailing list > libvir-list@xxxxxxxxxx > https://www.redhat.com/mailman/listinfo/libvir-list -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list