[sound-juicer/gnome-2-26] Fix reference counting in metadata getter
- From: Bastien Nocera <hadess src gnome org>
- To: svn-commits-list gnome org
- Cc:
- Subject: [sound-juicer/gnome-2-26] Fix reference counting in metadata getter
- Date: Thu, 26 Nov 2009 11:20:24 +0000 (UTC)
commit 0d58eaa0075f91076034525025e6c41783c9f3b3
Author: Bastien Nocera <hadess hadess net>
Date: Tue Nov 24 15:43:15 2009 +0000
Fix reference counting in metadata getter
Make sure that both the thread and the idle signals have
access to an object that's not destroyed yet.
libjuicer/sj-metadata-getter.c | 11 ++++++++---
1 files changed, 8 insertions(+), 3 deletions(-)
---
diff --git a/libjuicer/sj-metadata-getter.c b/libjuicer/sj-metadata-getter.c
index 9e5e6c9..8f16efe 100644
--- a/libjuicer/sj-metadata-getter.c
+++ b/libjuicer/sj-metadata-getter.c
@@ -164,6 +164,7 @@ fire_signal_idle (SjMetadataGetterSignal *signal)
g_object_unref (signal->metadata);
if (signal->error != NULL)
g_error_free (signal->error);
+ g_object_unref (signal->mdg);
g_free (signal);
return FALSE;
@@ -172,7 +173,6 @@ fire_signal_idle (SjMetadataGetterSignal *signal)
static gpointer
lookup_cd (SjMetadataGetter *mdg)
{
- SjMetadata *metadata;
guint i;
SjMetadataGetterPrivate *priv;
GError *error = NULL;
@@ -196,6 +196,7 @@ lookup_cd (SjMetadataGetter *mdg)
priv->url = NULL;
for (i = 0; i < G_N_ELEMENTS (types); i++) {
+ SjMetadata *metadata;
GList *albums;
metadata = g_object_new (types[i],
@@ -213,7 +214,7 @@ lookup_cd (SjMetadataGetter *mdg)
signal = g_new0 (SjMetadataGetterSignal, 1);
signal->albums = albums;
- signal->mdg = mdg;
+ signal->mdg = g_object_ref (mdg);
signal->metadata = metadata;
g_idle_add ((GSourceFunc)fire_signal_idle, signal);
break;
@@ -228,12 +229,14 @@ lookup_cd (SjMetadataGetter *mdg)
signal = g_new0 (SjMetadataGetterSignal, 1);
signal->error = error;
- signal->mdg = mdg;
+ signal->mdg = g_object_ref (mdg);
g_idle_add ((GSourceFunc)fire_signal_idle, signal);
break;
}
}
+ g_object_unref (mdg);
+
return NULL;
}
@@ -242,11 +245,13 @@ sj_metadata_getter_list_albums (SjMetadataGetter *mdg, GError **error)
{
GThread *thread;
+ g_object_ref (mdg);
thread = g_thread_create ((GThreadFunc)lookup_cd, mdg, TRUE, error);
if (thread == NULL) {
g_set_error (error,
SJ_ERROR, SJ_ERROR_INTERNAL_ERROR,
_("Could not create CD lookup thread"));
+ g_object_unref (mdg);
return FALSE;
}
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]