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

Tomáš Golembiovský posted 1 patch 7 years, 1 month 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ý 7 years, 1 month 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 7 years, 1 month 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