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

Commit 8f408362 authored by Chienyuan's avatar Chienyuan Committed by Jack He
Browse files

Remove device from filter accept list before delete device record 

Bug: 229199562
Tag: #refactor
Test: gd/cert/run
Test: Pair LE device, disconnect. Unpair device, check filter accept
      list.
Merged-In: Ife2aa3ae94985763e16176d5167319a0b43234b1
Change-Id: Ife2aa3ae94985763e16176d5167319a0b43234b1
(cherry picked from commit c6676404304a5355f1033e40f483b7309a534160)
parent 210a297a

system/main/shim/acl.cc

+4 −0
Original line number Diff line number Diff line
@@ -1338,12 +1338,16 @@ void shim::legacy::Acl::CancelClassicConnection(const hci::Address& address) { 
void shim::legacy::Acl::AcceptLeConnectionFrom(

    const hci::AddressWithType& address_with_type, bool is_direct,

    std::promise<bool> promise) {

  LOG_DEBUG("AcceptLeConnectionFrom %s",

            PRIVATE_ADDRESS(address_with_type.GetAddress()));

  handler_->CallOn(pimpl_.get(), &Acl::impl::accept_le_connection_from,

                   address_with_type, is_direct, std::move(promise));

}



void shim::legacy::Acl::IgnoreLeConnectionFrom(

    const hci::AddressWithType& address_with_type) {

  LOG_DEBUG("IgnoreLeConnectionFrom %s",

            PRIVATE_ADDRESS(address_with_type.GetAddress()));

  handler_->CallOn(pimpl_.get(), &Acl::impl::ignore_le_connection_from,

                   address_with_type);

}

system/stack/btm/btm_ble_bgconn.cc

+15 −2
Original line number Diff line number Diff line
@@ -205,8 +205,14 @@ bool BTM_AcceptlistAdd(const RawAddress& address) { 
    LOG_WARN("Controller does not support Le");

    return false;

  }



  tBTM_SEC_DEV_REC* p_dev_rec = btm_find_dev(address);

  if (p_dev_rec != NULL && p_dev_rec->device_type & BT_DEVICE_TYPE_BLE) {

    p_dev_rec->ble.in_controller_list |= BTM_ACCEPTLIST_BIT;

  }



  return bluetooth::shim::ACL_AcceptLeConnectionFrom(

      convert_to_address_with_type(address, btm_find_dev(address)),

      convert_to_address_with_type(address, p_dev_rec),

      /* is_direct */ false);

}
@@ -216,8 +222,15 @@ void BTM_AcceptlistRemove(const RawAddress& address) { 
    LOG_WARN("Controller does not support Le");

    return;

  }



  tBTM_SEC_DEV_REC* p_dev_rec = btm_find_dev(address);

  if (p_dev_rec != NULL && p_dev_rec->device_type & BT_DEVICE_TYPE_BLE) {

    p_dev_rec->ble.in_controller_list &= ~BTM_ACCEPTLIST_BIT;

  }



  bluetooth::shim::ACL_IgnoreLeConnectionFrom(

      convert_to_address_with_type(address, btm_find_dev(address)));

      convert_to_address_with_type(address, p_dev_rec));

  return;

}



/** Clear the acceptlist, end any pending acceptlist connections */

system/stack/btm/btm_dev.cc

+9 −0
Original line number Diff line number Diff line
@@ -136,6 +136,9 @@ void wipe_secrets_and_remove(tBTM_SEC_DEV_REC* p_dev_rec) { 
  list_remove(btm_cb.sec_dev_rec, p_dev_rec);

}



/** Removes the device from acceptlist */

extern void BTM_AcceptlistRemove(const RawAddress& address);



/** Free resources associated with the device associated with |bd_addr| address.

 *

 * *** WARNING ***
@@ -162,6 +165,12 @@ bool BTM_SecDeleteDevice(const RawAddress& bd_addr) { 
  if (p_dev_rec != NULL) {

    RawAddress bda = p_dev_rec->bd_addr;



    if (p_dev_rec->ble.in_controller_list & BTM_ACCEPTLIST_BIT) {

      LOG_INFO("Remove device %s from filter accept list before delete record",

               PRIVATE_ADDRESS(bd_addr));

      BTM_AcceptlistRemove(p_dev_rec->bd_addr);

    }



    /* Clear out any saved BLE keys */

    btm_sec_clear_ble_keys(p_dev_rec);

    wipe_secrets_and_remove(p_dev_rec);