From ff874b128a43e428f8acbb3869551b130fd44cee Mon Sep 17 00:00:00 2001 From: Stefan Schmidt Date: Fri, 28 Mar 2014 10:51:29 +0100 Subject: [PATCH] music-control: Do not leak the allocated metadata strings Fix for ca07e7d505274377635fda2a79d8ac8c61877ea6 Thanks are going to Davide Andreoli for spotting it. --- src/modules/music-control/e_mod_main.c | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/src/modules/music-control/e_mod_main.c b/src/modules/music-control/e_mod_main.c index b8e7244ef..36aaaa743 100644 --- a/src/modules/music-control/e_mod_main.c +++ b/src/modules/music-control/e_mod_main.c @@ -224,7 +224,8 @@ parse_metadata(E_Music_Control_Module_Context *ctxt, Eina_Value *array) for (i = 0; i < eina_value_array_count(array); i++) { - const char *key, *str_val, *str_markup; + const char *key, *str_val; + char *str_markup; Eina_Value st, subst; Efreet_Uri *uri; @@ -236,6 +237,7 @@ parse_metadata(E_Music_Control_Module_Context *ctxt, Eina_Value *array) eina_value_struct_get(&subst, "arg0", &str_val); str_markup = evas_textblock_text_utf8_to_markup(NULL, str_val); ctxt->meta_title = eina_stringshare_add(str_markup); + free(str_markup); eina_value_flush(&subst); } else if (!strcmp(key, "xesam:album")) @@ -244,6 +246,7 @@ parse_metadata(E_Music_Control_Module_Context *ctxt, Eina_Value *array) eina_value_struct_get(&subst, "arg0", &str_val); str_markup = evas_textblock_text_utf8_to_markup(NULL, str_val); ctxt->meta_album = eina_stringshare_add(str_markup); + free(str_markup); eina_value_flush(&subst); } else if (!strcmp(key, "xesam:artist")) @@ -254,6 +257,7 @@ parse_metadata(E_Music_Control_Module_Context *ctxt, Eina_Value *array) eina_value_array_get(&arr, 0, &str_val); str_markup = evas_textblock_text_utf8_to_markup(NULL, str_val); ctxt->meta_artist = eina_stringshare_add(str_markup); + free(str_markup); eina_value_flush(&arr); eina_value_flush(&subst); }