ayatana-commits team mailing list archive

[noreply@xxxxxxxxxxxxx]

Merge authors:
  Kees Cook (kees)
Related merge proposals:
  https://code.launchpad.net/~kees/indicator-messages/trunk-lp-533021/+merge/24304
  proposed by: Kees Cook (kees)
  review: Approve - Ted Gould (ted)
------------------------------------------------------------
revno: 187 [merge]
committer: Ted Gould <ted@xxxxxxxx>
branch nick: trunk
timestamp: Wed 2010-04-28 08:07:25 -0500
message:
  Patch from Kees to ensure the hashtable keeps its memory
modified:
  src/messages-service.c


--
lp:indicator-messages
https://code.launchpad.net/~indicator-applet-developers/indicator-messages/trunk

Your team ayatana-commits is subscribed to branch lp:indicator-messages.
To unsubscribe from this branch go to https://code.launchpad.net/~indicator-applet-developers/indicator-messages/trunk/+edit-subscription

=== modified file 'src/messages-service.c'
--- src/messages-service.c	2010-04-17 17:59:27 +0000
+++ src/messages-service.c	2010-04-28 10:33:57 +0000
@@ -315,7 +315,7 @@
 }
 
 /* Add a definition file into the black list and eclipse
-   and launchers that have the same file. */
+   any launchers that have the same file. */
 static gboolean
 blacklist_add (gpointer udata)
 {
@@ -359,7 +359,7 @@
 	}
 
 	/* Actually blacklist this thing */
-	g_hash_table_insert(blacklist, desktop, definition);
+	g_hash_table_insert(blacklist, g_strdup(desktop), g_strdup(definition));
 	g_debug("Adding Blacklist item '%s' for desktop '%s'", definition, desktop);
 
 	/* Go through and eclipse folks */