[Spice-commits] 2 commits - server/tests

[GitLab Mirror]

 server/tests/valgrind/glib.supp |   30 ++++++++++++++++++++++++++++++
 1 file changed, 30 insertions(+)

New commits:
commit 87965d8deab617c1f9a1999335b4247a88975f35
Author: Frediano Ziglio <fziglio at redhat.com>
Date:   Tue Jun 5 11:59:20 2018 +0100

    ci: Add some needed Valgrind suppression rule
    
    From Gitlab CI:
    
    =17955== 16 bytes in 1 blocks are possibly lost in loss record 725 of 2,079
    ==17955==    at 0x4C2DBAB: malloc (vg_replace_malloc.c:299)
    ==17955==    by 0x4011D17: tls_get_addr_tail.isra.0 (in /usr/lib64/ld-2.27.so)
    ==17955==    by 0x4017997: __tls_get_addr (in /usr/lib64/ld-2.27.so)
    ==17955==    by 0xEE4534B: gnutls_rnd (in /usr/lib64/libgnutls.so.30.20.2)
    ==17955==    by 0xEE1F254: ??? (in /usr/lib64/libgnutls.so.30.20.2)
    ==17955==    by 0xEE1F947: ??? (in /usr/lib64/libgnutls.so.30.20.2)
    ==17955==    by 0xEE231B5: ??? (in /usr/lib64/libgnutls.so.30.20.2)
    ==17955==    by 0xEE24D67: gnutls_handshake (in /usr/lib64/libgnutls.so.30.20.2)
    ==17955==    by 0xEBD4FEA: ??? (in /usr/lib64/gio/modules/libgiognutls.so)
    ==17955==    by 0x7463936: g_task_thread_pool_thread (gtask.c:1331)
    ==17955==    by 0x7A3E932: g_thread_pool_thread_proxy (gthreadpool.c:307)
    ==17955==    by 0x7A3DF29: g_thread_proxy (gthread.c:784)
    ==17955==    by 0x8284563: start_thread (in /usr/lib64/libpthread-2.27.so)
    ==17955==    by 0x859631E: clone (in /usr/lib64/libc-2.27.so)
    ==17955==
    ==17955== 32 bytes in 1 blocks are possibly lost in loss record 1,234 of 2,079
    ==17955==    at 0x4C2DBAB: malloc (vg_replace_malloc.c:299)
    ==17955==    by 0x4011D17: tls_get_addr_tail.isra.0 (in /usr/lib64/ld-2.27.so)
    ==17955==    by 0x4017997: __tls_get_addr (in /usr/lib64/ld-2.27.so)
    ==17955==    by 0xCAA5173: __cxa_get_globals (in /usr/lib64/libstdc++.so.6.0.25)
    ==17955==    by 0xCAA6186: __cxa_throw (in /usr/lib64/libstdc++.so.6.0.25)
    ==17955==    by 0xC601457: ??? (in /usr/lib64/libproxy.so.1.0.0)
    ==17955==    by 0xC5F6BB6: ??? (in /usr/lib64/libproxy.so.1.0.0)
    ==17955==    by 0xC5F7089: ??? (in /usr/lib64/libproxy.so.1.0.0)
    ==17955==    by 0xC5F7470: px_proxy_factory_get_proxies (in /usr/lib64/libproxy.so.1.0.0)
    ==17955==    by 0xC3E64E3: ??? (in /usr/lib64/gio/modules/libgiolibproxy.so)
    ==17955==    by 0x7463936: g_task_thread_pool_thread (gtask.c:1331)
    ==17955==    by 0x7A3E932: g_thread_pool_thread_proxy (gthreadpool.c:307)
    ==17955==    by 0x7A3DF29: g_thread_proxy (gthread.c:784)
    ==17955==    by 0x8284563: start_thread (in /usr/lib64/libpthread-2.27.so)
    ==17955==    by 0x859631E: clone (in /usr/lib64/libc-2.27.so)
    
    Signed-off-by: Frediano Ziglio <fziglio at redhat.com>
    Acked-by: Christophe Fergeau <cfergeau at redhat.com>

diff --git a/server/tests/valgrind/glib.supp b/server/tests/valgrind/glib.supp
index b5b540f5..fb2cb4a7 100644
--- a/server/tests/valgrind/glib.supp
+++ b/server/tests/valgrind/glib.supp
@@ -520,3 +520,23 @@
 	...
 	fun:px_proxy_factory_get_proxies
 }
+
+{
+	px_proxy_factory_get_proxies2
+	Memcheck:Leak
+	fun:malloc
+	...
+	fun:__tls_get_addr
+	...
+	fun:px_proxy_factory_get_proxies
+}
+
+{
+	gnutls_tls_rnd_leak
+	Memcheck:Leak
+	fun:malloc
+	...
+	fun:__tls_get_addr
+	...
+	fun:gnutls_rnd
+}
commit 337dad1110e5dc7a4b12b91109cd1a6d099269b9
Author: Frediano Ziglio <fziglio at redhat.com>
Date:   Tue Jun 5 12:07:05 2018 +0100

    ci: Merge new Valgrind suppression rule from official glib.supp
    
    Signed-off-by: Frediano Ziglio <fziglio at redhat.com>
    Acked-by: Christophe Fergeau <cfergeau at redhat.com>

diff --git a/server/tests/valgrind/glib.supp b/server/tests/valgrind/glib.supp
index c4625bba..b5b540f5 100644
--- a/server/tests/valgrind/glib.supp
+++ b/server/tests/valgrind/glib.supp
@@ -232,6 +232,16 @@
 	fun:_g_io_module_get_default
 }
 
+# memcheck checks that the third argument to ioctl() is a valid pointer, but
+# some ioctls use that argument as an integer
+{
+   ioctl-with-non-pointer-param
+   Memcheck:Param
+   ioctl(generic)
+   fun:ioctl
+   fun:btrfs_reflink_with_progress
+}
+
 {
 	g-private-get
 	drd:ConflictingAccess