]> Pileus Git - ~andy/gtk/commitdiff
entry: guard for != NULL before unreffing the pixbuf
authorCosimo Cecchi <cosimoc@gnome.org>
Sun, 8 Jul 2012 04:27:30 +0000 (00:27 -0400)
committerCosimo Cecchi <cosimoc@gnome.org>
Sun, 8 Jul 2012 04:34:33 +0000 (00:34 -0400)
In gtk_entry_get_icon_pixbuf() we unreference the pixbuf that
_gtk_icon_helper_ensure_pixbuf() gives us back, since the function
doesn't return a reference, and by doing so we're able to return the
reference owned by the cached icon helper.
Since the icon helper method can return NULL though, if no icon
properties have been set on it, guard for != NULL before unreffing the
pixbuf, as that would cause a critical warning.

https://bugzilla.gnome.org/show_bug.cgi?id=679537

gtk/gtkentry.c

index ef58c2e330b726e2e33e140f3a2ccfac08a1ad8f..db83f058d46e8f60d7e5bbbd2532f77fc5f32f8f 100644 (file)
@@ -7946,7 +7946,8 @@ gtk_entry_get_icon_pixbuf (GtkEntry             *entry,
    * the icon helper's cache ref directly.
    */
   pixbuf = gtk_entry_ensure_pixbuf (entry, icon_pos);
-  g_object_unref (pixbuf);
+  if (pixbuf)
+    g_object_unref (pixbuf);
 
   return pixbuf;
 }