Re: [spice-server PATCH 3/3] valgrind/spice.supp: add missing ...

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 8/11/19 2:57 PM, Frediano Ziglio wrote:

There may to be another function (cache_alt_names) between
   gnutls_x509_ext_import_subject_alt_names and
   gnutls_x509_crt_import

cache_alt_names is a static function in gnutls/lib/x509/x509.c
used only in gnutls_x509_crt_import and may be inlined by
the compiler.

Signed-off-by: Uri Lublin <uril@xxxxxxxxxx>

Acked.

If you have the exact environment where this happened worth adding
it in the commit message.

After a little investigation,
Turns out the "problem" was a gnutls-debuginfo package that
was installed on my machine.
With gnutls-debuginfo, valgrind fails for test-listen.
Removing it makes valgrind pass test-listen.

I'll add this information to the commit log.

Thanks,
    Uri.


---
  server/tests/valgrind/spice.supp | 1 +
  1 file changed, 1 insertion(+)

diff --git a/server/tests/valgrind/spice.supp
b/server/tests/valgrind/spice.supp
index aa64b61bc..284833a72 100644
--- a/server/tests/valgrind/spice.supp
+++ b/server/tests/valgrind/spice.supp
@@ -42,6 +42,7 @@
  	Memcheck:Cond
  	...
  	fun:gnutls_x509_ext_import_subject_alt_names
+	...
  	fun:gnutls_x509_crt_import
  	fun:gnutls_x509_trust_list_iter_get_ca
  }

Frediano


_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux Virtualization]     [Linux Virtualization]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]