diff options
| author | Xavier Claessens <xclaesse@gmail.com> | 2010-02-11 01:51:53 +0800 |
|---|---|---|
| committer | Xavier Claessens <xclaesse@gmail.com> | 2010-02-11 03:14:20 +0800 |
| commit | 41f9f0a717f8519cf8c69cb043198980fe503ebc (patch) | |
| tree | 3cff33d8c48cc69a2778f27f56b15372416decba /libempathy-gtk/empathy-contact-list-view.c | |
| parent | ddbfc6ec6a3b2ffab1f83471dbf1f82630292f8a (diff) | |
| download | gsoc2013-empathy-41f9f0a717f8519cf8c69cb043198980fe503ebc.tar gsoc2013-empathy-41f9f0a717f8519cf8c69cb043198980fe503ebc.tar.gz gsoc2013-empathy-41f9f0a717f8519cf8c69cb043198980fe503ebc.tar.bz2 gsoc2013-empathy-41f9f0a717f8519cf8c69cb043198980fe503ebc.tar.lz gsoc2013-empathy-41f9f0a717f8519cf8c69cb043198980fe503ebc.tar.xz gsoc2013-empathy-41f9f0a717f8519cf8c69cb043198980fe503ebc.tar.zst gsoc2013-empathy-41f9f0a717f8519cf8c69cb043198980fe503ebc.zip | |
Fix leaked GtkMenu
GtkMenu is initially unowned, which means ref_count is 1 and floating flag is set.
gtk_menu_popup temporarily adds a ref, that will be dropped when the menu is popped
down. So once the menu is popped up, we call g_object_ref_sink() to clear the floating
flag, and g_object_unref() to drop the initial ref. Only the ref from gtk_menu_popup
stay, and the menu will be finalized when it is popped down.
Fixes bug #609567
Diffstat (limited to 'libempathy-gtk/empathy-contact-list-view.c')
| -rw-r--r-- | libempathy-gtk/empathy-contact-list-view.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/libempathy-gtk/empathy-contact-list-view.c b/libempathy-gtk/empathy-contact-list-view.c index 958aaf0fd..0ed24bdf8 100644 --- a/libempathy-gtk/empathy-contact-list-view.c +++ b/libempathy-gtk/empathy-contact-list-view.c @@ -683,6 +683,8 @@ contact_list_view_popup_menu_idle_cb (gpointer user_data) gtk_menu_popup (GTK_MENU (menu), NULL, NULL, NULL, NULL, data->button, data->time); + g_object_ref_sink (menu); + g_object_unref (menu); } g_slice_free (MenuPopupData, data); @@ -795,6 +797,8 @@ contact_list_view_call_activated_cb ( gtk_widget_show (menu); gtk_menu_popup (GTK_MENU (menu), NULL, NULL, NULL, NULL, event->button, event->time); + g_object_ref_sink (menu); + g_object_unref (menu); g_object_unref (contact); } |