Set trusted profile/device in ACCEPT_ALWAYS authorization case

If ACCEPT_ALWAYS option is selected in authorization popup,
This change will set Profile as trusted if the profile is
PBAP, MAP or SAP. Otherwise it will put device in the
trusted list.
This will allow the user to connect automatically
in future connections without asking for authorization.

Change-Id: I2033f34b8436987d4c4ea87db57c2193cfa7c34c
Signed-off-by: Ayush Garg <ayush.garg@samsung.com>

diff --git a/bt-oal/bluez_hal/src/bt-hal-device-dbus-handler.c b/bt-oal/bluez_hal/src/bt-hal-device-dbus-handler.c
index fdf462c..a42671d 100644 (file)
--- a/bt-oal/bluez_hal/src/bt-hal-device-dbus-handler.c
+++ b/bt-oal/bluez_hal/src/bt-hal-device-dbus-handler.c
@@ -488,7 +488,7 @@ int _bt_hal_device_set_trust(const bt_bdaddr_t *bd_addr, uint8_t trust)
                return BT_STATUS_PARM_INVALID;
        }
 
-       DBG("Set authotrize [%d]", authorize);
+       DBG("Set authorize [%d]", authorize);
        result = g_dbus_proxy_call_sync(device_proxy, "Set",
                        g_variant_new("(ssv)", BT_HAL_DEVICE_INTERFACE, "Trusted", g_variant_new("b", authorize)),
                        G_DBUS_CALL_FLAGS_NONE,
@@ -1374,6 +1374,18 @@ static char* __bt_hal_get_trusted_profile_uuid(bt_trusted_profile_t profile)
        return NULL;
 }
 
+bt_trusted_profile_t _bt_hal_get_trusted_profile_enum(const char *uuid)
+{
+       if (g_strcmp0("0000112f-0000-1000-8000-00805f9b34fb", uuid) == 0)
+               return BT_TRUSTED_PROFILE_PBAP;
+       else if (g_strcmp0("00001132-0000-1000-8000-00805f9b34fb", uuid) == 0)
+               return BT_TRUSTED_PROFILE_MAP;
+       else if (g_strcmp0("0000112D-0000-1000-8000-00805f9b34fb", uuid) == 0)
+               return BT_TRUSTED_PROFILE_SAP;
+
+       return 0; /* 0 - Unknown Profile */
+}
+
 int _bt_hal_device_set_trusted_profile(const bt_bdaddr_t *bd_addr,
                bt_trusted_profile_t profile, uint8_t trust)
 {

diff --git a/bt-oal/bluez_hal/src/bt-hal-device-dbus-handler.h b/bt-oal/bluez_hal/src/bt-hal-device-dbus-handler.h
index 9aaf84a..a986f2c 100644 (file)
--- a/bt-oal/bluez_hal/src/bt-hal-device-dbus-handler.h
+++ b/bt-oal/bluez_hal/src/bt-hal-device-dbus-handler.h
@@ -69,6 +69,8 @@ int _bt_hal_device_get_service_connection_state(
 int _bt_hal_device_register_osp_server(uint32_t type, char *uuid, char *path, int fd);
 int _bt_hal_device_unregister_osp_server(uint32_t type, char *uuid);
 
+bt_trusted_profile_t _bt_hal_get_trusted_profile_enum(const char *uuid);
+
 int _bt_hal_device_set_trusted_profile(const bt_bdaddr_t *bd_addr,
                bt_trusted_profile_t profile, uint8_t trust);
 

diff --git a/bt-oal/bluez_hal/src/bt-hal-gap-agent.c b/bt-oal/bluez_hal/src/bt-hal-gap-agent.c
index 800b735..d6a32a5 100644 (file)
--- a/bt-oal/bluez_hal/src/bt-hal-gap-agent.c
+++ b/bt-oal/bluez_hal/src/bt-hal-gap-agent.c
@@ -41,6 +41,7 @@
 #include <dlog.h>
 
 #include "bt-hal-adapter-dbus-handler.h"
+#include "bt-hal-device-dbus-handler.h"
 #include "bt-hal-dbus-common-utils.h"
 
 static GDBusConnection *connection = NULL;
@@ -727,6 +728,7 @@ static void __bt_gap_agent_method(GDBusConnection *connection,
 
                priv->exec_type = GAP_AGENT_EXEC_AUTHORZATION;
                priv->reply_context = invocation;
+               priv->uuid = g_strdup(uuid);
 
                addr = strstr(path, "dev_");
                if (addr != NULL) {
@@ -1055,7 +1057,25 @@ gboolean gap_agent_reply_authorize(GapAgentPrivate *agent, const guint accept,
                if (accept == GAP_AGENT_ACCEPT) {
                        g_dbus_method_invocation_return_value(priv->reply_context, NULL);
                } else if (accept == GAP_AGENT_ACCEPT_ALWAYS) {
-                       /* TODO: Enable below logic after set authorization API implementation */
+                       bluetooth_device_address_t addr = { { 0, } };
+                       int result;
+                       bt_trusted_profile_t profile;
+
+                       _bt_hal_convert_addr_string_to_type(addr.addr,
+                                                       priv->authorize_addr);
+
+                       profile = _bt_hal_get_trusted_profile_enum(priv->uuid);
+
+                       if (profile)
+                               result = _bt_hal_device_set_trusted_profile((bt_bdaddr_t *)(&addr), profile, TRUE);
+                       else
+                               result = _bt_hal_device_set_trust((bt_bdaddr_t *)(&addr), TRUE);
+
+                       if (result == BT_STATUS_SUCCESS) {
+                               INFO("[%s] Profile added as trusted for Device[%s]",
+                                               priv->uuid, priv->authorize_addr);
+                       }
+
                        g_dbus_method_invocation_return_value(priv->reply_context, NULL);
                } else {
                        switch (accept) {
@@ -1089,6 +1109,10 @@ gboolean gap_agent_reply_authorize(GapAgentPrivate *agent, const guint accept,
        priv->reply_context = NULL;
        memset(priv->authorize_addr, 0x00, sizeof(priv->authorize_addr));
 
+       if (priv->uuid) {
+               g_free(priv->uuid);
+               priv->uuid = NULL;
+       }
        return ret;
 }
 

diff --git a/bt-oal/bluez_hal/src/bt-hal-gap-agent.h b/bt-oal/bluez_hal/src/bt-hal-gap-agent.h
index 67daab3..9abb61e 100644 (file)
--- a/bt-oal/bluez_hal/src/bt-hal-gap-agent.h
+++ b/bt-oal/bluez_hal/src/bt-hal-gap-agent.h
@@ -86,6 +86,7 @@ struct _GapAgentPrivate {
        GDBusProxy *dbus_proxy;
        GapAgentExecType exec_type;
        GDBusMethodInvocation *reply_context;
+       char *uuid;
 
        char pairing_addr[18];
        char authorize_addr[18];