OSDN Git Service

android-x86/system-bt.git
4 years agoFix potential stack overflow caused by integer overflow am: 4c0c93eb61
Jakub Pawlowski [Tue, 7 Apr 2020 19:17:17 +0000 (19:17 +0000)]
Fix potential stack overflow caused by integer overflow am: 4c0c93eb61

Change-Id: I4996d5fa51f94a588e9f8c22bb8338668de2b21b

4 years agoFix potential stack overflow caused by integer overflow
Jakub Pawlowski [Mon, 16 Mar 2020 11:09:15 +0000 (12:09 +0100)]
Fix potential stack overflow caused by integer overflow

Bug: 151155194
Change-Id: I0655b0b62301f78cd8705cc7b0e4fc11522f00ca

4 years agoMerge "A2DP: Do the connection task in the stack main thread" into rvc-dev am: 145214a381
TreeHugger Robot [Mon, 6 Apr 2020 15:08:00 +0000 (15:08 +0000)]
Merge "A2DP: Do the connection task in the stack main thread" into rvc-dev am: 145214a381

Change-Id: Id50c974d1615acdd5191b56670d059972497b0c0

4 years agoMerge "A2DP: Do the connection task in the stack main thread" into rvc-dev
TreeHugger Robot [Mon, 6 Apr 2020 14:49:41 +0000 (14:49 +0000)]
Merge "A2DP: Do the connection task in the stack main thread" into rvc-dev

4 years agoMerge "Add min_sdk_version:R to updatable apexes" into rvc-dev am: 9d11ca841c
Jooyung Han [Sun, 5 Apr 2020 00:32:14 +0000 (00:32 +0000)]
Merge "Add min_sdk_version:R to updatable apexes" into rvc-dev am: 9d11ca841c

Change-Id: Id444dc78976fd11c6e7b239002f2aa07dd340ef0

4 years agoMerge "Add min_sdk_version:R to updatable apexes" into rvc-dev
Jooyung Han [Sun, 5 Apr 2020 00:14:36 +0000 (00:14 +0000)]
Merge "Add min_sdk_version:R to updatable apexes" into rvc-dev

4 years agoMerge "BluetoothMetrics: Add SDP DI manufacturers info to Bluetooth config; Add SDP...
TreeHugger Robot [Fri, 3 Apr 2020 19:40:12 +0000 (19:40 +0000)]
Merge "BluetoothMetrics: Add SDP DI manufacturers info to Bluetooth config; Add SDP ID info and LMP info to Bluetooth log" into rvc-dev am: 91e46a2550

Change-Id: If11973b34ef9e3e15c4580bbeecd4545b7846408

4 years agoMerge "BluetoothMetrics: Add SDP DI manufacturers info to Bluetooth config; Add SDP...
TreeHugger Robot [Fri, 3 Apr 2020 19:22:52 +0000 (19:22 +0000)]
Merge "BluetoothMetrics: Add SDP DI manufacturers info to Bluetooth config; Add SDP ID info and LMP info to Bluetooth log" into rvc-dev

4 years agoA2DP: Do the connection task in the stack main thread
Cheney Ni [Tue, 31 Mar 2020 10:38:49 +0000 (18:38 +0800)]
A2DP: Do the connection task in the stack main thread

There is a profile queue to handle connection tasks sequentially, and
invokes the connect API in JNI thread while the previous task finishes.
However, since BtifAvStateMachine is running in the stack main thread,
it brings a race condition about the BtifAVPeer object between the JNI
and stack main threads. This change does A2DP connection task in the
main thread, so has no such race conditions.

Bug: 152595833
Test: make sure the opening event is in main thread manually
Change-Id: Ib40dc7c5fa3e839a894d6f7947ac8266eacd2022
Merged-In: Ib40dc7c5fa3e839a894d6f7947ac8266eacd2022
(cherry picked from commit 367c497abbc0ff5551a1b3210db35ad6540e2dfe)

4 years agoAdd rahulsabnis@ as an owner of bluetooth code to allow for owner am: f2cdc9e04f
Rahul Sabnis [Fri, 3 Apr 2020 02:57:20 +0000 (02:57 +0000)]
Add rahulsabnis@ as an owner of bluetooth code to allow for owner am: f2cdc9e04f

Change-Id: Iea2389e267ff196f22ed53e1f9216dd734878f27

4 years agoAdd rahulsabnis@ as an owner of bluetooth code to allow for owner
Rahul Sabnis [Fri, 3 Apr 2020 00:02:36 +0000 (17:02 -0700)]
Add rahulsabnis@ as an owner of bluetooth code to allow for owner
code reviews

Bug: 153115116
Test: Manual
Merged-In: I8153358384bbf18102f84f544551d358d95b45e7
Change-Id: I8153358384bbf18102f84f544551d358d95b45e7

4 years agoBluetoothMetrics: Add SDP DI manufacturers info to Bluetooth config;
Chen Chen [Wed, 1 Apr 2020 02:12:39 +0000 (19:12 -0700)]
BluetoothMetrics: Add SDP DI manufacturers info to Bluetooth config;
Add SDP ID info and LMP info to Bluetooth log

Bug: 142563377
Test: manually check Bluetooth config file
Change-Id: I4a7c348ac570a730cccc31e51bdc9669847f96a5

4 years agoAdd min_sdk_version:R to updatable apexes
Jooyung Han [Mon, 30 Mar 2020 04:27:34 +0000 (13:27 +0900)]
Add min_sdk_version:R to updatable apexes

APEXes introduced in R need to set min_sdk_version to ensure that they
are built against correct version(30 or R) of stubs (libc/liblog/...).

Bug: 152655956
Test: /vendor/google/build/build_mainline_modules.sh
Change-Id: I055f46f574c7349f46b8c1cb6bb39a515ae69967

4 years agoMerge "Replacing updateBleAppCount to enableBle and disableBle" into rvc-dev am:...
Ugo Yu [Tue, 31 Mar 2020 04:45:07 +0000 (04:45 +0000)]
Merge "Replacing updateBleAppCount to enableBle and disableBle" into rvc-dev am: eba4873382

Change-Id: I6ea53c3471ec730afabe0645561d8d6f20acbca8

4 years agoMerge "Replacing updateBleAppCount to enableBle and disableBle" into rvc-dev
Ugo Yu [Tue, 31 Mar 2020 04:41:43 +0000 (04:41 +0000)]
Merge "Replacing updateBleAppCount to enableBle and disableBle" into rvc-dev

4 years agoMerge "Retry service discovery for Avrcp if due to sdp connection failed" into rvc...
TreeHugger Robot [Mon, 30 Mar 2020 14:35:17 +0000 (14:35 +0000)]
Merge "Retry service discovery for Avrcp if due to sdp connection failed" into rvc-dev am: 48bafcaad4

Change-Id: Ie8c2a551034caf707831cceac105140430b8255b

4 years agoMerge "Retry service discovery for Avrcp if due to sdp connection failed" into rvc-dev
TreeHugger Robot [Mon, 30 Mar 2020 14:17:34 +0000 (14:17 +0000)]
Merge "Retry service discovery for Avrcp if due to sdp connection failed" into rvc-dev

4 years agoReplacing updateBleAppCount to enableBle and disableBle
Ugo Yu [Tue, 17 Mar 2020 11:52:41 +0000 (19:52 +0800)]
Replacing updateBleAppCount to enableBle and disableBle

Bug: 139454316
Test: enableBle() -> disableBle() and check Bluetooth is OFF.
Change-Id: Ifa0b26af7b5c1eef65658948cffe6e409a787344

4 years agoMerge "Bluetooth Metric Id: Add metric id allocation to Bluetooth metric calls" into...
Chen Chen [Sat, 28 Mar 2020 00:57:43 +0000 (00:57 +0000)]
Merge "Bluetooth Metric Id: Add metric id allocation to Bluetooth metric calls" into rvc-dev am: 5457af6d56

Change-Id: I08221f66a46fc8cefd6034f133917c571997a507

4 years agoMerge "Bluetooth Metric Id: Add metric id allocation to Bluetooth metric calls" into...
Chen Chen [Sat, 28 Mar 2020 00:54:26 +0000 (00:54 +0000)]
Merge "Bluetooth Metric Id: Add metric id allocation to Bluetooth metric calls" into rvc-dev

4 years agoMerge "BluetoothAudioHAL: Have full initialization for audio hardware objects" into...
TreeHugger Robot [Fri, 27 Mar 2020 21:37:23 +0000 (21:37 +0000)]
Merge "BluetoothAudioHAL: Have full initialization for audio hardware objects" into rvc-dev am: 453cb7ff12

Change-Id: Ib10522240bcfdf210c6c6d850e4e563fa2e8af1b

4 years agoMerge "BluetoothAudioHAL: Have full initialization for audio hardware objects" into...
TreeHugger Robot [Fri, 27 Mar 2020 21:06:22 +0000 (21:06 +0000)]
Merge "BluetoothAudioHAL: Have full initialization for audio hardware objects" into rvc-dev

4 years agoHFP: Cleanup bt_hf_callbacks on the jni thread. am: af8630eddd
weichinweng [Fri, 27 Mar 2020 08:02:47 +0000 (08:02 +0000)]
HFP: Cleanup bt_hf_callbacks on the jni thread. am: af8630eddd

Change-Id: I1ebb39a8a90b4a0a93b17472007b418bacc5423c

4 years ago[automerger skipped] Merge "DO NOT MERGE Fix potential overflow in btif_rc" into...
Chris Manton [Fri, 27 Mar 2020 01:07:55 +0000 (01:07 +0000)]
[automerger skipped] Merge "DO NOT MERGE Fix potential overflow in btif_rc" into qt-qpr1-dev am: 6a91b4d117 -s ours am: 3a484b3878

Change-Id: I2d402fa6f5f953bd1464364882c2f1b094d58aa6

4 years agoHFP: Cleanup bt_hf_callbacks on the jni thread.
weichinweng [Thu, 12 Mar 2020 06:47:27 +0000 (14:47 +0800)]
HFP: Cleanup bt_hf_callbacks on the jni thread.

For the race condition, the bt_hf_callbacks will be set null on main
thread when doing btif_hf_upstreams_evt on jni thread. Then it will
trigger crash due to btif_hf_upstreams_evt calling a null point.
The patch is cleanup bt_hf_callbacks on the jni thread to avoid the race
condition.

Bug: 140781191
Test: manual, unit test, auto script stress test.

Change-Id: I67400749120aebe13fa3e791f624c8e595d63379
Merged-In: I67400749120aebe13fa3e791f624c8e595d63379

4 years ago[automerger skipped] Merge "DO NOT MERGE Fix potential overflow in btif_rc" into...
Chris Manton [Fri, 27 Mar 2020 00:49:20 +0000 (00:49 +0000)]
[automerger skipped] Merge "DO NOT MERGE Fix potential overflow in btif_rc" into qt-qpr1-dev am: 6a91b4d117 -s ours

am skip reason: subject contains skip directive

Change-Id: Ieff23e6c5693306d2c8b9402ed7788808f812ac3

4 years agoMerge "DO NOT MERGE Fix potential overflow in btif_rc" into qt-qpr1-dev
Chris Manton [Fri, 27 Mar 2020 00:36:19 +0000 (00:36 +0000)]
Merge "DO NOT MERGE Fix potential overflow in btif_rc" into qt-qpr1-dev

4 years agoMerge "Fix potential OOB vulnerability when an HCI event is received" into qt-qpr1...
TreeHugger Robot [Thu, 26 Mar 2020 22:10:22 +0000 (22:10 +0000)]
Merge "Fix potential OOB vulnerability when an HCI event is received" into qt-qpr1-dev am: ee0e886a89 am: 45d1c806e3

Change-Id: I04a74c5d507bf64c2044e2db63192fe833e5e746

4 years ago[automerger skipped] Revert "DO NOT MERGE Ensure hci command status event has suffic...
Kelly Rossmoyer [Thu, 26 Mar 2020 22:09:51 +0000 (22:09 +0000)]
[automerger skipped] Revert "DO NOT MERGE  Ensure hci command status event has sufficient packet length" am: 006e8e481c -s ours am: 18c29357bd

Change-Id: Icf4d5414d51dffbf363e37d4bc02436d43ec83ef

4 years agoMerge "Fix potential OOB vulnerability when an HCI event is received" into qt-qpr1...
TreeHugger Robot [Thu, 26 Mar 2020 21:58:22 +0000 (21:58 +0000)]
Merge "Fix potential OOB vulnerability when an HCI event is received" into qt-qpr1-dev am: ee0e886a89

Change-Id: I18a2a32e6f34602f76d3e018a29e9bd24694c7c5

4 years ago[automerger skipped] Revert "DO NOT MERGE Ensure hci command status event has suffic...
Kelly Rossmoyer [Thu, 26 Mar 2020 21:58:08 +0000 (21:58 +0000)]
[automerger skipped] Revert "DO NOT MERGE  Ensure hci command status event has sufficient packet length" am: 006e8e481c -s ours

am skip reason: subject contains skip directive

Change-Id: Idb50aa70e42f832df92cdaaaf705a6cec38f67dd

4 years agoMerge "Fix potential OOB vulnerability when an HCI event is received" into qt-qpr1-dev
TreeHugger Robot [Thu, 26 Mar 2020 21:45:36 +0000 (21:45 +0000)]
Merge "Fix potential OOB vulnerability when an HCI event is received" into qt-qpr1-dev

4 years agoDO NOT MERGE Fix potential overflow in btif_rc
Chris Manton [Wed, 4 Mar 2020 04:04:51 +0000 (20:04 -0800)]
DO NOT MERGE Fix potential overflow in btif_rc

Bug: 142878416
Test: net_test_btif_rc

Change-Id: Ia263bd5c863644f2adde759a103d79b812a9a5de
(cherry picked from commit e5dd9c735a30188286c7c9b88c247e4f629bdd54)

4 years agoRevert "DO NOT MERGE Ensure hci command status event has sufficient packet length"
Kelly Rossmoyer [Thu, 26 Mar 2020 20:57:42 +0000 (20:57 +0000)]
Revert "DO NOT MERGE  Ensure hci command status event has sufficient packet length"

This reverts commit 179e1ee138a8811deffd49eae77f85246e5092a9.

Reason for revert: build breakage (b/152543627)

Change-Id: I437c99613732a5feb4ecbd082598f2efa5853c1e

4 years ago[automerger skipped] Merge "DO NOT MERGE Ensure hci command status event has suffici...
TreeHugger Robot [Thu, 26 Mar 2020 20:03:24 +0000 (20:03 +0000)]
[automerger skipped] Merge "DO NOT MERGE  Ensure hci command status event has sufficient packet length" into qt-qpr1-dev am: 466138189c -s ours am: 7eefb5094e

Change-Id: Ia9aa4011fbd88bcd013d167d26485ce256279010

4 years ago[automerger skipped] Merge "DO NOT MERGE Ensure hci command status event has suffici...
TreeHugger Robot [Thu, 26 Mar 2020 19:43:52 +0000 (19:43 +0000)]
[automerger skipped] Merge "DO NOT MERGE  Ensure hci command status event has sufficient packet length" into qt-qpr1-dev am: 466138189c -s ours

am skip reason: subject contains skip directive

Change-Id: I3e56d0584e861ec214834a7568cf48c8b2d65f19

4 years agoMerge "DO NOT MERGE Ensure hci command status event has sufficient packet length...
TreeHugger Robot [Thu, 26 Mar 2020 19:27:55 +0000 (19:27 +0000)]
Merge "DO NOT MERGE  Ensure hci command status event has sufficient packet length" into qt-qpr1-dev

4 years agoRetry service discovery for Avrcp if due to sdp connection failed
Ted Wang [Tue, 17 Mar 2020 12:35:02 +0000 (20:35 +0800)]
Retry service discovery for Avrcp if due to sdp connection failed

Some remote devices are not allow multiple l2cap connection with same
psm, so service discovery for Avrcp might failed due to this reason,
causing absolute volume not enable on this device.
Retry one time if the reason of service discovery fail reason is sdp
connection failed.

Bug: 150354122
Test: Manually Test
Merged-In: I3bb511434788211950884f72ff0a1256533aaa05
Change-Id: I3bb511434788211950884f72ff0a1256533aaa05

4 years agoBluetoothAudioHAL: Have full initialization for audio hardware objects
Cheney Ni [Wed, 25 Mar 2020 10:37:33 +0000 (18:37 +0800)]
BluetoothAudioHAL: Have full initialization for audio hardware objects

Audio frameworks usually check whether a module has specific features by
particular function pointers, and must be null if unsupported. If a
module is not fully initialized, some pointers would be undefined, but
causes abnormal crashes when accessed by audio frameworks.  

Bug: 152379666
Test: manually
Change-Id: Id6a79b651f2acbd35bd509ba36a80adea5c13e05
Merged-In: Id6a79b651f2acbd35bd509ba36a80adea5c13e05
(cherry picked from commit 7397c51ac706d3577b277d7b5bcdc78c78fcc6ac)

4 years agoA2DP: Remember every peer's AVDTP delay reporting am: c12dc4d9e0
Cheney Ni [Wed, 25 Mar 2020 15:51:06 +0000 (15:51 +0000)]
A2DP: Remember every peer's AVDTP delay reporting am: c12dc4d9e0

Change-Id: I56006c33aa266418598c0eb6180f3477b520677f

4 years agoA2DP: Remember every peer's AVDTP delay reporting
Cheney Ni [Mon, 23 Mar 2020 14:40:39 +0000 (22:40 +0800)]
A2DP: Remember every peer's AVDTP delay reporting

Almost every headset has its own delay reporting. This change remembers
those particular values individually, and applies when the peer is the
active device.

Bug: 152276107
Test: manually

Change-Id: I634dfd18a034fe4020f48139d7e0f14a1309098a
Merged-In: I634dfd18a034fe4020f48139d7e0f14a1309098a
(cherry picked from commit 0bf47fa2cf71188728aac2d232e2f02e8661d92c)

4 years agoBluetoothMetrics: Return true if SaveDevice is called twice. Add more am: 8b185ac050
Chen Chen [Wed, 25 Mar 2020 00:12:55 +0000 (00:12 +0000)]
BluetoothMetrics: Return true if SaveDevice is called twice. Add more am: 8b185ac050

Change-Id: I2ab5dc860a9a4e099a730ed945a051a24ec9a2cd

4 years agoBluetoothMetrics: Return true if SaveDevice is called twice. Add more
Chen Chen [Tue, 24 Mar 2020 21:52:52 +0000 (14:52 -0700)]
BluetoothMetrics: Return true if SaveDevice is called twice. Add more
logging

Test: atest BluetoothMetricIdAllocatorTest
Bug:152251473

Change-Id: I3d5ee8d086a4e589e84a766d3d0923d3ae2278ba

4 years ago[automerger skipped] Merge "AVDTP: Prevent OOB read when parsing rejected response...
TreeHugger Robot [Mon, 23 Mar 2020 06:19:01 +0000 (06:19 +0000)]
[automerger skipped] Merge "AVDTP: Prevent OOB read when parsing rejected response" into qt-qpr1-dev am: f021675ca6 -s ours am: 61c03eb553 -s ours

am skip reason: Change-Id Iddb887c79bd8a2caa2ae5f21af15219807f9dd63 with SHA-1 c49665a9f9 is in history

Change-Id: I4e8cb89ae72f0020939d337b1ea96122e2289448

4 years ago[automerger skipped] Merge "AVDTP: Prevent OOB read when parsing rejected response...
TreeHugger Robot [Mon, 23 Mar 2020 05:53:51 +0000 (05:53 +0000)]
[automerger skipped] Merge "AVDTP: Prevent OOB read when parsing rejected response" into qt-qpr1-dev am: f021675ca6 -s ours

am skip reason: Change-Id Iddb887c79bd8a2caa2ae5f21af15219807f9dd63 with SHA-1 c49665a9f9 is in history

Change-Id: I68f5adcc7dc7f005c3d484ae68472635cbda02b1

4 years agoMerge "AVDTP: Prevent OOB read when parsing rejected response" into qt-qpr1-dev
TreeHugger Robot [Mon, 23 Mar 2020 05:36:25 +0000 (05:36 +0000)]
Merge "AVDTP: Prevent OOB read when parsing rejected response" into qt-qpr1-dev

4 years agoRe-add binder method for getPriority in IBluetoothA2dp.aidl as well as am: d0ff7d0e22
Rahul Sabnis [Sat, 21 Mar 2020 07:02:43 +0000 (07:02 +0000)]
Re-add binder method for getPriority in IBluetoothA2dp.aidl as well as am: d0ff7d0e22

Change-Id: Id99260f4beb96e69e750fc599a0d296d4bdf7685

4 years agoRe-add binder method for getPriority in IBluetoothA2dp.aidl as well as
Rahul Sabnis [Fri, 20 Mar 2020 02:40:40 +0000 (19:40 -0700)]
Re-add binder method for getPriority in IBluetoothA2dp.aidl as well as
getPriority and setPriority in IBluetoothHeadset.aidl

Bug: 148689314
Test: Manual
Change-Id: Id9efe237b24dbf24e91491826d34f61cbb761991

4 years agoBluetooth Metrics: Init and Close MetricIdAllocator and use its APIs am: 4c23956486
Chen Chen [Fri, 20 Mar 2020 20:46:18 +0000 (20:46 +0000)]
Bluetooth Metrics: Init and Close MetricIdAllocator and use its APIs am: 4c23956486

Change-Id: I2c525e6942eb4ecfceb6c03f8dd33c3f84584d23

4 years agoBluetooth Metrics: Init and Close MetricIdAllocator and use its APIs
Chen Chen [Sat, 14 Mar 2020 00:00:24 +0000 (17:00 -0700)]
Bluetooth Metrics: Init and Close MetricIdAllocator and use its APIs

Test: Manually pair device and cat config file
Bug: 142563377
Change-Id: I4492dc9b4c9d810e7919e97ef36ea151b76b2dbc
Merged-In: I4492dc9b4c9d810e7919e97ef36ea151b76b2dbc
(cherry picked from commit f6a356ead8e711e3b94e0e915a5b94cfac978ffa)

4 years ago[automerger skipped] Merge changes I685873b0,Iac3078fa into qt-qpr1-dev am: 2c4f9ef51...
Joseph Pirozzo [Fri, 20 Mar 2020 17:45:23 +0000 (17:45 +0000)]
[automerger skipped] Merge changes I685873b0,Iac3078fa into qt-qpr1-dev am: 2c4f9ef514 -s ours am: 960d4a9f0b -s ours

am skip reason: Change-Id I685873b0c4c74ddb4e273e2a38307ec2af0bbd13 with SHA-1 daa34793f2 is in history

Change-Id: Ie2864e9d046754b6c90e5e8f5d75e1a1a5801e3a

4 years ago[automerger skipped] Merge changes I685873b0,Iac3078fa into qt-qpr1-dev am: 2c4f9ef51...
Joseph Pirozzo [Fri, 20 Mar 2020 17:28:47 +0000 (17:28 +0000)]
[automerger skipped] Merge changes I685873b0,Iac3078fa into qt-qpr1-dev am: 2c4f9ef514 -s ours

am skip reason: Change-Id I685873b0c4c74ddb4e273e2a38307ec2af0bbd13 with SHA-1 daa34793f2 is in history

Change-Id: Ia10cdd7d6309e64bae3d59de1a77620eccb6d86e

4 years agoMerge changes I685873b0,Iac3078fa into qt-qpr1-dev
Joseph Pirozzo [Fri, 20 Mar 2020 17:19:37 +0000 (17:19 +0000)]
Merge changes I685873b0,Iac3078fa into qt-qpr1-dev

* changes:
  HF_Client: Send BTA_HF_CLIENT_RFC_CLOSE_EVT when client_cb == NULL
  HF_Client: Free the RFC if the handle doesn't match

4 years ago[automerger skipped] Merge "Fix bluetooth can't turn off during network reset (3...
TreeHugger Robot [Fri, 20 Mar 2020 04:02:03 +0000 (04:02 +0000)]
[automerger skipped] Merge "Fix bluetooth can't turn off during network reset (3/3)" into qt-qpr1-dev am: 60af186b8e -s ours am: e57428ba69 -s ours

am skip reason: Change-Id Ic36ffd63c376f84e3c9f0388820da86f63465c8f with SHA-1 9adf47f6c8 is in history

Change-Id: If4f1fee702a7d5830c8e1e520ba78ffe4d5f8ae0

4 years agoMerge "avrc: Validating msg size before accessing fields" into qt-qpr1-dev am: 49437a...
TreeHugger Robot [Fri, 20 Mar 2020 04:01:53 +0000 (04:01 +0000)]
Merge "avrc: Validating msg size before accessing fields" into qt-qpr1-dev am: 49437a62bd am: 4230b134a3

Change-Id: I30a22eb3b5246254aa50a864f7e6f9b2d6e8cd7c

4 years ago[automerger skipped] Merge "Fix bluetooth can't turn off during network reset (3...
TreeHugger Robot [Fri, 20 Mar 2020 03:44:15 +0000 (03:44 +0000)]
[automerger skipped] Merge "Fix bluetooth can't turn off during network reset (3/3)" into qt-qpr1-dev am: 60af186b8e -s ours

am skip reason: Change-Id Ic36ffd63c376f84e3c9f0388820da86f63465c8f with SHA-1 9adf47f6c8 is in history

Change-Id: I20a62a958b5d6ce65712af48eb31fe7d0bf489c3

4 years agoMerge "avrc: Validating msg size before accessing fields" into qt-qpr1-dev am: 49437a62bd
TreeHugger Robot [Fri, 20 Mar 2020 03:43:16 +0000 (03:43 +0000)]
Merge "avrc: Validating msg size before accessing fields" into qt-qpr1-dev am: 49437a62bd

Change-Id: I6ff012808218c6026584d4dc9217b99d8c6a3169

4 years agoMerge "Fix bluetooth can't turn off during network reset (3/3)" into qt-qpr1-dev
TreeHugger Robot [Fri, 20 Mar 2020 03:33:52 +0000 (03:33 +0000)]
Merge "Fix bluetooth can't turn off during network reset (3/3)" into qt-qpr1-dev

4 years agoMerge "avrc: Validating msg size before accessing fields" into qt-qpr1-dev
TreeHugger Robot [Fri, 20 Mar 2020 03:29:10 +0000 (03:29 +0000)]
Merge "avrc: Validating msg size before accessing fields" into qt-qpr1-dev

4 years agoMerge "Fix potential OOB vulnerability when an HCI event is received" into rvc-dev...
TreeHugger Robot [Fri, 20 Mar 2020 03:19:05 +0000 (03:19 +0000)]
Merge "Fix potential OOB vulnerability when an HCI event is received" into rvc-dev am: aca4c75e63

Change-Id: I00f9ef278b27fb7800d017044c5ad9d881a6c757

4 years agoMerge "Fix potential OOB vulnerability when an HCI event is received" into rvc-dev
TreeHugger Robot [Fri, 20 Mar 2020 03:03:23 +0000 (03:03 +0000)]
Merge "Fix potential OOB vulnerability when an HCI event is received" into rvc-dev

4 years ago[automerger skipped] Merge changes from topic "AutomotiveQBTSync" into qt-qpr1-dev...
Joseph Pirozzo [Thu, 19 Mar 2020 18:18:47 +0000 (18:18 +0000)]
[automerger skipped] Merge changes from topic "AutomotiveQBTSync" into qt-qpr1-dev am: b204c00caa -s ours am: c5487c058a -s ours

am skip reason: Change-Id If5802418f6069dbc72824745307342776175d9d8 with SHA-1 7661c2a8b9 is in history

Change-Id: I230f2a77a12ee7b61346894ad8548e81a9f2d628

4 years ago[automerger skipped] Merge changes from topic "AutomotiveQBTSync" into qt-qpr1-dev...
Joseph Pirozzo [Thu, 19 Mar 2020 18:06:19 +0000 (18:06 +0000)]
[automerger skipped] Merge changes from topic "AutomotiveQBTSync" into qt-qpr1-dev am: b204c00caa -s ours

am skip reason: Change-Id If5802418f6069dbc72824745307342776175d9d8 with SHA-1 7661c2a8b9 is in history

Change-Id: I0c194570d0c8e3049a2c550f69c45865e033789c

4 years agoMerge changes from topic "AutomotiveQBTSync" into qt-qpr1-dev
Joseph Pirozzo [Thu, 19 Mar 2020 17:54:22 +0000 (17:54 +0000)]
Merge changes from topic "AutomotiveQBTSync" into qt-qpr1-dev

* changes:
  AVRCP Controller manage focus while disconnected
  AVRCP Controller refresh track data on Interim
  AVRCP Controller get play status
  ServiceDiscoveryProtocol DB Full
  AVRCP Position Changed Notification
  DO NOT MERGE: AVRCP Controller Shuffle/Repeat support

4 years agoFix potential OOB vulnerability when an HCI event is received
jonerlin [Mon, 2 Mar 2020 11:13:31 +0000 (19:13 +0800)]
Fix potential OOB vulnerability when an HCI event is received

* Check the minimum length of Hci Events to avoid OOB vulnerability

Bug: 142546104
Bug: 142546561
Bug: 142544089
Bug: 142638492
Bug: 142638392
Bug: 142544079
Bug: 142543497
Test: inject function, Bluetooth regression test PASS.
Change-Id: I761fc56dae45bd1fe24e55669adf6a9965717830

4 years agoFix potential OOB vulnerability when an HCI event is received
jonerlin [Mon, 2 Mar 2020 11:13:31 +0000 (19:13 +0800)]
Fix potential OOB vulnerability when an HCI event is received

* Check the minimum length of Hci Events to avoid OOB vulnerability

Bug: 142546104
Bug: 142546561
Bug: 142544089
Bug: 142638492
Bug: 142638392
Bug: 142544079
Bug: 142543497
Test: inject function, Bluetooth regression test PASS.
Change-Id: I761fc56dae45bd1fe24e55669adf6a9965717830

4 years agoMerge "Randomize the time between rotations of BLE Random Address" into rvc-dev am...
Jakub Pawlowski [Thu, 19 Mar 2020 10:44:45 +0000 (10:44 +0000)]
Merge "Randomize the time between rotations of BLE Random Address" into rvc-dev am: 2dcb0c2d59

Change-Id: I6b5e18f946bd2ce733b3030e62c73150a6f0f7ba

4 years agoMerge "Randomize the time between rotations of BLE Random Address" into rvc-dev
Jakub Pawlowski [Thu, 19 Mar 2020 10:19:13 +0000 (10:19 +0000)]
Merge "Randomize the time between rotations of BLE Random Address" into rvc-dev

4 years agoMerge "Notify remote name failed for LE device" into qt-qpr1-dev am: fba6af3495 am...
More Kuo [Thu, 19 Mar 2020 02:52:06 +0000 (02:52 +0000)]
Merge "Notify remote name failed for LE device" into qt-qpr1-dev am: fba6af3495 am: 2d2c3500dd

Change-Id: I35491a414ed91c130cfcf8c220df235663d6f7dd

4 years agoMerge "Notify remote name failed for LE device" into qt-qpr1-dev am: fba6af3495
More Kuo [Thu, 19 Mar 2020 02:28:43 +0000 (02:28 +0000)]
Merge "Notify remote name failed for LE device" into qt-qpr1-dev am: fba6af3495

Change-Id: I3f70c6d6f3c8ddace2b0e3dda47a09409d0313eb

4 years agoMerge "Notify remote name failed for LE device" into qt-qpr1-dev
More Kuo [Thu, 19 Mar 2020 02:05:39 +0000 (02:05 +0000)]
Merge "Notify remote name failed for LE device" into qt-qpr1-dev

4 years agoMerge "Make sure only valid packet fields are accessed in VendorPacketHandler" into...
TreeHugger Robot [Wed, 18 Mar 2020 23:26:41 +0000 (23:26 +0000)]
Merge "Make sure only valid packet fields are accessed in VendorPacketHandler" into qt-qpr1-dev am: 0535ce8c6a am: a411a5d9e1

Change-Id: I929a9795cc8129183d6d5342d8f9e716a4248509

4 years agoMerge "Make sure only valid packet fields are accessed in VendorPacketHandler" into...
TreeHugger Robot [Wed, 18 Mar 2020 23:05:17 +0000 (23:05 +0000)]
Merge "Make sure only valid packet fields are accessed in VendorPacketHandler" into qt-qpr1-dev am: 0535ce8c6a

Change-Id: Ib85f65466fd49c6290e2ed395ea56f63827d953b

4 years agoMerge "Make sure only valid packet fields are accessed in VendorPacketHandler" into...
TreeHugger Robot [Wed, 18 Mar 2020 22:52:14 +0000 (22:52 +0000)]
Merge "Make sure only valid packet fields are accessed in VendorPacketHandler" into qt-qpr1-dev

4 years agoRandomize the time between rotations of BLE Random Address
Jakub Pawlowski [Tue, 17 Mar 2020 15:37:36 +0000 (16:37 +0100)]
Randomize the time between rotations of BLE Random Address

Bug: 150394435
Bug: 151485516
Change-Id: I342f3e34f12a200db0a5c3aab23efabac5626825

4 years agoBluetooth Metric Id: Add metric id allocation to Bluetooth
Chen Chen [Mon, 9 Mar 2020 23:19:24 +0000 (16:19 -0700)]
Bluetooth Metric Id: Add metric id allocation to Bluetooth
metric calls

Test: Build successfully
Bug: 142563377
Change-Id: I5d34d745fc09fcfac38e583765c6762bc943d7b0

4 years agoHF_Client: Send BTA_HF_CLIENT_RFC_CLOSE_EVT when client_cb == NULL
Song Gao [Thu, 12 Dec 2019 18:32:27 +0000 (10:32 -0800)]
HF_Client: Send BTA_HF_CLIENT_RFC_CLOSE_EVT when client_cb == NULL

Bug: 146086992
Test: manual
Change-Id: I685873b0c4c74ddb4e273e2a38307ec2af0bbd13
Merged-In: I685873b0c4c74ddb4e273e2a38307ec2af0bbd13
(cherry picked from commit daa34793f2e076be3bf600e079128754a0cb8576)

4 years agoHF_Client: Free the RFC if the handle doesn't match
Song Gao [Thu, 12 Dec 2019 18:28:36 +0000 (10:28 -0800)]
HF_Client: Free the RFC if the handle doesn't match

Bug: 146086995
Test: manual
Change-Id: Iac3078fa2d10373f295fe9c6fd49ddc39f15eb98
Merged-In: Iac3078fa2d10373f295fe9c6fd49ddc39f15eb98
(cherry picked from commit 8da3a9ed0886821be7fb4a9fa71abbb112e7a95f)

4 years agoMerge "DO NOT MERGE Fix potential overflow for gatt process write" into rvc-dev am...
Automerger Merge Worker [Mon, 16 Mar 2020 16:40:00 +0000 (16:40 +0000)]
Merge "DO NOT MERGE Fix potential overflow for gatt process write" into rvc-dev am: 686a63f483

Change-Id: I31780c980932a198c7b215b0fbb3f79a5fc186f0

4 years agoDO NOT MERGE Fix potential overflow for gatt process write am: a6fcdfabeb
Automerger Merge Worker [Mon, 16 Mar 2020 16:39:59 +0000 (16:39 +0000)]
DO NOT MERGE Fix potential overflow for gatt process write am: a6fcdfabeb

Change-Id: I093aa39a96b03f4905f852555950466f74aa3b9b

4 years agoFix bond list mismatch between Java and native am: 8ffea167cb
Automerger Merge Worker [Mon, 16 Mar 2020 16:39:43 +0000 (16:39 +0000)]
Fix bond list mismatch between Java and native am: 8ffea167cb

Change-Id: I979372e34a3f3f80fa2c56f5c89a3ed3c9325813

4 years agoMerge "DO NOT MERGE Fix potential overflow for gatt process write" into rvc-dev
TreeHugger Robot [Mon, 16 Mar 2020 16:25:40 +0000 (16:25 +0000)]
Merge "DO NOT MERGE Fix potential overflow for gatt process write" into rvc-dev

4 years agoFix bond list mismatch between Java and native
Ugo Yu [Mon, 9 Mar 2020 06:54:14 +0000 (14:54 +0800)]
Fix bond list mismatch between Java and native

When a paired device is diconnected while authenticating, do not report
bond state change to Java since the stack does not remove the device in
this situation.

Bug: 142842685
Test: Manual
Change-Id: I308320b0c10387d1cce27f709c8b2348c14dc423
Merged-In: I308320b0c10387d1cce27f709c8b2348c14dc423

4 years agoNotify remote name failed for LE device
More Kuo [Fri, 13 Mar 2020 06:43:29 +0000 (14:43 +0800)]
Notify remote name failed for LE device

When cancel discovery for LE device in BTA_DM_DISCOVER_ACTIVE state,
there is no notification to indicate the remote name request of LE
device is cancelled and change the search state back to
BTA_DM_SEARCH_IDLE state. Call btm_inq_rmt_name_failed_cancelled()
to notify in this situation.

Bug: 146840780
Test: Manual
Change-Id: I4ab2a2f413e4ac19765476039f4127ce045e84d9

4 years agoDO NOT MERGE Ensure hci command status event has sufficient packet length
Chris Manton [Tue, 3 Mar 2020 18:33:42 +0000 (10:33 -0800)]
DO NOT MERGE  Ensure hci command status event has sufficient packet length

Bug: 141618611
Test: net_test_hci_native

Change-Id: I70a318b05d7781ddf8f82d7922a8ee7afc8d2e9f
(cherry picked from commit 6e25c5d81c4a43c2794a605c9fc8a194f37889af)

4 years agoDO NOT MERGE Fix potential overflow for gatt process write
Chris Manton [Wed, 4 Mar 2020 19:26:31 +0000 (11:26 -0800)]
DO NOT MERGE Fix potential overflow for gatt process write

Bug: 143604331
Test: net_test_stack_gatt_native

Change-Id: I88704d74ba0c265872fbfd20c8bcd3bc1dd60360
(cherry picked from commit af0919f967eef7fffaf698fa0a30f0efe7d941b5)

4 years agoSave reversed BR/EDR link key derived from LE LTK am: 0f13f7c4ca
Automerger Merge Worker [Fri, 13 Mar 2020 20:02:18 +0000 (20:02 +0000)]
Save reversed BR/EDR link key derived from LE LTK am: 0f13f7c4ca

Change-Id: I7ad0c1f19ae3a63ec42576450346beb616904063

4 years agoAVRCP Controller manage focus while disconnected
Joseph Pirozzo [Mon, 18 Nov 2019 19:44:37 +0000 (11:44 -0800)]
AVRCP Controller manage focus while disconnected

If Audio focus is lost(or gained)  while A2DP is not connected there is the
possibility for states to get out of sync.  Removing the check for
connectivity ensures that the state can be properly managed.

Bug: 144082798
Test: Disconnect BT device, lose audio focus, verify
btif_a2dp_sink_set_focus_state_event state gets updated.

Change-Id: If5802418f6069dbc72824745307342776175d9d8
(cherry picked from commit 7661c2a8b90bee7edcf7c1279abe06ed32e22a2c)

Change-Id: I22a7ee2a3e51c3dea441a6f185e1c993cd65c4a1
Merged-In: If5802418f6069dbc72824745307342776175d9d8

4 years agoAVRCP Controller refresh track data on Interim
Joseph Pirozzo [Thu, 14 Nov 2019 00:02:06 +0000 (16:02 -0800)]
AVRCP Controller refresh track data on Interim

Upon receiving an interim track changed event fetch the now playing
track information.  Resolves issues where track changes get out of sync.

Bug: 143954201
Test: connect a phone and change tracks rapidly on phone.
Change-Id: I1e4ce62df6839dd2b9cf40832556a9b2608593e7
(cherry picked from commit a3d62166f685f5249b813bd110e894682c84da54)

Change-Id: Ia6cc2a7a0430d6d9e3ae6e911e3fc8834221b599
Merged-In: I1e4ce62df6839dd2b9cf40832556a9b2608593e7

4 years agoAVRCP Controller get play status
Joseph Pirozzo [Mon, 21 Oct 2019 22:39:09 +0000 (15:39 -0700)]
AVRCP Controller get play status

Correct the parsing error in the GET_PLAY_STATUS message and call the
proper jni callback when it is received.

Bug: 139033614
Test: Connect AVRCP change media players, observe correct value parsed.
Change-Id: I8302b64efdd72bfa671cca5b8c512a436fa54ab3
(cherry picked from commit 86b473cae724e9db2efec9ed1ee50a71f2a2dd22)

Merged-In: 86b473cae724e9db2efec9ed1ee50a71f2a2dd22
Change-Id: I0c753fbc7bb62698cf23b9ded13497b6b60cda35
Merged-In: I8302b64efdd72bfa671cca5b8c512a436fa54ab3

4 years agoServiceDiscoveryProtocol DB Full
Joseph Pirozzo [Tue, 1 Oct 2019 21:13:29 +0000 (14:13 -0700)]
ServiceDiscoveryProtocol DB Full

Increase the size of the local SDP DB to cope with newer phones supporting
more Bluetooth profiles.

Bug: 141889288
Test: pair Bluetooth device verify no warning "SDP - DB full add_attr"
Change-Id: I823ea6e178f098a0441efde4fb7d8e7fb68e38c0
(cherry picked from commit e62a51a4d6a25de3629194bad7d1d42b739e231f)

Change-Id: I06259aa3aa9b1d034b8b60dd9dcbfffe0080bacc
Merged-In: I823ea6e178f098a0441efde4fb7d8e7fb68e38c0

4 years agoAVRCP Position Changed Notification
Joseph Pirozzo [Fri, 28 Jun 2019 18:07:52 +0000 (11:07 -0700)]
AVRCP Position Changed Notification

Correct position change notification to be received every 2000 ms
instead of every 2000 s.

Bug: 133400561
Test: Connect to device and monitor avrcp traffic.
Change-Id: Id8a33670b207f4a9ac03da957075acbac0c7c052
(cherry picked from commit f11c6baecf498b2f407c545ab8d9dcc34cc62c4d)

Change-Id: If5324d4ef5fe20c71c7b603d648fff82b81ce550
Merged-In: Id8a33670b207f4a9ac03da957075acbac0c7c052

4 years agoDO NOT MERGE: AVRCP Controller Shuffle/Repeat support
Joseph Pirozzo [Fri, 26 Jul 2019 19:26:01 +0000 (12:26 -0700)]
DO NOT MERGE: AVRCP Controller Shuffle/Repeat support

Fetch player settings when the addressed player changes.

Bug: 72495707
Test: AvrcpControllerStateMachineTest#testShuffle
      AvrcpControllerStateMachineTest#testRepeat

Change-Id: I4df1f130dafb96708ec7a0a2c13c776affcc6318
(cherry picked from commit 60a193dfeb155aeb8428fb341737b385f8260899)

Change-Id: I944419ad0bd5e49bad8a988851c1a753c0753d01
Merged-In: I4df1f130dafb96708ec7a0a2c13c776affcc6318

4 years agoSave reversed BR/EDR link key derived from LE LTK
IHLHO KIM [Tue, 5 Nov 2019 08:56:24 +0000 (08:56 +0000)]
Save reversed BR/EDR link key derived from LE LTK

Authentication is failed between Android Q and Android P or earlier when derived link key is used.

Repro steps:
1. Try to LE connect & pair(LE secure connection) with Android P device from Android Q device
2. Try to connect PAN from Android Q device to Android P device
3. Connection failed

This issue is caused by the following change.
https://android.googlesource.com/platform/system/bt/+/ae57211e7c6ef25788369052817403e30cc9f06f

This fix just reverts removed line from the previous change.
Bug: 143940249
Change-Id: I5d18cdd76b6c436dad010caf514ab0d78997f091

4 years ago[automerger skipped] Merge "While AVDT opened or its AVRC_TIMER_EVT expired, it is...
Automerger Merge Worker [Fri, 13 Mar 2020 04:54:49 +0000 (04:54 +0000)]
[automerger skipped] Merge "While AVDT opened or its AVRC_TIMER_EVT expired, it is no needed to start the 2nd AVDT stream" into qt-qpr1-dev am: 5e6231c7b2 -s ours am: 145941ff2f -s ours

am skip reason: Change-Id I75f39801e9779ee55fa574e30051e01966c61ea3 with SHA-1 ba432366f9 is in history

Change-Id: I7a0f8d891096ea794775ebc9aaf860d0badf7acb

4 years ago[automerger skipped] Merge "While AVDT opened or its AVRC_TIMER_EVT expired, it is...
Automerger Merge Worker [Fri, 13 Mar 2020 04:37:20 +0000 (04:37 +0000)]
[automerger skipped] Merge "While AVDT opened or its AVRC_TIMER_EVT expired, it is no needed to start the 2nd AVDT stream" into qt-qpr1-dev am: 5e6231c7b2 -s ours

am skip reason: Change-Id I75f39801e9779ee55fa574e30051e01966c61ea3 with SHA-1 ba432366f9 is in history

Change-Id: I182a3d57dca6c055487dc1cbd659e53eaab0f111

4 years agoMerge "While AVDT opened or its AVRC_TIMER_EVT expired, it is no needed to start...
TreeHugger Robot [Fri, 13 Mar 2020 04:21:17 +0000 (04:21 +0000)]
Merge "While AVDT opened or its AVRC_TIMER_EVT expired, it is no needed to start the 2nd AVDT stream" into qt-qpr1-dev

4 years agoHFP: fix collision for the outgoing and incoming connection am: 0f12c08a5f
Automerger Merge Worker [Fri, 13 Mar 2020 01:27:24 +0000 (01:27 +0000)]
HFP: fix collision for the outgoing and incoming connection am: 0f12c08a5f

Change-Id: Ie4cd0314547ce495b0c3aafa52d2a2520307f487

4 years agoHFP: fix collision for the outgoing and incoming connection
weichinweng [Wed, 11 Mar 2020 06:51:27 +0000 (14:51 +0800)]
HFP: fix collision for the outgoing and incoming connection

Since the HF incoming connection and the HF outgoing connection use the
same HF control block (p_scb) at the same time, the case will cause the
btif_hf_cb flag going wrong due to collision. Then it will trigger a
crash due to a check failure when executing BTA_AG_OPEN_EVT. The patch
will ignore the outgoing connection when the connection
collision is happening.

Bug: 143128949
Test: manual

Change-Id: Id5495bd3ecb1ed9e03de5bd74a61ab102604eaae
Merged-In: Id5495bd3ecb1ed9e03de5bd74a61ab102604eaae

4 years agoMerge "DO NOT MERGE Fix potential overflow in btif_rc" into rvc-dev am: f465348c7a
Automerger Merge Worker [Thu, 12 Mar 2020 02:35:56 +0000 (02:35 +0000)]
Merge "DO NOT MERGE Fix potential overflow in btif_rc" into rvc-dev am: f465348c7a

Change-Id: I3706d649e70469ecf377c03f9b7717c7fce27f51