On 12/04/2013 10:55 AM, Peter Krempa wrote: > The 'internal' variable holds only two states; convert it to a boolean > and the 'fail' label should be called 'cleanup'. > --- > tests/domainsnapshotxml2xmltest.c | 36 ++++++++++++++++++++---------------- > 1 file changed, 20 insertions(+), 16 deletions(-) >
> @@ -95,8 +97,10 @@ mymain(void)
> if ((driver.caps = testQemuCapsInit()) == NULL)
> return EXIT_FAILURE;
>
> - if (!(driver.xmlopt = virQEMUDriverCreateXMLConf(&driver)))
> + if (!(driver.xmlopt = virQEMUDriverCreateXMLConf(&driver))) {
> + virObjectUnref(driver.caps);
> return EXIT_FAILURE;
> + }
Is this a memory leak fix not mentioned in the commit message? Fine to
include it, but mentioning it would be nice.
ACK.
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
-- libvir-list mailing list [email protected] https://www.redhat.com/mailman/listinfo/libvir-list
