From: Minje Ahn <minje.ahn@samsung.com>
Date: Wed, 8 Aug 2018 03:42:17 +0000 (+0900)
Subject: Fix memory leak
X-Git-Tag: submit/tizen_4.0/20180808.060727^0
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=846c5544a6b7153cec6ab7b7e4d4b68b358ced9b;p=platform%2Fcore%2Fapi%2Fmedia-controller.git

Fix memory leak

Change-Id: Id41c86f180203f4c4f9e575760f0585ec10a77b6
Signed-off-by: Minje Ahn <minje.ahn@samsung.com>
---

diff --git a/src/media_controller_server.c b/src/media_controller_server.c
index c4abf86..6e4380f 100755
--- a/src/media_controller_server.c
+++ b/src/media_controller_server.c
@@ -957,11 +957,18 @@ int mc_server_update_playlist_done(mc_server_h server, mc_playlist_h playlist)
 	}
 
 	ret = mc_db_update_playlist(mc_server->db_handle, mc_server->server_name, _playlist->playlist_name, query_list);
-	mc_retvm_if(ret != MEDIA_CONTROLLER_ERROR_NONE, ret, "fail to mc_db_update_playlist");
+	if (ret != MEDIA_CONTROLLER_ERROR_NONE) {
+		mc_error("fail to mc_db_update_playlist");
+		goto RELEASE;
+	}
 
 	ret = __mc_server_send_message(mc_server, MC_DBUS_UPDATE_INTERFACE, MC_DBUS_SIGNAL_NAME_PLAYLIST, MC_PLAYLIST_UPDATED, 0, _playlist->playlist_name);
-	mc_retvm_if(ret != MEDIA_CONTROLLER_ERROR_NONE, ret, "fail to __mc_server_send_message");
+	if (ret != MEDIA_CONTROLLER_ERROR_NONE) {
+		mc_error("fail to __mc_server_send_message");
+		goto RELEASE;
+	}
 
+RELEASE:
 	for (idx = 0; idx < length; idx++) {
 		__mc_server_playlist_item_release((mc_playlist_item_s*)g_list_nth_data(_playlist->item_list, idx));
 	}