← Back to team overview

ayatana-commits team mailing list archive

[Merge] lp:~ted/indicator-application/damn-missed-codys-review into lp:indicator-application

 

Ted Gould has proposed merging lp:~ted/indicator-application/damn-missed-codys-review into lp:indicator-application.

Requested reviews:
  Indicator Applet Developers (indicator-applet-developers)


I missed Cody's review of the last branch, and this is the memory leak that he found.
-- 
https://code.launchpad.net/~ted/indicator-application/damn-missed-codys-review/+merge/32591
Your team ayatana-commits is subscribed to branch lp:indicator-application.
=== modified file 'src/application-service-appstore.c'
--- src/application-service-appstore.c	2010-08-11 20:55:25 +0000
+++ src/application-service-appstore.c	2010-08-13 14:52:42 +0000
@@ -298,6 +298,7 @@
 
 		g_hash_table_insert(hash, g_strdup(key), GINT_TO_POINTER(val));
 	}
+	g_strfreev(keys);
 	g_key_file_free(keyfile);
 
 	return;


Follow ups