summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Treehugger Robot <android-test-infra-autosubmit@system.gserviceaccount.com> 2024-06-25 08:52:14 +0000
committer Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> 2024-06-25 08:52:14 +0000
commitab14501f438fdceb6d6cc4e743f399474d2c64d1 (patch)
tree1bb693e231ecf85b1ed51bcd3ac81e326638cecf
parentef856c3287051f6afa09d8ba2e8a519cdfd922e4 (diff)
parent8b30820037eb6d3bb04854ef890f91c83088232a (diff)
Merge changes Ib291c9a6,I98ca045b into main am: 8b30820037
Original change: https://android-review.googlesource.com/c/platform/packages/modules/Bluetooth/+/3144476 Change-Id: I0a010fd21a95561519e6e06472ce37919181073f Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
-rw-r--r--flags/btif_dm.aconfig7
-rw-r--r--flags/security.aconfig7
-rw-r--r--system/btif/src/btif_dm.cc6
-rw-r--r--system/stack/smp/smp_act.cc7
4 files changed, 2 insertions, 25 deletions
diff --git a/flags/btif_dm.aconfig b/flags/btif_dm.aconfig
index 2da8590def..9000dfe30b 100644
--- a/flags/btif_dm.aconfig
+++ b/flags/btif_dm.aconfig
@@ -2,13 +2,6 @@ package: "com.android.bluetooth.flags"
container: "com.android.btservices"
flag {
- name: "force_bredr_for_sdp_retry"
- namespace: "bluetooth"
- description: "Force BR/EDR transport for retrying SDP service discovery"
- bug: "326656580"
-}
-
-flag {
name: "do_not_replace_existing_cod_with_uncategorized_cod"
namespace: "bluetooth"
description: "Don't replace an existing stored class of device with one determined to be uncategorized"
diff --git a/flags/security.aconfig b/flags/security.aconfig
index e3acba1d5d..3e35c2ad86 100644
--- a/flags/security.aconfig
+++ b/flags/security.aconfig
@@ -30,13 +30,6 @@ flag {
}
flag {
- name: "fix_pairing_failure_reason_from_remote"
- namespace: "bluetooth"
- description: "Correct the pairing failure reason reported from remote"
- bug: "320745565"
-}
-
-flag {
name: "fix_le_oob_pairing_bypass"
namespace: "bluetooth"
description: "Fix oob bypassing bug in SMP"
diff --git a/system/btif/src/btif_dm.cc b/system/btif/src/btif_dm.cc
index 87a0439f9f..e3fd42d3fd 100644
--- a/system/btif/src/btif_dm.cc
+++ b/system/btif/src/btif_dm.cc
@@ -1697,11 +1697,7 @@ static void btif_on_service_discovery_results(
if (pairing_cb.sdp_attempts) {
log::warn("SDP failed after bonding re-attempting for {}", bd_addr);
pairing_cb.sdp_attempts++;
- if (com::android::bluetooth::flags::force_bredr_for_sdp_retry()) {
- btif_dm_get_remote_services(bd_addr, BT_TRANSPORT_BR_EDR);
- } else {
- btif_dm_get_remote_services(bd_addr, BT_TRANSPORT_AUTO);
- }
+ btif_dm_get_remote_services(bd_addr, BT_TRANSPORT_BR_EDR);
} else {
log::warn("SDP triggered by someone failed when bonding");
}
diff --git a/system/stack/smp/smp_act.cc b/system/stack/smp/smp_act.cc
index f9c1580039..97fc68c165 100644
--- a/system/stack/smp/smp_act.cc
+++ b/system/stack/smp/smp_act.cc
@@ -535,12 +535,7 @@ void smp_proc_pair_fail(tSMP_CB* p_cb, tSMP_INT_DATA* p_data) {
p_cb->rcvd_cmd_len);
p_cb->status = SMP_INVALID_PARAMETERS;
} else {
- if (com::android::bluetooth::flags::
- fix_pairing_failure_reason_from_remote()) {
- p_cb->status = static_cast<tSMP_STATUS>(p_data->p_data[0]);
- } else {
- p_cb->status = p_data->status;
- }
+ p_cb->status = static_cast<tSMP_STATUS>(p_data->p_data[0]);
}
/* Cancel pending auth complete timer if set */