Fix the coverity issue (Dereference after null check)

[platform/core/connectivity/bluetooth-frwk.git] / bt-api / bt-common.c

diff --git a/bt-api/bt-common.c b/bt-api/bt-common.c
index 9d573f0..a4dbd38 100644 (file)
--- a/bt-api/bt-common.c
+++ b/bt-api/bt-common.c
@@ -133,9 +133,56 @@ bt_user_info_t *_bt_get_user_data(int type)
        return &user_info[type];
 }
 
+int _bt_common_send_rfcomm_rx_details(bluetooth_rfcomm_received_data_t *data)
+{
+       uid_t uid = getuid();
+       pid_t pid = getpid();
+       int size = data->buffer_size;
+       int ret = BLUETOOTH_ERROR_NONE;
+
+       BT_INFO("+");
+       BT_INIT_PARAMS();
+       BT_ALLOC_PARAMS(in_param1, in_param2, in_param3, in_param4, out_param);
+
+       g_array_append_vals(in_param1, &uid, sizeof(uid_t));
+       g_array_append_vals(in_param2, &pid, sizeof(pid_t));
+       g_array_append_vals(in_param3, &size, sizeof(int));
+
+       ret = _bt_send_request(BT_BLUEZ_SERVICE, BT_RFCOMM_SEND_RX_DETAILS,
+               in_param1, in_param2, in_param3, in_param4, &out_param);
+
+       BT_FREE_PARAMS(in_param1, in_param2, in_param3, in_param4, out_param);
+       BT_INFO("-");
+       return ret;
+}
+
+int _bt_common_send_rfcomm_tx_details(int len)
+{
+       uid_t uid = getuid();
+       pid_t pid = getpid();
+       int size = len;
+       int ret = BLUETOOTH_ERROR_NONE;
+
+       BT_INFO("+");
+       BT_INIT_PARAMS();
+       BT_ALLOC_PARAMS(in_param1, in_param2, in_param3, in_param4, out_param);
+
+       g_array_append_vals(in_param1, &uid, sizeof(uid_t));
+       g_array_append_vals(in_param2, &pid, sizeof(pid_t));
+       g_array_append_vals(in_param3, &size, sizeof(int));
+
+       ret = _bt_send_request(BT_BLUEZ_SERVICE, BT_RFCOMM_SEND_TX_DETAILS,
+               in_param1, in_param2, in_param3, in_param4, &out_param);
+
+       BT_FREE_PARAMS(in_param1, in_param2, in_param3, in_param4, out_param);
+       BT_DBG("-");
+       return ret;
+}
+
 void _bt_common_event_cb(int event, int result, void *param,
                                        void *callback, void *user_data)
 {
+       BT_DBG("+");
        bluetooth_event_param_t bt_event = { 0, };
        bt_event.event = event;
        bt_event.result = result;
@@ -488,8 +535,6 @@ const char *_bt_convert_service_function_to_string(int function)
                {BT_REMOVE_WHITE_LIST, "BT_REMOVE_WHITE_LIST"},
                {BT_CLEAR_WHITE_LIST, "BT_CLEAR_WHITE_LIST"},
                {BT_REGISTER_SCAN_FILTER, "BT_REGISTER_SCAN_FILTER"},
-               {BT_UNREGISTER_SCAN_FILTER, "BT_UNREGISTER_SCAN_FILTER"},
-               {BT_UNREGISTER_ALL_SCAN_FILTERS, "BT_UNREGISTER_ALL_SCAN_FILTERS"},
                {BT_IS_SCAN_FILTER_SUPPORTED, "BT_IS_SCAN_FILTER_SUPPORTED"},
                {BT_GET_PROFILE_CONNECTED_DEVICES, "BT_GET_PROFILE_CONNECTED_DEVICES"},
                {BT_ENABLE_FORCE_HCI_DUMP, "BT_ENABLE_FORCE_HCI_DUMP"},
@@ -1358,7 +1403,7 @@ void _bt_print_api_caller_name(void)
 
        if (fgets(buf, 256, fp) != NULL) {
                str_list = g_strsplit(buf, " ", -1);
-               if (str_list[0] != '\0')
+               if (str_list[0] != NULL)
                        BT_INFO("Caller : %s", str_list[0]);
                g_strfreev(str_list);
        }
@@ -2462,4 +2507,3 @@ BT_EXPORT_API int bluetooth_unregister_callback(void)
 
        return BLUETOOTH_ERROR_NONE;
 }
-