![](https://seccdn.libravatar.org/avatar/e2145bc5cf53dda95c308a3c75e8fef3.jpg?s=120&d=mm&r=g)
Hello community, here is the log from the commit of package virt-viewer for openSUSE:Factory checked in at 2012-05-03 11:01:26 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Comparing /work/SRC/openSUSE:Factory/virt-viewer (Old) and /work/SRC/openSUSE:Factory/.virt-viewer.new (New) ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Package is "virt-viewer", Maintainer is "CARNOLD@suse.com" Changes: -------- --- /work/SRC/openSUSE:Factory/virt-viewer/virt-viewer.changes 2012-03-29 11:42:50.000000000 +0200 +++ /work/SRC/openSUSE:Factory/.virt-viewer.new/virt-viewer.changes 2012-05-03 11:01:28.000000000 +0200 @@ -1,0 +2,8 @@ +Tue Apr 24 10:54:33 MDT 2012 - carnold@novell.com + +- Fix failure of viewer to close when guest is terminated + virtview-git-vnc-close.patch +- Additional upstream fix + virtview-git-uri-crash.patch + +------------------------------------------------------------------- New: ---- virtview-git-uri-crash.patch virtview-git-vnc-close.patch ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Other differences: ------------------ ++++++ virt-viewer.spec ++++++ --- /var/tmp/diff_new_pack.Qrjwvg/_old 2012-05-03 11:01:29.000000000 +0200 +++ /var/tmp/diff_new_pack.Qrjwvg/_new 2012-05-03 11:01:29.000000000 +0200 @@ -25,6 +25,8 @@ Url: http://www.virt-manager.org BuildRoot: %{_tmppath}/%{name}-%{version}-build Source0: virt-viewer-0.5.2.tar.gz +Patch0: virtview-git-uri-crash.patch +Patch1: virtview-git-vnc-close.patch Patch20: netcat.diff Patch21: windows-keycombo.patch Patch22: netware-keycombo.patch @@ -64,6 +66,8 @@ %prep %setup -q +%patch0 -p1 +%patch1 -p1 %patch20 -p1 %patch21 -p1 %patch22 -p1 ++++++ virtview-git-uri-crash.patch ++++++ Subject: Do not crash so easily when given invalid uri From: Marc-André Lureau marcandre.lureau@gmail.com Fri Mar 16 13:17:09 2012 +0100 Date: Fri Mar 16 17:16:57 2012 +0100: Git: e06c822023ac2f5d2b2dae50dd813f4a0d5e908b 'remote-viewer foobar' shouldn't crash Index: virt-viewer-0.5.2/src/remote-viewer.c =================================================================== --- virt-viewer-0.5.2.orig/src/remote-viewer.c +++ virt-viewer-0.5.2/src/remote-viewer.c @@ -728,7 +728,7 @@ remote_viewer_start(VirtViewerApp *app) DEBUG_LOG("Opening display to %s", guri); g_object_set(app, "title", guri, NULL); - if (virt_viewer_util_extract_host(guri, &type, NULL, NULL, NULL, NULL) < 0) { + if (virt_viewer_util_extract_host(guri, &type, NULL, NULL, NULL, NULL) < 0 || type == NULL) { virt_viewer_app_simple_message_dialog(app, _("Cannot determine the connection type from URI")); goto cleanup; } Index: virt-viewer-0.5.2/src/virt-viewer-app.c =================================================================== --- virt-viewer-0.5.2.orig/src/virt-viewer-app.c +++ virt-viewer-0.5.2/src/virt-viewer-app.c @@ -669,6 +669,7 @@ virt_viewer_app_create_session(VirtViewe g_return_val_if_fail(VIRT_VIEWER_IS_APP(self), -1); VirtViewerAppPrivate *priv = self->priv; g_return_val_if_fail(priv->session == NULL, -1); + g_return_val_if_fail(type != NULL, -1); #ifdef HAVE_GTK_VNC if (g_ascii_strcasecmp(type, "vnc") == 0) { Index: virt-viewer-0.5.2/src/virt-viewer-util.c =================================================================== --- virt-viewer-0.5.2.orig/src/virt-viewer-util.c +++ virt-viewer-0.5.2/src/virt-viewer-util.c @@ -81,7 +81,7 @@ virt_viewer_util_extract_host(const char int *port) { xmlURIPtr uri; - char *offset; + char *offset = NULL; if (uristr == NULL || !g_ascii_strcasecmp(uristr, "xen")) @@ -107,16 +107,17 @@ virt_viewer_util_extract_host(const char if (port) *port = uri->port; - offset = strchr(uri->scheme, '+'); + if (uri->scheme) + offset = strchr(uri->scheme, '+'); if (transport) { if (offset) - *transport = g_strdup(offset+1); + *transport = g_strdup(offset + 1); else *transport = NULL; } - if (scheme) { + if (scheme && uri->scheme) { if (offset) *scheme = g_strndup(uri->scheme, offset - uri->scheme); else ++++++ virtview-git-vnc-close.patch ++++++ Subject: Fix close of VNC displays From: Daniel P. Berrange berrange@redhat.com Tue Apr 3 17:31:39 2012 +0100 Date: Wed Apr 4 10:41:32 2012 +0100: Git: 831ebf5dabd23d11186af9707631c29a1ecb3578 When clicking the close button on a virt-viewer window with a VNC session open, while the VNC session terminates, the window does not go away. The problem is that the virt_viewer_session_vnc_disconnected method never gets invoked. The close button triggers a call to virt_viewer_session_clear_displays which unrefs the VirtViewerDisplayVnc instance. This in turn triggers a call to gtk_container_destroy, which destroys all widgets it contains, ie the VncDisplay * object. With the VncDisplay object in its dispose phase, no signals will ever be emitted, thus the 'vnc-disconnected' signal never gets seen. The design issue is that VirtViewerDisplayVnc is assuming it owns the VncDisplay, whereas in fact the real owner is the VirtViewerSessionVnc object. The solution is to introduce a new virt_viewer_display_close method which can be used to de-parent the widget before VirtViewerDisplay is unref'd. The VirtViewerSessionVnc object also needs to hold a full ref on the VncDisplay object, not merely a floating reference * virt-viewer-display-spice.c, virt-viewer-display.c, virt-viewer-display.h: Add virt_viewer_display_close * virt-viewer-display-vnc.c: Deparent VNC widget in virt_viewer_display_close impl * virt-viewer-session-vnc.c: Improve logging * virt-viewer-session.c: Call virt_viewer_display_close before unrefing display * virt-viewer-window.c: Improve logging Signed-off-by: Daniel P. Berrange <berrange@redhat.com> Index: virt-viewer-0.5.2/src/virt-viewer-display-spice.c =================================================================== --- virt-viewer-0.5.2.orig/src/virt-viewer-display-spice.c +++ virt-viewer-0.5.2/src/virt-viewer-display-spice.c @@ -46,6 +46,7 @@ static void virt_viewer_display_spice_se int nkeyvals); static GdkPixbuf *virt_viewer_display_spice_get_pixbuf(VirtViewerDisplay *display); static void virt_viewer_display_spice_release_cursor(VirtViewerDisplay *display); +static void virt_viewer_display_spice_close(VirtViewerDisplay *display G_GNUC_UNUSED); static void virt_viewer_display_spice_finalize(GObject *obj) @@ -69,6 +70,7 @@ virt_viewer_display_spice_class_init(Vir dclass->send_keys = virt_viewer_display_spice_send_keys; dclass->get_pixbuf = virt_viewer_display_spice_get_pixbuf; dclass->release_cursor = virt_viewer_display_spice_release_cursor; + dclass->close = virt_viewer_display_spice_close; g_type_class_add_private(klass, sizeof(VirtViewerDisplaySpicePrivate)); } @@ -239,6 +241,12 @@ virt_viewer_display_spice_release_cursor } +static void +virt_viewer_display_spice_close(VirtViewerDisplay *display G_GNUC_UNUSED) +{ +} + + /* * Local variables: * c-indent-level: 4 Index: virt-viewer-0.5.2/src/virt-viewer-display-vnc.c =================================================================== --- virt-viewer-0.5.2.orig/src/virt-viewer-display-vnc.c +++ virt-viewer-0.5.2/src/virt-viewer-display-vnc.c @@ -39,6 +39,7 @@ struct _VirtViewerDisplayVncPrivate { static void virt_viewer_display_vnc_send_keys(VirtViewerDisplay* display, const guint *keyvals, int nkeyvals); static GdkPixbuf *virt_viewer_display_vnc_get_pixbuf(VirtViewerDisplay* display); +static void virt_viewer_display_vnc_close(VirtViewerDisplay *display); static void virt_viewer_display_vnc_finalize(GObject *obj) @@ -61,6 +62,7 @@ virt_viewer_display_vnc_class_init(VirtV dclass->send_keys = virt_viewer_display_vnc_send_keys; dclass->get_pixbuf = virt_viewer_display_vnc_get_pixbuf; + dclass->close = virt_viewer_display_vnc_close; g_type_class_add_private(klass, sizeof(VirtViewerDisplayVncPrivate)); } @@ -153,7 +155,6 @@ virt_viewer_display_vnc_new(VncDisplay * display = g_object_new(VIRT_VIEWER_TYPE_DISPLAY_VNC, NULL); g_object_ref(vnc); - g_object_ref(vnc); /* Because gtk_container_add steals the first ref */ display->priv->vnc = vnc; gtk_container_add(GTK_CONTAINER(display), GTK_WIDGET(display->priv->vnc)); @@ -188,6 +189,19 @@ virt_viewer_display_vnc_new(VncDisplay * return GTK_WIDGET(display); } + +static void +virt_viewer_display_vnc_close(VirtViewerDisplay *display) +{ + VirtViewerDisplayVnc *vnc = VIRT_VIEWER_DISPLAY_VNC(display); + + /* We're not the real owner, so we shouldn't be letting the container + * destroy the widget. There are still signals that need to be + * propagated to the VirtViewerSession + */ + gtk_container_remove(GTK_CONTAINER(display), GTK_WIDGET(vnc->priv->vnc)); +} + /* * Local variables: * c-indent-level: 4 Index: virt-viewer-0.5.2/src/virt-viewer-display.c =================================================================== --- virt-viewer-0.5.2.orig/src/virt-viewer-display.c +++ virt-viewer-0.5.2/src/virt-viewer-display.c @@ -579,6 +579,19 @@ void virt_viewer_display_release_cursor( klass->release_cursor(self); } + +void virt_viewer_display_close(VirtViewerDisplay *self) +{ + VirtViewerDisplayClass *klass; + + g_return_if_fail(VIRT_VIEWER_IS_DISPLAY(self)); + + klass = VIRT_VIEWER_DISPLAY_GET_CLASS(self); + g_return_if_fail(klass->close != NULL); + + klass->close(self); +} + /* * Local variables: * c-indent-level: 4 Index: virt-viewer-0.5.2/src/virt-viewer-display.h =================================================================== --- virt-viewer-0.5.2.orig/src/virt-viewer-display.h +++ virt-viewer-0.5.2/src/virt-viewer-display.h @@ -75,6 +75,8 @@ struct _VirtViewerDisplayClass { GdkPixbuf *(*get_pixbuf)(VirtViewerDisplay *display); void (*release_cursor)(VirtViewerDisplay *display); + void (*close)(VirtViewerDisplay *display); + /* signals */ void (*display_pointer_grab)(VirtViewerDisplay *display); void (*display_pointer_ungrab)(VirtViewerDisplay *display); @@ -112,6 +114,8 @@ void virt_viewer_display_set_auto_resize gboolean virt_viewer_display_get_auto_resize(VirtViewerDisplay *display); void virt_viewer_display_release_cursor(VirtViewerDisplay *display); +void virt_viewer_display_close(VirtViewerDisplay *display); + G_END_DECLS #endif /* _VIRT_VIEWER_DISPLAY_H */ Index: virt-viewer-0.5.2/src/virt-viewer-session-vnc.c =================================================================== --- virt-viewer-0.5.2.orig/src/virt-viewer-session-vnc.c +++ virt-viewer-0.5.2/src/virt-viewer-session-vnc.c @@ -104,6 +104,7 @@ static void virt_viewer_session_vnc_disconnected(VncDisplay *vnc G_GNUC_UNUSED, VirtViewerSessionVnc *session) { + DEBUG_LOG("Disconnected"); g_signal_emit_by_name(session, "session-disconnected"); /* TODO perhaps? */ /* virt_viewer_display_set_show_hint(VIRT_VIEWER_DISPLAY(session->priv->vnc), */ @@ -234,6 +235,7 @@ virt_viewer_session_vnc_close(VirtViewer g_return_if_fail(self != NULL); + DEBUG_LOG("close vnc=%p", self->priv->vnc); if (self->priv->vnc != NULL) { virt_viewer_session_clear_displays(session); vnc_display_close(self->priv->vnc); @@ -241,6 +243,7 @@ virt_viewer_session_vnc_close(VirtViewer } self->priv->vnc = VNC_DISPLAY(vnc_display_new()); + g_object_ref_sink(self->priv->vnc); g_signal_connect(self->priv->vnc, "vnc-connected", G_CALLBACK(virt_viewer_session_vnc_connected), session); @@ -271,6 +274,7 @@ virt_viewer_session_vnc_new(GtkWindow *m session = g_object_new(VIRT_VIEWER_TYPE_SESSION_VNC, NULL); session->priv->vnc = VNC_DISPLAY(vnc_display_new()); + g_object_ref_sink(session->priv->vnc); session->priv->main_window = g_object_ref(main_window); g_signal_connect(session->priv->vnc, "vnc-connected", Index: virt-viewer-0.5.2/src/virt-viewer-session.c =================================================================== --- virt-viewer-0.5.2.orig/src/virt-viewer-session.c +++ virt-viewer-0.5.2/src/virt-viewer-session.c @@ -280,8 +280,10 @@ void virt_viewer_session_clear_displays( GList *tmp = session->priv->displays; while (tmp) { - g_signal_emit_by_name(session, "session-display-removed", tmp->data); - g_object_unref(tmp->data); + VirtViewerDisplay *display = VIRT_VIEWER_DISPLAY(tmp->data); + g_signal_emit_by_name(session, "session-display-removed", display); + virt_viewer_display_close(display); + g_object_unref(display); tmp = tmp->next; } g_list_free(session->priv->displays); Index: virt-viewer-0.5.2/src/virt-viewer-window.c =================================================================== --- virt-viewer-0.5.2.orig/src/virt-viewer-window.c +++ virt-viewer-0.5.2/src/virt-viewer-window.c @@ -670,6 +670,7 @@ virt_viewer_window_delete(GtkWidget *src void *dummy G_GNUC_UNUSED, VirtViewerWindow *self) { + DEBUG_LOG("Window closed"); virt_viewer_app_window_set_visible(self->priv->app, self, FALSE); return TRUE; } -- To unsubscribe, e-mail: opensuse-commit+unsubscribe@opensuse.org For additional commands, e-mail: opensuse-commit+help@opensuse.org