Two statements in virt_viewer_session_spice_main_channel_event() are wrapped in a { } block, but this is unneeded. Signed-off-by: Christophe Fergeau <cfergeau@xxxxxxxxxx> --- src/virt-viewer-session-spice.c | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/src/virt-viewer-session-spice.c b/src/virt-viewer-session-spice.c index 9b52ec0..5f326aa 100644 --- a/src/virt-viewer-session-spice.c +++ b/src/virt-viewer-session-spice.c @@ -694,12 +694,10 @@ virt_viewer_session_spice_main_channel_event(SpiceChannel *channel, gchar *host = NULL; g_debug("main channel: auth failure (wrong username/password?)"); - { - error = spice_channel_get_error(channel); - username_required = g_error_matches(error, - SPICE_CLIENT_ERROR, - SPICE_CLIENT_ERROR_AUTH_NEEDS_PASSWORD_AND_USERNAME); - } + error = spice_channel_get_error(channel); + username_required = g_error_matches(error, + SPICE_CLIENT_ERROR, + SPICE_CLIENT_ERROR_AUTH_NEEDS_PASSWORD_AND_USERNAME); if (self->priv->pass_try > 0) g_signal_emit_by_name(session, "session-auth-refused", -- 2.9.3 _______________________________________________ virt-tools-list mailing list virt-tools-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/virt-tools-list