[Spice-devel] [PATCH spice-server] test-leaks: Checks some leaks using TLS

Frediano Ziglio fziglio at redhat.com
Wed Mar 8 12:20:26 UTC 2017


> 
> On Tue, Mar 07, 2017 at 04:56:00PM +0000, Frediano Ziglio wrote:
> > Verify stuff are freed correctly (like TLS context).
> > The different PKI file required are generated with
> > base values (localhost and rsa 1024).
> > 
> > Signed-off-by: Frediano Ziglio <fziglio at redhat.com>
> > ---
> >  server/tests/Makefile.am         |  1 +
> >  server/tests/pki/ca-cert.pem     | 16 ++++++++++++++++
> >  server/tests/pki/server-cert.pem | 18 ++++++++++++++++++
> >  server/tests/pki/server-key.pem  | 15 +++++++++++++++
> >  server/tests/test-leaks.c        |  9 +++++++++
> >  5 files changed, 59 insertions(+)
> >  create mode 100644 server/tests/pki/ca-cert.pem
> >  create mode 100644 server/tests/pki/server-cert.pem
> >  create mode 100644 server/tests/pki/server-key.pem
> > 
> > diff --git a/server/tests/Makefile.am b/server/tests/Makefile.am
> > index db23177..3f61faa 100644
> > --- a/server/tests/Makefile.am
> > +++ b/server/tests/Makefile.am
> > @@ -1,6 +1,7 @@
> >  NULL =
> >  
> >  AM_CPPFLAGS =					\
> > +	-DSPICE_TOP_SRCDIR=\"$(abs_top_srcdir)\"\
> 
> Should we use g_test_build_filename() instead?
> 
> Acked-by: Christophe Fergeau <cfergeau at redhat.com>
> 
> Christophe
> 

I considered it but I found overkilling at the moment:
- requires to include a glib.mk (which I didn't find on my installation
  but only on glib sources). And this file should be then updated;
- requires to use different Makefile macros;
- requires to setup some additional environment variable to
  run the tests manually. This make a bit harder to debug the tests.

Frediano


More information about the Spice-devel mailing list