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

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

Use return condition to stack/l2cap/l2c_ble::l2cble_conn_comp am: 99d92c3d am: 8912607b 

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1456346

Change-Id: Ia57514e3a2f0074bb4be2f085741a5ab5bae1dbc
parents 6a09d2fc 8912607b

system/stack/acl/ble_acl.cc

+13 −2
Original line number Diff line number Diff line
@@ -51,8 +51,19 @@ static bool acl_ble_common_connection(const tBLE_BD_ADDR& address_with_type, 
  btm_ble_increment_link_topology_mask(role);



  // Inform l2cap of a potential connection.

  l2cble_conn_comp(handle, role, address_with_type.bda, address_with_type.type,

                   conn_interval, conn_latency, conn_timeout);

  if (!l2cble_conn_comp(handle, role, address_with_type.bda,

                        address_with_type.type, conn_interval, conn_latency,

                        conn_timeout)) {

    btm_sec_disconnect(handle, HCI_ERR_NO_CONNECTION);

    LOG_WARN("Unable to complete l2cap connection");

    return false;

  }



  btm_ble_disable_resolving_list(BTM_BLE_RL_INIT, true);



  /* Tell BTM Acl management about the link */

  btm_acl_created(address_with_type.bda, handle, role, BT_TRANSPORT_LE);



  return true;

}

system/stack/l2cap/l2c_ble.cc

+0 −7
Original line number Diff line number Diff line
@@ -279,12 +279,10 @@ bool l2cble_conn_comp(uint16_t handle, uint8_t role, const RawAddress& bda, 
  if (!p_lcb) {

    p_lcb = l2cu_allocate_lcb(bda, false, BT_TRANSPORT_LE);

    if (!p_lcb) {

      btm_sec_disconnect(handle, HCI_ERR_NO_CONNECTION);

      LOG_ERROR("Unable to allocate link resource for le acl connection");

      return false;

    } else {

      if (!l2cu_initialize_fixed_ccb(p_lcb, L2CAP_ATT_CID)) {

        btm_sec_disconnect(handle, HCI_ERR_NO_CONNECTION);

        LOG_ERROR("Unable to allocate channel resource for le acl connection");

        return false;

      }
@@ -317,15 +315,10 @@ bool l2cble_conn_comp(uint16_t handle, uint8_t role, const RawAddress& bda, 
  p_lcb->latency = conn_latency;

  p_lcb->conn_update_mask = L2C_BLE_NOT_DEFAULT_PARAM;



  /* Tell BTM Acl management about the link */

  btm_acl_created(bda, handle, p_lcb->LinkRole(), BT_TRANSPORT_LE);



  p_lcb->peer_chnl_mask[0] = L2CAP_FIXED_CHNL_ATT_BIT |

                             L2CAP_FIXED_CHNL_BLE_SIG_BIT |

                             L2CAP_FIXED_CHNL_SMP_BIT;



  btm_ble_disable_resolving_list(BTM_BLE_RL_INIT, true);



  if (role == HCI_ROLE_SLAVE) {

    if (!controller_get_interface()

             ->supports_ble_peripheral_initiated_feature_exchange()) {