On Thu, 2016-12-01 at 11:09 +0000, Frediano Ziglio wrote:
> Signed-off-by: Frediano Ziglio <fziglio@xxxxxxxxxx>
> ---
> server/reds.c | 7 +++++++
> 1 file changed, 7 insertions(+)
>
> diff --git a/server/reds.c b/server/reds.c
> index 17e5ada..d350331 100644
> --- a/server/reds.c
> +++ b/server/reds.c
> @@ -3592,6 +3592,8 @@ SPICE_GNUC_VISIBLE void
> spice_server_destroy(SpiceServer *reds)
> servers = g_list_remove(servers, reds);
> pthread_mutex_unlock(&global_reds_lock);
>
> + g_list_free_full(reds->qxl_instances,
> (GDestroyNotify)red_qxl_destroy);
> +
> if (reds->inputs_channel) {
> reds_unregister_channel(reds, RED_CHANNEL(reds-
> >inputs_channel));
> red_channel_destroy(RED_CHANNEL(reds->inputs_channel));
> @@ -3600,6 +3602,11 @@ SPICE_GNUC_VISIBLE void
> spice_server_destroy(SpiceServer *reds)
> red_channel_destroy(RED_CHANNEL(reds->main_channel));
> }
> reds_core_timer_remove(reds, reds->mig_timer);
> +
> + if (reds->main_dispatcher) {
> + g_object_unref(reds->main_dispatcher);
> + }
> +
This chunk appears unrelated to the qxl instances and is not mentioned
anywhere in the commit log. I'm not necessarily opposed to adding both
of these things in the same patch, but then the log should mention them
both.
Jonathon
> reds_cleanup(reds);
> #ifdef RED_STATISTICS
> stat_file_free(reds->stat_file);
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel