[virt-tools-list] [virt-viewer v2 2/4] timed-revealer: Don't release reference we don't own

Fabiano Fidêncio fidencio at redhat.com
Tue Jun 28 15:23:00 UTC 2016


From: Christophe Fergeau <cfergeau at redhat.com>

VirtViewerTimedRevealer::evBox is created in
virt_viewer_timed_revealer_new() and will be passed to
gtk_container_add() through gtk_overlay_add_overlay(overlay,
virt_viewer_timed_revealer_get_overlay_widget(priv->revealer))

This means VirtViewerTimedRevealer does not own a reference on evBox,
and that it should not try to release it in
VirtViewerTimedRevealer::dispose()
---
 src/virt-viewer-timed-revealer.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/virt-viewer-timed-revealer.c b/src/virt-viewer-timed-revealer.c
index cad9ede..61b2d20 100644
--- a/src/virt-viewer-timed-revealer.c
+++ b/src/virt-viewer-timed-revealer.c
@@ -125,7 +125,7 @@ virt_viewer_timed_revealer_dispose(GObject *object)
     VirtViewerTimedRevealer *self = VIRT_VIEWER_TIMED_REVEALER(object);
     VirtViewerTimedRevealerPrivate *priv = self->priv;
 
-    g_clear_object(&priv->evBox);
+    priv->evBox = NULL;
     priv->revealer = NULL;
 
     if (priv->timeout_id) {
-- 
2.7.4




More information about the virt-tools-list mailing list