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

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

 



On 09/21/2017 04:58 PM, Peter Krempa wrote:
> On Thu, Sep 21, 2017 at 16:08:05 +0200, Michal Privoznik wrote:
>> On 09/21/2017 04:01 PM, Ján Tomko wrote:
>>> On Thu, Sep 21, 2017 at 02:55:12PM +0200, Michal Privoznik wrote:
> 
> [...]
> 
>>>>
>>>
>>> ACK
>>>
>>> Looks like aiforaf in libvirt_nss.c could use the same treatment.
>>
>> I've no idea what aiforaf mean.
> 
> $ git grep aiforaf
> 
> tools/nss/libvirt_nss.c:aiforaf(const char *name, int af, struct addrinfo *pai, struct addrinfo **aip)
> tools/nss/libvirt_nss.c:        aiforaf(name, AF_INET6, ai, &cur);
> tools/nss/libvirt_nss.c:        aiforaf(name, AF_INET, ai, &cur);
> 

Ah, aiforaf(). This is why I put brackets after function names so that
it's visible what I mean. Anyway, I was wondering why I haven't hit that
memleak and it's happening only on BSDs. I'll post a patch shortly.

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