diff options
author | Arjun Garg <arjgarg@google.com> | 2019-07-15 12:43:34 -0700 |
---|---|---|
committer | Arjun Garg <arjgarg@google.com> | 2019-07-15 12:43:34 -0700 |
commit | 2acb8e2b1ce511bcf73ba131822bef592e9d5981 (patch) | |
tree | 21c6dd213762e71523869e0c077e768de58337fe | |
parent | 4d57cc3df9d3330b55f2ee50fd5b5250e10d3b1c (diff) | |
download | bt-nougat-mr2-security-release.tar.gz |
Revert "DO NOT MERGE Separate SDP procedure from bonding state (1/2)"android-7.1.2_r39nougat-mr2-security-release
This reverts commit 669c21e17874a11394668b2a927b04d03850d237.
-rw-r--r-- | btif/src/btif_dm.c | 74 |
1 files changed, 20 insertions, 54 deletions
diff --git a/btif/src/btif_dm.c b/btif/src/btif_dm.c index 154263add..f54b64277 100644 --- a/btif/src/btif_dm.c +++ b/btif/src/btif_dm.c @@ -191,7 +191,6 @@ typedef struct #define BTA_SERVICE_ID_TO_SERVICE_MASK(id) (1 << (id)) #define UUID_HUMAN_INTERFACE_DEVICE "00001124-0000-1000-8000-00805f9b34fb" -#define UUID_EMPTY "00000000-0000-0000-0000-000000000000" #define MAX_BTIF_BOND_EVENT_ENTRIES 15 @@ -271,11 +270,6 @@ static bool is_empty_128bit(uint8_t *data) return !memcmp(zero, data, sizeof(zero)); } -static bool is_bonding_or_sdp() { - return pairing_cb.state == BT_BOND_STATE_BONDING || - (pairing_cb.state == BT_BOND_STATE_BONDED && pairing_cb.sdp_attempts); -} - static void btif_dm_data_copy(uint16_t event, char *dst, char *src) { tBTA_DM_SEC *dst_dm_sec = (tBTA_DM_SEC*)dst; @@ -557,14 +551,15 @@ static void bond_state_changed(bt_status_t status, bt_bdaddr_t *bd_addr, bt_bond HAL_CBACK(bt_hal_cbacks, bond_state_changed_cb, status, bd_addr, state); - if (state == BT_BOND_STATE_BONDING || - (state == BT_BOND_STATE_BONDED && pairing_cb.sdp_attempts > 0)) + if (state == BT_BOND_STATE_BONDING) { - // Save state for the device is bonding or SDP. pairing_cb.state = state; bdcpy(pairing_cb.bd_addr, bd_addr->address); } else { - memset(&pairing_cb, 0, sizeof(pairing_cb)); + if (!pairing_cb.sdp_attempts) + memset(&pairing_cb, 0, sizeof(pairing_cb)); + else + BTIF_TRACE_DEBUG("%s: BR-EDR service discovery active", __func__); } } @@ -1217,14 +1212,6 @@ static void btif_dm_auth_cmpl_evt (tBTA_DM_AUTH_CMPL *p_auth_cmpl) // Ensure inquiry is stopped before attempting service discovery btif_dm_cancel_discovery(); - if (is_crosskey) { - // If bonding occurred due to cross-key pairing, send bonding callback - // for static address now - LOG_INFO(LOG_TAG, - "%s: send bonding state update for static address", __func__); - bond_state_changed(BT_STATUS_SUCCESS, &bd_addr, BT_BOND_STATE_BONDING); - } - bond_state_changed(BT_STATUS_SUCCESS, &bd_addr, BT_BOND_STATE_BONDED); /* Trigger SDP on the device */ pairing_cb.sdp_attempts = 1; @@ -1518,7 +1505,7 @@ static void btif_dm_search_services_evt(UINT16 event, char *p_param) BTIF_TRACE_DEBUG("%s:(result=0x%x, services 0x%x)", __FUNCTION__, p_data->disc_res.result, p_data->disc_res.services); if ((p_data->disc_res.result != BTA_SUCCESS) && - (pairing_cb.state == BT_BOND_STATE_BONDED) && + (pairing_cb.state == BT_BOND_STATE_BONDING ) && (pairing_cb.sdp_attempts < BTIF_DM_MAX_SDP_ATTEMPTS_AFTER_PAIRING)) { BTIF_TRACE_WARNING("%s:SDP failed after bonding re-attempting", __FUNCTION__); @@ -1543,37 +1530,21 @@ static void btif_dm_search_services_evt(UINT16 event, char *p_param) /* onUuidChanged requires getBondedDevices to be populated. ** bond_state_changed needs to be sent prior to remote_device_property */ - if ((pairing_cb.state == BT_BOND_STATE_BONDED && pairing_cb.sdp_attempts) && + if ((pairing_cb.state == BT_BOND_STATE_BONDING) && ((bdcmp(p_data->disc_res.bd_addr, pairing_cb.bd_addr) == 0) || - (bdcmp(p_data->disc_res.bd_addr, pairing_cb.static_bdaddr.address) == 0))) + (bdcmp(p_data->disc_res.bd_addr, pairing_cb.static_bdaddr.address) == 0)) && + pairing_cb.sdp_attempts > 0) { - LOG_INFO(LOG_TAG, "%s Remote Service SDP done.", __FUNCTION__); + BTIF_TRACE_DEBUG("%s Remote Service SDP done. Call bond_state_changed_cb BONDED", + __FUNCTION__); pairing_cb.sdp_attempts = 0; - // Both SDP and bonding are done, clear pairing control block - memset(&pairing_cb, 0, sizeof(pairing_cb)); - - // Send one empty UUID to Java to unblock pairing intent when SDP failed - // or no UUID is discovered - if (p_data->disc_res.result == BTA_SUCCESS || - p_data->disc_res.num_uuids == 0) { - LOG_INFO(LOG_TAG, "%s: SDP failed, send empty UUID to unblock bonding", - __func__); - bt_property_t prop; - bt_uuid_t uuid = {}; - char uuid_str[128] = UUID_EMPTY; - - string_to_uuid(uuid_str, &uuid); - - prop.type = BT_PROPERTY_UUIDS; - prop.val = uuid.uu; - prop.len = MAX_UUID_SIZE; + // If bonding occured due to cross-key pairing, send bonding callback + // for static address now + if (bdcmp(p_data->disc_res.bd_addr, pairing_cb.static_bdaddr.address) == 0) + bond_state_changed(BT_STATUS_SUCCESS, &bd_addr, BT_BOND_STATE_BONDING); - /* Send the event to the BTIF */ - HAL_CBACK(bt_hal_cbacks, remote_device_properties_cb, - BT_STATUS_SUCCESS, &bd_addr, 1, &prop); - break; - } + bond_state_changed(BT_STATUS_SUCCESS, &bd_addr, BT_BOND_STATE_BONDED); } if (p_data->disc_res.num_uuids != 0) @@ -1797,7 +1768,7 @@ static void btif_dm_upstreams_evt(UINT16 event, char* p_param) break; case BTA_DM_BOND_CANCEL_CMPL_EVT: - if (is_bonding_or_sdp()) + if (pairing_cb.state == BT_BOND_STATE_BONDING) { bdcpy(bd_addr.address, pairing_cb.bd_addr); btm_set_bond_type_dev(pairing_cb.bd_addr, BOND_TYPE_UNKNOWN); @@ -2464,7 +2435,7 @@ bt_status_t btif_dm_cancel_bond(const bt_bdaddr_t *bd_addr) ** 1. Restore scan modes ** 2. special handling for HID devices */ - if (is_bonding_or_sdp()) + if (pairing_cb.state == BT_BOND_STATE_BONDING) { #if (defined(BLE_INCLUDED) && (BLE_INCLUDED == TRUE)) @@ -2526,7 +2497,7 @@ bt_status_t btif_dm_cancel_bond(const bt_bdaddr_t *bd_addr) void btif_dm_hh_open_failed(bt_bdaddr_t *bdaddr) { - if (is_bonding_or_sdp() && + if (pairing_cb.state == BT_BOND_STATE_BONDING && bdcmp(bdaddr->address, pairing_cb.bd_addr) == 0) { bond_state_changed(BT_STATUS_FAIL, bdaddr, BT_BOND_STATE_NONE); @@ -3166,11 +3137,6 @@ static void btif_dm_ble_auth_cmpl_evt (tBTA_DM_AUTH_CMPL *p_auth_cmpl) break; } } - if (state == BT_BOND_STATE_BONDED && - (bdcmp(bd_addr.address, pairing_cb.static_bdaddr.address) != 0)) { - // Report RPA bonding state to Java in crosskey paring - bond_state_changed(status, &bd_addr, BT_BOND_STATE_BONDING); - } bond_state_changed(status, &bd_addr, state); } @@ -3521,7 +3487,7 @@ bt_status_t btif_le_test_mode(uint16_t opcode, uint8_t *buf, uint8_t len) void btif_dm_on_disable() { /* cancel any pending pairing requests */ - if (is_bonding_or_sdp()) + if (pairing_cb.state == BT_BOND_STATE_BONDING) { bt_bdaddr_t bd_addr; |