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

Re: [libvirt] [PATCH] tests: Update valgrind suppressions file



2011/4/30 Eric Blake <eblake redhat com>:
> On 04/30/2011 05:13 AM, Matthias Bolte wrote:
>> ---
>>  tests/.valgrind.supp |   58 +++++++++++++++++++++++++++++++++++++++++++------
>>  1 files changed, 51 insertions(+), 7 deletions(-)
>>
>> diff --git a/tests/.valgrind.supp b/tests/.valgrind.supp
>> index 4af10b1..68cfa0c 100644
>> --- a/tests/.valgrind.supp
>> +++ b/tests/.valgrind.supp
>> @@ -258,14 +258,58 @@
>>     Memcheck:Param
>>     capget(data)
>>     fun:capget
>> +   fun:*
>>     fun:capng_clear
>> +   fun:virClearCapabilities
>>     fun:__virExec
>>     fun:virExecWithHook
>> -   fun:virExec
>
>
> Makes sense.
>
>> +{
>> +   libnlMemoryLeak1
>> +   Memcheck:Leak
>> +   fun:malloc
>> +   fun:strdup
>> +   obj:/usr/lib/libnl.so.1.1
>> +}
>
> Nice.  It would be nicer if these other programs plugged their leaks,
> but doing this makes it easier to find just the libvirt leaks.

well, the world isn't perfect and for example the
VBoxNsxpNS_InitXPCOM2 suppression probably isn't a real leak, it's
just initializing tons of global stuff that later on isn't freed and
leaks on purpose. But this clutters valgrind's output massively.

> ACK.
>

Thanks, pushed.

Matthias


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