[Spice-commits] 2 commits - server/tests

[GitLab Mirror]

 server/tests/valgrind/glib.supp |   64 +++++++++++++++++++++++++++++++++-------
 1 file changed, 53 insertions(+), 11 deletions(-)

New commits:
commit 91b15b6de3f86b1750c0b41df671e040fa98b81f
Author: Frediano Ziglio <fziglio at redhat.com>
Date:   Fri Nov 9 09:25:14 2018 +0000

    ci: Ignore leak from GnuTLS p11 call
    
    Fix Fedora 29 one-time leak.
    
    Signed-off-by: Frediano Ziglio <fziglio at redhat.com>
    Acked-by: Eduardo Lima (Etrunko) <etrunko at redhat.com>

diff --git a/server/tests/valgrind/glib.supp b/server/tests/valgrind/glib.supp
index e95f1cb2..ca0684d6 100644
--- a/server/tests/valgrind/glib.supp
+++ b/server/tests/valgrind/glib.supp
@@ -572,3 +572,13 @@
 	...
 	fun:gnutls_rnd
 }
+
+{
+	gnutls_tls_p11_kit_leak
+	Memcheck:Leak
+	fun:malloc
+	...
+	fun:__tls_get_addr
+	...
+	fun:p11_kit_modules_load
+}
commit e02d58f0a7b33014b9b93561021de0a51f1c30af
Author: Frediano Ziglio <fziglio at redhat.com>
Date:   Fri Nov 9 09:22:52 2018 +0000

    ci: Merge new glib.supp file
    
    Merge from GLib master repository.
    
    Signed-off-by: Frediano Ziglio <fziglio at redhat.com>
    Acked-by: Eduardo Lima (Etrunko) <etrunko at redhat.com>

diff --git a/server/tests/valgrind/glib.supp b/server/tests/valgrind/glib.supp
index fb2cb4a7..e95f1cb2 100644
--- a/server/tests/valgrind/glib.supp
+++ b/server/tests/valgrind/glib.supp
@@ -17,7 +17,7 @@
 # This file should be updated if GLib introduces a new deliberate one-time leak,
 # or another false race positive in Valgrind: please file bugs at:
 #
-# https://bugzilla.gnome.org/enter_bug.cgi?product=glib
+# https://gitlab.gnome.org/GNOME/glib/issues/new
 
 {
 	gnutls-init-calloc
@@ -51,6 +51,36 @@
 	fun:initialize_module_inlock_reentrant
 }
 
+# One-time allocation from libc for getpwnam() results
+{
+	g-local-vfs-getpwnam
+	Memcheck:Leak
+	fun:malloc
+	...
+	fun:getpwnam
+	fun:g_local_vfs_parse_name
+}
+
+{
+	glib-init-malloc
+	Memcheck:Leak
+	fun:malloc
+	...
+	fun:g_quark_init
+	...
+	fun:glib_init_ctor
+}
+
+{
+	glib-init-calloc
+	Memcheck:Leak
+	fun:calloc
+	...
+	fun:g_quark_init
+	...
+	fun:glib_init_ctor
+}
+
 {
 	gobject-init-malloc
 	Memcheck:Leak
@@ -232,6 +262,18 @@
 	fun:_g_io_module_get_default
 }
 
+# One-time getaddrinfo() configuration loading
+{
+	g-threaded-resolver-getaddrinfo-config
+	Memcheck:Leak
+	fun:malloc
+	...
+	fun:__resolv_conf_allocate
+	...
+	fun:getaddrinfo
+	fun:do_lookup_by_name
+}
+
 # memcheck checks that the third argument to ioctl() is a valid pointer, but
 # some ioctls use that argument as an integer
 {
@@ -503,16 +545,6 @@
 }
 
 {
-	getaddrinfo
-	Memcheck:Leak
-	fun:malloc
-	...
-	fun:__resolv_conf_allocate
-	...
-	fun:getaddrinfo
-}
-
-{
 	px_proxy_factory_get_proxies
 	Memcheck:Leak
 	fun:malloc