[libvirt] [PATCH] testutilsqemu: avoid uninitialized variable
Jiri Denemark
jdenemar at redhat.com
Thu Apr 22 17:38:12 UTC 2010
> Gcc on F12 caught this, sorry I didn't catch it in the review:
>
> > - if ((machines = testQemuAllocMachines(&nmachines)) == NULL)
> > + if ((caps->host.cpu = virCPUDefCopy(&host_cpu)) == NULL ||
> > + (machines = testQemuAllocMachines(&nmachines)) == NULL)
> > goto cleanup;
>
> This change meant that we could then use machines inside the
> cleanup: label without having initialized it; freeing uninitialized
> data is never a good idea, even it if is unlikely to have happened.
>
> I'm pushing it under the obvious rule, since it breaks builds with
> --enable-compiler-warnings=error.
Hmm, my gcc didn't catch it :( I always configure with warnings=error and I
even built from scratch everything just before pushing... Thanks for the fix.
Jirka
More information about the libvir-list
mailing list