On Thu, Apr 09, 2015 at 01:36:50PM +0200, Fabiano Fidêncio wrote:
> Whenever we reach this branch while connecting we will create a new
> spice session (from the dialog showed to the user). So, destroying the
> channel on this situation seems sane enough.
> It also avoids an error dialog to be popped out twice with (basically)
> the same information.
>
> Related to: rhbz#1085216
> ---
> src/virt-viewer-session-spice.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/virt-viewer-session-spice.c b/src/virt-viewer-session-spice.c
> index 851d2f6..10dd149 100644
> --- a/src/virt-viewer-session-spice.c
> +++ b/src/virt-viewer-session-spice.c
> @@ -610,7 +610,7 @@ virt_viewer_session_spice_main_channel_event(SpiceChannel
> *channel,
> spice_session_connect(self->priv->session);
> }
> } else {
> - g_signal_emit_by_name(session, "session-disconnected", error ?
> error->message : NULL);
> + virt_viewer_session_spice_channel_destroy(NULL, channel,
> session);Are the SPICE_CHANNEL_ERROR_IO, SPICE_CHANNEL_ERROR_LINK, SPICE_CHANNEL_ERROR_TLS below different or should they also destroy the channel rather than emitting a "session-disconnected" signal? Christophe
pgpvRQsw9gQJh.pgp
Description: PGP signature
_______________________________________________ virt-tools-list mailing list [email protected] https://www.redhat.com/mailman/listinfo/virt-tools-list
