Re: [libvirt] [PATCH] tests: Update valgrind suppressions file
2011/4/30 Eric Blake <eblake(a)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