From: Kyeonghun Lee <kh9090.lee@samsung.com>
Date: Fri, 30 Dec 2016 05:33:26 +0000 (+0900)
Subject: fix memory leak
X-Git-Tag: submit/tizen/20170104.061339~3
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=refs%2Fchanges%2F18%2F107918%2F1;p=platform%2Fcore%2Fmessaging%2Fmsg-service.git

fix memory leak

Change-Id: I2c0e33b0b0d693eb19c75c49fbf89ae74f53ac1c
Signed-off-by: Kyeonghun Lee <kh9090.lee@samsung.com>
---

diff --git a/manager/src/msg-manager.cpp b/manager/src/msg-manager.cpp
index 4f4ffc3..3f66681 100644
--- a/manager/src/msg-manager.cpp
+++ b/manager/src/msg-manager.cpp
@@ -158,6 +158,8 @@ void _incoming_msg_func(app_control_h app_control)
 	msg_err = msg_get_message(msg_handle, msg_id, msg, opt);
 	if (msg_err != MSG_SUCCESS) {
 		MSG_MGR_ERR("msg_get_message() failed [%d]", msg_err);
+		msg_release_struct(&msg);
+		msg_release_struct(&opt);
 		return;
 	}
 
@@ -219,6 +221,8 @@ void _outgoing_msg_func(app_control_h app_control)
 	msg_err = msg_get_message(msg_handle, msg_id, msg, opt);
 	if (msg_err != MSG_SUCCESS) {
 		MSG_MGR_ERR("msg_get_message() failed [%d]", msg_err);
+		msg_release_struct(&msg);
+		msg_release_struct(&opt);
 		return;
 	}
 
@@ -338,6 +342,8 @@ void _add_noti_func(app_control_h app_control)
 	msg_err = msg_get_message(msg_handle, msg_id, msg, opt);
 	if (msg_err != MSG_SUCCESS) {
 		MSG_MGR_ERR("msg_get_message() failed [%d]", msg_err);
+		msg_release_struct(&msg);
+		msg_release_struct(&opt);
 		return;
 	}
 
@@ -445,6 +451,8 @@ void _add_report_noti_func(app_control_h app_control)
 	msg_err = msg_get_message(msg_handle, msg_id, msg, opt);
 	if (msg_err != MSG_SUCCESS) {
 		MSG_MGR_ERR("msg_get_message() failed [%d]", msg_err);
+		msg_release_struct(&msg);
+		msg_release_struct(&opt);
 		return;
 	}
 
@@ -523,6 +531,8 @@ void _insert_only_active_noti_func(app_control_h app_control)
 	msg_err = msg_get_message(msg_handle, msg_id, msg, opt);
 	if (msg_err != MSG_SUCCESS) {
 		MSG_MGR_ERR("msg_get_message() failed [%d]", msg_err);
+		msg_release_struct(&msg);
+		msg_release_struct(&opt);
 		return;
 	}