Re: [PATCH 2/3] qemuxml2argvtest: Don't initialize qemuCaps twice

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, May 30, 2018 at 18:04:28 +0200, Michal Privoznik wrote:
> There's no point in calling testInitQEMUCaps() (which sets
> info.qemuCaps) only to overwrite (and leak) it on the very next
> line.
> 
> ==12962== 296 (208 direct, 88 indirect) bytes in 1 blocks are definitely lost in loss record 265 of 331
> ==12962==    at 0x4C2CF26: calloc (vg_replace_malloc.c:711)
> ==12962==    by 0x5D28D9F: virAllocVar (viralloc.c:560)
> ==12962==    by 0x5D96AB4: virObjectNew (virobject.c:239)
> ==12962==    by 0x56DB7C7: virQEMUCapsNew (qemu_capabilities.c:1480)
> ==12962==    by 0x112A5B: testInitQEMUCaps (qemuxml2argvtest.c:361)
> ==12962==    by 0x1371C8: mymain (qemuxml2argvtest.c:2871)
> ==12962==    by 0x13AD0B: virTestMain (testutils.c:1120)
> ==12962==    by 0x1372FD: main (qemuxml2argvtest.c:2883)
> 
> Signed-off-by: Michal Privoznik <mprivozn@xxxxxxxxxx>
> ---
>  tests/qemuxml2argvtest.c | 2 --
>  1 file changed, 2 deletions(-)
> 
> diff --git a/tests/qemuxml2argvtest.c b/tests/qemuxml2argvtest.c
> index ddd2b88c0a..6f421ce8f5 100644
> --- a/tests/qemuxml2argvtest.c
> +++ b/tests/qemuxml2argvtest.c
> @@ -699,8 +699,6 @@ mymain(void)
>              (flags), parseFlags, false, NULL \
>          }; \
>          info.skipLegacyCPUs = skipLegacyCPUs; \
> -        if (testInitQEMUCaps(&info, gic) < 0) \
> -            return EXIT_FAILURE; \


This makes the 'gic' macro argument unused. You probably need to replace
it with testQemuCapsSetGIC after the caps are parsed if the parser does
not do that.

>          if (!(info.qemuCaps = qemuTestParseCapabilitiesArch(virArchFromString(arch), \
>                                                              capsfile))) \
>              return EXIT_FAILURE; \
> -- 
> 2.16.1
> 
> --
> libvir-list mailing list
> libvir-list@xxxxxxxxxx
> https://www.redhat.com/mailman/listinfo/libvir-list

Attachment: signature.asc
Description: PGP signature

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

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux