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 <u...@redhat.com>

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@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/spice-devel

Reply via email to