fix memory leak.submit/tizen_wearable/20160523.042657submit/tizen_mobile/20160523.042646submit/tizen_common/20160530.122415accepted/tizen/wearable/20160523.103222accepted/tizen/mobile/20160523.103227accepted/tizen/common/20160530.132227

Change-Id: Ic82182bfecfca047919cb07f840857d7f94b6782
Signed-off-by: jiseob.jang <jiseob.jang@samsung.com>

2 files changed, 40 insertions, 2 deletions

diff --git a/server/src/account-server-db.c b/server/src/account-server-db.c
index 9ee589b..807837d 100644
--- a/server/src/account-server-db.c
+++ b/server/src/account-server-db.c
@@ -4040,6 +4040,7 @@ int _account_type_query_by_provider_feature_from_global_db(const char* key, GSLi
 		*account_type_list_all = g_slist_append(*account_type_list_all, account_type);
 		_INFO("add account_type");
 	}
+	g_slist_free(account_type_list);
 
 	error_code = _ACCOUNT_ERROR_NONE;
 
@@ -4233,6 +4234,7 @@ int _account_type_query_all_from_global_db(GSList **account_type_list_all)
 		account_type = (account_type_s*)iter->data;
 		*account_type_list_all = g_slist_append(*account_type_list_all, account_type);
 	}
+	g_slist_free(account_type_list);
 
 	error_code = _ACCOUNT_ERROR_NONE;
 CATCH:
diff --git a/server/src/account-server.c b/server/src/account-server.c
index 1c38a52..a43c409 100644
--- a/server/src/account-server.c
+++ b/server/src/account-server.c
@@ -308,6 +308,7 @@ RETURN:
 		_ERR("Account SVC is returning error [%d]", return_code);
 		GError* error = g_error_new (__ACCOUNT_ERROR_quark(), return_code, "RecordNotFound");
 		g_dbus_method_invocation_return_gerror (invocation, error);
+		g_error_free(error);
 	}
 	else
 	{
@@ -385,6 +386,7 @@ gboolean account_manager_account_query_all(AccountManager *obj, GDBusMethodInvoc
 	return_code = 0;
 	_INFO("before calling marshal_account_list");
 	account_list_variant = marshal_account_list(account_list);
+	_account_gslist_account_free(account_list);
 	_INFO("after calling marshal_account_list");
 
 RETURN:
@@ -393,6 +395,7 @@ RETURN:
 	{
 		GError* error = g_error_new (__ACCOUNT_ERROR_quark(), return_code, "RecordNotFound");
 		g_dbus_method_invocation_return_gerror (invocation, error);
+		g_error_free(error);
 	}
 	else
 	{
@@ -467,6 +470,7 @@ gboolean account_manager_account_type_query_all(AccountManager *obj, GDBusMethod
 	return_code = 0;
 	_INFO("before calling marshal_account_type_list");
 	account_type_list_variant = marshal_account_type_list(account_type_list);
+	_account_type_gslist_account_type_free(account_type_list);
 	_INFO("after calling marshal_account_type_list");
 
 RETURN:
@@ -475,6 +479,7 @@ RETURN:
 	{
 		GError* error = g_error_new (__ACCOUNT_ERROR_quark(), return_code, "RecordNotFound");
 		g_dbus_method_invocation_return_gerror (invocation, error);
+		g_error_free(error);
 	}
 	else
 	{
@@ -564,6 +569,7 @@ RETURN:
 		_ERR("Account SVC is returning error [%d]", return_code);
 		GError* error = g_error_new (__ACCOUNT_ERROR_quark(), return_code, "RecordNotFound");
 		g_dbus_method_invocation_return_gerror (invocation, error);
+		g_error_free(error);
 	}
 	else
 	{
@@ -648,6 +654,7 @@ RETURN:
 		_ERR("Account SVC is returning error [%d]", return_code);
 		GError* error = g_error_new (__ACCOUNT_ERROR_quark(), return_code, "RecordNotFound");
 		g_dbus_method_invocation_return_gerror (invocation, error);
+		g_error_free(error);
 	}
 	else
 	{
@@ -731,6 +738,7 @@ RETURN:
 		_ERR("Account SVC is returning error [%d]", return_code);
 		GError* error = g_error_new (__ACCOUNT_ERROR_quark(), return_code, "RecordNotFound");
 		g_dbus_method_invocation_return_gerror (invocation, error);
+		g_error_free(error);
 	}
 	else
 	{
@@ -817,6 +825,7 @@ RETURN:
 		_ERR("Account SVC is returning error [%d]", return_code);
 		GError* error = g_error_new (__ACCOUNT_ERROR_quark(), return_code, "RecordNotFound");
 		g_dbus_method_invocation_return_gerror (invocation, error);
+		g_error_free(error);
 	}
 	else
 	{
@@ -910,6 +919,7 @@ RETURN:
 		_ERR("Account SVC is returning error [%d]", return_code);
 		GError* error = g_error_new (__ACCOUNT_ERROR_quark(), return_code, "RecordNotFound");
 		g_dbus_method_invocation_return_gerror (invocation, error);
+		g_error_free(error);
 	}
 	else
 	{
@@ -1006,6 +1016,7 @@ RETURN:
 		_ERR("Account SVC is returning error [%d]", return_code);
 		GError* error = g_error_new (__ACCOUNT_ERROR_quark(), return_code, "RecordNotFound");
 		g_dbus_method_invocation_return_gerror (invocation, error);
+		g_error_free(error);
 	}
 	else
 	{
@@ -1087,11 +1098,13 @@ RETURN:
 		_ERR("Account SVC is returning error [%d]", return_code);
 		GError* error = g_error_new (__ACCOUNT_ERROR_quark(), return_code, "RecordNotFound");
 		g_dbus_method_invocation_return_gerror (invocation, error);
+		g_error_free(error);
 	}
 	else
 	{
 		account_manager_complete_account_type_query_label_by_locale(object, invocation, label_name);
 	}
+	_ACCOUNT_FREE(label_name);
 
 	return_code = _account_db_close();
 	if (return_code != _ACCOUNT_ERROR_NONE)
@@ -1177,6 +1190,7 @@ account_manager_handle_account_type_query_by_provider_feature(AccountManager *ob
 
 	_INFO("before calling marshal_account_type_list");
 	account_type_list_variant = marshal_account_type_list(account_type_list);
+	_account_type_gslist_account_type_free(account_type_list);
 	_INFO("after calling marshal_account_type_list");
 
 RETURN:
@@ -1185,6 +1199,7 @@ RETURN:
 	{
 		GError* error = g_error_new (__ACCOUNT_ERROR_quark(), _ACCOUNT_ERROR_RECORD_NOT_FOUND, "RecordNotFound");
 		g_dbus_method_invocation_return_gerror (invocation, error);
+		g_error_free(error);
 	}
 	else
 	{
@@ -1260,6 +1275,7 @@ RETURN:
 		_ERR("Account SVC is returning error [%d]", return_code);
 		GError* error = g_error_new (__ACCOUNT_ERROR_quark(), return_code, "RecordNotFound");
 		g_dbus_method_invocation_return_gerror (invocation, error);
+		g_error_free(error);
 	}
 	else
 	{
@@ -1349,6 +1365,7 @@ RETURN:
 	{
 		GError* error = g_error_new (__ACCOUNT_ERROR_quark(), return_code, "RecordNotFound");
 		g_dbus_method_invocation_return_gerror (invocation, error);
+		g_error_free(error);
 	}
 	else
 	{
@@ -1430,6 +1447,7 @@ account_manager_handle_account_query_account_by_user_name(AccountManager *obj,
 
 	_INFO("before calling marshal_account_list_double");
 	account_list_variant = marshal_account_list_double(account_list);
+	_account_glist_account_free(account_list);
 	_INFO("after calling marshal_account_list_double");
 
 	if (return_code != _ACCOUNT_ERROR_NONE)
@@ -1443,6 +1461,7 @@ RETURN:
 	{
 		GError* error = g_error_new (__ACCOUNT_ERROR_quark(), _ACCOUNT_ERROR_RECORD_NOT_FOUND, "RecordNotFound");
 		g_dbus_method_invocation_return_gerror (invocation, error);
+		g_error_free(error);
 	}
 	else
 	{
@@ -1521,6 +1540,7 @@ account_manager_handle_account_query_account_by_package_name(AccountManager *obj
 	_INFO("account_list length= [%d]", g_list_length(account_list));
 
 	account_list_variant = marshal_account_list_double(account_list);
+	_account_glist_account_free(account_list);
 
 	if (return_code != _ACCOUNT_ERROR_NONE)
 	{
@@ -1533,8 +1553,8 @@ RETURN:
 	if (account_list_variant == NULL)
 	{
 		GError* error = g_error_new (__ACCOUNT_ERROR_quark(), return_code, "RecordNotFound");
-		_INFO("sending error Domain[%d] Message[%s] Code[%d]", error->domain, error->message, error->code);
 		g_dbus_method_invocation_return_gerror (invocation, error);
+		g_error_free(error);
 	}
 	else
 	{
@@ -1618,6 +1638,7 @@ account_manager_handle_account_query_account_by_capability(AccountManager *obj,
 	_INFO("account_list length= [%d]", g_list_length(account_list));
 
 	account_list_variant = marshal_account_list_double(account_list);
+	_account_glist_account_free(account_list);
 
 	if (return_code != _ACCOUNT_ERROR_NONE)
 	{
@@ -1631,6 +1652,7 @@ RETURN:
 	{
 		GError* error = g_error_new (__ACCOUNT_ERROR_quark(), _ACCOUNT_ERROR_RECORD_NOT_FOUND, "RecordNotFound");
 		g_dbus_method_invocation_return_gerror (invocation, error);
+		g_error_free(error);
 	}
 	else
 	{
@@ -1711,6 +1733,7 @@ account_manager_handle_account_query_account_by_capability_type(AccountManager *
 
 	_INFO("before calling marshal_account_list_double");
 	account_list_variant = marshal_account_list_double(account_list);
+	_account_glist_account_free(account_list);
 	_INFO("after calling marshal_account_list_double");
 
 	if (return_code != _ACCOUNT_ERROR_NONE)
@@ -1725,6 +1748,7 @@ RETURN:
 	{
 		GError* error = g_error_new (__ACCOUNT_ERROR_quark(), _ACCOUNT_ERROR_RECORD_NOT_FOUND, "RecordNotFound");
 		g_dbus_method_invocation_return_gerror (invocation, error);
+		g_error_free(error);
 	}
 	else
 	{
@@ -1805,6 +1829,7 @@ account_manager_handle_account_query_capability_by_account_id(AccountManager *ob
 
 	_INFO("before calling marshal_capability_list");
 	capability_list_variant = marshal_capability_list(capability_list);
+	_account_gslist_capability_free(capability_list);
 	_INFO("after calling marshal_capability_list");
 
 	if (return_code != _ACCOUNT_ERROR_NONE)
@@ -1819,6 +1844,7 @@ RETURN:
 	{
 		GError* error = g_error_new (__ACCOUNT_ERROR_quark(), _ACCOUNT_ERROR_RECORD_NOT_FOUND, "RecordNotFound");
 		g_dbus_method_invocation_return_gerror (invocation, error);
+		g_error_free(error);
 	}
 	else
 	{
@@ -1903,6 +1929,7 @@ RETURN:
 		_ERR("Account SVC is returning error [%d]", return_code);
 		GError* error = g_error_new (__ACCOUNT_ERROR_quark(), return_code, "RecordNotFound");
 		g_dbus_method_invocation_return_gerror (invocation, error);
+		g_error_free(error);
 	}
 	else
 	{
@@ -1978,7 +2005,7 @@ gboolean account_manager_handle_account_type_query_provider_feature_by_app_id(Ac
 	}
 
 	feature_record_list_variant = provider_feature_list_to_variant(feature_record_list);
-	_INFO("%s", g_variant_print(feature_record_list_variant, true));
+	_account_type_gslist_feature_free(feature_record_list);
 
 	if (return_code != _ACCOUNT_ERROR_NONE)
 	{
@@ -1991,6 +2018,7 @@ RETURN:
 	{
 		GError* error = g_error_new (__ACCOUNT_ERROR_quark(), return_code, "RecordNotFound");
 		g_dbus_method_invocation_return_gerror (invocation, error);
+		g_error_free(error);
 	}
 	else
 	{
@@ -2069,6 +2097,7 @@ RETURN:
 	{
 		GError* error = g_error_new (__ACCOUNT_ERROR_quark(), return_code, "RecordNotFound");
 		g_dbus_method_invocation_return_gerror (invocation, error);
+		g_error_free(error);
 	}
 	else
 	{
@@ -2156,6 +2185,7 @@ RETURN:
 	{
 		GError* error = g_error_new (__ACCOUNT_ERROR_quark(), return_code, "RecordNotFound");
 		g_dbus_method_invocation_return_gerror (invocation, error);
+		g_error_free(error);
 	}
 	else
 	{
@@ -2240,6 +2270,7 @@ RETURN:
 	{
 		GError* error = g_error_new (__ACCOUNT_ERROR_quark(), return_code, "RecordNotFound");
 		g_dbus_method_invocation_return_gerror (invocation, error);
+		g_error_free(error);
 	}
 	else
 	{
@@ -2315,12 +2346,14 @@ gboolean account_manager_handle_account_type_query_label_by_app_id (AccountManag
 	}
 
 	label_list_variant = label_list_to_variant (label_list);
+	_account_type_gslist_label_free(label_list);
 
 RETURN:
 	if (return_code != _ACCOUNT_ERROR_NONE)
 	{
 		GError* error = g_error_new (__ACCOUNT_ERROR_quark(), return_code, "RecordNotFound");
 		g_dbus_method_invocation_return_gerror (invocation, error);
+		g_error_free(error);
 	}
 	else
 	{
@@ -2409,6 +2442,7 @@ RETURN:
 		_ERR("Account SVC is returning error [%d]", return_code);
 		GError* error = g_error_new (__ACCOUNT_ERROR_quark(), return_code, "RecordNotFound");
 		g_dbus_method_invocation_return_gerror (invocation, error);
+		g_error_free(error);
 	}
 	else
 	{
@@ -2488,6 +2522,7 @@ RETURN:
 		_ERR("Account SVC is returning error [%d]", return_code);
 		GError* error = g_error_new (__ACCOUNT_ERROR_quark(), return_code, "RecordNotFound");
 		g_dbus_method_invocation_return_gerror (invocation, error);
+		g_error_free(error);
 	}
 	else
 	{
@@ -2581,6 +2616,7 @@ RETURN:
 		_ERR("Account SVC is returning error [%d]", return_code);
 		GError* error = g_error_new (__ACCOUNT_ERROR_quark(), return_code, "RecordNotFound");
 		g_dbus_method_invocation_return_gerror (invocation, error);
+		g_error_free(error);
 	}
 	else
 	{