Fix misc-macro-parentheses warnings in system/bt. am: fc25b19606
am: 435140c19f

Change-Id: I0161b80115cbeb6a5754e54c2c3c7cc89b0ca821

Fix misc-macro-parentheses warnings in system/bt.
am: fc25b19606

Change-Id: I7ef5175514d7f1483743c20c916854f1e4cce687

Fix misc-macro-parentheses warnings in system/bt.

* Use NOLINT to suppress missing parentheses warnings around bitmask.

Bug: 28705665
Test: make with WITH_TIDY=1 WITH_TIDY_CHECKS=-*,misc-macro-* \
      WITH_TIDY_FLAGS=-header-filter=system/bt/.*

Change-Id: I4f095898d49eafdea9eb72f9ffc9eac87f68a4c8

SDP: Bounds check 'id' parameter for free_sdp_slot()
am: b413f1b136

Change-Id: I731b84859091c5580b358678830b59914dad5f42

Fix errors in handling RawAddresses am: 882aec320e
am: 1a6a640a36

Change-Id: I52bc8277153bc439f652311c2eff00f34f214480

Fix errors in handling RawAddresses
am: 882aec320e

Change-Id: If23c7e9bf4231ec3398d0de21c0656a240935209

Fix errors in handling RawAddresses

* In change I8d1bd6914aec55bb53495b1d0d5e3d37b86865e6
  memcmp(a, b, LEN) != 0 should be translated to
  A != B
* memcpy should not be applied to RawAddress objects.
  Assignment operator should be used instead.
* memset should not be applied to RawAddress objects.
  Assignment to RawAddress::kEmpty should be used.
* Fixed a crash in GATT

Bug: 64316340
Test: Unit test, pair with device and transmit
Change-Id: Iceefab821c1d45a88194d87a43a192afa5f263fd

Merge "test_vendor: Add function to extract bytes from iterators" am: 9a7f50c041
am: 1dcc643f74

Change-Id: I5d1f05416df73f3d3f6432a6e1f506fc544a238f

Merge "test_vendor: Add function to extract bytes from iterators"
am: 9a7f50c041

Change-Id: I588d3a7fd6dce5c2eaf1aea9944e4d45663e3dd0

Merge "test_vendor: Add function to extract bytes from iterators"

test_vendor: Add function to extract bytes from iterators

The extract function extracts a number of bytes from the type that the
iterator is pointing to. The iterator will be incremented to the value
after the last byte after extract is called.

Fixed the bounds of the for loop in the preIncrementTest so that it
does not try to dereference the sentinel after the last value.

Test: Passes unit tests in test/iterator_test.cc
Change-Id: I07944641b829c820bbe40d612d7f065e514516b3

osi: Suppress length warning in config parsing am: 8b7968085b
am: 1b63c106d4

Change-Id: I6d38fee565d635ba2a9ca89ac3b078b654e3d182

osi: Suppress length warning in config parsing
am: 8b7968085b

Change-Id: I338ec794dcc5db22ee4615ef139500c44eb1cf2b

osi: Suppress length warning in config parsing

Bug: 38178471
Test: Built without seeing the warning at the line
Change-Id: I1f8a5fc58313456921891700a3039cad1715b4d0

resolve merge conflicts of 0a5d9ebb0b15976b47eaff77bb66a257b8b114bb to stage-aosp-master
am: ca61556a76

Change-Id: I7d93c7bfd2fdd3bb0b80e0ef5400dd2432e035ac

resolve merge conflicts of 0a5d9ebb0b15976b47eaff77bb66a257b8b114bb to stage-aosp-master

Test: this fixes merge conflict that I skipped
Change-Id: Ie829182b01d01e5ae648165f30cfad7647a97db6
Merged-In: Ie829182b01d01e5ae648165f30cfad7647a97db5

Bluetooth: move AIDL files related to Bluetooth into system/bt (2/3) am: 0a5d9ebb0b
am: 0cc11eea12

Change-Id: Ia96a68b8c9fe6d070a917a7749ade226e7994d9d

Bluetooth: move AIDL files related to Bluetooth into system/bt (2/3)
am: 0a5d9ebb0b

Change-Id: I6a921b05aa32dbb9bf196635c17118de5d39d2b0

SDP: Bounds check 'id' parameter for free_sdp_slot()

Test: manual
Fixes: 37502513
Change-Id: I34e8296ec7ec6b4ffbe1fa0452754f2a421e6ec7

Bluetooth: move AIDL files related to Bluetooth into system/bt (2/3)

This patch moves *.aidl files from
frameworks/base/core/java/android/bluetooth into system/bt/binder. This
is in preparation to convert the Bluetooth deamon into native
implementation piece by piece.
In order to do that, one must have C++ header files, and paths to them
with AIDL files, and */java/* folder didn't seem as proper place for
that. Additionally, keeping AIDL files out of framework/base will not
require creating dependency on this huge project, which should help
keeping the compilation fast.

Test: compilation test
Change-Id: I4c992e6ee16eea3173a49b1d37e961af3d3c9ac6
Merged-In: I4c992e6ee16eea3173a49b1d37e961af3d3c9ac6
(cherry picked from commit 8a12de49177a341a118c95435177fba4f12fe412)

Merge "Call bta_dm_sp_cback if BTM_LOCAL_IO_CAPS == BTM_IO_CAP_NONE" am: 9d098408ce
am: 4a09400364

Change-Id: I45dd43d6311339a67da18007176bd8c620c19487

Merge "Call bta_dm_sp_cback if BTM_LOCAL_IO_CAPS == BTM_IO_CAP_NONE"
am: 9d098408ce

Change-Id: I0988507f1ed8bd3a9138ecb80171d7cfd18a4031

Merge "Call bta_dm_sp_cback if BTM_LOCAL_IO_CAPS == BTM_IO_CAP_NONE"

Use a mutex to protect the scheduling of tBTA_PM_TIMER am: 9e702a6aef
am: a6ba2c3c0a

Change-Id: Ifc72953b703d4203b5271a922a64802a9d27e255

Use a mutex to protect the scheduling of tBTA_PM_TIMER
am: 9e702a6aef

Change-Id: I4f41eee8b9e5dd129b3aeffb287a6e8b9c7269c4

Use a mutex to protect the scheduling of tBTA_PM_TIMER

Bug: 63689331
Test: Code compilation
Change-Id: If062f7b3e2c17cb5a1ac2c8ef0813a2fe5c5a5f5

Merge "test_vendor: Add Iterator tests" am: c427c0bd30
am: ef20e0dbb8

Change-Id: I17f91bc17978fc21f68d464c5b7c6df979ecbccb

Merge "test_vendor: Add Iterator tests"
am: c427c0bd30

Change-Id: I81d51453b88f707463b140861dfd0b6e0dcefb50

Merge "test_vendor: Add Iterator tests"

test_vendor: Use Custom Iterators in L2CAP Classes am: 6f1401153f
am: 590df2b3f1

Change-Id: I1949b8181ea1c3c5b4bebbb111a316f2e2e9b76d

test_vendor: Use Custom Iterators in L2CAP Classes
am: 6f1401153f

Change-Id: Ie5fcdbe89d35ab391fd756f675b264d1dd2233ed

test_vendor: Use Custom Iterators in L2CAP Classes

Use custom bounded iterators instead of vector iterators in L2capPacket and
L2capSdu.

Test: Passes unit tests in test/l2cap_test.cc
Change-Id: I892083ec6155b18c4b222f3c38d17d2d9fef686c

test_vendor: Add Iterator tests

Add Iterator tests to check that the overloaded operators and bounded iterators
are implemented correctly.

Test: Run the unit tests:
out/host/linux-x86/nativetest64/test-vendor_test_host/\
test-vendor_test_host --gtest_filter=IteratorTest.*
Change-Id: I6402cc36fe6818fa972524ac20aed526128a8553

Merge "test_vendor: Add HciPacket and Iterator classes" am: 3e1a3d0ddc
am: ff5beab4fd

Change-Id: Ie44e518117764d92cb67d23a0b3d1a059e708540

Protect the btif config state with a lock during cleanup am: a4febd6d04
am: 407754ba3a

Change-Id: I8cb723ca43c3917faa690c191078e0575aa21625

Merge "test_vendor: Add HciPacket and Iterator classes"
am: 3e1a3d0ddc

Change-Id: Ide1490901c49ae9e38e69ba6315c69df92c7bb27

Protect the btif config state with a lock during cleanup
am: a4febd6d04

Change-Id: I47a3b5efc680954afc11e58f16c440284ba016a9

Merge "test_vendor: Add HciPacket and Iterator classes"

Protect the btif config state with a lock during cleanup

Bug: 64186300
Test: Code compilation, enable/disable Bluetooth
Change-Id: Ic2397a128d59c9cc69cbd8252734b69eb7c02501

Advertisement parsing fix for zero padding am: 35a9ea352d
am: b3bf1285f4

Change-Id: Ica2250b89a1f72bf15bf16d1eaf159fe71049c44

Advertisement parsing fix for zero padding
am: 35a9ea352d

Change-Id: I59bf8524b2c95c5f8c7f4261166057078dd25b7c

Advertisement parsing fix for zero padding

When AD data is zero padded, and Scan Response is appended at end, data
becomes invalid. Instead, zero paddning must be removed first.

Test: AdvertiseDataParserTest.RemoveTrailingZeros
Bug: 38489707
Change-Id: I229ca3db6c92bc06bc1429e72412417010721063

Added another entry to the INTEROP_2MBPS_LINK_ONLY interoperability database am: 427da91563
am: b7471f0173

Change-Id: I953492a7be219d4b15f9b7ae581af3bceb5f47ef

Added another entry to the INTEROP_2MBPS_LINK_ONLY interoperability database
am: 427da91563

Change-Id: I7412713b0c0104309b8e5fcffa7df2fde1eaa7f2

Added another entry to the INTEROP_2MBPS_LINK_ONLY interoperability database

Bug: 64357126
Test: Audio streaming to the affected carkit
Change-Id: I160eb3775c77fe400798f038d2aae7dd7825aacb
(cherry picked from commit a5f0a7717d129ca8164a0712e63f41cc4b60dd72)

test_vendor: Add HciPacket and Iterator classes

HciPacket is the abstract base class for all packet types.

Iterator is the custom iterator class that all packet types will use
to iterate through. The iterators in this class are bounded from
[start_of_packet, length_of_packet]. This prevents incrementing the iterator
passed the end sentinel.

Test: Passes all of the tests in test/iterator_test.cc
Change-Id: If9fec9ae32d9a23575ad3e9219b8249d164bc0d5

Don't reuse AVDTP internal state for different purposes am: 8da87bea5e
am: 42544cb250

Change-Id: I5245bb4d02d11efbdd1bc06fbaf0ef865a9a9a94

Add A2DP_DumpCodecInfo() API am: 219b85f136
am: ae550b31c2

Change-Id: I711e1ceac6c0c44d34f9e5e8b0d2508a8e6af6f5

Don't reuse AVDTP internal state for different purposes
am: 8da87bea5e

Change-Id: I1b35a8e6ba0f3d109731a1128c4ccb6363a9404c

Add A2DP_DumpCodecInfo() API
am: 219b85f136

Change-Id: I1bc85fcf214ab75e130954245d585e3015b08bec

Don't reuse AVDTP internal state for different purposes

A number of issues were exposed by carkits that proactively try
to discover the codecs supported by the Phone and explicitly select
the A2DP codec.

* Don't reuse p_scb->p_cap for storing the most recent fetched capability
  and for storing the result configuration while reconfiguring the stream.
  Instead, use p_scb->cfg for storing the result configuration.
* Don't try to reconfigure the audio stream before all peer's capabilities
  have been retrieved.
* Don't reset p_scb->sep_info_idx after fetching all capabilities inside
  bta_av_save_caps(), otherwise the stream reconfiguration logic is
  confused. Apparently, p_scb->sep_info_idx is overused for both:
  (a) Pointing where to store the next capability while fetching the
      capabilities.
  (b) Storing the selected the SEP after the codec selection is completed.
  Resetting p_scb->sep_info_idx to 0 creates logical confusion when the
  actual selected SEP index is 0.
* Add extra debug log messages

Test: Connecting to and streaming with various carkits
Bug: 64128712
Change-Id: I5f57e9069d5402bcbb17613470ac0d29c8e45069
(cherry picked from commit c64dc1d28496b5e917d0f82317d40d3c60e6495d)

Add A2DP_DumpCodecInfo() API

The API can be used to display A2DP codec info when using |LOG_DEBUG|.

Test: Code compilation
Bug: 64128712
Change-Id: I04379b45f24adb7008365d38ccc8aa7f49ed4b73
(cherry picked from commit 81c2234305d4c84eb38f0d5a92605e3b42366f84)

Workaround for Traxxas am: 02d1ec171f
am: 9f4e534074

Change-Id: I34c64ec33c5cb09d839416f81110b3594e125bb4

Workaround for Traxxas
am: 02d1ec171f

Change-Id: If547b412aabc0cf86194116c9a0641e700209336

Workaround for Traxxas

Bug: 64252588
Test: net_test_stack_ad_parser
Change-Id: I1471e9e5f0f36ec931f8d86c927abbc1137e2b94

Merge "AVRCP Controller request metadata on playback." am: 39b6c45264
am: 9cc7daf831

Change-Id: Ie860ca5d4c8b2330f77279ec0eba84496da48f54

Merge "AVRCP Controller request metadata on playback."
am: 39b6c45264

Change-Id: Ia5000440315c0f8a1bc75142e4463606e9fc931c

Merge "AVRCP Controller request metadata on playback."

Merge "AE: Set secondary adv max skip to zero" am: b92d9f3530
am: 149053fe3d

Change-Id: Ie7f516abf4a778b4e288ef5ab069b9a64fa0ca50

Merge "AE: Set secondary adv max skip to zero"
am: b92d9f3530

Change-Id: I54710829a6ba81d69bbee4571c00492d4a612e32

Merge "AE: Set secondary adv max skip to zero"

AVRCP Controller request metadata on playback.

Track metadata is only provided when requested, and the current
implementation only requested it when there was a track change
notification.  Add a request for metadata whenever there is an
update to indicate that the play status is playing.

Bug: 63998350
Test: Start playing media, make a phone call, end a phonecall observe
metadata is correct.

Change-Id: Iea5cdacfe95c1a98cdaf73836ee6359a923f481b

test_vendor: Rename discoverable devices. am: 974b1f9a91
am: 6e928fcdd7

Change-Id: I483622c7d27b5acb12d8040199f96496cdad998f

test_vendor: Rename discoverable devices.
am: 974b1f9a91

Change-Id: I2252dd34fa376c4aa3e6da49b0f5f75f59d05837

test_vendor: Rename discoverable devices.

Test: compiles
Change-Id: I1fc1f6845995e9e816556679622db6f6d447b7b3

AE: Set secondary adv max skip to zero

Setting secondary adv max skip to zero to guarantee
when the AUX Tx happens. This way the App can use the
primary advertising interval to dictate the AUX Tx
interval.

Bug: 62552121
Change-Id: Id66222e4f8a583c42acafb6532f81a817504e606

Merge "Fix processing of Inquiry results data" am: 67da9a5e19
am: 6718f5d924

Change-Id: I78efe07301f3f6747dcdae95c5e5bab6cc4801f2

Merge "Fix processing of Inquiry results data"
am: 67da9a5e19

Change-Id: If6b7523e9096c114263baf7a065f023186818c82

Merge "Fix processing of Inquiry results data"

Use the correct AVDTP version-specific features am: 9cd9ef4c38
am: 3a7e4f8a30

Change-Id: If55547328805868f3cbb84af5b96f85f6dde791c

Use the correct AVDTP version-specific features
am: 9cd9ef4c38

Change-Id: I85e816084a49a0a3438c8cc88fe2ab0ef32097f2

DO NOT MERGE ANYWHERE Allow the Bluetooth MAC address to be updated asynchronously (3/3) am: fead72d124  -s ours am: 672ca0faf5  -s ours
am: 0e379aaef1  -s ours

Change-Id: I1fdf1d2c32c4ef79d7e56d2a3b34f8ecc6687bfb

DO NOT MERGE ANYWHERE Allow the Bluetooth MAC address to be updated asynchronously (3/3) am: fead72d124  -s ours
am: 672ca0faf5  -s ours

Change-Id: Icf9b2a90b0adb3d9645bd6839ed9c244ba8c8c26

DO NOT MERGE ANYWHERE Allow the Bluetooth MAC address to be updated asynchronously (3/3)
am: fead72d124  -s ours

Change-Id: I96daa5c94b8ac75e2e7faf00911274ba72dff51b

Use the correct AVDTP version-specific features

Failure:
A2DP is not reconnected after power cycling DUT BT.

Root cause:
AVDTP v1.3 feature request (AVDT_GetAllCapReq) sent by DUT
as remote supports the same, but this 1.3 feature is not
completely supported by DUT which leads to erroneous behavior.

Fix:
Use AVDT_GetAllCapReq request only when both host and remote
AVDTP version are 1.3 and above.

Test: 1.Pair to CarKit; 2.Connect to CarKit; 3.Do BT power cycle @ DUT
Bug: 35657623
Change-Id: I66c2d7d8fe8506b74282bcca93595e1cbf5c2565

DO NOT MERGE ANYWHERE Allow the Bluetooth MAC address to be updated asynchronously (3/3)

There are intermittent issues where either the returned Bluetooth
MAC address to Java framework is uninitialized or this address update
arrives too late. This fix will do 2 things:
(1) Returns error when MAC address is unavailable in the native code.
(2) Updates the MAC address later by adding a new broadcast event.

Test: Check address for these cases: factory reset, system reboot, and
Bluetooth re-enable.
Bug: 36709382

(cherry picked from commit 7dc0525365025e17e289869288b1b7d5146306cc)
Merged-In: I7b1c526ee227897c003ac921078f317f96b92604

Change-Id: I9810484a136c69d487a4b415c9265913db6c5cce

Fix processing of Inquiry results data

Process inquiry results data based on the data
length sent from the lower stack layers and not
by using HCI_EXT_INQ_RESPONSE_LEN.

Bug: 62589724
Change-Id: Iee4a2c8f0dfb459404b28ba21061f7ced9f8455c

Fix the MAC address for INTEROP_2MBPS_LINK_ONLY entry am: b60955f9c7
am: 7093baab90

Change-Id: I0e90bc82991f31c520be480c3f14a61901fa2827

Fix the MAC address for INTEROP_2MBPS_LINK_ONLY entry
am: b60955f9c7

Change-Id: I06a99e795b0ddfd6978fca2ccca09f20de3d299f

Fix the MAC address for INTEROP_2MBPS_LINK_ONLY entry

Bug: 62394764
Test: Code compilation
Change-Id: I3e6d8bb8258d6cc56ce05d7f17dda445859f011a

Add interoperability mechanism to restrict the AVDTP MTU for AAC codec am: b866da0bc1
am: ec7b336583

Change-Id: Id94f4643bec9e052d51e54950bab9ef8b062a13a

Add interoperability mechanism to restrict the AVDTP MTU for AAC codec
am: b866da0bc1

Change-Id: I79d73807f18ca5f21050f72b16c0e6f6f972f0fb

Add interoperability mechanism to restrict the AVDTP MTU for AAC codec

Also, add an interoperability entry for devices that are known
to have AAC-related issues.

Test: A2DP streaming to Headsets
Bug: 63725151
Change-Id: Ic9b6640da44ebc28bfa63a19a87f0543d69bc576

Retry authentication in key-missing or pairing collision use case am: a95a6a3608
am: 27457a8b85

Change-Id: I343dbedf4a3aefe61c4e37fe8d0e2f0f26b02a49

Retry authentication in key-missing or pairing collision use case
am: a95a6a3608

Change-Id: Iad1f7b2b5cc32fe6bb7e7237a4652ba7b2ed4e78

Retry authentication in key-missing or pairing collision use case

Problem:
A2DP connection failing with connection timeout

Steps:
1. Pair and connect car kit from DUT
2. Disconnect the car kit from DUT
3  Unpair DUT from car kit menu and initiate pairing from car kit.
   Pairing and A2DP connection should be successful.

Failure:
Paring and A2DP connections fail

Root Cause:
As the stack posting authentication failure to upper layers
when SOC gives key missing error code remote removed from
DUT paired list, but same time lower layers keep retrying
for missing key and leading to connection failure.

Fix:
Don't post authentication complete to upper layers when stack
re trying for security in key missing or transaction collision cases

Test: manual
Bug: 35448920
Change-Id: I970c8679bf27801fe46f8dd757d6435ed500f77f
(cherry picked from commit 35752e32fb5d859c012f6d06f2c22fa6b1f84174)

Merge "Revert "bt: Update libchrome APIS to r456626."" am: 6241dd5522
am: afc7f05f72

Change-Id: I2f8ceb0afa199d5faee032b4b6f03206589d5421

Merge "Revert "bt: Update libchrome APIS to r456626.""
am: 6241dd5522

Change-Id: I2ab440ec5601a01717428b0ad8faf135437af971

Merge "Use a mutex to protect the consistency of the tBTA_PM_TIMER state" am: 02ce3f8f52
am: 6baf3931ae

Change-Id: I2d2f4cb5b997af3dcf18eb6d1792163ff0aa90d5

Merge "Revert "bt: Update libchrome APIS to r456626.""

Merge "Use a mutex to protect the consistency of the tBTA_PM_TIMER state"
am: 02ce3f8f52

Change-Id: I61dbfa463046af52f30385bb2593709b0f8e7730

Merge "Use a mutex to protect the consistency of the tBTA_PM_TIMER state"

Merge "HFP Client connect Audio" am: d568679105
am: d9401dfa06

Change-Id: I985b2d6e8d09e35bde57eb75f54024f8e51d438c

Merge "HFP Client connect Audio"
am: d568679105

Change-Id: Ie8b2a8b3b0d54d10fb99017d6f34649004033a5c

Merge "HFP Client connect Audio"

Merge "Remove unwanted space characters from dialing number" am: ac3faef9da
am: 86470eebc8

Change-Id: I46b2bc514c28a59e59e2ff2542c99a18bd8e71b5

Merge "Remove unwanted space characters from dialing number"
am: ac3faef9da

Change-Id: I1209591467468e7b5b5cc77b04c759b02b3b7b67

Merge "Remove unwanted space characters from dialing number"

Revert "bt: Update libchrome APIS to r456626."

This reverts commit 014890b74878022db7d23be47db179ba06e01856.

Reason for revert: Broke the mac_sdk
Exempt-From-Owner-Approval: Fixing mac_sdk

Change-Id: I11652839faf8430a7ee6ad64af0a7bd6985478e8

Use a mutex to protect the consistency of the tBTA_PM_TIMER state

Bug: 63689331
Test: Code compilation
Change-Id: Id8ab78473e1f766d5a76b967024b7ff4af499667