Apply tizen 3.0 based product patchsets
[platform/core/connectivity/bluetooth-frwk.git] / bt-api / bt-hid-device.c
index d58edbd..db0f6bb 100644 (file)
 #define BT_HID_HSHK_ERR_UNKNOWN        0x0E
 #define BT_HID_HSHK_ERR_FATAL  0x0F
 
-typedef struct {
-       guint object_id;
-       gchar *path;
-       int id;
-       char *uuid;
-       GSList *device_list;
-} hid_info_t;
+#define BT_HID_SERVICE_NAME "org.bluez.hid_agent"
+#define BT_HID_AGENT_OBJECT_PATH "/org/bluez/hid_agent"
+#define BT_HID_SERVICE_INTERFACE "org.tizen.HidApp"
 
 typedef struct {
        int ctrl_fd;
@@ -81,7 +77,7 @@ struct reports {
        guint8 rep_data[20];
 } __attribute__((__packed__));
 
-static hid_info_t *hid_info = NULL;
+static GSList *device_list;
 
 /* Variable for privilege, only for write API,
   before we should reduce time to bt-service dbus calling
@@ -135,15 +131,15 @@ int _bt_hid_device_get_fd(const char *address, int *ctrl, int *intr)
                if (err != NULL) {
                        g_dbus_error_strip_remote_error(err);
                        BT_ERR("INPUT server register Error: %s\n", err->message);
-                       if (g_strcmp0(err->message, "Already Exists") == 0) {
+                       if (g_strcmp0(err->message, "Already Exists") == 0)
                                ret = BLUETOOTH_ERROR_ALREADY_INITIALIZED;
-                       } else {
+                       else
                                ret = BLUETOOTH_ERROR_INTERNAL;
-                       }
+
                        g_error_free(err);
                }
        } else {
-               g_variant_get (result, "(hh)", &index1, &index2);
+               g_variant_get(result, "(hh)", &index1, &index2);
                int fd1 = g_unix_fd_list_get(out_fd_list, index1, NULL);
                int fd2 = g_unix_fd_list_get(out_fd_list, index2, NULL);
 
@@ -156,11 +152,36 @@ int _bt_hid_device_get_fd(const char *address, int *ctrl, int *intr)
        return ret;
 }
 
+static GVariant* __bt_hid_agent_dbus_send(const char *path,
+                               const char *interface,  const char *method,
+                               GError **err, GVariant *parameters)
+{
+       GVariant *reply = NULL;
+       GDBusProxy *proxy = NULL;
+       GDBusConnection *conn = NULL;
+
+       conn = _bt_gdbus_get_system_gconn();
+       retv_if(conn == NULL, NULL);
+
+       proxy = g_dbus_proxy_new_sync(conn, G_DBUS_PROXY_FLAGS_NONE,
+               NULL, BT_HID_SERVICE_NAME, path, interface, NULL, err);
+       if (proxy == NULL) {
+               BT_ERR("Unable to allocate new proxy");
+               return NULL;
+       }
+
+       reply = g_dbus_proxy_call_sync(proxy, method, parameters,
+                               G_DBUS_CALL_FLAGS_NONE, -1, NULL, err);
+
+       g_object_unref(proxy);
+       return reply;
+}
+
 static hid_connected_device_info_t *__find_hid_info_with_address(const char *remote_addr)
 {
        GSList *l;
 
-       for (l = hid_info->device_list; l != NULL; l = l->next) {
+       for (l = device_list; l != NULL; l = l->next) {
                hid_connected_device_info_t *info = l->data;
                if (g_strcmp0((const char *)info->address, (const char *)remote_addr) == 0)
                        return info;
@@ -183,7 +204,7 @@ static void __hid_connected_cb(hid_connected_device_info_t *info,
                conn_info.socket_fd = info->intr_fd;
        else
                conn_info.socket_fd = info->ctrl_fd;
-       _bt_convert_addr_string_to_type (conn_info.device_addr.addr , info->address);
+       _bt_convert_addr_string_to_type(conn_info.device_addr.addr, info->address);
 
        if (result == BLUETOOTH_ERROR_NONE)
                BT_INFO_C("Connected [HID Device]");
@@ -245,7 +266,7 @@ static gboolean __hid_disconnect(hid_connected_device_info_t *info)
        bt_event_info_t *event_info;
 
        BT_INFO_C("Disconnected [HID Device]");
-       hid_info->device_list = g_slist_remove(hid_info->device_list, info);
+       device_list = g_slist_remove(device_list, info);
        if (info->ctrl_data_id > 0) {
                g_source_remove(info->ctrl_data_id);
                info->ctrl_data_id = 0;
@@ -290,27 +311,11 @@ static gboolean __hid_disconnect(hid_connected_device_info_t *info)
        return FALSE;
 }
 
-void __free_hid_info(hid_info_t *info)
+static gboolean __is_error_by_disconnect(GError *err)
 {
-       BT_DBG("");
-
-       _bt_unregister_gdbus(info->object_id);
-
-       while (info->device_list) {
-               hid_connected_device_info_t *dev_info = NULL;
-               dev_info = (hid_connected_device_info_t *)info->device_list->data;
-
-               if (dev_info->disconnect_idle_id > 0) {
-                       BT_INFO("Disconnect idle still not process remove source");
-                       g_source_remove(dev_info->disconnect_idle_id);
-                       dev_info->disconnect_idle_id = 0;
-               }
-               __hid_disconnect(dev_info);
-       }
-
-       g_free(info->path);
-       g_free(info->uuid);
-       g_free(info);
+       return !g_strcmp0(err->message, "Connection reset by peer") ||
+                       !g_strcmp0(err->message, "Connection timed out") ||
+                       !g_strcmp0(err->message, "Software caused connection abort");
 }
 
 static gboolean __received_cb(GIOChannel *chan, GIOCondition cond,
@@ -318,7 +323,7 @@ static gboolean __received_cb(GIOChannel *chan, GIOCondition cond,
 {
        hid_connected_device_info_t *info = data;
        GIOStatus status = G_IO_STATUS_NORMAL;
-       char buffer[20];
+       char buffer[BT_RFCOMM_BUFFER_LEN];
        gsize len = 0;
        GError *err = NULL;
        guint8  header, type, param;
@@ -327,14 +332,15 @@ static gboolean __received_cb(GIOChannel *chan, GIOCondition cond,
 
        if (cond & (G_IO_NVAL | G_IO_HUP | G_IO_ERR)) {
                BT_ERR_C("HID  disconnected: %d", info->ctrl_fd);
-               if (info->disconnect_idle_id > 0) {
-                       BT_INFO("Disconnect idle still not process remove source");
-                       g_source_remove(info->disconnect_idle_id);
-                       info->disconnect_idle_id = 0;
-               }
+                       if (info->disconnect_idle_id > 0) {
+                               BT_INFO("Disconnect idle still not process remove source");
+                               g_source_remove(info->disconnect_idle_id);
+                               info->disconnect_idle_id = 0;
+                       }
                __hid_disconnect(info);
                return FALSE;
        }
+
        status = g_io_channel_read_chars(chan, buffer, BT_RFCOMM_BUFFER_LEN,
                        &len, &err);
        if (status == G_IO_STATUS_NORMAL) {
@@ -352,138 +358,174 @@ static gboolean __received_cb(GIOChannel *chan, GIOCondition cond,
 
                data.address = g_strdup(info->address);
                switch (type) {
-                       case BT_HID_TRANS_HANDSHAKE:
-                               BT_INFO("TRANS HANDSHAKE");
-                               data.type = HTYPE_TRANS_HANDSHAKE;
-                               data.buffer_size = len;
-                               data.buffer = (char *) malloc(sizeof(char) * len);
-                               if (data.buffer)
-                                       memcpy(data.buffer, buffer, len);
-                       break;
-                       case BT_HID_TRANS_HID_CONTROL:
-                               BT_INFO("HID CONTROL");
-                               data.type = HTYPE_TRANS_HID_CONTROL;
-                               data.buffer_size = len;
-                               data.buffer = (char *) malloc(sizeof(char) * len);
-                               if (data.buffer)
-                                       memcpy(data.buffer, buffer, len);
-                       break;
-                       case BT_HID_TRANS_DATA:
-                               BT_INFO("TRANS DATA");
-                               data.type = HTYPE_TRANS_DATA;
-                               if (param & BT_HID_DATA_RTYPE_INPUT) {
-                                       BT_INFO("Input Report");
-                                       data.param = PTYPE_DATA_RTYPE_INPUT;
-                                       data.buffer_size = len;
-                                       data.buffer = (char *) malloc(sizeof(char) * len);
-                                       if (data.buffer)
-                                               memcpy(data.buffer, buffer, len);
-                               } else {
-                                       BT_INFO("Out Report");
-                                       data.param = PTYPE_DATA_RTYPE_OUTPUT;
-                                       data.buffer_size = len;
-                                       data.buffer = (char *) malloc(sizeof(char) * len);
-                                       if (data.buffer)
-                                               memcpy(data.buffer, buffer, len);
-                               }
-                       break;
-                       case BT_HID_TRANS_GET_REPORT: {
-                               BT_INFO("Get Report");
-                               data.type = HTYPE_TRANS_GET_REPORT;
-                               if (param & BT_HID_DATA_RTYPE_INPUT) {
-                                       BT_INFO("Input Report");
-                                       data.param = PTYPE_DATA_RTYPE_INPUT;
-                               } else {
-                                       BT_INFO("Output Report");
-                                       data.param = PTYPE_DATA_RTYPE_OUTPUT;
-                               }
+               case BT_HID_TRANS_HANDSHAKE:
+                       BT_INFO("TRANS HANDSHAKE");
+                       data.type = HTYPE_TRANS_HANDSHAKE;
+                       data.buffer_size = len;
+                       data.buffer = (char *) malloc(sizeof(char) * len);
+                       if (data.buffer)
+                               memcpy(data.buffer, buffer, len);
+               break;
+
+               case BT_HID_TRANS_HID_CONTROL:
+                       BT_INFO("HID CONTROL");
+                       data.type = HTYPE_TRANS_HID_CONTROL;
+                       data.buffer_size = len;
+                       data.buffer = (char *) malloc(sizeof(char) * len);
+                       if (data.buffer)
+                               memcpy(data.buffer, buffer, len);
+               break;
+
+               case BT_HID_TRANS_DATA:
+                       BT_INFO("TRANS DATA");
+                       data.type = HTYPE_TRANS_DATA;
+                       if (param & BT_HID_DATA_RTYPE_INPUT) {
+                               BT_INFO("Input Report");
+                               data.param = PTYPE_DATA_RTYPE_INPUT;
                                data.buffer_size = len;
                                data.buffer = (char *) malloc(sizeof(char) * len);
                                if (data.buffer)
                                        memcpy(data.buffer, buffer, len);
-                               break;
-                       }
-                       case BT_HID_TRANS_SET_REPORT: {
-                               BT_INFO("Set Report");
-                               data.type = HTYPE_TRANS_SET_REPORT;
-                               if (param & BT_HID_DATA_RTYPE_INPUT) {
-                                       BT_INFO("Input Report");
-                                       data.param = PTYPE_DATA_RTYPE_INPUT;
-                               } else {
-                                       BT_INFO("Output Report");
-                                       data.param = PTYPE_DATA_RTYPE_OUTPUT;
-                               }
+                       } else {
+                               BT_INFO("Out Report");
+                               data.param = PTYPE_DATA_RTYPE_OUTPUT;
                                data.buffer_size = len;
                                data.buffer = (char *) malloc(sizeof(char) * len);
                                if (data.buffer)
                                        memcpy(data.buffer, buffer, len);
-                               break;
                        }
-                       case BT_HID_TRANS_GET_PROTOCOL:{
-                               BT_INFO("Get_PROTOCOL");
-                               data.type = HTYPE_TRANS_GET_PROTOCOL;
+               break;
+
+               case BT_HID_TRANS_GET_REPORT: {
+                       BT_INFO("Get Report");
+                       data.type = HTYPE_TRANS_GET_REPORT;
+                       if (param & BT_HID_DATA_RTYPE_INPUT) {
+                               BT_INFO("Input Report");
                                data.param = PTYPE_DATA_RTYPE_INPUT;
-                               data.buffer_size = len;
-                               data.buffer = (char *) malloc(sizeof(char) * len);
-                               if (data.buffer)
-                                       memcpy(data.buffer, buffer, len);
-                               break;
+                       } else {
+                               BT_INFO("Output Report");
+                               data.param = PTYPE_DATA_RTYPE_OUTPUT;
                        }
-                       case BT_HID_TRANS_SET_PROTOCOL:{
-                               BT_INFO("Set_PROTOCOL");
-                               data.type = HTYPE_TRANS_SET_PROTOCOL;
+                       data.buffer_size = len;
+                       data.buffer = (char *) malloc(sizeof(char) * len);
+                       if (data.buffer)
+                               memcpy(data.buffer, buffer, len);
+                       break;
+               }
+
+               case BT_HID_TRANS_SET_REPORT: {
+                       BT_INFO("Set Report");
+                       data.type = HTYPE_TRANS_SET_REPORT;
+                       if (param & BT_HID_DATA_RTYPE_INPUT) {
+                               BT_INFO("Input Report");
                                data.param = PTYPE_DATA_RTYPE_INPUT;
-                               data.buffer_size = len;
-                               data.buffer = (char *) malloc(sizeof(char) * len);
-                               if (data.buffer)
-                                       memcpy(data.buffer, buffer, len);
-                               break;
-                       }
-                       default: {
-                               BT_INFO("unsupported HIDP control message");
-                               BT_ERR("Send Handshake Message");
-                               guint8 type = BT_HID_TRANS_HANDSHAKE |
-                                       BT_HID_HSHK_ERR_UNSUPPORTED_REQUEST;
-                               data.type = HTYPE_TRANS_UNKNOWN;
-                               int fd = g_io_channel_unix_get_fd(chan);
-                               int bytes = write(fd,  &type, sizeof(type));
-                               BT_INFO("Bytes Written %d", bytes);
-                               break;
+                       } else {
+                               BT_INFO("Output Report");
+                               data.param = PTYPE_DATA_RTYPE_OUTPUT;
                        }
+                       data.buffer_size = len;
+                       data.buffer = (char *) malloc(sizeof(char) * len);
+                       if (data.buffer)
+                               memcpy(data.buffer, buffer, len);
+                       break;
+               }
+
+               case BT_HID_TRANS_GET_PROTOCOL:{
+                       BT_INFO("Get_PROTOCOL");
+                       data.type = HTYPE_TRANS_GET_PROTOCOL;
+                       data.param = PTYPE_DATA_RTYPE_INPUT;
+                       data.buffer_size = len;
+                       data.buffer = (char *) malloc(sizeof(char) * len);
+                       if (data.buffer)
+                               memcpy(data.buffer, buffer, len);
+                       break;
+               }
+
+               case BT_HID_TRANS_SET_PROTOCOL:{
+                       BT_INFO("Set_PROTOCOL");
+                       data.type = HTYPE_TRANS_SET_PROTOCOL;
+                       data.param = PTYPE_DATA_RTYPE_INPUT;
+                       data.buffer_size = len;
+                       data.buffer = (char *) malloc(sizeof(char) * len);
+                       if (data.buffer)
+                               memcpy(data.buffer, buffer, len);
+                       break;
                }
 
-               _bt_common_event_cb(BLUETOOTH_HID_DEVICE_DATA_RECEIVED,
+               case BT_HID_TRANS_GET_IDLE:{
+                       BT_INFO("Get_IDLE");
+                       data.type = HTYPE_TRANS_GET_IDLE;
+                       data.param = PTYPE_DATA_RTYPE_INPUT;
+                       data.buffer_size = len;
+                       data.buffer = (char *) malloc(sizeof(char) * len);
+                       if (data.buffer)
+                               memcpy(data.buffer, buffer, len);
+                       break;
+               }
+
+               case BT_HID_TRANS_SET_IDLE:{
+                       BT_INFO("Set_IDLE");
+                       data.type = HTYPE_TRANS_SET_IDLE;
+                       data.param = PTYPE_DATA_RTYPE_INPUT;
+                       data.buffer_size = len;
+                       data.buffer = (char *) malloc(sizeof(char) * len);
+                       if (data.buffer)
+                               memcpy(data.buffer, buffer, len);
+                       break;
+               }
+
+               default: {
+                       BT_INFO("unsupported HIDP control message");
+                       BT_ERR("Send Handshake Message");
+                       guint8 type = BT_HID_TRANS_HANDSHAKE |
+                               BT_HID_HSHK_ERR_UNSUPPORTED_REQUEST;
+                       data.type = HTYPE_TRANS_UNKNOWN;
+                       int fd = g_io_channel_unix_get_fd(chan);
+                       int bytes = write(fd,  &type, sizeof(type));
+                       BT_INFO("Bytes Written %d", bytes);
+                       break;
+               }
+       }
+
+       _bt_common_event_cb(BLUETOOTH_HID_DEVICE_DATA_RECEIVED,
                                BLUETOOTH_ERROR_NONE, &data,
                                event_info->cb, event_info->user_data);
-               if (data.buffer)
-                       g_free(data.buffer);
-               if (data.address)
-                       g_free((char *)data.address);
+       if (data.buffer)
+               g_free(data.buffer);
+
+       if (data.address)
+               g_free((char *)data.address);
        } else {
-               BT_INFO("Error while reading data");
+               BT_ERR("Error while reading data %d [%s]", status, info->address);
+               if (err) {
+                       BT_ERR("IO Channel read error [%s]", err->message);
+                       if (status == G_IO_STATUS_ERROR &&
+                                       __is_error_by_disconnect(err)) {
+                               BT_DBG("cond : %d", cond);
+                               g_error_free(err);
+                               __hid_disconnect(info);
+                               return FALSE;
+                       }
+                       g_error_free(err);
+               } else if (status == G_IO_STATUS_EOF) {
+                       __hid_disconnect(info);
+                       return FALSE;
+               }
        }
        return TRUE;
 }
 
 int new_hid_connection(const char *path, int fd, bluetooth_device_address_t *addr)
 {
-       hid_info_t *info = NULL;
        hid_connected_device_info_t *dev_info = NULL;
        char address[18];
-       info = hid_info;
 
-       if (info == NULL)
-               return -1;
        _bt_convert_addr_type_to_string((char *)address, addr->addr);
        BT_INFO("Address [%s]", address);
        dev_info = __find_hid_info_with_address(address);
        if (dev_info == NULL) {
                dev_info = (hid_connected_device_info_t *)
                        g_malloc0(sizeof(hid_connected_device_info_t));
-               if (dev_info == NULL) {
-                       BT_ERR("Fail to allocation memory");
-                       return -1;
-               }
 
                dev_info->intr_fd = -1;
                dev_info->ctrl_fd = -1;
@@ -496,7 +538,7 @@ int new_hid_connection(const char *path, int fd, bluetooth_device_address_t *add
                dev_info->intr_data_id = g_io_add_watch(dev_info->intr_data_io,
                                G_IO_IN | G_IO_HUP | G_IO_ERR | G_IO_NVAL,
                                __received_cb, dev_info);
-               hid_info->device_list = g_slist_append(hid_info->device_list, dev_info);
+               device_list = g_slist_append(device_list, dev_info);
        } else {
                dev_info->ctrl_fd = fd;
                dev_info->ctrl_data_io = g_io_channel_unix_new(dev_info->ctrl_fd);
@@ -507,30 +549,32 @@ int new_hid_connection(const char *path, int fd, bluetooth_device_address_t *add
                                G_IO_IN | G_IO_HUP | G_IO_ERR | G_IO_NVAL,
                                __received_cb, dev_info);
        }
-       if (dev_info->ctrl_fd != -1 && dev_info->intr_fd != -1) {
+
+       if (dev_info->ctrl_fd != -1 && dev_info->intr_fd != -1)
                __hid_connected_cb(dev_info, BLUETOOTH_ERROR_NONE);
-       }
 
        return 0;
 }
-static hid_info_t *__register_method()
+
+static void __free_hid_info(void *data)
 {
-       int object_id;
-       hid_info_t *info = NULL;
-       char *path = NULL;
-       path = g_strdup_printf("/org/socket/server/%d", getpid());
+       BT_DBG("");
 
-       object_id = _bt_register_new_conn(path, new_hid_connection);
-       if (object_id < 0) {
-               return NULL;
+       hid_connected_device_info_t *dev_info = (hid_connected_device_info_t *)data;
+
+       if (dev_info->disconnect_idle_id > 0) {
+               BT_INFO("Disconnect idle still not process remove source");
+               g_source_remove(dev_info->disconnect_idle_id);
+               dev_info->disconnect_idle_id = 0;
        }
-       info = g_new(hid_info_t, 1);
-       info->object_id = (guint)object_id;
-       info->path = path;
-       info->id = 0;
-       info->device_list = NULL;
+       __hid_disconnect(dev_info);
+}
+
+void _bluetooth_hid_free_hid_info(void)
+{
+       g_slist_free_full(device_list, __free_hid_info);
 
-       return info;
+       device_list = NULL;
 }
 
 BT_EXPORT_API int bluetooth_hid_device_init(hid_cb_func_ptr callback_ptr, void *user_data)
@@ -570,8 +614,10 @@ BT_EXPORT_API int bluetooth_hid_device_deinit(void)
 
 BT_EXPORT_API int bluetooth_hid_device_activate(void)
 {
-       bt_register_profile_info_t profile_info;
-       int result = BLUETOOTH_ERROR_NONE;
+       GVariant *reply;
+       GError *err = NULL;
+
+       BT_CHECK_ENABLED(return);
 
        if (_bt_check_privilege(BT_CHECK_PRIVILEGE, BT_HID_DEVICE_ACTIVATE)
                 == BLUETOOTH_ERROR_PERMISSION_DEINED) {
@@ -579,43 +625,64 @@ BT_EXPORT_API int bluetooth_hid_device_activate(void)
                return BLUETOOTH_ERROR_PERMISSION_DEINED;
        }
 
-       if (hid_info != NULL)
-               return BLUETOOTH_ERROR_IN_PROGRESS;
-
-       hid_info = __register_method();
-       if (hid_info == NULL)
-               return BLUETOOTH_ERROR_INTERNAL;
-
-       hid_info->uuid = g_strdup(HID_DEVICE_UUID);
-
-       profile_info.authentication = TRUE;
-       profile_info.authorization = TRUE;
-       profile_info.obj_path = hid_info->path;
-       profile_info.role = g_strdup("Hid");
-       profile_info.service = hid_info->uuid;
-       profile_info.uuid = hid_info->uuid;
+       reply = __bt_hid_agent_dbus_send(BT_HID_AGENT_OBJECT_PATH,
+                       BT_HID_SERVICE_INTERFACE,
+                       "RegisterApplication", &err, NULL);
+
+       if (!reply) {
+               int ret = BLUETOOTH_ERROR_INTERNAL;
+               BT_ERR("Error returned in method call");
+               if (err) {
+                       BT_ERR("Error = %s", err->message);
+                       if (strcmp(err->message, BT_ERROR_ALREADY_EXIST) == 0)
+                               ret = BLUETOOTH_ERROR_IN_PROGRESS;
+                       else
+                               ret = BLUETOOTH_ERROR_INTERNAL;
+                       g_clear_error(&err);
+               }
+               return ret;
+       }
 
-       BT_INFO("uuid %s", profile_info.uuid);
-       result = _bt_register_profile_platform(&profile_info, FALSE);
+       g_variant_unref(reply);
 
-       return result;
+       return BLUETOOTH_ERROR_NONE;
 }
 
 BT_EXPORT_API int bluetooth_hid_device_deactivate(void)
 {
+       GVariant *reply;
+       GError *err = NULL;
+
+       BT_CHECK_ENABLED(return);
+
        if (_bt_check_privilege(BT_CHECK_PRIVILEGE, BT_HID_DEVICE_DEACTIVATE)
                 == BLUETOOTH_ERROR_PERMISSION_DEINED) {
                BT_ERR("Don't have a privilege to use this API");
                return BLUETOOTH_ERROR_PERMISSION_DEINED;
        }
 
-       if (hid_info == NULL)
-               return BLUETOOTH_ERROR_NOT_IN_OPERATION;
+       reply = __bt_hid_agent_dbus_send(BT_HID_AGENT_OBJECT_PATH,
+                       BT_HID_SERVICE_INTERFACE,
+                       "UnregisterApplication", &err, NULL);
+
+       if (!reply) {
+               int ret = BLUETOOTH_ERROR_INTERNAL;
+               BT_ERR("Error returned in method call");
+               if (err) {
+                       BT_ERR("Error = %s", err->message);
+                       if (strcmp(err->message, BT_ERROR_NOT_AVAILABLE) == 0)
+                               ret = BLUETOOTH_ERROR_NOT_IN_OPERATION;
+                       else
+                               ret = BLUETOOTH_ERROR_INTERNAL;
+                       g_clear_error(&err);
+               }
+               return ret;
+       }
+
+       g_variant_unref(reply);
 
-       _bt_unregister_profile(hid_info->path);
+       _bluetooth_hid_free_hid_info();
 
-       __free_hid_info(hid_info);
-       hid_info = NULL;
        return BLUETOOTH_ERROR_NONE;
 }
 
@@ -627,6 +694,8 @@ BT_EXPORT_API int bluetooth_hid_device_connect(const char *remote_addr)
        BT_DBG("+");
        BT_CHECK_PARAMETER(remote_addr, return);
 
+       BT_CHECK_ENABLED(return);
+
        info = __find_hid_info_with_address(remote_addr);
        if (info) {
                BT_ERR("Connection Already Exists");
@@ -646,6 +715,10 @@ BT_EXPORT_API int bluetooth_hid_device_connect(const char *remote_addr)
 }
 BT_EXPORT_API int bluetooth_hid_device_disconnect(const char *remote_addr)
 {
+       BT_CHECK_PARAMETER(remote_addr, return);
+
+       BT_CHECK_ENABLED(return);
+
        if (_bt_check_privilege(BT_CHECK_PRIVILEGE, BT_HID_DEVICE_DISCONNECT)
                 == BLUETOOTH_ERROR_PERMISSION_DEINED) {
                BT_ERR("Don't have a privilege to use this API");
@@ -672,6 +745,8 @@ BT_EXPORT_API int bluetooth_hid_device_send_mouse_event(const char *remote_addr,
        int written = 0;
        hid_connected_device_info_t *info = NULL;
 
+       BT_CHECK_PARAMETER(remote_addr, return);
+
        switch (privilege_token_send_mouse) {
        case 0:
                result = _bt_check_privilege(BT_CHECK_PRIVILEGE, BT_HID_DEVICE_SEND_MOUSE_EVENT);
@@ -719,6 +794,8 @@ BT_EXPORT_API int bluetooth_hid_device_send_key_event(const char *remote_addr,
        int written = 0;
        hid_connected_device_info_t *info = NULL;
 
+       BT_CHECK_PARAMETER(remote_addr, return);
+
        switch (privilege_token_send_key) {
        case 0:
                result = _bt_check_privilege(BT_CHECK_PRIVILEGE, BT_HID_DEVICE_SEND_KEY_EVENT);
@@ -808,9 +885,7 @@ BT_EXPORT_API int bluetooth_hid_device_send_custom_event(const char *remote_addr
        else
                socket_fd = info->ctrl_fd;
 
-       send_event = (char*)g_malloc0(data_len + 2);
-       if (send_event == NULL)
-               return BLUETOOTH_ERROR_OUT_OF_MEMORY;
+       send_event = g_malloc0(data_len + 2);
 
        send_event[0] = (char)btcode;
        send_event[1] = (char)report_id;
@@ -831,7 +906,7 @@ BT_EXPORT_API int bluetooth_hid_device_reply_to_report(const char *remote_addr,
 {
        int result;
        struct reports output_report = { 0 };
-       int bytes = 0;
+       int bytes = BLUETOOTH_ERROR_INTERNAL;
        hid_connected_device_info_t *info = NULL;
        info = __find_hid_info_with_address(remote_addr);
        if (info == NULL) {
@@ -839,6 +914,8 @@ BT_EXPORT_API int bluetooth_hid_device_reply_to_report(const char *remote_addr,
                return BLUETOOTH_ERROR_INVALID_PARAM;
        }
 
+       BT_CHECK_PARAMETER(remote_addr, return);
+
        switch (privilege_token_reply) {
        case 0:
                result = _bt_check_privilege(BT_CHECK_PRIVILEGE, BT_HID_DEVICE_SEND_REPLY_TO_REPORT);
@@ -864,51 +941,74 @@ BT_EXPORT_API int bluetooth_hid_device_reply_to_report(const char *remote_addr,
        }
 
        BT_INFO("htype %d ptype %d", htype, ptype);
-       switch(htype) {
-               case HTYPE_TRANS_GET_REPORT: {
-                       switch(ptype) {
-                               case PTYPE_DATA_RTYPE_INPUT: {
-                                       output_report.type = BT_HID_TRANS_DATA |
-                                                       BT_HID_DATA_RTYPE_INPUT;
-                                       memcpy(output_report.rep_data, data, data_len);
-                                       bytes = write(info->intr_fd, &output_report,
-                                                               sizeof(output_report));
-                                       BT_DBG("Bytes Written %d", bytes);
-                                       break;
-                               }
-                               default:
-                                       BT_INFO("Not Supported");
-                                       break;
-                       }
-                       break;
-               case HTYPE_TRANS_GET_PROTOCOL: {
-                       BT_DBG("Replying to Get_PROTOCOL");
-                       output_report.type = BT_HID_TRANS_DATA | BT_HID_DATA_RTYPE_OUTPUT;
-                       output_report.rep_data[0] = data[0];
-                       bytes = write(info->intr_fd, &output_report, 2);
-                       BT_DBG("Bytes Written %d", bytes);
-                       break;
-               }
-               case HTYPE_TRANS_SET_PROTOCOL: {
-                       BT_DBG("Reply to Set_Protocol");
-                       output_report.type = BT_HID_TRANS_DATA | BT_HID_DATA_RTYPE_INPUT;
+       switch (htype) {
+       case HTYPE_TRANS_GET_REPORT: {
+               switch (ptype) {
+               case PTYPE_DATA_RTYPE_INPUT: {
+                       output_report.type = BT_HID_TRANS_DATA |
+                                       BT_HID_DATA_RTYPE_INPUT;
                        memcpy(output_report.rep_data, data, data_len);
-                       bytes = write(info->ctrl_fd, &output_report,
-                                       sizeof(output_report));
+                       bytes = write(info->intr_fd, &output_report,
+                                               sizeof(output_report));
                        BT_DBG("Bytes Written %d", bytes);
                        break;
                }
-               case HTYPE_TRANS_HANDSHAKE: {
-                       BT_DBG("Replying Handshake");
-                       output_report.type = BT_HID_TRANS_HANDSHAKE | data[0];
-                       memset(output_report.rep_data, 0, sizeof(output_report.rep_data));
-                       bytes = write(info->intr_fd,  &output_report.type,
-                                       sizeof(output_report.type));
-                       BT_DBG("Bytes Written %d", bytes);
+               default:
+                       BT_INFO("Not Supported");
                        break;
-               }
-                       default:
-                               break;
+       }
+       break;
+
+       case HTYPE_TRANS_GET_PROTOCOL: {
+               BT_DBG("Replying to Get_PROTOCOL");
+               output_report.type = BT_HID_TRANS_DATA | BT_HID_DATA_RTYPE_OUTPUT;
+               output_report.rep_data[0] = data[0];
+               bytes = write(info->intr_fd, &output_report, 2);
+               BT_DBG("Bytes Written %d", bytes);
+               break;
+       }
+
+       case HTYPE_TRANS_SET_PROTOCOL: {
+               BT_DBG("Reply to Set_Protocol");
+               output_report.type = BT_HID_TRANS_DATA | BT_HID_DATA_RTYPE_INPUT;
+               memcpy(output_report.rep_data, data, data_len);
+               bytes = write(info->ctrl_fd, &output_report,
+                               sizeof(output_report));
+               BT_DBG("Bytes Written %d", bytes);
+               break;
+       }
+
+       case HTYPE_TRANS_HANDSHAKE: {
+               BT_DBG("Replying Handshake");
+               output_report.type = BT_HID_TRANS_HANDSHAKE | data[0];
+               memset(output_report.rep_data, 0, sizeof(output_report.rep_data));
+               bytes = write(info->intr_fd,  &output_report.type,
+                               sizeof(output_report.type));
+               BT_DBG("Bytes Written %d", bytes);
+               break;
+       }
+
+       case HTYPE_TRANS_GET_IDLE: {
+               BT_DBG("Replying to Get_IDLE");
+               output_report.type = BT_HID_TRANS_DATA | BT_HID_DATA_RTYPE_OUTPUT;
+               output_report.rep_data[0] = data[0];
+               bytes = write(info->intr_fd, &output_report, 2);
+               BT_DBG("Bytes Written %d", bytes);
+               break;
+       }
+
+       case HTYPE_TRANS_SET_IDLE: {
+               BT_DBG("Reply to Set_IDLE");
+               output_report.type = BT_HID_TRANS_DATA | BT_HID_DATA_RTYPE_INPUT;
+               memcpy(output_report.rep_data, data, data_len);
+               bytes = write(info->ctrl_fd, &output_report,
+                               sizeof(output_report));
+               BT_DBG("Bytes Written %d", bytes);
+               break;
+       }
+
+       default:
+               break;
                }
        }
        return bytes;