Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 7c8d3195 authored by Bill Chang's avatar Bill Chang
Browse files

Revert "Reconfigure Address policy on last bond removed" 

This reverts commit a4675b98.

Reason for revert: broken test b/241860519

Test: SystemAppCheck#system_app_native_crash

Change-Id: I66359f65ad56b0161434015e2c29fb9f710789e6
parent a4675b98

system/bta/dm/bta_dm_act.cc

+7 −14
Original line number Diff line number Diff line
@@ -652,6 +652,13 @@ void bta_dm_remove_device(const RawAddress& bd_addr) { 
  if (!other_address_connected && !other_address.IsEmpty()) {

    bta_dm_process_remove_device(other_address);

  }



  /* Check the length of the paired devices, and if 0 then reset IRK */

  auto paired_devices = btif_config_get_paired_devices();

  if (paired_devices.empty()) {

    LOG_INFO("Last paired device removed, resetting IRK");

    btm_ble_reset_id();

  }

}



/*******************************************************************************
@@ -4123,20 +4130,6 @@ void bta_dm_set_event_filter_inquiry_result_all_devices() { 
  bluetooth::shim::BTM_SetEventFilterInquiryResultAllDevices();

}



/*******************************************************************************

 *

 * Function         bta_dm_ble_reset_id

 *

 * Description      Reset the local adapter BLE keys.

 *

 * Parameters:

 *

 ******************************************************************************/

void bta_dm_ble_reset_id(void) {

  VLOG(1) << "bta_dm_ble_reset_id in bta_dm_act";

  bluetooth::shim::BTM_BleResetId();

}



/*******************************************************************************

 *

 * Function         bta_dm_gattc_callback

system/bta/dm/bta_dm_api.cc

+0 −14
Original line number Diff line number Diff line
@@ -706,17 +706,3 @@ void BTA_DmSetEventFilterInquiryResultAllDevices() { 
      FROM_HERE,

      base::Bind(bta_dm_set_event_filter_inquiry_result_all_devices));

}
 


/*******************************************************************************

 *

 * Function         BTA_DmBleResetId

 *

 * Description      This function resets the ble keys such as IRK

 *

 * Returns          void

 *

 ******************************************************************************/

void BTA_DmBleResetId(void) {

  APPL_TRACE_API("BTA_DmBleResetId");

  do_in_main_thread(FROM_HERE, base::Bind(bta_dm_ble_reset_id));

}

system/bta/dm/bta_dm_int.h

+0 −2
Original line number Diff line number Diff line
@@ -550,8 +550,6 @@ extern void bta_dm_restore_filter_accept_list(); 
extern void bta_dm_set_default_event_mask();

extern void bta_dm_set_event_filter_inquiry_result_all_devices();



extern void bta_dm_ble_reset_id(void);



uint8_t bta_dm_search_get_state();

void bta_dm_search_set_state(uint8_t state);

system/bta/include/bta_api.h

+0 −11
Original line number Diff line number Diff line
@@ -1256,15 +1256,4 @@ extern void BTA_DmSetDefaultEventMask(); 
 *******************************************************************************/

extern void BTA_DmSetEventFilterInquiryResultAllDevices();



/*******************************************************************************

 *

 * Function         BTA_DmBleResetId

 *

 * Description      This function resets the ble keys such as IRK

 *

 * Returns          void

 *

 ******************************************************************************/

extern void BTA_DmBleResetId(void);



#endif /* BTA_API_H */

system/btif/src/btif_storage.cc

+1 −11
Original line number Diff line number Diff line
@@ -897,13 +897,6 @@ bt_status_t btif_storage_remove_bonded_device( 


  /* write bonded info immediately */

  btif_config_flush();



  /* Check the length of the paired devices, and if 0 then reset IRK */

  auto paired_devices = btif_config_get_paired_devices();

  if (paired_devices.empty()) {

    LOG_INFO("Last paired device removed, resetting IRK");

    BTA_DmBleResetId();

  }

  return ret ? BT_STATUS_SUCCESS : BT_STATUS_FAIL;

}
@@ -1293,10 +1286,7 @@ bt_status_t btif_storage_add_ble_local_key(const Octet16& key, 
      return BT_STATUS_FAIL;

  }

  int ret = btif_config_set_bin("Adapter", name, key.data(), key.size());

  // Had to change this to flush to get it to work on test.

  // Seems to work in the real world on a phone... but not sure why there's a

  // race in test. Investigate b/239828132

  btif_config_flush();

  btif_config_save();

  return ret ? BT_STATUS_SUCCESS : BT_STATUS_FAIL;

}