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

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

 



> 
> 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@xxxxxxxxxx>
> > ---
> >  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@xxxxxxxxxx>
> 
> 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
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel




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