Read the correct amount of attributes am: 636b754cfa am: 5ec93991d7 am: 6e7c360c85
am: a51aaf2366

Change-Id: Ie8dfa4cff72aef91a87b4dbbc5b82f88213994e2

Read the correct amount of attributes am: 8eb6493ad5  -s ours
am: 95b2e35d5d

Change-Id: I0956d72a77cfd13dd6dca526cad54c402991ab44

Read the correct amount of attributes am: 636b754cfa am: 5ec93991d7
am: 6e7c360c85

Change-Id: Icb1240a22b2cfc7fa32b3ab23652fa1510b3c590

Read the correct amount of attributes
am: 8eb6493ad5  -s ours

Change-Id: I73a8ff63af23863be10d8cdd4da73234fcd25e3d

Merge "Read the correct amount of attributes" into oc-dev
am: 83aee7b3ba

Change-Id: I633e4d7fbcb48387ca092e5e19a6f5c0ba015072

Merge "Read the correct amount of attributes" into oc-dev

Read the correct amount of attributes am: 636b754cfa
am: 5ec93991d7

Change-Id: I69bd3d4e294765feade3840d9f1710cfa1cdc69b

Read the correct amount of attributes
am: 636b754cfa

Change-Id: I43948bbb53b3075e7e7b016b9127f38245a8f9da

Fixed the implementation of std::equal_to<bt_bdaddr_t>
am: 181144a501

Change-Id: Ic8e35d75ebe14eccd7fd169572fe5bff39d150f6

Fixed the implementation of std::equal_to<bt_bdaddr_t>

Now we correctly suppress the processing of repetitive
BLE scan results.

Test: Manual with extra log messages
Bug: 65174158
Bug: 64975965
Change-Id: Ic4386c5e2fbabdc218f6ec675c5b298a7af41e5a
Merged-In: Ie0694843ad5fbd2a80b310c5f532e5e5a9548043

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.

Bug: 37160362
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>

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.

Bug: 37160362
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>

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.

Bug: 37160362
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>

Fix build error for AVRCP13 compat fix
am: 544f7e1be9

Change-Id: I4acd192aea9c8c72d3ad1ab6c7b3f3ccf9bb868f

Change our AVRCP capabilities if the remote device only supports 1.3
am: c8ab11a06a

Change-Id: I0aee338bc6c387ea6060637c74d89fa0745c40e5

Fix build error for AVRCP13 compat fix

Bug: 37943083
Test: manual
Change-Id: I3974192dd007804f90f35a724a39824ed093a633
Merged-In: I6d041590dc51d7e8711b17fb1cb9c880b640052a

Change our AVRCP capabilities if the remote device only supports 1.3

This prevents issues with devices that only support 1.3 but can not
handle forward compatability like some Alpine Carkits.

Bug: 37943083
Test: Connect to Alpine carkit that only supports 1.3 and see new features
      are used.

Change-Id: I6d041590dc51d7e8711b17fb1cb9c880b640052a
Merged-In: I6d041590dc51d7e8711b17fb1cb9c880b640052a
(cherry picked from commit f9f1c8b449efb9cb7894a64fe2977f875679ba52)
(cherry picked from commit ff2d86ed3af91099597d033a76a143437543d6fd)

GAP: Set service_id before calling gap_release_ccb
am: 4c6208fd2d

Change-Id: I705d78e301414308a7150a20576f61a9e95f35c3

GAP: Set service_id before calling gap_release_ccb

Calling gap_release_ccb with a service_id == 0, which in turn
calls BTM_SecClrService with an ID of 0.

From the documentation for BTM_SecClrService:

  Service ID - Id of the service to remove. '0' removes all
               service records (except SDP).

Bug: 65695769, 65223508
Test: BLE connection and characteristic read
Change-Id: Icf309807f02e1faa273cf9bad9c09d9221a8bbfd
(cherry picked from commit 5c5c10683e2a11162838297fc8054b15837c4f32)
(cherry picked from commit 4023c6731d5e991c6202d4e499b9dd43d642fb3c)

Fix build error for AVRCP13 compat fix

Bug: 37943083
Test: manual
Change-Id: I3974192dd007804f90f35a724a39824ed093a633
Merged-In: I6d041590dc51d7e8711b17fb1cb9c880b640052a

Change our AVRCP capabilities if the remote device only supports 1.3

This prevents issues with devices that only support 1.3 but can not
handle forward compatability like some Alpine Carkits.

Bug: 37943083
Test: Connect to Alpine carkit that only supports 1.3 and see new features
      are used.

Change-Id: I6d041590dc51d7e8711b17fb1cb9c880b640052a
Merged-In: I6d041590dc51d7e8711b17fb1cb9c880b640052a
(cherry picked from commit f9f1c8b449efb9cb7894a64fe2977f875679ba52)

DO NOT MERGE Fix unknown type issue in unit tests

Bug: 63790458
Test: build, unit test
Change-Id: If5e403907292aa25c8d39b17f2f1818d493433e6
Merged-In: I28288c295b7ca0259b2112c11b4e5a81d6f2e33c

Clean-up BTIF profile queue on profile shutdown

* Add btif_profile_cleanup(uuid) method to remove pending connection
  requests for individual UUIDs
* Call the above method in each profile's clean-up method
* Add unit tests for btif_profile_queue

Bug: 63790458
Test: make, unit tests, pair and connect car kits
Change-Id: I28288c295b7ca0259b2112c11b4e5a81d6f2e33c
Merged-In: I28288c295b7ca0259b2112c11b4e5a81d6f2e33c
(cherry picked from commit d366807102bce97e494e83570d757ebcb6a33b3c)

Merge "Don't reject notifications and wait until new addressed player is ready (2/2)" into oc-dr1-dev

GAP: Set service_id before calling gap_release_ccb

Calling gap_release_ccb with a service_id == 0, which in turn
calls BTM_SecClrService with an ID of 0.

From the documentation for BTM_SecClrService:

  Service ID - Id of the service to remove. '0' removes all
               service records (except SDP).

Bug: 65695769, 65223508
Test: BLE connection and characteristic read
Change-Id: Icf309807f02e1faa273cf9bad9c09d9221a8bbfd
Merged-In: Icf309807f02e1faa273cf9bad9c09d9221a8bbfd
(cherry picked from commit 5c5c10683e2a11162838297fc8054b15837c4f32)
(cherry picked from commit 4023c6731d5e991c6202d4e499b9dd43d642fb3c)

Don't reject notifications and wait until new addressed player is ready (2/2)

There are some carkits like the ones found in the 2016 Honda CRZ and some
Audi's that do not follow the spec and do not honor reject pending notification
messages after switching players. This causes an issue whenever you switch
players, the metadata freezes due to the fact that the carkit never re-registers
for new track changed notifications. This patch removes the reject notification
and reorders the current notifications.

Bug: 64142363
Test: Test with Audi S7 and 2016 Honda CRZ and see that switching players works
      TestTracker: 105391/3975
Change-Id: Iaec70863594e13217916ab740d529f526d27c2d1
Merged-In: Iaec70863594e13217916ab740d529f526d27c2d1
(cherry picked from commit dd535e70c767fe214c1634b46618d0fb5ed4385d)

Ensure that services are cleaned from the GattServer HandleMap.
am: eb215402e3

Change-Id: Ic509b64189fccf8c48afaf947d625a8998b389af

Ensure that services are cleaned from the GattServer HandleMap.

The incorrect service handle was being plumbed up to onServiceDeleted.
This was causing stale entries to stick around forever in the HandleMap,
which could later cause failures to find callback references in
ContextMap if the connection ID changed for a given device.

Bug: 65463237
Test: unit tests modified and run
Change-Id: I2e22858b447f4e6b5a4fbceee4c406191c84a67d
(cherry picked from commit e631789075f5625fd79c774678f4af0bf102c7d1)

SDP: Bounds check 'id' parameter for free_sdp_slot() am: 294cc61f4f am: 5dc24c7b41 am: 8498351370 am: 615de1d1fe  -s ours am: 1ee8850324  -s ours am: 27e67ba277  -s ours am: 5af8ec6e85  -s ours am: bc2ddd916c  -s ours am: ad12cf8070  -s ours
am: eb80110ae4  -s ours

Change-Id: Ie41f709b2f47fcf629022c561b3c712ae90ac572

SDP: Bounds check 'id' parameter for free_sdp_slot() am: 294cc61f4f am: 5dc24c7b41 am: 8498351370 am: 615de1d1fe  -s ours am: 1ee8850324  -s ours am: 27e67ba277  -s ours am: 5af8ec6e85  -s ours am: bc2ddd916c  -s ours
am: ad12cf8070  -s ours

Change-Id: I98a440d9bfa53e413c531b2afa510b52c02104cc

SDP: Bounds check 'id' parameter for free_sdp_slot() am: 294cc61f4f am: 5dc24c7b41 am: 8498351370 am: 615de1d1fe  -s ours am: 1ee8850324  -s ours am: 27e67ba277  -s ours am: 5af8ec6e85  -s ours
am: bc2ddd916c  -s ours

Change-Id: Iad7cf407257de1788102f83e7ceafaf9167655c9

SDP: Bounds check 'id' parameter for free_sdp_slot() am: 294cc61f4f am: 5dc24c7b41 am: 8498351370 am: 615de1d1fe  -s ours am: 1ee8850324  -s ours am: 27e67ba277  -s ours
am: 5af8ec6e85  -s ours

Change-Id: I8971c22e6641d46e2d1fba13fe55407e3386acb2

SDP: Bounds check 'id' parameter for free_sdp_slot() am: 294cc61f4f am: 5dc24c7b41 am: 8498351370 am: 615de1d1fe  -s ours am: 1ee8850324  -s ours
am: 27e67ba277  -s ours

Change-Id: I5fb42670d4a641b39a14e538057a0193d33afb0e

SDP: Bounds check 'id' parameter for free_sdp_slot() am: 294cc61f4f am: 5dc24c7b41 am: 8498351370 am: 615de1d1fe  -s ours
am: 1ee8850324  -s ours

Change-Id: I5f54fca3ea0d17cad9a5e5439265c9b40340036c

SDP: Bounds check 'id' parameter for free_sdp_slot() am: 294cc61f4f am: 5dc24c7b41 am: 8498351370
am: 615de1d1fe  -s ours

Change-Id: I2e6425620ccf65636ba0b0ba6c783c8bf8764ec0

SDP: Bounds check 'id' parameter for free_sdp_slot() am: 294cc61f4f am: 5dc24c7b41
am: 8498351370

Change-Id: I171cafa603f3bf9d6a91895aba461fa96711c18c

SDP: Bounds check 'id' parameter for free_sdp_slot() am: 294cc61f4f
am: 5dc24c7b41

Change-Id: I27dffd6583706912f79cfd730eca390e930e9294

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

Change-Id: Ief8d911221408ff79a5addc7a7d58c7ca1a38808

SDP: Bounds check 'id' parameter for free_sdp_slot() am: 82e4754aaa am: 3b90fd7b0c am: 33aee8d11e am: d772aec66b am: c8d256c052  -s ours
am: 7fe92a7abf  -s ours

Change-Id: Ie383a65f8f6b2593f4c4bf32cea0179521e351b3

SDP: Bounds check 'id' parameter for free_sdp_slot() am: 82e4754aaa am: 3b90fd7b0c am: 33aee8d11e am: d772aec66b
am: c8d256c052  -s ours

Change-Id: I0c563fdf2b28e8b50abab9f6d9a1b1851a13a2ab

SDP: Bounds check 'id' parameter for free_sdp_slot() am: 82e4754aaa am: 3b90fd7b0c am: 33aee8d11e
am: d772aec66b

Change-Id: I41ed5d1d9f8efa504941b7084aa955496dfe0ec3

SDP: Bounds check 'id' parameter for free_sdp_slot() am: 82e4754aaa am: 3b90fd7b0c
am: 33aee8d11e

Change-Id: Ie3991cdfd834d3d0ff736562af300251fd70f96c

SDP: Bounds check 'id' parameter for free_sdp_slot() am: 82e4754aaa
am: 3b90fd7b0c

Change-Id: I2e9769b83ab39cfc51320b1116959780dd5d92da

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

Change-Id: Ica0050f493b45e216daa4c31e6eea38a4921d356

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

Change-Id: I2a51738b15ab49d419a95a1f14abf87aa2542af1

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

Merged-In: I34e8296ec7ec6b4ffbe1fa0452754f2a421e6ec7

Test: manual
Fixes: 37502513
Change-Id: I34e8296ec7ec6b4ffbe1fa0452754f2a421e6ec7
(cherry picked from commit b413f1b1365af4273647727e497848f95312d0ec)

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

Merged-In: I34e8296ec7ec6b4ffbe1fa0452754f2a421e6ec7

Test: manual
Fixes: 37502513
Change-Id: I34e8296ec7ec6b4ffbe1fa0452754f2a421e6ec7
(cherry picked from commit b413f1b1365af4273647727e497848f95312d0ec)

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

Test: manual
Fixes: 37502513
Change-Id: I34e8296ec7ec6b4ffbe1fa0452754f2a421e6ec7

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
Merged-In: I229ca3db6c92bc06bc1429e72412417010721063
(cherry picked from commit 35a9ea352d59e6d261eba1254814f7238e3ed022)

Use a mutex to protect the scheduling of tBTA_PM_TIMER

Bug: 63689331
Test: Code compilation
Change-Id: If062f7b3e2c17cb5a1ac2c8ef0813a2fe5c5a5f5
(cherry picked from commit a34d33ff2a3fe3562c1e003d870a0716f83568bf)

Protect the btif config state with a lock during cleanup

Bug: 64186300
Test: Code compilation, enable/disable Bluetooth
Change-Id: Ic2397a128d59c9cc69cbd8252734b69eb7c02501
(cherry picked from commit a4febd6d0460ee0f5834822ca406a4b8bab28cf1)

Merge "Added another entry to the INTEROP_2MBPS_LINK_ONLY interoperability database" into oc-dr1-dev

Added another entry to the INTEROP_2MBPS_LINK_ONLY interoperability database

Bug: 64357126
Test: Audio streaming to the affected carkit
Change-Id: I160eb3775c77fe400798f038d2aae7dd7825aacb

Merge "Revert "Allow collection of Bluetooth firmware dumps in bugreports (3/3)"" into oc-dr1-dev

Revert "Allow collection of Bluetooth firmware dumps in bugreports (3/3)"

This reverts commit 0580dd639cb10fafb6953d8a724db030df5030db.

Change-Id: I37f9ba5a9ace655d1d12404926a158fb9a4001a7

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

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

Merge "AE: Set secondary adv max skip to zero" into oc-dr1-dev

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 "Use the correct AVDTP version-specific features" into oc-dr1-dev

Merge "Fix processing of Inquiry results data" into oc-dr1-dev

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
(cherry picked from commit 9cd9ef4c3818d2c7949f52703b3c2ba0e0f43f2f)

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: I7cf536bdf1ab3c56ca5e1357749ef1b939dd703a

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

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
(cherry picked from commit 9bd6250eba5e624c94c66f053f5603dca8e0e4f1)

Fix the MAC address for INTEROP_2MBPS_LINK_ONLY entry

Bug: 62394764
Test: Code compilation
Change-Id: I3e6d8bb8258d6cc56ce05d7f17dda445859f011a
(cherry picked from commit b60955f9c72d8481a1a2d92d46d59e2fd31139c9)

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
(cherry picked from commit b866da0bc1254103146912077bcb49128963dee7)

Merge "Retry authentication in key-missing or pairing collision use case" into oc-dr1-dev

Merge "Remove unwanted space characters from dialing number" into oc-dr1-dev

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)

Use a mutex to protect the consistency of the tBTA_PM_TIMER state

Bug: 63689331
Test: Code compilation
Change-Id: Id8ab78473e1f766d5a76b967024b7ff4af499667
(cherry picked from commit f470def4e91301b7b161b88b0428476ecce4f324)

Remove unwanted space characters from dialing number

Certain car kits might send an AT command with space characters in
the dialing number. This commit adds a workaround to trim these unwanted
space characters.

Bug: 6340999
Test: Manual test with a car kit but purposely added numbers with spaces.
Change-Id: I9c238bc625fff8503c45cbbfaeb8bdd15b8a7d9a
(cherry picked from commit 54a1461a29a8c71dd010d418ec1b23012bf32805)

Merge "Revert "Always restrict the AVDTP MTU for SBC codec to MAX_2MBPS_AVDTP_MTU(663)"" into oc-dr1-dev

Merge "DO NOT MERGE ANYWHERE: Revert "Revert "Fix the interoperability mechanism for 2Mbps-only links for audio""" into oc-dr1-dev

DO NOT MERGE ANYWHERE: Revert "Revert "Fix the interoperability mechanism for 2Mbps-only links for audio""

This reverts commit ff1c56c3675079b3905739abffc3a6035529a248.

The original issue in oc-mr1-dev has been resolved by:
https://googleplex-android-review.git.corp.google.com/#/c/2576565/

Change-Id: I8bac8dca7939bac1865fe820e3c7b8252ce99478
Merged-In: Idaa25e18c0b017d8859aab723244fdc2613fc159

Reduce error conditions which cause link key deletion on auth complete

This inverts previous conditions which attempt to limit what error
conditions cause link key deletion and instead of excluding error codes
it provides a shorter list of codes which actually DO cause link key
deletion.

As a result, temporary failures caused by page timeouts, random
disconnects etc. should no longer cause link key deletion by accident.

Fixes: 63507123
Test: manual
Change-Id: I8d51403b8d233a3c335313b8f257a562c3d01109
(cherry picked from commit 51fb9c0e0cfbda559bf6377c183928ae48a24629)

Revert "Fix the interoperability mechanism for 2Mbps-only links for audio"

This reverts commit 039f9a8f09ab25b4b16acfd9aca3733108d02b28.

Reason for revert: Failing build in oc-mr1-dev

Change-Id: I186c116d4b26cd4b92938066309a898b82ef90bc
Merged-In: Idaa25e18c0b017d8859aab723244fdc2613fc159

Revert "Always restrict the AVDTP MTU for SBC codec to MAX_2MBPS_AVDTP_MTU(663)"

This reverts commit 9eee3a772b3f23e5c33845422203f0dec07be234.

Not needed anymore - handled by the interoperability database.

Bug: 62394764
Bug: 34127145
Test: Audio streaming to headset that supports 3Mbps
Change-Id: I143c07c171e0fe9ef882c5e506545187f9e41498
(cherry picked from commit a97b0381c8626320151c246cfd9a71dedcf635ff)

Fix the interoperability mechanism for 2Mbps-only links for audio

Also, added two new entries to the INTEROP_2MBPS_LINK_ONLY
interoperability database.

Bug: 62394764
Test: Audio streaming to headset that supports 3Mbps
Change-Id: Idaa25e18c0b017d8859aab723244fdc2613fc159
Merged-In: Idaa25e18c0b017d8859aab723244fdc2613fc159
(cherry picked from commit 0dec1807c6770a2e5f8b673588922046ed272255)

Do not delete pairing on HCI connection timeout

Change-Id: I67121f3f6d34cd82e243af11ecd735684c4ca67d
(cherry picked from commit f54d7e2719e36016f8dfa70757ae750909765776)
Fixes: 63796670
Test: manual

Merge "Add missing extension length check while parsing BNEP control packets" into mnc-dev am: fdcad53f8e am: fd94c15a5a am: 0cea290eee am: 97922c7401 am: c0008b6c2f am: 42b29f1ce6 am: 5328c7a32d am: 33f4a8065e am: 471e964d69
am: 1c780fabd5

Change-Id: Id09d66ad7aed3b9d88c8ed2426a190cb01e4e440

Free p_pending_data from tBNEP_CONN to avoid potential memory leaks am: 8f18afd26c am: ffa87779a1 am: 7eda996189 am: ba12732aad  -s ours am: 107bb0f68d  -s ours am: 73df1dfa11  -s ours am: 3fe34fe3fb  -s ours am: 6d3312cdf0  -s ours am: 515c909b33  -s ours
am: a7020eafc3  -s ours

Change-Id: Ifd89fb852bcaa3fc4028d2cc9bedea73e8018e73

Merge "Add missing extension length check while parsing BNEP control packets" into mnc-dev am: fdcad53f8e am: fd94c15a5a am: 0cea290eee am: 97922c7401 am: c0008b6c2f am: 42b29f1ce6 am: 5328c7a32d am: 33f4a8065e
am: 471e964d69

Change-Id: I2285382ec2b4f7629fca84740933bb85b22fbfab

Free p_pending_data from tBNEP_CONN to avoid potential memory leaks am: 8f18afd26c am: ffa87779a1 am: 7eda996189 am: ba12732aad  -s ours am: 107bb0f68d  -s ours am: 73df1dfa11  -s ours am: 3fe34fe3fb  -s ours am: 6d3312cdf0  -s ours
am: 515c909b33  -s ours

Change-Id: Ifffcbca70228a8bdac4bb00991742ced3a0c8e35

Merge "Add missing extension length check while parsing BNEP control packets" into nyc-dev am: c3619e1436 am: d647097c74 am: 0a9673bceb am: 88f2ae07f4 am: 384e1dff72  -s ours
am: cc56e94905

Change-Id: Iff7e1c0262caeb0217d67bd8f66e9baffabe6735

Merge "Add missing extension length check while parsing BNEP control packets" into mnc-dev am: fdcad53f8e am: fd94c15a5a am: 0cea290eee am: 97922c7401 am: c0008b6c2f am: 42b29f1ce6 am: 5328c7a32d
am: 33f4a8065e

Change-Id: I460a7805ba5d6ce28751bfba59245026ab09d49b

Free p_pending_data from tBNEP_CONN to avoid potential memory leaks am: 8f18afd26c am: ffa87779a1 am: 7eda996189 am: ba12732aad  -s ours am: 107bb0f68d  -s ours am: 73df1dfa11  -s ours am: 3fe34fe3fb  -s ours
am: 6d3312cdf0  -s ours

Change-Id: Idb3ed55f34818ce3af08e56787ac51dff49d6d92

Merge "Add missing extension length check while parsing BNEP control packets" into nyc-dev am: c3619e1436 am: d647097c74 am: 0a9673bceb am: 88f2ae07f4
am: 384e1dff72  -s ours

Change-Id: I041e907145d3f299b59c3fcbd8ae25c86176622d

Merge "Add missing extension length check while parsing BNEP control packets" into mnc-dev am: fdcad53f8e am: fd94c15a5a am: 0cea290eee am: 97922c7401 am: c0008b6c2f am: 42b29f1ce6
am: 5328c7a32d

Change-Id: I825dc2f50a83d94b54baf1f430e71c63f6bfcb3e

Free p_pending_data from tBNEP_CONN to avoid potential memory leaks am: 8f18afd26c am: ffa87779a1 am: 7eda996189 am: ba12732aad  -s ours am: 107bb0f68d  -s ours am: 73df1dfa11  -s ours
am: 3fe34fe3fb  -s ours

Change-Id: Id43622ce2ff6520e12bc80d4d034c59d3ab909ca

Merge "Add missing extension length check while parsing BNEP control packets" into mnc-dev am: fdcad53f8e am: fd94c15a5a am: 0cea290eee am: 97922c7401 am: c0008b6c2f
am: 42b29f1ce6

Change-Id: Idea1e755dd079731745a899aa7668a0f7179cb9e

Free p_pending_data from tBNEP_CONN to avoid potential memory leaks am: f8fc7f7d11 am: 2b53cd05f2 am: 28d58cb2c8 am: 9fa3d4f2c7 am: 92744bc2ce  -s ours
am: 27835e4e3e  -s ours

Change-Id: Ic8c4e4d473fb474626073b94c16af93ee1093d70

Free p_pending_data from tBNEP_CONN to avoid potential memory leaks am: 8f18afd26c am: ffa87779a1 am: 7eda996189 am: ba12732aad  -s ours am: 107bb0f68d  -s ours
am: 73df1dfa11  -s ours

Change-Id: If35fa07c33914a1df863f0c72b0d3061ae463678

Merge "Add missing extension length check while parsing BNEP control packets" into mnc-dev am: fdcad53f8e am: fd94c15a5a am: 0cea290eee am: 97922c7401
am: c0008b6c2f

Change-Id: Id14d11e553b6ba2f3fa688302415c79a537940a2

Free p_pending_data from tBNEP_CONN to avoid potential memory leaks am: f8fc7f7d11 am: 2b53cd05f2 am: 28d58cb2c8 am: 9fa3d4f2c7
am: 92744bc2ce  -s ours

Change-Id: I2e18a2b625023a318da67b7a5140fac5fb9ee590

Free p_pending_data from tBNEP_CONN to avoid potential memory leaks am: 8f18afd26c am: ffa87779a1 am: 7eda996189 am: ba12732aad  -s ours
am: 107bb0f68d  -s ours

Change-Id: Ib74d462308ed0e99f75aaca7e9344de3b07ba5a8

Merge "Add missing extension length check while parsing BNEP control packets" into nyc-dev am: c3619e1436 am: d647097c74 am: 0a9673bceb
am: 88f2ae07f4

Change-Id: I05f7864e4eb435e5d0cc7f5275fd4f34deb724b8

Merge "Add missing extension length check while parsing BNEP control packets" into mnc-dev am: fdcad53f8e am: fd94c15a5a am: 0cea290eee
am: 97922c7401

Change-Id: Id1569df9156b8de893dcb5c28997ec4a924eb82e