resolved the code rule warnings
[platform/core/connectivity/bluetooth-frwk.git] / bt-oal / bluez_hal / src / bt-hal-device-dbus-handler.c
index eac64f3..b1f4671 100644 (file)
@@ -56,9 +56,9 @@ static handle_stack_msg event_cb = NULL;
 static void __bt_hal_bond_device_cb(GDBusProxy *proxy, GAsyncResult *res, gpointer user_data);
 
 static void __bt_hal_unbond_device_cb(GDBusProxy *proxy, GAsyncResult *res,
-                                        gpointer user_data);
+               gpointer user_data);
 static void __bt_hal_device_service_search_cb(GDBusProxy *proxy, GAsyncResult *res,
-                                        gpointer user_data);
+               gpointer user_data);
 int __bt_hal_dbus_enquire_remote_device_services(char *address);
 
 static void __bt_device_parse_services(GVariant *result);
@@ -241,7 +241,7 @@ int _bt_hal_device_cancel_bond(const bt_bdaddr_t *bd_addr)
        DBG("+");
 
        result = _bt_hal_agent_reply_cancellation();
-       if (result != BT_HAL_ERROR_NONE){
+       if (result != BT_HAL_ERROR_NONE) {
                ERR("Fail to call reply cancellation");
                return BT_STATUS_FAIL;
        }
@@ -251,7 +251,7 @@ int _bt_hal_device_cancel_bond(const bt_bdaddr_t *bd_addr)
 }
 
 int _bt_hal_device_legacy_pin_reply(const bt_bdaddr_t *bd_addr,
-                                gboolean accept, uint8_t pin_len, char *pincode)
+               gboolean accept, uint8_t pin_len, char *pincode)
 {
        GapAgentPrivate *agent = _bt_hal_get_adapter_agent();
        DBG("+");
@@ -262,9 +262,9 @@ int _bt_hal_device_legacy_pin_reply(const bt_bdaddr_t *bd_addr,
        DBG("pin_len [0x%x]", pin_len);
        DBG("pincode [%s]", pincode);
 
-       if (accept) {
+       if (accept)
                gap_agent_reply_pin_code(agent, GAP_AGENT_ACCEPT, pincode, NULL);
-       else
+       else
                gap_agent_reply_pin_code(agent, GAP_AGENT_REJECT, NULL, NULL);
 
        DBG("-");
@@ -272,7 +272,7 @@ int _bt_hal_device_legacy_pin_reply(const bt_bdaddr_t *bd_addr,
 }
 
 int _bt_hal_device_ssp_reply(const bt_bdaddr_t *bd_addr, bt_ssp_variant_t variant,
-                uint8_t accept, uint32_t passkey)
+               uint8_t accept, uint32_t passkey)
 {
        GapAgentPrivate *agent = _bt_hal_get_adapter_agent();
        DBG("+");
@@ -281,31 +281,31 @@ int _bt_hal_device_ssp_reply(const bt_bdaddr_t *bd_addr, bt_ssp_variant_t varian
                return BT_STATUS_FAIL;
 
        switch (variant) {
-               case BT_SSP_VARIANT_PASSKEY_CONFIRMATION:
-                       DBG("SSP: PASSKEY_CONFIRMATION");
-                       if (accept)
-                               gap_agent_reply_confirmation(agent, GAP_AGENT_ACCEPT, NULL);
-                       else
-                               gap_agent_reply_confirmation(agent, GAP_AGENT_REJECT, NULL);
-                       break;
-               case BT_SSP_VARIANT_PASSKEY_NOTIFICATION:
-                       DBG("SSP: PASSKEY_NOTIFICATION");
-                       break;
-               case BT_SSP_VARIANT_PASSKEY_ENTRY:
-                       DBG("SSP: PASSKEY_ENTRY");
-                       if (accept) {
-                               char str_passkey[7];
-                               snprintf(str_passkey, sizeof(str_passkey), "%.6d", passkey);
-                               DBG("Passkey [%s]", str_passkey);
-                               gap_agent_reply_passkey(agent, GAP_AGENT_ACCEPT, str_passkey, NULL);
-                       } else
-                               gap_agent_reply_passkey(agent, GAP_AGENT_REJECT, NULL, NULL);
-                       break;
-               case BT_SSP_VARIANT_CONSENT:
-                       DBG("SSP: VARIANT_CONSENT: Unhandled!");
-                       break;
-               default:
-                       break;
+       case BT_SSP_VARIANT_PASSKEY_CONFIRMATION:
+               DBG("SSP: PASSKEY_CONFIRMATION");
+               if (accept)
+                       gap_agent_reply_confirmation(agent, GAP_AGENT_ACCEPT, NULL);
+               else
+                       gap_agent_reply_confirmation(agent, GAP_AGENT_REJECT, NULL);
+               break;
+       case BT_SSP_VARIANT_PASSKEY_NOTIFICATION:
+               DBG("SSP: PASSKEY_NOTIFICATION");
+               break;
+       case BT_SSP_VARIANT_PASSKEY_ENTRY:
+               DBG("SSP: PASSKEY_ENTRY");
+               if (accept) {
+                       char str_passkey[7];
+                       snprintf(str_passkey, sizeof(str_passkey), "%.6d", passkey);
+                       DBG("Passkey [%s]", str_passkey);
+                       gap_agent_reply_passkey(agent, GAP_AGENT_ACCEPT, str_passkey, NULL);
+               } else
+                       gap_agent_reply_passkey(agent, GAP_AGENT_REJECT, NULL, NULL);
+               break;
+       case BT_SSP_VARIANT_CONSENT:
+               DBG("SSP: VARIANT_CONSENT: Unhandled!");
+               break;
+       default:
+               break;
        }
 
        DBG("-");
@@ -381,7 +381,7 @@ fail:
 }
 
 int _bt_hal_device_authorize_response(const bt_bdaddr_t *bd_addr, bt_service_id_t service_id,
-                                    uint8_t authorize, uint8_t save_settings)
+               uint8_t authorize, uint8_t save_settings)
 {
        int reply = GAP_AGENT_ACCEPT;
        GapAgentPrivate *agent = _bt_hal_get_adapter_agent();
@@ -510,7 +510,7 @@ int _bt_hal_device_set_trust(const bt_bdaddr_t *bd_addr, uint8_t trust)
 }
 
 static void __bt_hal_device_service_search_cb(GDBusProxy *proxy, GAsyncResult *res,
-                                        gpointer user_data)
+               gpointer user_data)
 {
        /* Buffer and propety count management */
        uint8_t buf[BT_HAL_MAX_PROPERTY_BUF_SIZE];
@@ -528,25 +528,25 @@ static void __bt_hal_device_service_search_cb(GDBusProxy *proxy, GAsyncResult *r
 
        /* Check event pointer */
        if (!event_cb)
-                event_cb = _bt_hal_get_stack_message_handler();
-        if (!event_cb) {
-                ERR("event_cb is NULL, can not send Service search results to HAL User");
-                goto cleanup;
-        }
+               event_cb = _bt_hal_get_stack_message_handler();
+       if (!event_cb) {
+               ERR("event_cb is NULL, can not send Service search results to HAL User");
+               goto cleanup;
+       }
 
        if (err != NULL) {
                g_dbus_error_strip_remote_error(err);
                ERR("Error occured in Proxy call [%s]\n", err->message);
 
-               if (g_strrstr("Operation canceled", err->message)) {
+               if (g_strrstr("Operation canceled", err->message))
                        result = BT_HAL_ERROR_CANCEL_BY_USER;
-               } else if (g_strrstr("In Progress", err->message)) {
+               else if (g_strrstr("In Progress", err->message))
                        result = BT_HAL_ERROR_IN_PROGRESS;
-               } else if (g_strrstr("Host is down", err->message)) {
+               else if (g_strrstr("Host is down", err->message))
                        result = BT_HAL_ERROR_HOST_DOWN;
-               } else {
+               else
                        result = BT_HAL_ERROR_CONNECTION_ERROR;
-               }
+
 
                if (result == BT_HAL_ERROR_HOST_DOWN ||
                                result == BT_HAL_ERROR_CONNECTION_ERROR) {
@@ -566,7 +566,7 @@ done:
        ev->num_props = 0;
        size = sizeof(*ev);
        ERR("Error: Failed to get Remote device properties after SDP,"
-                       " Num Prop [%d] total size [%d]",ev->num_props, size);
+                       " Num Prop [%d] total size [%d]", ev->num_props, size);
        event_cb(HAL_EV_REMOTE_DEVICE_PROPS, (void*) buf, size);
 
 cleanup:
@@ -603,7 +603,7 @@ static void __bt_device_parse_services(GVariant *result)
                        char **uuid_value;
                        int uuid_count = 0;
                        gsize size1 = 0;
-                       int i =0;
+                       int i = 0;
                        size1 = g_variant_get_size(value);
                        int num_props_tmp = ev->num_props;
                        if (size1 > 0) {
@@ -633,7 +633,7 @@ static void __bt_device_parse_services(GVariant *result)
        }
 
        DBG("Send Remote Device services to HAL,"
-                       " Num Prop [%d] total size [%d]",ev->num_props, size);
+                       " Num Prop [%d] total size [%d]", ev->num_props, size);
        event_cb(HAL_EV_REMOTE_DEVICE_PROPS, (void*) buf, size);
 
        g_variant_unref(result);
@@ -703,7 +703,7 @@ int __bt_hal_dbus_enquire_remote_device_services(char *address)
 }
 
 static void __bt_hal_bond_device_cb(GDBusProxy *proxy, GAsyncResult *res,
-                                        gpointer user_data)
+               gpointer user_data)
 {
        GError *err = NULL;
        const char *device_path;
@@ -792,7 +792,7 @@ static void __bt_hal_bond_device_cb(GDBusProxy *proxy, GAsyncResult *res,
 }
 
 static void __bt_hal_unbond_device_cb(GDBusProxy *proxy, GAsyncResult *res,
-                                        gpointer user_data)
+               gpointer user_data)
 {
        GError *err = NULL;
        char *device_path = NULL;
@@ -845,8 +845,8 @@ static gboolean __bt_device_bonded_device_info_cb(gpointer user_data)
        const gchar *name = NULL;
        unsigned int cod = 0;
        gint rssi = 0;
-       uint8_t trust=0;
-       uint8_t paired=0;
+       uint8_t trust = 0;
+       uint8_t paired = 0;
        int connected = 0;
        GByteArray *manufacturer_data = NULL;
        const gchar *key;
@@ -862,7 +862,7 @@ static gboolean __bt_device_bonded_device_info_cb(gpointer user_data)
 
        g_variant_get(result, "(a{sv})", &property_iter);
        while (g_variant_iter_loop(property_iter, "{sv}", &key, &value)) {
-               if(!g_strcmp0(key, "Address")) {
+               if (!g_strcmp0(key, "Address")) {
                        address = g_variant_get_string(value, NULL);
                        DBG("Address [%s]", address);
                        _bt_hal_convert_addr_string_to_type(ev->bdaddr, address);
@@ -884,7 +884,7 @@ static gboolean __bt_device_bonded_device_info_cb(gpointer user_data)
                        size += __bt_insert_hal_properties(buf + size, HAL_PROP_DEVICE_CONNECTED,
                                        sizeof(unsigned int), &connected);
                        ev->num_props++;
-               } else if (!g_strcmp0(key,"Paired")) {
+               } else if (!g_strcmp0(key, "Paired")) {
                        paired = (g_variant_get_boolean(value) ? 1 : 0);
                        DBG("Paired [%d]", paired);
                        size += __bt_insert_hal_properties(buf + size,
@@ -912,7 +912,7 @@ static gboolean __bt_device_bonded_device_info_cb(gpointer user_data)
                        char **uuid_value;
                        int uuid_count = 0;
                        gsize size1 = 0;
-                       int i =0;
+                       int i = 0;
                        size1 = g_variant_get_size(value);
                        int num_props_tmp = ev->num_props;
                        if (size1 > 0) {
@@ -942,9 +942,8 @@ static gboolean __bt_device_bonded_device_info_cb(gpointer user_data)
                } else if (!g_strcmp0(key, "ManufacturerData")) {
                        manufacturer_data = g_byte_array_new();
                        g_variant_get(value, "ay", &char_value_iter);
-                       while(g_variant_iter_loop(char_value_iter, "y",  &char_value)) {
+                       while (g_variant_iter_loop(char_value_iter, "y", &char_value))
                                g_byte_array_append(manufacturer_data, &char_value, 1);
-                       }
 
                        if (manufacturer_data) {
                                if (manufacturer_data->len > 0) {
@@ -979,12 +978,12 @@ static gboolean __bt_device_bonded_device_info_cb(gpointer user_data)
                ev->num_props = 0;
                size = sizeof(*ev);
                DBG("Send Remote Device properties event to HAL,"
-                               " Num Prop [%d] total size [%d]",ev->num_props, size);
+                               " Num Prop [%d] total size [%d]", ev->num_props, size);
                event_cb(HAL_EV_REMOTE_DEVICE_PROPS, (void*) buf, size);
        } else {
                if (size > 2) {
                        DBG("Send Remote Device properties event to HAL,"
-                               " Num Prop [%d] total size [%d]",ev->num_props, size);
+                               " Num Prop [%d] total size [%d]", ev->num_props, size);
                        event_cb(HAL_EV_REMOTE_DEVICE_PROPS, (void*) buf, size);
                }
        }
@@ -1003,7 +1002,7 @@ int _bt_hal_dbus_get_remote_device_properties(bt_bdaddr_t *remote_addr)
        GDBusConnection *conn;
        GVariant *result;
 
-       if(!remote_addr) {
+       if (!remote_addr) {
                ERR("Invalid device address ptr received");
                return BT_STATUS_PARM_INVALID;
        }