From: DoHyun Pyun <dh79.pyun@samsung.com>
Date: Wed, 21 Sep 2016 08:11:32 +0000 (+0900)
Subject: Fix svace issues
X-Git-Tag: accepted/tizen/common/20160922.120604^0
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=7809c2d9ea7c2755bd91cc145ad5a862f658ef05;p=platform%2Fcore%2Fconnectivity%2Fbluetooth-frwk.git

Fix svace issues

Change-Id: If63dc2722cf56bda295d26df5fc56fef76cb0218
Signed-off-by: DoHyun Pyun <dh79.pyun@samsung.com>
---

diff --git a/bt-api/bt-gatt-service.c b/bt-api/bt-gatt-service.c
index 20bffd8..a79cf40 100644
--- a/bt-api/bt-gatt-service.c
+++ b/bt-api/bt-gatt-service.c
@@ -329,6 +329,12 @@ static void __bt_gatt_manager_method_call(GDBusConnection *connection,
 			}
 			register_pending_cnt--;
 
+			if (l1 == NULL) {
+				BT_ERR("gatt service list is NULL");
+				g_dbus_method_invocation_return_value(invocation, NULL);
+				return;
+			}
+
 			struct gatt_service_info *serv_info = l1->data;
 			if (serv_info == NULL) {
 				BT_ERR("service info value is NULL");
diff --git a/bt-service/bt-service-avrcp-controller.c b/bt-service/bt-service-avrcp-controller.c
index 2b0bdfc..f443319 100644
--- a/bt-service/bt-service-avrcp-controller.c
+++ b/bt-service/bt-service-avrcp-controller.c
@@ -640,6 +640,10 @@ void _bt_handle_avrcp_control_event(GVariant *reply, const char *path)
 				/* Free key and value because of break unless free not required */
 				free((char *)property);
 				g_variant_unref(value);
+				g_free((char *)metadata.title);
+				g_free((char *)metadata.artist);
+				g_free((char *)metadata.album);
+				g_free((char *)metadata.genre);
 				break;
 			}