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

Commit ab767916 authored by Chris Manton's avatar Chris Manton Committed by Automerger Merge Worker
Browse files

Ensure address removed from filter accept list am: 70b63e45 

Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/modules/Bluetooth/+/19240036



Change-Id: Ide4be07696a84ab51a5da7d00f269d94abb12ef1
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 69fb05f3 70b63e45

system/stack/gatt/gatt_utils.cc

+20 −3
Original line number Diff line number Diff line
@@ -1398,7 +1398,12 @@ void gatt_sr_update_prep_cnt(tGATT_TCB& tcb, tGATT_IF gatt_if, bool is_inc, 
/** Cancel LE Create Connection request */

bool gatt_cancel_open(tGATT_IF gatt_if, const RawAddress& bda) {

  tGATT_TCB* p_tcb = gatt_find_tcb_by_addr(bda, BT_TRANSPORT_LE);

  if (!p_tcb) return true;

  if (!p_tcb) {

    LOG_WARN(

        "Unable to cancel open for unknown connection gatt_if:%hhu peer:%s",

        gatt_if, PRIVATE_ADDRESS(bda));

    return true;

  }



  if (gatt_get_ch_state(p_tcb) == GATT_CH_OPEN) {

    LOG(ERROR) << __func__ << ": link connected Too late to cancel";
@@ -1407,9 +1412,21 @@ bool gatt_cancel_open(tGATT_IF gatt_if, const RawAddress& bda) { 


  gatt_update_app_use_link_flag(gatt_if, p_tcb, false, false);



  if (p_tcb->app_hold_link.empty()) gatt_disconnect(p_tcb);

  if (p_tcb->app_hold_link.empty()) {

    LOG_DEBUG(

        "Client reference count is zero disconnecting device gatt_if:%hhu "

        "peer:%s",

        gatt_if, PRIVATE_ADDRESS(bda));

    gatt_disconnect(p_tcb);

  }



  connection_manager::direct_connect_remove(gatt_if, bda);

  if (!connection_manager::direct_connect_remove(gatt_if, bda)) {

    BTM_AcceptlistRemove(bda);

    LOG_INFO(

        "GATT connection manager has no record but removed filter acceptlist "

        "gatt_if:%hhu peer:%s",

        gatt_if, PRIVATE_ADDRESS(bda));

  }

  return true;

}