Convert HID and Device Manager BTA code to C++
am: b7938c1866

Change-Id: I9ddcdc2683d63ba84cd5f8e9de044dc78b0d6521

Convert HID and Device Manager BTA code to C++

These are the only places that use BTA GATTC API. After this change it's
possible to change GATT API to use C++ types as arguments.

Bug: 28485365
Change-Id: Idf83396f4bc3a2cc8a9b41f69da5d033af1be678

Fix unused source warning
am: b172552537

Change-Id: I2c2191358dd60a087cb1d39fa22dd65ab2eb7371

Fix unused source warning

These were using the wrong c++ extension, so the build system was not
picking them up.

Change-Id: I3a767a236bca05f04dd069bb1f40796f41cb8b60

Fix missing GATT Characteristic from last service
am: 6c81482c2d

Change-Id: I1f103942867c539eb97ce4768812558cf3bf9844

Use UINT16 for GATT attribute id field
am: d407e1afca

Change-Id: I890bc4f7bde854578c837dcc3fb0eaace92e411c

Fix missing GATT Characteristic from last service

GATT Service is contained between start and end handle. If
characteristic definition is at end handle, and it's value definition is
after end handle, it will not be properly discovered. That's because we
use value_handle instead of attribute_handle to identify
characteristics.

As a workaround, increase service boundary if value is defined after
it's definition.

Bug: 29253825
Change-Id: Ib145aea4f5cf38a1fbb977c301136e16f8f900f7
(chery picked from commit 552e630435a68d8f51d458dccf8697c625cf1510)

Add missing curly braces in clear notification
am: 82e2fa17b4

Change-Id: I4185c983b253dcd06dade4e762b304e2594984f0

Use UINT16 for GATT attribute id field

Id field is equal to attribute handle, so it must be 16bits.
Otherwise only piece of GATT database can be used.

Bug: 29253825
Change-Id: Ia88e3ad3eb96ca8e97349aa5e852e66c57072b49
(cherry picked from commit e6b1cb43525e27f95de85dc43150d7368b9d4c35)

Add missing curly braces in clear notification

Change-Id: I59b13486f76735e5dd0c07d57b2671c746169bd8

HOGP: Clear pending operations for a given device on disconnect am: f978a57103
am: 6818e548cb

Change-Id: Ife41744dba7574aaa462cb0a2ac545d9382544ca

HOGP: Clear pending operations for a given device on disconnect
am: f978a57103

Change-Id: Ica1e0caf127e3baccb20075e2f6fd95b1d9c03ba

HOGP: Clear pending operations for a given device on disconnect

If there are any GATT operations interrupted by a disconnect, mark the
device as no longer executing. Otherwise we'll stil receive data from
the device, but we'll be unable to send any HID commands to it.

Bug: 29184976
Change-Id: I489f41c970abad0ff1145005d3250beafa83d27e

HOGP: Clear pending operations for a given device on disconnect
am: 22fc38259f

Change-Id: Icac19c9f691843e5df70fdd0051c6d2441bf5582

HOGP: Clear pending operations for a given device on disconnect

If there are any GATT operations interrupted by a disconnect, mark the
device as no longer executing. Otherwise we'll stil receive data from
the device, but we'll be unable to send any HID commands to it.

Bug: 29184976
Change-Id: I489f41c970abad0ff1145005d3250beafa83d27e

Do not request AVRC_CAP_COMPANY_ID if A2DP Sink is not enabled am: 1c512b851d
am: 6f01bc2a6d

Change-Id: I219aad79a41fd9499e040ecf1d551fb6fda058a1

Do not request AVRC_CAP_COMPANY_ID if A2DP Sink is not enabled
am: 1c512b851d

Change-Id: Ic30615e2cf916ab2311deb1377d868e3f3a661a7

Do not request AVRC_CAP_COMPANY_ID if A2DP Sink is not enabled
am: 470466e6be

Change-Id: If478c20f576e201f3443ae8ae61768f8ba6e80d8

Do not request AVRC_CAP_COMPANY_ID if A2DP Sink is not enabled

Also fixed AVRC_PDU_GET_PLAY_STATUS timeout response.

Bug: 28177785
Change-Id: Icde64a5c1806453850abe101f9707fff61566566
(cherry picked from commit 1c512b851d76deac860fce1232e4a1fa1a097f1c)

Do not request AVRC_CAP_COMPANY_ID if A2DP Sink is not enabled

Also fixed AVRC_PDU_GET_PLAY_STATUS timeout response.

Bug: 28177785
Change-Id: Icde64a5c1806453850abe101f9707fff61566566

Set disable timer in case of any active ACL connection
am: 70035577fd

Change-Id: I05c0e02b4149cc8f943612617f5166d3fffafd59

Set disable timer in case of any active ACL connection

Use Case: Disable time out when atleast one active ACL connections

Steps:
1. Turn ON Bluetooth
2. Connect to any Headset
3. Turn OFF bluetooth.

Failure: Turn OFF bluetooth is failed when atleast one active ACL connections

Root Cause:i If we have one or more active ACL links, so it is not notitying
BTA_DM_DISABLE_EVT to upper layer.

Fix: Set disable timer in case of any active ACL connection

Bug: 29217385

Change-Id: Ica47717119f1ab9fa9f4f5244e8483c408276a2a
(cherry picked from commit 3e4f01ada8c1627bd48f31cd0c8269984b8e5f96)

Merge \"btm: Restrict EXTENDED_INQUIRY_RESULT to 1 result\" into nyc-dev
am: 59bc2de906

Change-Id: I5cefab6f346731ea11085ce613c9c4c95665ba4c

Merge "btm: Restrict EXTENDED_INQUIRY_RESULT to 1 result" into nyc-dev

Set disable timer in case of any active ACL connection am: 3e4f01ada8
am: b5d0d36154

Change-Id: I968db4a71934e298d91e8e3815c21d697157efb9

Set disable timer in case of any active ACL connection
am: 3e4f01ada8

Change-Id: Ia22d37db6f47cf634a6c38a3f169140054b95ee8

Set disable timer in case of any active ACL connection

Use Case: Disable time out when atleast one active ACL connections

Steps:
1. Turn ON Bluetooth
2. Connect to any Headset
3. Turn OFF bluetooth.

Failure: Turn OFF bluetooth is failed when atleast one active ACL connections

Root Cause:i If we have one or more active ACL links, so it is not notitying
BTA_DM_DISABLE_EVT to upper layer.

Fix: Set disable timer in case of any active ACL connection

Change-Id: Ica47717119f1ab9fa9f4f5244e8483c408276a2a

Merge \"Fix payload size for GATT Read by type request\" into nyc-dev
am: 73ee3314ed

Change-Id: I5893f5aa1c551469d17dd0dc556ebafa5c47266c

Merge \"Reject L2CAP INFO RSP and ECHO RSP on the LE signaling channel\" into nyc-dev
am: e67263c5e6

Change-Id: I4856ec39f37afee18bc6d5154bdffe23a57c454d

Merge "Fix payload size for GATT Read by type request" into nyc-dev

Merge "Reject L2CAP INFO RSP and ECHO RSP on the LE signaling channel" into nyc-dev

Remote device changes for SMP certification
am: 9ded3b71de

Change-Id: If254bb87236ea88405433ccd88b874e8672fa3dd

Merge \\"Reject L2CAP INFO RSP and ECHO RSP on the LE signaling channel\\" am: fe7f53d6b3
am: f92afd2378

Change-Id: Ie49e2a2f6f4d0d7751f1d1e378f9753e08d14370

Merge \\"Fix payload size for GATT Read by type request\\" am: 89b997468c
am: 556f238bac

Change-Id: I399635806ba80c08019efa7aeb8b4d3566803bd5

Merge \"Reject L2CAP INFO RSP and ECHO RSP on the LE signaling channel\"
am: fe7f53d6b3

Change-Id: Ida2dd0bff8210d63e3aa02f8261a9633352f0204

Merge \"Fix payload size for GATT Read by type request\"
am: 89b997468c

Change-Id: I26c3889d95d5e7d0b1b94cbabdbbe91404fd6530

Merge "Reject L2CAP INFO RSP and ECHO RSP on the LE signaling channel"

Merge "Fix payload size for GATT Read by type request"

btm: Restrict EXTENDED_INQUIRY_RESULT to 1 result am: 1a011544dc
am: 2386dc5177

Change-Id: I506fe9f24710cf212e397b4142052f8a33c0ad6d

btm: Restrict EXTENDED_INQUIRY_RESULT to 1 result
am: 1a011544dc

Change-Id: Ibd967e028594f6cbb64b4040016012fb53bb047d

Fix payload size for GATT Read by type request

Payload size for GATT Read by type request for UUID32 and UUID128
are incorrectly allocated. This leads to memory overflow when
Read by type request is sent for them and eventually results
in crash during free. This change makes sure that the payload
can accomodate upto 128 bit UUID.

This crash is observed while running TC_GAR_CL_BV_03_C Qual test
case.

Bug: 29011042
Change-Id: Ib2b41b769b394670099f4549f204e2972f7df876
(cherry picked from commit 7da98621a4dfa8196b8d20e8955ad508f540e5d9)

Fix payload size for GATT Read by type request

Payload size for GATT Read by type request for UUID32 and UUID128
are incorrectly allocated. This leads to memory overflow when
Read by type request is sent for them and eventually results
in crash during free. This change makes sure that the payload
can accomodate upto 128 bit UUID.

This crash is observed while running TC_GAR_CL_BV_03_C Qual test
case.

Bug: 29011042
Change-Id: Ib2b41b769b394670099f4549f204e2972f7df876

Reject L2CAP INFO RSP and ECHO RSP on the LE signaling channel

Steps:
Execute TC_LE_REJ_BI_02_C

Failure:
PTS stuck after sending INFO RSP, ECHO RSP on the LE signaling channel

Root Cause:
The Bluetooth stack is not rejecting INFO RSP, ECHO RSP even though
these commands are invalid for the LE signaling Channel.

Fix:
Reject the INFO RSP, ECHO RSP with error L2CAP_CMD_REJ_NOT_UNDERSTOOD.

Bug: 27852645
Change-Id: Ie09f658d606206cbdbea928e8cb6d10332743cf4
(cherry picked from commit d35bb40dd563e7f37df0a2fa9c0f61d0958b9601)

Reject L2CAP INFO RSP and ECHO RSP on the LE signaling channel

Steps:
Execute TC_LE_REJ_BI_02_C

Failure:
PTS stuck after sending INFO RSP, ECHO RSP on the LE signaling channel

Root Cause:
The Bluetooth stack is not rejecting INFO RSP, ECHO RSP even though
these commands are invalid for the LE signaling Channel.

Fix:
Reject the INFO RSP, ECHO RSP with error L2CAP_CMD_REJ_NOT_UNDERSTOOD.

Bug: 27852645
Change-Id: Ie09f658d606206cbdbea928e8cb6d10332743cf4

btm: Restrict EXTENDED_INQUIRY_RESULT to 1 result

The spec only allows one result per event.
Abort processing and return if the spec is not followed.

Bug: 29141745
Change-Id: Ic414b28adadec274779b0d953d12d499a619aff6
Signed-off-by: Myles Watson <mylesgw@google.com>

btm: Restrict EXTENDED_INQUIRY_RESULT to 1 result

The spec only allows one result per event.
Abort processing and return if the spec is not followed.

Bug: 29141745
Change-Id: Ic414b28adadec274779b0d953d12d499a619aff6
Signed-off-by: Myles Watson <mylesgw@google.com>

Remote device changes for SMP certification

Use Case:
Certification test cases for SMP require various scenarios
where the remote device needs to show a specific behavior
where it fails the pairing in a certain way, and the DUT is
required to abort the pairing properly.
In abcense of a proper PTS suite to execute these test cases,
we can use another device running the same host by configuring
certain run time property.

Test Cases:
TP/SCJW/BI-02-C
TP/SCJW/BV-02-C
TP/SCPK/BI-03-C
TP/SCPK/BI-04-C
TP/SCPK/BV-02-C
TP/SCPK/BV-03-C
TP/SCJW/BI-01-C
TP/SCCT/BV-01-C
TP/SCCT/BV-02-C
TP/SCPK/BI-01-C
TP/SCPK/BI-02-C
TP/SCPK/BV-04-C
TP/SCPK/BV-01-C

Fix:
Added a property in the bt_stack.conf file. The property name
is "SmpFailureCase". The values 2 to 6(inclusive), are forcausing SMP
failures with various failure reasons.
Failure case 1 and 9 are for producing error "Confirm value failure".
Cases 7 and 8 are for generating specific errors at pair cancel.

Note:
The default use of this feature is controlled using a compile
time flag BTM_BLE_SMP_CERTIFICATION.
The BTM_BLE_SMP_CERTIFICATION = TRUE is needed only while we
wait for the PTS support for the LE Secure connections.

Bug: 27852645
Change-Id: I1f7a8ff2659d85b5978b75870c57162a34d394d0
(cherry picked from commit 0bd0c8fc88a7141691a7ca839b11cb711945ee33)

Fix invalid BDA parameter passing for connection parameter updates am: 3021c5a5aa
am: 68750afdb5

* commit '68750afdb59191037c651571099f542122f5361f':
  Fix invalid BDA parameter passing for connection parameter updates

Change-Id: I7571c6277ccb7142e486b2c1588ab687c2b79a69

Fix invalid BDA parameter passing for connection parameter updates
am: 3021c5a5aa

* commit '3021c5a5aac9760e8c4885cbf9c220d1b04dbdb3':
  Fix invalid BDA parameter passing for connection parameter updates

Change-Id: I649b1b08ad84056dbdd44ca4258280bc57639215

Fix invalid BDA parameter passing for connection parameter updates

The BD_ADDR type is not a struct, just an array of bytes. Therefore,
Bind does not know how to store it for later execution, and just stores
the pointer to the first element, which is not valid at time of
execution.

Instead use bt_bdaddr_t, which is a struct. Bind will copy it's content
properly.

Change-Id: I4875aae4d7cbb809821d91a13554b8fdf310a7fe

Merge "Revert "DO NOT MERGE Proper construction of AVDTP general reject message"" into nyc-dev
am: a3f785a138

* commit 'a3f785a138ab67c6533de4f427e777c222e4352e':
  Revert "DO NOT MERGE Proper construction of AVDTP general reject message"

Change-Id: Ib8b48f4805efa30b46bbfdd3b760ced75a0b7a59

Merge "Clear BLE block from device records at unpair" into nyc-dev
am: 8c919fb37a

* commit '8c919fb37a56f3a6a2e30564e1356bf93c468b5a':
  Clear BLE block from device records at unpair

Change-Id: I7866ce6a767cc7b2ce5727a8e971f45808c5e2c0

Merge "Revert "DO NOT MERGE Proper construction of AVDTP general reject message"" into nyc-dev

Merge "Clear BLE block from device records at unpair" into nyc-dev

Remote device changes for SMP certification am: 0bd0c8fc88
am: a81770e617

* commit 'a81770e617b6f39107f8b1aa0aa519523f94ae39':
  Remote device changes for SMP certification

Change-Id: I85f54999363801ddac7a58d89bb31f51e4e3d442

Remote device changes for SMP certification
am: 0bd0c8fc88

* commit '0bd0c8fc88a7141691a7ca839b11cb711945ee33':
  Remote device changes for SMP certification

Change-Id: Id8e7028ae84257b6831bb6ae058c1495e63ea10a

Fix for issue introduced due to rebase of LE L2CAP COC
am: be3f7f9d24

* commit 'be3f7f9d244ca39a9a5a27afd3b117f83b69e0e4':
  Fix for issue introduced due to rebase of LE L2CAP COC

Change-Id: Iac2d2fd11ec63a4bc6af70ffb5e72174ed940cbf

Remote device changes for SMP certification

Use Case:
Certification test cases for SMP require various scenarios
where the remote device needs to show a specific behavior
where it fails the pairing in a certain way, and the DUT is
required to abort the pairing properly.
In abcense of a proper PTS suite to execute these test cases,
we can use another device running the same host by configuring
certain run time property.

Test Cases:
TP/SCJW/BI-02-C
TP/SCJW/BV-02-C
TP/SCPK/BI-03-C
TP/SCPK/BI-04-C
TP/SCPK/BV-02-C
TP/SCPK/BV-03-C
TP/SCJW/BI-01-C
TP/SCCT/BV-01-C
TP/SCCT/BV-02-C
TP/SCPK/BI-01-C
TP/SCPK/BI-02-C
TP/SCPK/BV-04-C
TP/SCPK/BV-01-C

Fix:
Added a property in the bt_stack.conf file. The property name
is "SmpFailureCase". The values 2 to 6(inclusive), are forcausing SMP
failures with various failure reasons.
Failure case 1 and 9 are for producing error "Confirm value failure".
Cases 7 and 8 are for generating specific errors at pair cancel.

Note:
The default use of this feature is controlled using a compile
time flag BTM_BLE_SMP_CERTIFICATION.
The BTM_BLE_SMP_CERTIFICATION = TRUE is needed only while we
wait for the PTS support for the LE Secure connections.

Bug: 27852645
Change-Id: I1f7a8ff2659d85b5978b75870c57162a34d394d0

Clear BLE block from device records at unpair am: 266ff1d759
am: df95c615cb

* commit 'df95c615cbbfae8303b100f163e006fab533a834':
  Clear BLE block from device records at unpair

Change-Id: I4b7168e64d5e9935237725c43ae0440ad35ab9f3

Clear BLE block from device records at unpair
am: 266ff1d759

* commit '266ff1d759e7713b957cb73c8d0c7a0c0b70c7dd':
  Clear BLE block from device records at unpair

Change-Id: I5cf9341cae1887226d7947a69b985d1d43f52254

Clear BLE block from device records at unpair

Use Case:
Pair and unpair LE remote device repeatedly. The repair
intermittently fails with the error "DHKey Check Failure"

Failure:
Pairing failure is observed when pairing is performed
after unpairing, without performing a BT reset.
Further analysis of the SMP logs indicate that DUT uses the
static address of the remote device for calculation of the
MacKey value. At the time of pairing, the remote RPA should
be used (and the static address has not yet been distributed either).
The problem is caused by the previous dev record which is left in
the stack, and at the time of repair, stack picks the old device
record and reuses it.

Fix:
This change removes the entire BLE block from the device record,
after the unpairing is performed, so that a new device record
is allocated, after the repairing is done.

Bug: 27852645
Change-Id: I8d605d1bb3b1c32061a8d8a7a7fe0fe200030abc

Clear BLE block from device records at unpair

Use Case:
Pair and unpair LE remote device repeatedly. The repair
intermittently fails with the error "DHKey Check Failure"

Failure:
Pairing failure is observed when pairing is performed
after unpairing, without performing a BT reset.
Further analysis of the SMP logs indicate that DUT uses the
static address of the remote device for calculation of the
MacKey value. At the time of pairing, the remote RPA should
be used (and the static address has not yet been distributed either).
The problem is caused by the previous dev record which is left in
the stack, and at the time of repair, stack picks the old device
record and reuses it.

Fix:
This change removes the entire BLE block from the device record,
after the unpairing is performed, so that a new device record
is allocated, after the repairing is done.

Bug: 27852645
Change-Id: I8d605d1bb3b1c32061a8d8a7a7fe0fe200030abc

Revert "DO NOT MERGE Proper construction of AVDTP general reject message"

This reverts commit 2ed10d442266eade44be121cd24fd473c85007d0.

Original change needs to be reverted to align with Errata
ESR04 where "Section 8.17, General Reject Page 71" of adopted
AvDTP Spec is changed to have "Invalid Signal Identifier"
in first 6 bits of Octet 1.
This change helps passing PTS case TP/SIG/SMG/ESR04/BI-28-C

Bug: 27852645
Change-Id: Idfc960ce00662e8aacc26f3b3fdca9deca8a2aff

Fix for issue introduced due to rebase of LE L2CAP COC

During 6721232 patch, some code got missed while doing manual rebase.
This patch is adding that missing code.

Bug: 27852645
Change-Id: I2525908c29e3288e0ace1ee26c5f2aa6cfd59d68
Signed-off-by: Navin Kochar <navin.kochar@intel.com>

Reset max_interval to BTM_BLE_CONN_INT_MIN_LIMIT am: 3bd286a411
am: c2148b38cb

* commit 'c2148b38cb53f577d5732bb093d70776fad59f66':
  Reset max_interval to BTM_BLE_CONN_INT_MIN_LIMIT

Change-Id: Ic938e3424e66ddc6906b53d0ac9b08c375b447e4

Reset max_interval to BTM_BLE_CONN_INT_MIN_LIMIT
am: 3bd286a411

* commit '3bd286a4112de2c68e7188c0001d884454686992':
  Reset max_interval to BTM_BLE_CONN_INT_MIN_LIMIT

Change-Id: I688b68898a107ad28678a096739a6dde5e074c24

Reset max_interval to BTM_BLE_CONN_INT_MIN_LIMIT

In case of min_interval < BTM_BLE_CONN_INT_MIN_LIMIT
and max_interval < BTM_BLE_CONN_INT_MIN_LIMIT,
min_interval is reset to BTM_BLE_CONN_INT_MIN_LIMIT
but max_interval is not reset.
Then valid connection parameter updates can be rejected.

To avoid this, reset max_interval to BTM_BLE_CONN_INT_MIN_LIMIT
when max_interval is smaller than BTM_BLE_INT_MIN_LIMIT.

Change-Id: I5f54ddc0276f40d667ff433557b74ea4b100058d

Merge "bt: Update libchrome APIs to r395517" am: cb3c6f3cd0
am: 7156015e90

* commit '7156015e902c2c4b33079f354143f597cdd9c28b':
  bt: Update libchrome APIs to r395517

Change-Id: I9fe6ea63138a35e9925ed339cf1f993ea059f475

net_test_bluetooth: RFCOMM test refactor am: fa717c0b47
am: 7390de6ace

* commit '7390de6ace68340ced99c84835e13aaf5c0af4a5':
  net_test_bluetooth: RFCOMM test refactor

Change-Id: I436f760e8e2d57283da0abff75d137837c5fe13d

Merge "bt: Update libchrome APIs to r395517"
am: cb3c6f3cd0

* commit 'cb3c6f3cd0b908e36443d41c2d736b88d742ee1a':
  bt: Update libchrome APIs to r395517

Change-Id: I579baf27f8e777b16c559ba30bf649fc3958b94e

net_test_bluetooth: RFCOMM test refactor
am: fa717c0b47

* commit 'fa717c0b47bb1fed04da295eaa580a92bc08c622':
  net_test_bluetooth: RFCOMM test refactor

Change-Id: I4f5d85e43462e666ead4714b92b77cf95db3bdff

Merge "bt: Update libchrome APIs to r395517"

net_test_bluetooth: RFCOMM test refactor

Properly implement RFCOMM tests into net_test_rfcomm using GUnit.
Contains all the RFCOMM tests and the base class that the tests use.
This test currently requires the DUT to be paired with an HFP capable
device.

Bug: 25793348
Change-Id: I683db7c5f2779373b2e343c7ad21268e4969d014

resolve merge conflicts of 32d6669 to nyc-dev-plus-aosp

Change-Id: I949ead7d023a70561ab498c9772e8abbe17a01a6

Remove currently unused peer device module and tests am: bc472821fe
am: 21594289ea

* commit '21594289ea9a88046e74caedfaac646aa26667d9':
  Remove currently unused peer device module and tests

Change-Id: I016b23ad8017f9d51459fb1d5f5935b9f3f23d93

Remove currently unused peer device module and tests
am: bc472821fe

* commit 'bc472821fe8aac8fca089082496f5b7d067ffae8':
  Remove currently unused peer device module and tests

Change-Id: I6f9893bbb49f077b250b17c6ab73ead56ab8b60f

Remove currently unused peer device module and tests

The module is currently not used and it is currently not obvious if this
should be a module in the first place.

Filed separate bug (#29057391) to re-visit the concept in a future
refactor, but removing unused code for now.

Bug: 29000421
Change-Id: Ie369574e62281fa1d7c3284c86c4509de558fa15

Bump HDP version to 1.1 am: bb838444c3
am: 49f9eb0bb6

* commit '49f9eb0bb66dc8d9ba9226c47c88610cc24aec55':
  Bump HDP version to 1.1

Change-Id: If2e944edaf11dafd69ba10da802afe519a36b8d7

Bump HDP version to 1.1
am: bb838444c3

* commit 'bb838444c37642b7272e3ae4acaaa7a06a568cdb':
  Bump HDP version to 1.1

Change-Id: I7e39e0c6caace2077133c8cf6dd1ae9aa6b82673

Bump HDP version to 1.1

Change-Id: I0bdec6fffa097dc919ee495216a2f272cc0dcf1e

resolve merge conflicts of 934747d to nyc-dev-plus-aosp

Change-Id: Ifea8e5daf03ae684ad22e0dc790ae1bd26ba0035

Limit number of SBC frames to fit into single MTU
am: e9a52af36d

* commit 'e9a52af36d087ea68ec7ba5ed3f25c24fb3cdf64':
  Limit number of SBC frames to fit into single MTU

Change-Id: Ia96d418dfa7968981e1585f7d4f078035c5a317b

Limit number of SBC frames to fit into single MTU

- Calculate number of SBC frames to fit into single MTU for EDR capable
  remote devices. This makes sure the maximum utilization of the
  available bandwidth, keeping the bitrate at high quality.
- This change helps to address the discrepancy seen with a few available
  2 Mbps headsets which set the AVDTP media MTU size greater than the
  2DH5 packet payload size which eventually forces AVDTP packets to get
  fragmented in the controller. This change ensures packing of the SBC
  frames in a way that AVDTP packet size fits within the 2DH5 payload
  size.
- As packet size is restricted to meet the required bit rate more than
  one AVDTP packet can be sent to the controller in a given iteration
  based on the bit rate requirement.

Bug: 24875861
Change-Id: Icae7656ad563bbbdef65ecbfa15a89b26cf9edb2

HID: Check for EINTR in uhid_event() and fix return value evaluation

uhid_event() returns 0 in the normal case. Commit 3a2c2d61
results in the polling thread to be exited.

Bug: 28942565
Change-Id: Id0aff2958bc7c2704ba340aa0ff7848afb8dcf80

Update the HCI unit test if BT_CLEAN_TURN_ON_DISABLED is TRUE
am: 6834570262

* commit '6834570262116974a176cd8598b6d0ca6d031aca':
  Update the HCI unit test if BT_CLEAN_TURN_ON_DISABLED is TRUE

Change-Id: I406540a4a8fdf864717cd3ebf89ca814ba1ba991

Merge "Update the HCI unit test if BT_CLEAN_TURN_ON_DISABLED is TRUE" am: 692c63af2e
am: b6426a8d3c

* commit 'b6426a8d3ca4ccc036f5130b36f90a8db84d4b78':
  Update the HCI unit test if BT_CLEAN_TURN_ON_DISABLED is TRUE

Change-Id: I220d5c502dad4f08c8a7631e74ac7c4ad4d78f0d

Merge "Update the HCI unit test if BT_CLEAN_TURN_ON_DISABLED is TRUE"
am: 692c63af2e

* commit '692c63af2e1a8adc5bb98084b16e26ba8445ecfe':
  Update the HCI unit test if BT_CLEAN_TURN_ON_DISABLED is TRUE

Change-Id: Id962d5c6fac28fd352ca0f4b191a86c42460cba2

Merge "Update the HCI unit test if BT_CLEAN_TURN_ON_DISABLED is TRUE"

Fix misc-macro-parentheses warnings in embdrv/sbc. am: 7b7b057828
am: 89ebdc41bf

* commit '89ebdc41bf3ab8f285ed34dde63a6f628bd94aa6':
  Fix misc-macro-parentheses warnings in embdrv/sbc.

Change-Id: I723d7868ddeb7513dec76ffa503419f9ebdf699a

Fix misc-macro-parentheses warnings in embdrv/sbc.
am: 7b7b057828

* commit '7b7b0578285281c762bd52255072cb34800ec7ea':
  Fix misc-macro-parentheses warnings in embdrv/sbc.

Change-Id: Ifd3ad6cfb1b0f12230d8367f9cf2a8b84c1b7fd9

Update the HCI unit test if BT_CLEAN_TURN_ON_DISABLED is TRUE

Bug: 28614494
Change-Id: I3f4a923f9e14faef57b2d694bfc253fd10749414
(cherry picked from commit 7496372239773b7121359a34291681b749beda5f)

Update the HCI unit test if BT_CLEAN_TURN_ON_DISABLED is TRUE

Bug: 28614494
Change-Id: I3f4a923f9e14faef57b2d694bfc253fd10749414

Fix misc-macro-parentheses warnings am: 63b0519c3f
am: 8638a130fb

* commit '8638a130fb5f9ceb91dff8847f636f8e03c7f377':
  Fix misc-macro-parentheses warnings

Change-Id: I84e020ae253c783fa2c9006042e486f2ad805b50

Fix misc-macro-parentheses warnings in embdrv/sbc.

When compiled with WITH_TIDY=1, clang-tidy warns about
missing parameters around macro parameters.

Bug: 28705665
Change-Id: I79d234b88e090137412936b80303d21f0c59b0ae

Fix misc-macro-parentheses warnings
am: 63b0519c3f

* commit '63b0519c3f0d1c0cc523e0e9553c60fc68651acc':
  Fix misc-macro-parentheses warnings

Change-Id: If0986131e3e16f67f2de535db7825b12101c05a1

Fix misc-macro-parentheses warnings

When compiled with WITH_TIDY=1, clang-tidy warns about
missing parameters around macro parameters.

Bug: 28705665
Change-Id: I89c23cfe6eaefae5ef3c648a7c123c0501662c13

DO NOT MERGE Randomize allocation canary am: 3387396e16  -s ours
am: 6a1637e129  -s ours

* commit '6a1637e129456565d58056122428f619796c3eb9':
  DO NOT MERGE Randomize allocation canary

Change-Id: I35e0817f13c2fb47de6f1819d518197384c0fa88

DO NOT MERGE Randomize allocation canary
am: 3387396e16  -s ours

* commit '3387396e167819510c499b5a5486b499b3f4868f':
  DO NOT MERGE Randomize allocation canary

Change-Id: I87d650f2da46121e7b436a92f21d302f9870e5ee

Add minimum LE connection timeout that can be used for HID devices
am: bfc90ee868

* commit 'bfc90ee868b042aa245eed8305047b35bddfdc5c':
  Add minimum LE connection timeout that can be used for HID devices

Change-Id: I25aba0c882f7fe420e90f8ebd7f860410606d9d0

DO NOT MERGE Randomize allocation canary

A static allocation canary can be susceptible to buffer overflow
exploit code bypassing, so this randomizes it on every run.

Bug: 27411268
Change-Id: I81b06f89951a012c08d846042653ec957f3e9127
(cherry picked from commit 7c054350fb7da9da1fdb86e7f7b9a801cf1c39e4)