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

Commit 07b975f6 authored by Himanshu Rawat's avatar Himanshu Rawat Committed by Automerger Merge Worker
Browse files

Merge "Don't reset btif pairing state due to unrelated service discovery" into main am: 62e719a3 

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



Change-Id: Id5b7f79f1285b369635d6e9f7a53897def68ad0a
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 7ef53bc2 62e719a3

system/btif/src/btif_dm.cc

+23 −9
Original line number Diff line number Diff line
@@ -1678,6 +1678,28 @@ static bool btif_should_ignore_uuid(const Uuid& uuid) { 
  return uuid.IsEmpty() || uuid.IsBase();

}



static bool btif_is_gatt_service_discovery_post_pairing(const RawAddress bd_addr) {

  if (!IS_FLAG_ENABLED(reset_pairing_only_for_related_service_discovery)) {

    if (bd_addr == pairing_cb.bd_addr || bd_addr == pairing_cb.static_bdaddr) {

      if (pairing_cb.gatt_over_le !=

          btif_dm_pairing_cb_t::ServiceDiscoveryState::SCHEDULED) {

        LOG_ERROR("gatt_over_le should be SCHEDULED, did someone clear the "

                  "control block for %s ?",

                  ADDRESS_TO_LOGGABLE_CSTR(bd_addr));

      }



      return true;

    }



    return false;

  }



 return ((bd_addr == pairing_cb.bd_addr ||

          bd_addr == pairing_cb.static_bdaddr) &&

         (pairing_cb.gatt_over_le ==

          btif_dm_pairing_cb_t::ServiceDiscoveryState::SCHEDULED));

}



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

 *

 * Function         btif_dm_search_services_evt
@@ -1874,15 +1896,7 @@ static void btif_dm_search_services_evt(tBTA_DM_SEARCH_EVT event, 
                 ADDRESS_TO_LOGGABLE_CSTR(bd_addr));

        BTM_LogHistory(kBtmLogTag, bd_addr,

                       "Discovered GATT services using LE transport");

        if ((bd_addr == pairing_cb.bd_addr ||

             bd_addr == pairing_cb.static_bdaddr)) {

          if (pairing_cb.gatt_over_le !=

              btif_dm_pairing_cb_t::ServiceDiscoveryState::SCHEDULED) {

            LOG_ERROR(

                "gatt_over_le should be SCHEDULED, did someone clear the "

                "control block for %s ?",

                ADDRESS_TO_LOGGABLE_CSTR(bd_addr));

          }

        if (btif_is_gatt_service_discovery_post_pairing(bd_addr)) {

          pairing_cb.gatt_over_le =

              btif_dm_pairing_cb_t::ServiceDiscoveryState::FINISHED;