Fix Coverity issues in Bluetooth-frwk

[platform/core/connectivity/bluetooth-frwk.git] / bt-service-adaptation / services / adapter / bt-service-core-adapter-le.c

diff --git a/bt-service-adaptation/services/adapter/bt-service-core-adapter-le.c b/bt-service-adaptation/services/adapter/bt-service-core-adapter-le.c
index 8084df5..27c9219 100644 (file)
--- a/bt-service-adaptation/services/adapter/bt-service-core-adapter-le.c
+++ b/bt-service-adaptation/services/adapter/bt-service-core-adapter-le.c
@@ -617,9 +617,7 @@ static void __bt_le_event_handler(int event_type, gpointer event_data)
        }
        case OAL_EVENT_BLE_SERVER_INSTANCE_INITIALISED: {
                BT_INFO("OAL Event: Server Instance Registered");
-               event_gatts_register_t* event = g_memdup(event_data, sizeof(event_gatts_register_t));
-               /* GATT Server Registered event is handled in MAIN thread context */
-               __bt_le_handle_server_instance_registered(event);
+               __bt_le_handle_server_instance_registered((event_gatts_register_t *)event_data);
                break;
        }
        case OAL_EVENT_BLE_ADVERTISING_STARTED: {
@@ -632,23 +630,17 @@ static void __bt_le_event_handler(int event_type, gpointer event_data)
        }
        case OAL_EVENT_BLE_MULTI_ADVERTISING_ENABLE: {
                BT_INFO("OAL Event: Advertising Enabled");
-               event_ble_multiadv_status* event = (event_ble_multiadv_status*)g_memdup(event_data,
-                                                               sizeof(event_ble_multiadv_status));
-               __bt_le_multi_advertising_enabled(event);
+               __bt_le_multi_advertising_enabled((event_ble_multiadv_status *)event_data);
                break;
        }
        case OAL_EVENT_BLE_MULTI_ADVERTISING_DISABLE: {
                BT_INFO("OAL Event: Advertising Disabled");
-               event_ble_multiadv_status* event = (event_ble_multiadv_status*)g_memdup(event_data,
-                                                               sizeof(event_ble_multiadv_status));
-               __bt_le_multi_advertising_disabled(event);
+               __bt_le_multi_advertising_disabled((event_ble_multiadv_status *)event_data);
                break;
        }
        case OAL_EVENT_BLE_MULTI_ADVERTISING_SET_INST_DATA: {
                BT_INFO("OAL Event: Advertising Data set successfully");
-               event_ble_multiadv_status* event = (event_ble_multiadv_status*)g_memdup(event_data,
-                                                               sizeof(event_ble_multiadv_status));
-               __bt_le_multi_advertising_set_data(event);
+               __bt_le_multi_advertising_set_data((event_ble_multiadv_status *)event_data);
                break;
        }
        case OAL_EVENT_BLE_MULTI_ADVERTISING_UPDATE: {
@@ -1261,6 +1253,8 @@ int _bt_set_advertising_data(const char *sender, int adv_handle,
                                        BLUETOOTH_EVENT_ADVERTISING_MANUFACTURER_DATA_CHANGED,
                                        param);
                }
+
+               g_free(old_mdata);
        }
 
        /* Time to update new ADV data completely in Table */