[virt-tools-list] [virt-viewer][PATCH 4/5 v2] Do not wait for a guest that will never show up
Jonathon Jongsma
jjongsma at redhat.com
Thu Mar 26 16:06:38 UTC 2015
On Thu, 2015-03-26 at 16:35 +0100, Fabiano Fidêncio wrote:
> All the situations where virt_viewer_update_display() can fail are
> those when we won't be able to connect regardless of what changes on the
> remote host. So, propagate the error instead of keep waiting.
>
> Related: rhbz#1085216
> ---
> src/virt-viewer.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/virt-viewer.c b/src/virt-viewer.c
> index 3a570bb..386cdcc 100644
> --- a/src/virt-viewer.c
> +++ b/src/virt-viewer.c
> @@ -775,7 +775,7 @@ virt_viewer_initial_connect(VirtViewerApp *app, GError **error)
> }
>
> if (!virt_viewer_update_display(self, dom, &err))
> - goto wait;
> + goto cleanup;
>
> ret = VIRT_VIEWER_APP_CLASS(virt_viewer_parent_class)->initial_connect(app, &err);
> if (ret || err)
ACK
More information about the virt-tools-list
mailing list