OSDN Git Service

android-x86/system-bt.git
4 years agobtsnoop: Fix btsnoop_net runs unexpected am: dc57138de5
KimJeongYeon [Tue, 21 Apr 2020 21:45:35 +0000 (21:45 +0000)]
btsnoop: Fix btsnoop_net runs unexpected am: dc57138de5

Change-Id: I5cfcf4f2da8145c3a5765f2f7c04a3ce5e2fd37e

4 years agobtsnoop: Fix btsnoop_net runs unexpected
KimJeongYeon [Tue, 10 Mar 2020 04:00:40 +0000 (13:00 +0900)]
btsnoop: Fix btsnoop_net runs unexpected

By default, btsnoop_net was disabled using "BT_NET_DEBUG" macro.
But, it enabled actually and manages socket connection for sending
btsnoop logs. The reason was, TRUE/FALSE macro were not defined.
This fix including "bt_types.h" to work macro properly.

Bug: 154647071
Test: Turn on btsnoop logging and observe btsnoop_net runs or not.

Signed-off-by: KimJeongYeon <jeongyeon.kim@samsung.com>
Change-Id: I595d06510170d27fbf81068fa7ce8769eabd0992
Merged-In: I595d06510170d27fbf81068fa7ce8769eabd0992
(cherry picked from commit bc6072c798a597624bda3e999c3432d384d851a0)

4 years agoMerge "Send the only song in queue as now playing song when get item attributes of...
TreeHugger Robot [Mon, 20 Apr 2020 14:36:21 +0000 (14:36 +0000)]
Merge "Send the only song in queue as now playing song when get item attributes of now playing" into rvc-dev am: b5693ec293

Change-Id: I2dd5b9e6464982b31559e56466d268946a6f4d65

4 years agoMerge "Send the only song in queue as now playing song when get item attributes of...
TreeHugger Robot [Mon, 20 Apr 2020 14:13:41 +0000 (14:13 +0000)]
Merge "Send the only song in queue as now playing song when get item attributes of now playing" into rvc-dev

4 years agoA2DP: Fix integer sanitizer in SBC encoder am: 7f973d8726
Cheney Ni [Mon, 20 Apr 2020 13:13:45 +0000 (13:13 +0000)]
A2DP: Fix integer sanitizer in SBC encoder am: 7f973d8726

Change-Id: I73787acdbe3b951a7a3239a4301c77deaf206262

4 years ago[automerger skipped] DO NOT MERGE Check a2dp packet length is zero am: d61cf5fe04...
Chris Manton [Fri, 17 Apr 2020 19:46:50 +0000 (19:46 +0000)]
[automerger skipped] DO NOT MERGE Check a2dp packet length is zero am: d61cf5fe04 -s ours

am skip reason: Change-Id I105b445293c02cb4f37c759fd5b05758fd4e3646 with SHA-1 96392b0f2c is in history

Change-Id: I588fee43c2c097a38a1f14e9f201e6e69e0dafa6

4 years agoA2DP: Fix integer sanitizer in SBC encoder
Cheney Ni [Sat, 11 Apr 2020 21:19:31 +0000 (05:19 +0800)]
A2DP: Fix integer sanitizer in SBC encoder

There were three potential integer overflow within SBC software encoder:

* embdrv/sbc/encoder/srce/sbc_packing.c:144:38: runtime error: unsigned
  integer overflow: 4294967231 + 8192 cannot be represented in type
  'unsigned int'
* embdrv/sbc/encoder/srce/sbc_packing.c:147:9: runtime error: signed
  integer overflow: 37932 * 65535 cannot be represented in type 'int'
* embdrv/sbc/encoder/srce/sbc_packing.c:147:9: runtime error: signed
  integer overflow: 178177545 + 2146959360 cannot be represented in type
  'int'

They were caught by the integer sanitizer, and

1. (*ps32SbPtr >> 2) is either greater than 0xFF00,0000 or less than
   0x007F,FFFF, so just cast to a signed integer explicitly.
2. Positive integer between 0x8000,0000 ~ 0xFFFF,FFFF can't be
   represented in type 'int', but is still feasible in 32-bits.
3. s32OutLow is the lower byte of a 64 bits integer, but can't have the
   carry values which is only for the higher byte.

This change gives the compiler a signed 64-bits variable, and trusts it
to do better optimization at multiplication.

Bug: 153402404
Test: make sure there are no integer sanitization errors.
Change-Id: I5046a42f9927c1aa7c25da2828c4f921ba7a5021
Merged-In: I5046a42f9927c1aa7c25da2828c4f921ba7a5021
(cherry picked from commit a42db783434da238e4daade95ce2adb1bca0f138)

4 years agoDO NOT MERGE Check a2dp packet length is zero
Chris Manton [Tue, 10 Mar 2020 15:59:02 +0000 (08:59 -0700)]
DO NOT MERGE Check a2dp packet length is zero

already in master

Bug: 142546668
Test: net_test_stack_a2dp_native

Change-Id: I105b445293c02cb4f37c759fd5b05758fd4e3646
Merged-In: I105b445293c02cb4f37c759fd5b05758fd4e3646
(cherry picked from commit 96392b0f2cfb2adc72cc7cad0d74dec8f4041582)

4 years agoRevert "Mark updatable APEXes" am: e3ff9a59f8
Jiyong Park [Fri, 17 Apr 2020 05:36:42 +0000 (05:36 +0000)]
Revert "Mark updatable APEXes" am: e3ff9a59f8

Change-Id: Iaba5dc471efa3997d3444ff5d106d34136b182e2

4 years agoRevert "Mark updatable APEXes"
Jiyong Park [Fri, 17 Apr 2020 03:55:59 +0000 (03:55 +0000)]
Revert "Mark updatable APEXes"

This reverts commit f3d123910b4c899b3aed8d82272fa5d24b6dc426.

Reason for revert: the BT apex is not yet updatable
Bug: 149805758
Test: m
Change-Id: I92cf6ac12f989726a797fec6a0c0f121346c188e

4 years agoSend the only song in queue as now playing song when get item attributes
Ted Wang [Thu, 26 Mar 2020 10:11:26 +0000 (18:11 +0800)]
Send the only song in queue as now playing song when get item attributes
of now playing

Bug: 152007084
Test: Manully
Change-Id: I1d31612f20fe21e8364c578718ca282eeb75eac0
Merged-In: I1d31612f20fe21e8364c578718ca282eeb75eac0

4 years agoMerge "Fix potential OOB vulnerability when SBC framedata is prefetched" into rvc...
Joseph Pirozzo [Tue, 14 Apr 2020 23:12:07 +0000 (23:12 +0000)]
Merge "Fix potential OOB vulnerability when SBC framedata is prefetched" into rvc-dev am: 633b811269

Change-Id: Ibe7ea77fb263d5571ac141f02136fb0dd0943e37

4 years agoMerge "Fix potential OOB vulnerability when SBC framedata is prefetched" into rvc-dev
Joseph Pirozzo [Tue, 14 Apr 2020 22:58:56 +0000 (22:58 +0000)]
Merge "Fix potential OOB vulnerability when SBC framedata is prefetched" into rvc-dev

4 years agoMerge "AVCRP Controller support Get Item Attributes" into rvc-dev am: 6c8e7b0de3
Sal Savage [Tue, 14 Apr 2020 19:42:56 +0000 (19:42 +0000)]
Merge "AVCRP Controller support Get Item Attributes" into rvc-dev am: 6c8e7b0de3

Change-Id: I38e422ed3001cde8351e12ca6ff569aa70aa3a80

4 years agoMerge "AVCRP Controller support Get Item Attributes" into rvc-dev
Sal Savage [Tue, 14 Apr 2020 19:22:21 +0000 (19:22 +0000)]
Merge "AVCRP Controller support Get Item Attributes" into rvc-dev

4 years agoA2DP: Wait for previous AVDTP instance to be clean up before re-enabling am: f2cf5ebc7f
Cheney Ni [Tue, 14 Apr 2020 15:00:13 +0000 (15:00 +0000)]
A2DP: Wait for previous AVDTP instance to be clean up before re-enabling am: f2cf5ebc7f

Change-Id: I76f4fa637fd61030ed2c7d80792b40221b463983

4 years agoA2DP: Workaround to clean up previous AVDTP instance while re-enabling am: 3537765c21
Cheney Ni [Tue, 14 Apr 2020 15:00:10 +0000 (15:00 +0000)]
A2DP: Workaround to clean up previous AVDTP instance while re-enabling am: 3537765c21

Change-Id: I88d4af5d20cf09bbc973d646aaaee8ddfd668877

4 years agoA2DP: Be more careful in allocating a BtifAvPeer instance am: f92966049c
Cheney Ni [Tue, 14 Apr 2020 15:00:07 +0000 (15:00 +0000)]
A2DP: Be more careful in allocating a BtifAvPeer instance am: f92966049c

Change-Id: I98c77dadc9031c78a0f319485d8811ff3403f164

4 years agoA2DP: Wait for previous AVDTP instance to be clean up before re-enabling
Cheney Ni [Wed, 19 Feb 2020 09:41:59 +0000 (17:41 +0800)]
A2DP: Wait for previous AVDTP instance to be clean up before re-enabling

Because more connections spend more time to disconnect, the stack may
still be busy at disabling AVDTP, if we re-enable the A2dpService
quickly. This change will defer BTA_AV_API_ENABLE and BTA_AV_API_REG
events if the stack is still disabling the previous instance, so gives
the stack half more seconds to handle its previous task.

Bug: 135655859
Test: manually
Change-Id: I8e700ce195aa634819ea1ad8cf5892cc35669e10
Merged-In: I8e700ce195aa634819ea1ad8cf5892cc35669e10
(cherry picked from commit 21fe257db23887b3000a6c8d38ee339fcccbdc2b)

4 years agoA2DP: Workaround to clean up previous AVDTP instance while re-enabling
Cheney Ni [Wed, 18 Mar 2020 08:37:29 +0000 (16:37 +0800)]
A2DP: Workaround to clean up previous AVDTP instance while re-enabling

If the user disables and re-enables A2dpService quickly, the stack has
chances to be still busy at dropping those out-of-date links, and this
is highly dependent on the handset. If the handset responds more slowly, it
spends more time to disable, but causes the new AVDTP instance won't
work. This change checks the previous disabling flag, and cleans up the
old instance up before re-enabling.

Bug: 135655859
Test: manually
Change-Id: I625cfffab6d4df327fb8159aa4b759a09d90e076
Merged-In: I625cfffab6d4df327fb8159aa4b759a09d90e076
(cherry picked from commit 07689d65f4c6fbc8b20dd0890657936fc6d6cb7b)

4 years agoA2DP: Be more careful in allocating a BtifAvPeer instance
Cheney Ni [Fri, 27 Mar 2020 03:54:28 +0000 (11:54 +0800)]
A2DP: Be more careful in allocating a BtifAvPeer instance

While a new peer is coming either by in-coming or out-going connection,
the stack first tries to find or create an associated BtifAvPeer object
for it, but uses an unknown BTA handle if BTA_AV is still registering.
This handle will be updated after BTA_AV registered, but causes
a mismatch in the handle usage between BTIF and BTA layers since the
register process just finished. This change disallows to use an unknown
BTA handle, so it prevents such abnormal cases. Besides, it adds more
log messages about BTA_AV registering, and also makes sure the disabling
flag won't be activated after disabled.

Bug: 135655859
Bug: 152597903
Test: Switch BT state quickly between STATE_BLE_ON and STATE_ON
Change-Id: I9df3d64f301dffbecdeaf3de18dd455be1c63ce2
Merged-In: I9df3d64f301dffbecdeaf3de18dd455be1c63ce2
(cherry picked from commit 714e639cb7291ecfb9feb12acad9bc7c8f455ba0)

4 years agoAVCRP Controller support Get Item Attributes
Sal Savage [Tue, 3 Mar 2020 22:43:06 +0000 (14:43 -0800)]
AVCRP Controller support Get Item Attributes

Add support for the GetItemAttributes command when the browsing chanel
is connected.

Bug: 149411071
Test: Play a song from a paired phone compare metadata and HCI logs.
Change-Id: I0ac3c3bb7c9d1c3c72f0e2bad13d276c836ecf6f
Merged-In: I0ac3c3bb7c9d1c3c72f0e2bad13d276c836ecf6f

4 years agoFix potential OOB vulnerability when SBC framedata is prefetched
Joseph Pirozzo [Tue, 31 Mar 2020 20:57:21 +0000 (13:57 -0700)]
Fix potential OOB vulnerability when SBC framedata is prefetched

Macros were filling a value buffer when reading bit stream data in
advance of any fetch being called, updated to only fetch data into
buffer when needed.

Bug: 110107252
Bug: 146032016
Bug: 146398979
Test: sbcdecoder_fuzzer
Change-Id: Ibb7793c9a015eaddcc8faa6ccfd8ec998d03ed86

4 years ago[automerger skipped] Merge "Check a2dp packet length is zero" into qt-dev am: c909de6...
Sterling Huber [Thu, 9 Apr 2020 17:54:09 +0000 (17:54 +0000)]
[automerger skipped] Merge "Check a2dp packet length is zero" into qt-dev am: c909de62bf -s ours am: db1917d8ae am: e134c6f179

Change-Id: I7b59a34358ec5178219188a0158e66ab05133850

4 years ago[automerger skipped] Merge "Check a2dp packet length is zero" into qt-dev am: c909de6...
Sterling Huber [Thu, 9 Apr 2020 17:39:35 +0000 (17:39 +0000)]
[automerger skipped] Merge "Check a2dp packet length is zero" into qt-dev am: c909de62bf -s ours am: db1917d8ae

Change-Id: Id1c8c3155d96ee1e050dd1d308ebeca838a22c8e

4 years ago[automerger skipped] Merge "Check a2dp packet length is zero" into qt-dev am: c909de6...
Sterling Huber [Thu, 9 Apr 2020 17:17:37 +0000 (17:17 +0000)]
[automerger skipped] Merge "Check a2dp packet length is zero" into qt-dev am: c909de62bf -s ours

am skip reason: skipped by user cmanton

Change-Id: I8ff4f933b88c68473b378d0894b89ae8b10431fd

4 years agoMerge "btm: fixing oob write in multi-adv SetData." into qt-qpr1-dev am: aeeb427c94...
TreeHugger Robot [Wed, 8 Apr 2020 17:24:04 +0000 (17:24 +0000)]
Merge "btm: fixing oob write in multi-adv SetData." into qt-qpr1-dev am: aeeb427c94 am: 873f23b9be

Change-Id: I8a6ea3ad108de14fe22ccdff10e5b52d217b99ad

4 years agoMerge "btm: fixing oob write in multi-adv SetData." into qt-qpr1-dev am: aeeb427c94
TreeHugger Robot [Wed, 8 Apr 2020 17:00:45 +0000 (17:00 +0000)]
Merge "btm: fixing oob write in multi-adv SetData." into qt-qpr1-dev am: aeeb427c94

Change-Id: Ibb621c8984997088b6ec96cc61fc65733e73c673

4 years agoMerge "btm: fixing oob write in multi-adv SetData." into qt-qpr1-dev
TreeHugger Robot [Wed, 8 Apr 2020 16:40:55 +0000 (16:40 +0000)]
Merge "btm: fixing oob write in multi-adv SetData." into qt-qpr1-dev

4 years ago[automerger skipped] Fix potential stack overflow caused by integer overflow am:...
Jakub Pawlowski [Wed, 8 Apr 2020 00:24:50 +0000 (00:24 +0000)]
[automerger skipped] Fix potential stack overflow caused by integer overflow am: 1570b62c88 am: 5b4bf745e9 am: 6360e91bb3 am: 3cd74938d8 -s ours am: db0c360dce -s ours am: ca7a1eabe0 -s ours am: 67fa765f93 -s ours

am skip reason: Change-Id I0655b0b62301f78cd8705cc7b0e4fc11522f00ca with SHA-1 ec0d507ed2 is in history

Change-Id: Id21decf03db19bd2e9d771268bf21378ef92f6d2

4 years agoMerge "Check a2dp packet length is zero" into qt-dev
Sterling Huber [Wed, 8 Apr 2020 00:13:33 +0000 (00:13 +0000)]
Merge "Check a2dp packet length is zero" into qt-dev

4 years ago[automerger skipped] Fix potential stack overflow caused by integer overflow am:...
Jakub Pawlowski [Wed, 8 Apr 2020 00:10:53 +0000 (00:10 +0000)]
[automerger skipped] Fix potential stack overflow caused by integer overflow am: 1570b62c88 am: 5b4bf745e9 am: 6360e91bb3 am: 3cd74938d8 -s ours am: db0c360dce -s ours am: ca7a1eabe0 -s ours

am skip reason: Change-Id I0655b0b62301f78cd8705cc7b0e4fc11522f00ca with SHA-1 ec0d507ed2 is in history

Change-Id: Id31d6d296c0c76921d0b581362e7e1e0a75f7f48

4 years ago[automerger skipped] Fix potential stack overflow caused by integer overflow am:...
Jakub Pawlowski [Tue, 7 Apr 2020 23:50:35 +0000 (23:50 +0000)]
[automerger skipped] Fix potential stack overflow caused by integer overflow am: 1570b62c88 am: 5b4bf745e9 am: 6360e91bb3 am: 3cd74938d8 -s ours am: db0c360dce -s ours

am skip reason: Change-Id I0655b0b62301f78cd8705cc7b0e4fc11522f00ca with SHA-1 ec0d507ed2 is in history

Change-Id: I2748027eb3a50e6ed5130be43861fa69d794bee8

4 years ago[automerger skipped] Fix potential stack overflow caused by integer overflow am:...
Jakub Pawlowski [Tue, 7 Apr 2020 23:38:36 +0000 (23:38 +0000)]
[automerger skipped] Fix potential stack overflow caused by integer overflow am: 1570b62c88 am: 5b4bf745e9 am: 6360e91bb3 am: 3cd74938d8 -s ours

am skip reason: Change-Id I0655b0b62301f78cd8705cc7b0e4fc11522f00ca with SHA-1 ec0d507ed2 is in history

Change-Id: Ife4a1d737088dbf51ac29f0a6411d9468b456593

4 years agoFix potential stack overflow caused by integer overflow am: 1570b62c88 am: 5b4bf745e9...
Jakub Pawlowski [Tue, 7 Apr 2020 23:24:21 +0000 (23:24 +0000)]
Fix potential stack overflow caused by integer overflow am: 1570b62c88 am: 5b4bf745e9 am: 6360e91bb3

Change-Id: I1e23e75fd05e41b9cd31bb93b945135bef21c04f

4 years agoFix potential stack overflow caused by integer overflow am: 1570b62c88 am: 5b4bf745e9
Jakub Pawlowski [Tue, 7 Apr 2020 23:09:11 +0000 (23:09 +0000)]
Fix potential stack overflow caused by integer overflow am: 1570b62c88 am: 5b4bf745e9

Change-Id: I91417fff40a89042993a9fcbfaa6b7f45c162d89

4 years ago[automerger skipped] Fix potential stack overflow caused by integer overflow am:...
Jakub Pawlowski [Tue, 7 Apr 2020 22:55:20 +0000 (22:55 +0000)]
[automerger skipped] Fix potential stack overflow caused by integer overflow am: ec0d507ed2 am: 8ccc0c8d5f -s ours am: 7a1190ef0c -s ours

am skip reason: Change-Id I0655b0b62301f78cd8705cc7b0e4fc11522f00ca with SHA-1 4c0c93eb61 is in history

Change-Id: I696b3093d3452bd25c592ddb226a2e8aabf70139

4 years agoFix potential stack overflow caused by integer overflow am: 1570b62c88
Jakub Pawlowski [Tue, 7 Apr 2020 22:51:21 +0000 (22:51 +0000)]
Fix potential stack overflow caused by integer overflow am: 1570b62c88

Change-Id: Iaf75d5b63297d50e3115422f15fa8511133ef45a

4 years ago[automerger skipped] Fix potential stack overflow caused by integer overflow am:...
Jakub Pawlowski [Tue, 7 Apr 2020 22:43:47 +0000 (22:43 +0000)]
[automerger skipped] Fix potential stack overflow caused by integer overflow am: ec0d507ed2 am: 8ccc0c8d5f -s ours

am skip reason: Change-Id I0655b0b62301f78cd8705cc7b0e4fc11522f00ca with SHA-1 4c0c93eb61 is in history

Change-Id: I2d54dc3becde918d809bfc2f970bc192bb562b1b

4 years agoFix potential stack overflow caused by integer overflow am: ec0d507ed2
Jakub Pawlowski [Tue, 7 Apr 2020 22:31:00 +0000 (22:31 +0000)]
Fix potential stack overflow caused by integer overflow am: ec0d507ed2

Change-Id: Ia8ae7d5d144e3a5e94728dc236d9e7af6ddd3dd2

4 years agoCheck a2dp packet length is zero
Chris Manton [Tue, 10 Mar 2020 15:59:02 +0000 (08:59 -0700)]
Check a2dp packet length is zero

Bug: 142546668
Test: net_test_stack_a2dp_native

Change-Id: I105b445293c02cb4f37c759fd5b05758fd4e3646
Merged-In: I105b445293c02cb4f37c759fd5b05758fd4e3646

4 years agoMerge "Remove BTA_HF_CLIENT_SCO_LISTEN_E fallthrough" into rvc-dev am: a1d7af831b
Joseph Pirozzo [Tue, 7 Apr 2020 19:40:22 +0000 (19:40 +0000)]
Merge "Remove BTA_HF_CLIENT_SCO_LISTEN_E fallthrough" into rvc-dev am: a1d7af831b

Change-Id: Ic108c9fe910a2c877c1c1d16a95b4089c4a1ad7a

4 years agoMerge "Remove BTA_HF_CLIENT_SCO_LISTEN_E fallthrough" into rvc-dev
Joseph Pirozzo [Tue, 7 Apr 2020 19:32:41 +0000 (19:32 +0000)]
Merge "Remove BTA_HF_CLIENT_SCO_LISTEN_E fallthrough" into rvc-dev

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 agoRemove BTA_HF_CLIENT_SCO_LISTEN_E fallthrough
Joseph Pirozzo [Tue, 7 Apr 2020 16:00:03 +0000 (09:00 -0700)]
Remove BTA_HF_CLIENT_SCO_LISTEN_E fallthrough

Remove unintended fallthrough.
Bug: 143901894
Test: connect SCO from both HF and AG.

Change-Id: Ifa5363479d4c844ed0b6a8585c25147b23156599
Merged-In: Ifa5363479d4c844ed0b6a8585c25147b23156599

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 agoFix potential stack overflow caused by integer overflow
Jakub Pawlowski [Fri, 20 Mar 2020 14:24:00 +0000 (15:24 +0100)]
Fix potential stack overflow caused by integer overflow

Bug: 151155194
Merged-In: I0655b0b62301f78cd8705cc7b0e4fc11522f00ca
Change-Id: I0655b0b62301f78cd8705cc7b0e4fc11522f00ca

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
Merged-In: I0655b0b62301f78cd8705cc7b0e4fc11522f00ca
Change-Id: I0655b0b62301f78cd8705cc7b0e4fc11522f00ca

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