From: anuj.bhumiya <anuj.bhumiya@samsung.com>
Date: Tue, 25 May 2021 08:58:41 +0000 (+0530)
Subject: Fix coverty issues
X-Git-Tag: accepted/tizen/unified/20210607.124412~3
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=0e6841c958b14e2cae44bfe77ef9c9449e4312bf;p=platform%2Fcore%2Fconnectivity%2Fbluetooth-frwk.git

Fix coverty issues

Change-Id: I06b246df43582a066d071b899558b9a39b9be57d
Signed-off-by: anuj.bhumiya <anuj.bhumiya@samsung.com>
---

diff --git a/bt-api/bt-event-handler.c b/bt-api/bt-event-handler.c
index f4e56e1..e1396b7 100644
--- a/bt-api/bt-event-handler.c
+++ b/bt-api/bt-event-handler.c
@@ -3733,6 +3733,8 @@ static void __bt_otp_event_filter(GDBusConnection *connection,
 		BT_DBG("Fd  [%d]", fd);
 
 		otc_info = g_malloc0(sizeof(bluetooth_otc_info_t));
+		if (!otc_info)
+			return;
 		otc_info->fd = fd;
 		otc_info->connected = connected;
 		otc_info->address = g_strdup(address);
diff --git a/bt-oal/bluez_hal/src/bt-hal-gatt-server.c b/bt-oal/bluez_hal/src/bt-hal-gatt-server.c
index 0dbdce5..376fbe6 100644
--- a/bt-oal/bluez_hal/src/bt-hal-gatt-server.c
+++ b/bt-oal/bluez_hal/src/bt-hal-gatt-server.c
@@ -2803,6 +2803,7 @@ static bt_status_t gatt_server_add_descriptor(int slot, int service_handle, bt_u
 	desc_handle = __bt_hal_assign_gatt_handle(BT_GATT_TYPE_DESC);
 	if (desc_handle == -1) {
 		ERR("Too many gatt desc handles are created");
+		g_strfreev(line_argv);
 		g_free(serv_path);
 		return BT_STATUS_FAIL;
 	}
diff --git a/bt-oal/bluez_hal/src/bt-hal-mesh-dbus-handler.c b/bt-oal/bluez_hal/src/bt-hal-mesh-dbus-handler.c
index 14b60f2..be91742 100644
--- a/bt-oal/bluez_hal/src/bt-hal-mesh-dbus-handler.c
+++ b/bt-oal/bluez_hal/src/bt-hal-mesh-dbus-handler.c
@@ -428,6 +428,7 @@ static gint __compare_dbus_path(gconstpointer data, gconstpointer user_data)
 	}
 	char **strings =  g_strsplit(path, "/elem", 2);
 	ret = g_strcmp0(strings[0], app->path);
+	g_strfreev(strings);
 	return ret;
 }
 
diff --git a/bt-service/services/mesh/bt-service-mesh-cdb.c b/bt-service/services/mesh/bt-service-mesh-cdb.c
index c3abc19..cdf1ab8 100644
--- a/bt-service/services/mesh/bt-service-mesh-cdb.c
+++ b/bt-service/services/mesh/bt-service-mesh-cdb.c
@@ -1609,8 +1609,10 @@ bool _bt_mesh_conf_add_model_config_data(_bt_mesh_cdb_t *cfg, uint16_t unicast,
 	if (!jgroup)
 		return false;
 
-	if (!__mesh_write_uint16_hex(jgroup, "sub-addr", group_addr))
+	if (!__mesh_write_uint16_hex(jgroup, "sub-addr", group_addr)) {
+		json_object_put(jgroup);
 		return false;
+	}
 
 	json_object_array_add(jgroups, jgroup);
 
@@ -1727,8 +1729,10 @@ bool _bt_mesh_conf_overwrite_model_config_data(_bt_mesh_cdb_t *cfg, uint16_t uni
 	if (!jgroup)
 		return false;
 
-	if (!__mesh_write_uint16_hex(jgroup, "sub-addr", group_addr))
+	if (!__mesh_write_uint16_hex(jgroup, "sub-addr", group_addr)) {
+		json_object_put(jgroup);
 		return false;
+	}
 
 	json_object_array_add(jgroups, jgroup);