On 02/06/17 09:57, Pavel Grunt wrote: > On Fri, 2017-06-02 at 14:05 +0200, Victor Toso wrote: >> From: Victor Toso <me@xxxxxxxxxxxxxx> >> >> Mainly an issue for kiosk mode. >> >> Related: https://bugzilla.redhat.com/show_bug.cgi?id=1446161 >> Signed-off-by: Victor Toso <victortoso@xxxxxxxxxx> >> --- >> src/virt-viewer-session-spice.c | 4 ++++ >> 1 file changed, 4 insertions(+) >> >> diff --git a/src/virt-viewer-session-spice.c b/src/virt-viewer- >> session-spice.c >> index 5f326aa..106abd1 100644 >> --- a/src/virt-viewer-session-spice.c >> +++ b/src/virt-viewer-session-spice.c >> @@ -725,6 +725,10 @@ >> virt_viewer_session_spice_main_channel_event(SpiceChannel *channel, >> g_free(host); >> if (!ret) { >> g_signal_emit_by_name(session, "session-cancelled"); >> + /* ret is false when dialog did not return >> GTK_RESPONSE_OK. We >> + * should ignore auth error dialog if user has >> cancelled or closed >> + * the dialog */ >> + self->priv->pass_try = 0; > > I'd first change the value and after that emit the signal. > > Acked-by: Pavel Grunt <pgrunt@xxxxxxxxxx> > The patch per se is 'ok-ish' for me, but I don't really understand why would we not want to show the message dialog before the authentication dialog again? I think this could be explained with more details on the commit message. Other than that, I also think the case described in the bug is trying a little bit too hard to be considered a valid use case. -- Eduardo de Barros Lima (Etrunko) Software Engineer - RedHat etrunko@xxxxxxxxxx _______________________________________________ virt-tools-list mailing list virt-tools-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/virt-tools-list
