commit 56c5f98929fde24e31283059faa769e85c057b6e
Author: James Burton <burtonjam...@googlemail.com>
Date: Sun Jul 3 18:31:58 2011 +0200
fix itdb_free crash
I've seen a few libgpod crashes in itdb_free() which appear to be due to a
double free of itdb->priv->genius_cuid. Also I assume the check before the
first g_free() can be removed since g_free does nothing if passed NULL.
src/itdb_itunesdb.c | 4 +---
1 files changed, 1 insertions(+), 3 deletions(-)
---
diff --git a/src/itdb_itunesdb.c b/src/itdb_itunesdb.c
index 446e43f..8e47c26 100644
--- a/src/itdb_itunesdb.c
+++ b/src/itdb_itunesdb.c
@@ -1355,8 +1355,7 @@ void itdb_free (Itdb_iTunesDB *itdb)
(GFunc)(itdb_playlist_free), NULL);
}
- if (itdb->priv->genius_cuid)
- g_free(itdb->priv->genius_cuid);
+ g_free(itdb->priv->genius_cuid);
}
g_list_free (itdb->playlists);
@@ -1367,7 +1366,6 @@ void itdb_free (Itdb_iTunesDB *itdb)
itdb_device_free (itdb->device);
if (itdb->userdata && itdb->userdata_destroy)
(*itdb->userdata_destroy) (itdb->userdata);
- g_free (itdb->priv->genius_cuid);
g_free (itdb->priv);
g_free (itdb);
}
------------------------------------------------------------------------------
All of the data generated in your IT infrastructure is seriously valuable.
Why? It contains a definitive record of application performance, security
threats, fraudulent activity, and more. Splunk takes this data and makes
sense of it. IT sense. And common sense.
http://p.sf.net/sfu/splunk-d2d-c2
_______________________________________________
gtkpod-cvs2 mailing list
gtkpod-cvs2@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/gtkpod-cvs2
