From e91f14cb87db2d4549dd387e4ff2a973f8ab1c27 Mon Sep 17 00:00:00 2001
From: John Ferlan <jferlan@redhat.com>
Date: Tue, 23 Jul 2013 10:29:10 -0400
Subject: [PATCH] valgrind.supp: Add more valgrind suppression paths

Update based on recent run/failures seen
---
 tests/.valgrind.supp | 60 ++++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 60 insertions(+)

diff --git a/tests/.valgrind.supp b/tests/.valgrind.supp
index 10cc3c0dc7..f04912d0a2 100644
--- a/tests/.valgrind.supp
+++ b/tests/.valgrind.supp
@@ -75,3 +75,63 @@
     ...
     obj:*/lib*/libc-2.*so*
 }
+#
+# commandtest validates the various threaded commands. The
+# virThreadCreate() routine allocates and passes args to the
+# new thread which now owns the 'args' and thus cannot be free'd
+#
+{
+    commandtestLeak1
+    Memcheck:Leak
+    fun:calloc
+    fun:virAlloc
+    fun:virThreadCreate
+    fun:mymain
+    fun:virtTestMain
+}
+#
+# The Error code requires static memory that is never free'd
+# for thread local storage to store error message/data
+#
+{
+    commandtestLeak2
+    Memcheck:Leak
+    fun:calloc
+    fun:virAlloc
+    ...
+    fun:vir*LastError*
+    fun:virEventRunDefaultImpl
+    fun:virCommandThreadWorker
+    fun:virThreadHelper
+    fun:start_thread
+    fun:clone
+}
+#
+# Some of the commandtests (test0, test1, test4, & test18) cause the
+# following traceback although it appears the memory is properly freed
+#
+{
+    commandtestLeak3
+    Memcheck:Leak
+    fun:calloc
+    fun:virAllocN
+    fun:virEventPollRunOnce
+    fun:virEventRunDefaultImpl
+    fun:virCommandThreadWorker
+    fun:virThreadHelper
+    fun:start_thread
+    fun:clone
+}
+#
+# seclabeltest relies on 'selabel_close' which is not in libvirt
+#
+{
+    seclabeltestcond1
+    Memcheck:Cond
+    obj:/usr/lib64/libselinux.so.1
+    fun:selabel_close
+    fun:virSecuritySELinuxSecurityDriverClose
+    fun:virSecurityManagerDispose
+    fun:virObjectUnref
+    fun:main
+}