AuroraMiddleware/gtk
1
0
Fork 1
mirror of https://gitlab.gnome.org/GNOME/gtk.git synced 2024-11-06 08:40:08 +00:00
Code Issues Projects Releases Wiki Activity

[filechooser] Unref cancellables we got from the model, not the ones that come into the callback

Browse Source 
The model_cancellables have an extra ref when we get them from gtk_tree_model_get().
So, we need to unref them always.  On the other hand, the cancellables that get
passed as arguments to the callbacks in question are memory-managed by
GtkFileSystemModel.

Reported by Morten Welinder <mortenw@gnome.org>

https://bugzilla.gnome.org/show_bug.cgi?id=646460
This commit is contained in:
Federico Mena Quintero 2011-04-01 16:06:51 -06:00
parent a24f61fb37
commit 58609cfdaf
2 changed files with 13 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
gtk/gtkfilechooserbutton.c
View File

@ -1399,7 +1399,7 @@ set_info_get_info_cb (GCancellable *cancellable,
GdkPixbuf *pixbuf;
GtkTreePath *path;
GtkTreeIter iter;
GCancellable *model_cancellable;
GCancellable *model_cancellable = NULL;
struct SetDisplayNameData *data = callback_data;
gboolean is_folder;
@ -1452,7 +1452,8 @@ out:
gtk_tree_row_reference_free (data->row_ref);
g_free (data);
g_object_unref (cancellable);
if (model_cancellable)
g_object_unref (model_cancellable);
}
static void
@ -1578,7 +1579,7 @@ model_add_special_get_info_cb (GCancellable *cancellable,
GtkTreeIter iter;
GtkTreePath *path;
GdkPixbuf *pixbuf;
GCancellable *model_cancellable;
GCancellable *model_cancellable = NULL;
struct ChangeIconThemeData *data = user_data;
gchar *name;
@ -1631,7 +1632,8 @@ out:
gtk_tree_row_reference_free (data->row_ref);
g_free (data);
g_object_unref (cancellable);
if (model_cancellable)
g_object_unref (model_cancellable);
}
static inline void

10
gtk/gtkfilechooserdefault.c
View File

@ -763,7 +763,10 @@ shortcuts_free_row_data (GtkFileChooserDefault *impl,
-1);
if (cancellable)
g_cancellable_cancel (cancellable);
{
g_cancellable_cancel (cancellable);
g_object_unref (cancellable);
}
if (!(shortcut_type == SHORTCUT_TYPE_FILE ||
shortcut_type == SHORTCUT_TYPE_VOLUME) ||
@ -1457,7 +1460,7 @@ get_file_info_finished (GCancellable *cancellable,
GdkPixbuf *pixbuf;
GtkTreePath *path;
GtkTreeIter iter;
GCancellable *model_cancellable;
GCancellable *model_cancellable = NULL;
struct ShortcutsInsertRequest *request = data;
path = gtk_tree_row_reference_get_path (request->row_ref);
@ -1560,7 +1563,8 @@ out:
g_free (request->label_copy);
g_free (request);
g_object_unref (cancellable);
if (model_cancellable)
g_object_unref (model_cancellable);
}
/* FIXME: GtkFileSystem needs a function to split a remote path