From: Gilbok Lee <gilbok.lee@samsung.com>
Date: Thu, 10 Sep 2020 02:55:17 +0000 (+0900)
Subject: [0.6.239] Fix memory leak
X-Git-Tag: accepted/tizen/unified/20200915.014531^0
X-Git-Url: http://review.tizen.org/git/?p=platform%2Fcore%2Fmultimedia%2Flibmm-player.git;a=commitdiff_plain;h=88c7b7ae174c0de56c378c3ed15bd82bd1f74f85

[0.6.239] Fix memory leak

Change-Id: Ifd794cae579d92ba8bb1d52a0f676e397039f20e
---

diff --git a/packaging/libmm-player.spec b/packaging/libmm-player.spec
index 2ef3859..f226d39 100644
--- a/packaging/libmm-player.spec
+++ b/packaging/libmm-player.spec
@@ -1,6 +1,6 @@
 Name:       libmm-player
 Summary:    Multimedia Framework Player Library
-Version:    0.6.238
+Version:    0.6.239
 Release:    0
 Group:      Multimedia/Libraries
 License:    Apache-2.0
diff --git a/src/mm_player_priv.c b/src/mm_player_priv.c
index d10bfa4..6b19906 100644
--- a/src/mm_player_priv.c
+++ b/src/mm_player_priv.c
@@ -6784,9 +6784,9 @@ _mmplayer_set_next_uri(MMHandleType hplayer, const char *uri, bool is_first_path
 			player->uri_info.uri_list = g_list_append(player->uri_info.uri_list, g_strdup(uri));
 			SECURE_LOGD("add original path : %s", uri);
 		} else {
-			player->uri_info.uri_list = g_list_delete_link(player->uri_info.uri_list, g_list_nth(player->uri_info.uri_list, 0));
-			player->uri_info.uri_list = g_list_insert(player->uri_info.uri_list, g_strdup(uri), 0);
-
+			g_free(g_list_nth_data(player->uri_info.uri_list, 0));
+			player->uri_info.uri_list = g_list_prepend(
+				g_list_delete_link(player->uri_info.uri_list, player->uri_info.uri_list), g_strdup(uri));
 			SECURE_LOGD("change original path : %s", uri);
 		}
 	} else {
@@ -7766,7 +7766,7 @@ __mmplayer_release_misc(mmplayer_t *player)
 static void
 __mmplayer_release_misc_post(mmplayer_t *player)
 {
-	char *original_uri = NULL;
+	gchar *original_uri = NULL;
 	MMPLAYER_FENTER();
 
 	/* player->pipeline is already released before. */
@@ -7803,7 +7803,7 @@ __mmplayer_release_misc_post(mmplayer_t *player)
 
 		mm_player_set_attribute((MMHandleType)player, NULL, "profile_uri",
 				original_uri, (original_uri) ? strlen(original_uri) : (0), NULL);
-
+		MMPLAYER_FREEIF(original_uri);
 	}
 
 	/* clear the audio stream buffer list */