Re: [PATCH] qemuBuildHostNetStr: Don't leak @addr

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

 



On 09/21/2017 04:01 PM, Ján Tomko wrote:
> On Thu, Sep 21, 2017 at 02:55:12PM +0200, Michal Privoznik wrote:
>> The virSocketAddrFormat() allocates the string and it's caller
>> responsibility to free it afterwards.
>>
> 
> Introduced by not-yet-released commit 8703813
> 
>> ==28857== 11 bytes in 1 blocks are definitely lost in loss record 37
>> of 168
>> ==28857==    at 0x4C2BEDF: malloc (vg_replace_malloc.c:299)
>> ==28857==    by 0x9A81D79: strdup (in /lib64/libc-2.23.so)
>> ==28857==    by 0x5DA3BF0: virStrdup (virstring.c:902)
>> ==28857==    by 0x5D96182: virSocketAddrFormatFull (virsocketaddr.c:427)
>> ==28857==    by 0x5D95E13: virSocketAddrFormat (virsocketaddr.c:352)
>> ==28857==    by 0x5706890: qemuBuildHostNetStr (qemu_command.c:3891)
>> ==28857==    by 0x57138D3: qemuBuildInterfaceCommandLine
>> (qemu_command.c:8597)
>> ==28857==    by 0x5713D6A: qemuBuildNetCommandLine (qemu_command.c:8699)
>> ==28857==    by 0x57176F6: qemuBuildCommandLine (qemu_command.c:10027)
>> ==28857==    by 0x5769D61: qemuProcessCreatePretendCmd
>> (qemu_process.c:6004)
>> ==28857==    by 0x4056EC: testCompareXMLToArgv (qemuxml2argvtest.c:502)
>> ==28857==    by 0x41DF40: virTestRun (testutils.c:180)
>>
>> Signed-off-by: Michal Privoznik <mprivozn@xxxxxxxxxx>
>> ---
>> src/qemu/qemu_command.c | 1 +
>> 1 file changed, 1 insertion(+)
>>
> 
> ACK
> 
> Looks like aiforaf in libvirt_nss.c could use the same treatment.

I've no idea what aiforaf mean.

Michal

--
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