Re: [Spice-devel] [PATCH 1/9] Use spice_server_destroy() at exit

2016-11-01 Thread Jonathon Jongsma
On Tue, 2016-11-01 at 06:28 -0400, Frediano Ziglio wrote: > > > > > > On Mon, 2016-10-31 at 10:44 -0400, Frediano Ziglio wrote: > > > > > > > > > > > > > > > > > > > The destructor function was calling reds_cleanup() for each > > > > server > > > > at > > > > application exit. But reds_cleanu

Re: [Spice-devel] [PATCH 1/9] Use spice_server_destroy() at exit

2016-11-01 Thread Frediano Ziglio
> > On Mon, 2016-10-31 at 10:44 -0400, Frediano Ziglio wrote: > > > > > > > > > The destructor function was calling reds_cleanup() for each server > > > at > > > application exit. But reds_cleanup() only cleans up a couple of > > > statistics-related variables. We should actually free the server

Re: [Spice-devel] [PATCH 1/9] Use spice_server_destroy() at exit

2016-10-31 Thread Jonathon Jongsma
On Mon, 2016-10-31 at 10:44 -0400, Frediano Ziglio wrote: > > > > > > The destructor function was calling reds_cleanup() for each server > > at > > application exit. But reds_cleanup() only cleans up a couple of > > statistics-related variables. We should actually free the servers > > by > > call

Re: [Spice-devel] [PATCH 1/9] Use spice_server_destroy() at exit

2016-10-31 Thread Frediano Ziglio
> > The destructor function was calling reds_cleanup() for each server at > application exit. But reds_cleanup() only cleans up a couple of > statistics-related variables. We should actually free the servers by > calling spice_server_destroy(). This left reds_cleanup() unused, so the > contents of

[Spice-devel] [PATCH 1/9] Use spice_server_destroy() at exit

2016-10-28 Thread Jonathon Jongsma
The destructor function was calling reds_cleanup() for each server at application exit. But reds_cleanup() only cleans up a couple of statistics-related variables. We should actually free the servers by calling spice_server_destroy(). This left reds_cleanup() unused, so the contents of this functio