[libvirt] [PATCH] valgrind: add suppression for bash memory leak

Tomáš Golembiovský posted 1 patch 146 weeks ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/7dd30112f3e50280023d8f4243cefc373762adc2.1487026088.git.tgolembi@redhat.com
tests/.valgrind.supp | 9 +++++++++
1 file changed, 9 insertions(+)

[libvirt] [PATCH] valgrind: add suppression for bash memory leak

Posted by Tomáš Golembiovský 146 weeks ago
Add suppression for memory leak in bash observerd with bash 4.4.011 on
Arch Linux.

Signed-off-by: Tomáš Golembiovský <tgolembi@redhat.com>
---
 tests/.valgrind.supp | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/tests/.valgrind.supp b/tests/.valgrind.supp
index d4fef857b..d474d63e9 100644
--- a/tests/.valgrind.supp
+++ b/tests/.valgrind.supp
@@ -32,6 +32,15 @@
    ...
    obj:*/bin/bash
 }
+{
+   bashMemoryLeak4
+   Memcheck:Leak
+   match-leak-kinds: definite
+   fun:malloc
+   fun:xmalloc
+   fun:set_default_locale
+   fun:main
+}
 #
 # Failure seen in /usr/lib64/ld-2.15.so
 #
-- 
2.11.1

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

Re: [libvirt] [PATCH] valgrind: add suppression for bash memory leak

Posted by Martin Kletzander 146 weeks ago
On Mon, Feb 13, 2017 at 11:49:06PM +0100, Tomáš Golembiovský wrote:
>Add suppression for memory leak in bash observerd with bash 4.4.011 on
>Arch Linux.
>
>Signed-off-by: Tomáš Golembiovský <tgolembi@redhat.com>
>---
> tests/.valgrind.supp | 9 +++++++++
> 1 file changed, 9 insertions(+)
>
>diff --git a/tests/.valgrind.supp b/tests/.valgrind.supp
>index d4fef857b..d474d63e9 100644
>--- a/tests/.valgrind.supp
>+++ b/tests/.valgrind.supp
>@@ -32,6 +32,15 @@
>    ...
>    obj:*/bin/bash
> }
>+{
>+   bashMemoryLeak4
>+   Memcheck:Leak
>+   match-leak-kinds: definite
>+   fun:malloc
>+   fun:xmalloc
>+   fun:set_default_locale
>+   fun:main
>+}

I can confirm this works for me and it is bash.  I added:
   ...
   obj:*/bin/bash

to make sure and also to go with the other definitions and pushed it.
Thanks.

Another question is, are you using 'make valgrind'?  Lot of tests are
failing for me.

> #
> # Failure seen in /usr/lib64/ld-2.15.so
> #
>--
>2.11.1
>
>--
>libvir-list mailing list
>libvir-list@redhat.com
>https://www.redhat.com/mailman/listinfo/libvir-list
--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list