Merge "Merge "Make RawAddress into a class (1/3)" am: 4a1fc85744" into stage-aosp-master

Merge "Make RawAddress into a class (1/3)"
am: 4a1fc85744

Change-Id: I61d249aa17044cd55b6834bc9563f648362621b6

Merge "Make RawAddress into a class (1/3)"
am: 4a1fc85744

Change-Id: I05b23ab3aff4d61dd8b04a3f011e832305203ce5

Merge "Make RawAddress into a class (1/3)"

Merge "Remove unused BDADDR definitions am: 525e743277" into stage-aosp-master

Remove unused BDADDR definitions
am: 525e743277

Change-Id: I211d82d42a53fe9bb2a6b930c7d99d5187b4f850

Remove unused BDADDR definitions
am: 525e743277

Change-Id: I13b1c9dd558bf8662f5f9af0008df0740ec333ae

Make RawAddress into a class (1/3)

* Add libbluetooth-types - library containing types implementation, that
is common between stystem/bt and packages/apps/Bluetooth. It must be
included in every project using btif interface.
* Put Raw Address implementation into libbluetooth-types
* Unify all "to/from string" helper methods into ToString and FromString
* bd_addr_empty -> RawAddress::kEmpty
* bd_addr_any -> RawAddress::kAny

Also fix leaks in jni str2addr by adding ReleaseStringUTFChars

Test: types_unittest
Change-Id: Ie0694843ad5fbd2a80b310c5f532e5e5a9548043
Merged-In: Ie0694843ad5fbd2a80b310c5f532e5e5a9548043

Remove unused BDADDR definitions

The code that used these moved to the HAL.

Test: build
Change-Id: I4a25b525ff60c060143ea913c800c4dea8f676e1

Merge "Remove bta_closure in favor of posting messages to a message loop"
am: 1c6f5e6a5c

Change-Id: I8a90de11fea85a01747d35e655e24d9c3f0ad41b

Merge "Remove bta_closure in favor of posting messages to a message loop"

Merge "Add missing lock protection when freeing RFCOMM port queues"
am: ea5319f3e0

Change-Id: Idee0119a6aae76df40a65d91618fe42130768f65

Merge "Add missing lock protection when freeing RFCOMM port queues"

Merge "Merge "A2DP: Add mandatory elements for SBC SRC capabilities" am: e0a0913672" into stage-aosp-master

Merge "A2DP: Add mandatory elements for SBC SRC capabilities"
am: e0a0913672

Change-Id: I28432a70beeb57771174a32edeb0629d97200bfc

Merge "A2DP: Add mandatory elements for SBC SRC capabilities"
am: e0a0913672

Change-Id: I018b14e6c7a1849da743ce19db053265e19af890

Merge "A2DP: Add mandatory elements for SBC SRC capabilities"

Merge "Don't initiate role switch for blacklisted devices"
am: 14eac9efa0

Change-Id: I87ffd1fcd4ff76c702eb895a528598c585aeaebb

Merge "Don't initiate role switch for blacklisted devices"

Add missing lock protection when freeing RFCOMM port queues

Bug: 63100223
Test: Code compilation
Change-Id: Ibbafb9586e9807688276c9eb2f6399cb3ee487c9

Merge "Generic Bluetooth HAL for standard Linux hci interface"
am: 6c34c8adc1

Change-Id: I0b2b523f8467b3f0cebbe96d997416baadf48cc7

Merge "Generic Bluetooth HAL for standard Linux hci interface"

A2DP: Add mandatory elements for SBC SRC capabilities

* The SBC items below are mandatory for SBC SRC capabilities
(see A2DP_SPEC_V13.pdf, Section 4.3.2):
 - Channel Mode - mono
 - Block Length - 4, 8, 12, 16
* Updated/fixed the corresponding unit tests

Bug: 63260735
Test: Manual - A2DP SBC streaming
Change-Id: I04c4f0b830411bc736d7c402b86006c182af02e7

Remove bta_closure in favor of posting messages to a message loop

Test: Sanity test with Bluetooth
Change-Id: Ifda27a3bd88d82f884c4d8af6383f1c692b69e85

Dynamic blacklist device for role switch
am: 942648ef5a

Change-Id: I95da63d1e6cc3952e53fcdb78760b86b27e6f4a9

Generic Bluetooth HAL for standard Linux hci interface

This acts as common interface for bluetooth devices (including
USB ones) using the standard Linux bluetooth hci interface.

Much of this code is pulled from hardware/interface/bluetooth
and system/bt/vendor_libs/linux.

Original code used from "system/bt/vendor_libs/linux" was from:
Samuel Ortiz <sameo@linux.intel.com>
Cristian Iorga <cristian.iorga@intel.com>
Pavlin Radoslavov <pavlin@google.com>
Jakub Pawlowski <jpawlowski@google.com>
Myles Watson <mylesgw@google.com>

Orginal code taken from hardware/interface/bluetooth was
contributed by:
Myles Watson <mylesgw@google.com>
Zach Johnson <zachoverflow@google.com>
Treehugger Robot <treehugger-gerrit@google.com>

Tested:
Hikey + on board bluetooth and Hikey + USB bluetooth dongle

Change-Id: I03da10cadfae6ffb298cfccc98e3337ab7db1967
Signed-off-by: Satish Patel <satish.patel@linaro.org>
[jstultz: Migrated code back to system/bt/vendor_libs/linux/.
 Included required sepolicy changes. Reworded and simplified
 commit message. ]
Signed-off-by: John Stultz <john.stultz@linaro.org>

Don't initiate role switch for blacklisted devices

Some devices always reject role switches and role switch
retries from the DUT may lead to A2DP glitches and LMP timeouts.

Test: manual
Bug: 33000157
Change-Id: I535a2c45ced0e14cf5530f78f3367569b306b1f4

Dynamic blacklist device for role switch

- Blacklist the device if it's rejected the role switch for
   max number of times. Same is added to the interop_database.

- Maximum number of failed (BTM_MAX_SW_ROLE_FAILED_ATTEMPTS)
  attempts set to 3 and it is configurable.

- Same blacklisted device is referred by the BTM module when
  any other module like profile is requesting for role switch
  and BTM module returns BTM_REPEATED_ATTEMPTS as the switch
  role status.

Test: Code compilation
Bug: 32999957
Change-Id: Ie999104a2c8ca507605ef3e1d454865cee98dcd2

Merge "Add stdint.h for uint*_t types"
am: ca6aea976d

Change-Id: I6bff96fd99d2091a34ade1101350fc798e2ba574

Merge "Add stdint.h for uint*_t types"

Read the correct amount of attributes
am: 972c885cc9

Change-Id: I01cb7b412650c7d0e97646539b3b86a5166f4d84

Merge changes from topic 'refactor_register'
am: 50e621f219

Change-Id: Ib9198bb482da3b67dcbeb1b517374643c14f4b45

gofmt -w build/fluoride.go
am: 2cf6e71b0a

Change-Id: Iaec2ee79f8930db7a75b8e8fc9c7434de0233a93

Read the correct amount of attributes

bta_gattc_cache_load currently attempts to read 0xFF attributes into an
allocation sized to num_attr attributes, which can be smaller than 0xFF.

There aren't more than num_attr bytes in correct data, but this breaks
with dynamic buffer overflow checking in CopperheadOS for the read
system call since fread ends up calling read, which obtains the size of
the allocation from the malloc implementation and then aborts due to the
(potential) overflow.

This would also fail with the default enabled _FORTIFY_SOURCE=2 feature
in the Android Open Source Project if osi_malloc was marked with the
alloc_size attribute. The way it wraps malloc loses that information so
fortify checks aren't done for calls like this.

Change-Id: I68bd170d5378c9d9d21cbda376083bc0b857e15c
Signed-off-by: Scott Bauer <sbauer@plzdonthack.me>
[migrated to C++ file, added 0xFFFF limit and wrote commit message]
Signed-off-by: Daniel Micay <danielmicay@gmail.com>

Merge changes from topic 'refactor_register'

* changes:
  Refactor module registration
  gofmt -w build/fluoride.go

Add stdint.h for uint*_t types

Mising stdint.h causes problem when building with BOARD_VNDK_VERSION,
where global include path is not provided.

Bug: 37342627
Test: BOARD_VNDK_VERSION=current m -j libbt-vendor
Change-Id: I55786baed229a3c9b4282816081a503d18a5a6f0

Allow full dev consolidation after pairing
am: 76eba3ad19

Change-Id: I4f118649f780a5302b69fdcfd0fc19793af730a4

Allow full dev consolidation after pairing

When BR/EDR and LE RPA device records are present
in the device records and the same remote device is
paired on a new RPA, the device consolidation happens
after the IRK is received. The current consolidation
breaks out of the loop after consolidating one device
record. In such cases, BR dev record is left and when
BR link key is derived, it gets updated in a separate
device record. Since two device records are present at
the same time for the same remote, the keys and static
addresses get picked up from incorrect node and issues
such as "remove device from resolving list seen with {0}
static address" are seen.

The fix includes allowing the host to complete
the consolidation for all device records rather than
breaking at the fist match.

Change-Id: Ic8ea4b831a7ba971eb6a0f54573934a6079b45d5

Bluetooth - Add tracing for TX queue and LDAC ABR
am: c02b5d4d68

Change-Id: Id2281ab270ee6e0a0d8416ce8d8cbff689838448

Bluetooth - Add tracing for TX queue and LDAC ABR

For debugging, add tracing to show the length of the TX queue and the
LDAC ABR level.

Test:  Ran bluetooth, took a systrace

Change-Id: Ia23764d68f27ae8209e37c4a0836852d73cf6cbc

Rename bt_bdaddr_t into RawAddress (3/3)
am: a484a88819

Change-Id: I43e5e71569281683c0a33000746e235391420714

Rename bt_bdaddr_t into RawAddress (3/3)

Test: compilation test
Change-Id: I4571721a0c6165a66450ee157a53d8d15bfc45d0

Merge "Get rid of BD_ADDR (again)"
am: abc4a993c8

Change-Id: Ic4da09c13391f7b13ce3797b42188c6b9ce1289a

Merge "Get rid of BD_ADDR (again)"

Set the A2DP media channel as a high priority L2CAP link
am: 624ba681a6

Change-Id: Ie43bd0c37d05bbe8ad3072a2918b4a500635bbfd

Get rid of BD_ADDR (again)

BD_ADDR and BD_ADDR_LEN were defined multiple times. Get rid of the
redefiniton.

Test: compilation test
Change-Id: Ib6e922ce88132045c13ddc28d681a0f673d87407

Set the A2DP media channel as a high priority L2CAP link

Use dynamic ACL buffer allocation for high priority links:
1. Ensure dedicated ACL buffers for high priority links
2. Adjust buffer allocation if the priority of the link changes

Test: Manually tested with 4 HID devices and A2DP streaming scenario
Bug: 30787081
Change-Id: Id3c4a76e360aee322e9f8073bd399807e7987990

LE State simplification
am: 5e43c8035c

Change-Id: I61a1cad20efa6eba7ba737458850c0b985eb3725

LE State simplification

Test: sl4a GattReadTest
Change-Id: I53d9b9a25f39b06e3593311ed9af5b1df10637ed

Merge "Fix a inappropriate return of local var address"
am: 7aefc01942

Change-Id: I6adc54b4d905322e865ea8e455f52c3a2012b944

Merge "Fix a inappropriate return of local var address"

HCI feature/command support check simplification
am: ec9f2f5ed0

Change-Id: I64cf6631447711660567cfdb8e65ee3f6572ebfd

Fix a inappropriate return of local var address

Local variable address is returned through giving to
"p_db->p_attrs", which needs to be available after
returned from the function.

Test: mm -j8
Change-Id: I68744d0c925e3ea384a07ad6491fe831d59ee2fd

HCI feature/command support check simplification

Don't use one-time long constants
Test: compilation test

Change-Id: Iaab8039269215b44608193290de4d140e1399e66

Refactor module registration

Refactor module registration so that it doesn't need to separately
track properties.

Test: builds, no change to out/soong/build.ninja
Change-Id: I4215b710a37f53ea1d5664bb0f7abbd42d25a3e9

gofmt -w build/fluoride.go

Test: builds
Change-Id: I158eff9274c97eb59e4e3a63a39d0d5df501cee4

Don't send the LE Set Privacy Mode command if it's not supported
am: ba78601034

Change-Id: I18bcdfd58af00a6557871bee702b1ce8ad3bbe0e

Don't send the LE Set Privacy Mode command if it's not supported

LE Set Privacy Mode is currently guarded by LE Privacy feature, which is
enabled on some 4.x controllers, that don't support LE Set Privcay Mode
that was added in 5.0 spec.

Bug: 62809491
Test: Turn Bluetooth on/off with Angler (4.x) and check for
      Set Privacy Mode
Change-Id: Ie9a164d98558db2e1fe76d90eb6ab93e4cec89d0

btif: Pass UIDs changed up to the Java layer
am: 57e508f56b

Change-Id: I0db73b5801afe6b72a438c0728273775cb14684b

bluetoothtbd common library reconfiguration
am: 5f88533feb

Change-Id: I98a21c91355737f09c348f90098640870b507699

btif: Pass UIDs changed up to the Java layer

Previously we were simply responding with an INTERIM response and
leaving it at that.

Test: connect to BMW kit and check that interim is sent from Java
Bug: 62775732
Change-Id: Idb7de4417438f0c9697ee5c838b511cdd2223e80

bluetoothtbd common library reconfiguration

This patch change how code in service/common directory is compiled. Up
till now, .cc files were generated from .aidl, and the .cc files were
build for each target that used them. From now on, static lib is build
from .aidl and .cc files, and it is included into all targets that
require them.

This reduces build steps from around 1240 to 849.

Test: compilation test
Change-Id: Iebaf0c75497bc435dcc013be09a1f82e75547aa3

btif: Pass UIDs changed up to the Java layer
am: 39f664e629

Change-Id: I7d53647e60644bd52cdc53ccaf22e8e726a6e4c5

Fix crashes on stack shutdown am: 6884c4c68b
am: 1a1e0fb3c7

Change-Id: I6f50e1ba5c8cf1cbe8389931a8892e61771e0fb3

Fix crashes on stack shutdown
am: 6884c4c68b

Change-Id: Id8a02f8359f6141bca21f17cf09e6a94bad87c72

Fix crashes on stack shutdown

During stack shutdown, Unregister would sometimes get called after the
stack is already cleaned up, and the AdvertiseManager::Get would crash
on null check assertion. Ensure that stack is initialized before
Unregister call.

Bug: 37888261
Test: none
Change-Id: I7a4bbcd24ec7d24d21d04fe842985baaac899321

btif: Pass UIDs changed up to the Java layer

Previously we were simply responding with an INTERIM response and
leaving it at that.

Test: connect to BMW kit and check that interim is sent from Java
Bug: 62775732
Change-Id: Idb7de4417438f0c9697ee5c838b511cdd2223e80
(cherry picked from commit 3d7c99dc6db3f2a0392a0b273140eb03e0050ee5)

Fix BLE remote address resolution
am: 5235a8c4cc  -s ours

Change-Id: Iaa3a2d1fc9a4e9d10f72c681c1cfef2ab66a3659

Fix BLE remote address resolution

The hash segment of the resolvable device address needs to be extracted
for comparison.

Bug: 62589903
Test: manual
(cherry picked from commit 7c58ca38433bc923c653c334b712fb05d25cab36)

Merged-In: I979a8ad3074b692503b0c882bd90041ba4fbfb4d
Change-Id: I5db8f266c9ee3022b2a8a882540870e5eecd0736

Fix BLE remote address resolution am: 7c58ca3843
am: ca1e1b224f

Change-Id: I13b56d630b446658df12498dc89dd4d545258e0c

Fix BLE remote address resolution
am: 7c58ca3843

Change-Id: I55b2929fb6474df3341e3489d44d93e84988c230

Fix BLE remote address resolution

The hash segment of the resolvable device address needs to be extracted
for comparison.

Bug: 62589903
Test: manual
Change-Id: I979a8ad3074b692503b0c882bd90041ba4fbfb4d

Use references in GATT client/server API (2/3) am: 96ac0a3c16
am: 2371ed343e

Change-Id: Ic507c8da15a19639a540242feb409f040d09d746

Use references in GATT client/server API (2/3)
am: 96ac0a3c16

Change-Id: Ib63537b210587e55d37e10f3f7d87a07c7dcb6d3

Use references in GATT client/server API (2/3)

This allows to get rid of some unnecessary null checks, and guarantee
proper values are passed.

Test: compilation test
Change-Id: I1b790dba0cfc3ab02984c3911d83a6f413a1e7e6

Fix peer address type in LE extended create connection command
am: 77ecddc3a4

Change-Id: Ie894931c63cf8cba238c7d5c4bea477ba8240c69

Fix peer address type in LE extended create connection command am: 015cbf1d2b
am: 34253bdd97

Change-Id: I15bfdd2fbd031872e99608a4575bbe5f4905b765

Fix peer address type in LE extended create connection command
am: 015cbf1d2b

Change-Id: I2c52745bbcecc613e3ef51d70e0b7cd4bb29aa22

Fix peer address type in LE extended create connection command

As per BT Spec 5.0, peer address type in LE extended create
connection command can have only either 0x00 or 0x01 as valid
values. This change fixes setting of incorrect peer address
type values in extended create connection command.

Bug: 62578331
Bug: 62358300
Change-Id: I792d48be7637fc571727e7e51de1fc397158b4d5

Fix peer address type in LE extended create connection command

As per BT Spec 5.0, peer address type in LE extended create
connection command can have only either 0x00 or 0x01 as valid
values. This change fixes setting of incorrect peer address
type values in extended create connection command.

Bug: 62578331
Bug: 62358300
Change-Id: I792d48be7637fc571727e7e51de1fc397158b4d5

Merge "Make GATT client and server assumption about addres type in open identical" am: ff32a739e1
am: 6b60d1cf82

Change-Id: I13fbb165601a169bb41a228068b4944a8d642db6

Unify the empty and any address usage am: b7b7473692
am: 6de9c97dea

Change-Id: I4d0391a32473ca17c61d2f912832b6740cec13d2

Merge "Make GATT client and server assumption about addres type in open identical"
am: ff32a739e1

Change-Id: Id6b33c411fef62eb49df6516cbf2184c492ba387

Unify the empty and any address usage
am: b7b7473692

Change-Id: Ic53589c69ae4ae25bdfd8c8ea7e8bd8c4a4214b1

Merge "Make GATT client and server assumption about addres type in open identical"

Add a mechanism to avoid using AVDTP RECONFIGURE for blacklisted devices
am: b5c932fe13  -s ours

Change-Id: I21e5f33e805babdae7d5e8e96bebef00a7a8d121

Make GATT client and server assumption about addres type in open identical

When attempting to open GATT server to device of unknown type, it should
make same assumption about the type as GATT client, which is LE device.

Test: compilation test
Bug: 62827460
Change-Id: Ic96de97445a843ef3c482fd5b54098b325fb0f9c

Add a mechanism to avoid using AVDTP RECONFIGURE for blacklisted devices

Some carkits report SUCCESS for AVDTP RECONFIGURE commands when
changing A2DP codec configuration. However, there is no audio coming
from the carkit.

Bug: 37625892
Test: Manual - A2DP Codec reconfiguration while streaming audio
Change-Id: I01f37a2514e490986a20e96ace78c92463403396
Merged-In: I01f37a2514e490986a20e96ace78c92463403396
(cherry picked from commit e91297a4d1a77ae0c367fbe756090b7357e9893a)

Disable 48.0kHz sampling rate for AAC - update the unit test
am: 78120a1a24

Change-Id: I1960ac4878a46a6e13669070f480a4b4d5243b4e

Unify the empty and any address usage

Test: compilation test
Change-Id: Ic7fae4aa6c6c32e08fbca257dd3629f4f8df92f0

Merge "Disable 48.0kHz sampling rate for AAC - update the unit test" am: 8fdac3b3e1
am: 44e67085eb

Change-Id: I330b2daa48ecbec24e12286d90b7c381d060006f

Merge "Disable 48.0kHz sampling rate for AAC - update the unit test"
am: 8fdac3b3e1

Change-Id: I4bd614d933bd4d14fd45ac878557298104876ee9

Merge "Disable 48.0kHz sampling rate for AAC - update the unit test"

Get rid of BD_ADDR am: b8a477e9ac
am: 61f2657d9c

Change-Id: I473f09be8420c9c509717cffb07b4d71efcf5cc3

Get rid of bdcpy, bdcmp and other BD_ADDR helpers am: 78b81c6db0
am: 2c4b02135e

Change-Id: I6a9b2624683ea603991926ce49fbe1f7aef0bbdc

Replace BT_ADDR with bd_addr_t in bta_sys related code am: c752e135d6
am: 44c207025f

Change-Id: Ifa9427166d13801617a04557723fb5efa286292a

Replace BD_ADDR with bt_addr_t in MCE related code am: 98b1b9498d
am: c033d79b42

Change-Id: I0917f64d8e2d3107fdb8de43472ab46c7afb077e

Build fix for AOSP am: 04801862d0
am: e84c1ce0e9

Change-Id: I292db699b34ff521059062d809ead53c56f0ac3d

Replace BD_ADDR with bt_bdaddr_t in RFCOMM related code am: 1979fa31d2
am: fe80f3379e

Change-Id: I6deb3a93035d026666f86822ec2e677e4bbb7d7a

Replace BD_ADDR with bt_bdaddr_t in SDP and A2DP related code am: 135b7f68e4
am: 2fded74ead

Change-Id: I1ffd4ac27af3f73ad5606eb1f32617b256c4851a