Merge "Ignore the Class Of Device if the value is invalid am: 7fe86917d7" into oc-dev-plus-aosp
am: dd48e58446

Change-Id: I3b473edc01e9b09b5aac6b5002626170a673fd1e

Merge "GATT write improvements" am: 34feb41643 am: 3455ef0449
am: e8ed18e203

Change-Id: Ieae1986e37891870078fb0ec9d9fd61bd9d9d0d0

Merge "Ignore the Class Of Device if the value is invalid am: 7fe86917d7" into oc-dev-plus-aosp

Ignore the Class Of Device if the value is invalid
am: 7fe86917d7

Change-Id: Iadd935ac9901bfa64d2b27ed2705e0fbf977d193

Merge "GATT write improvements" am: 34feb41643
am: 3455ef0449

Change-Id: I12196450ec3b4e9f91fb8b9a00d85e96d3f5b8f5

Merge "GATT write improvements"
am: 34feb41643

Change-Id: I9b950c052af3968138d0476f93b8899d89f31a96

Merge "GATT write improvements"

Ignore the Class Of Device if the value is invalid am: a052757882 am: e5056f6592
am: f3ea3df13d

Change-Id: I3a7908f8afa40707b4dd12ff54fb95a7455f141b

Ignore the Class Of Device if the value is invalid am: a052757882
am: e5056f6592

Change-Id: Ib00d2b697a1790df2040f48a34a7f61b3dfb6190

Ignore the Class Of Device if the value is invalid
am: a052757882

Change-Id: I1b28fb403a5231539fedf05d8b4af9cc5d3304ae

GATT write improvements

When doing GATT write, crash if p_clcb->p_attr_buf is null, or if
op_subtype is unknown. Both of this cases signal possible memory
corruption, and should be treated as fatal.
Properly handle GATT_CONGESTED error when doing long write.

Test: compilation test
Change-Id: Ia56f3bbd06cde2978b0ce2210ebe58042504938c

Ignore the Class Of Device if the value is invalid

If a Bluetooth Device is dual-mode and is discoverable as both
a Classic and a LE device, ignore unknown "zero" values
triggered by the LE advertisements. Otherwise, those could overwrite
a valid (previously discovered) Class Of Device.

Bug: 37615863
Test: Manual with LG HBS1100 headset
Change-Id: I5a0f6ef56248b41d973b850a5542475e45e0312d
(cherry picked from commit a052757882927c042399d614bc298f9704ce07a3)

Ignore the Class Of Device if the value is invalid

If a Bluetooth Device is dual-mode and is discoverable as both
a Classic and a LE device, ignore unknown "zero" values
triggered by the LE advertisements. Otherwise, those could overwrite
a valid (previously discovered) Class Of Device.

Bug: 37615863
Test: Manual with LG HBS1100 headset
Change-Id: I5a0f6ef56248b41d973b850a5542475e45e0312d

Merge "Add tracking of periodic_enable in adv_inst" am: a50c91ad0d am: c7b55d6d5d
am: e22ecd7839

Change-Id: Id87b4ca5cc7ea307a71831dffb8694de90fae064

Merge "Add tracking of periodic_enable in adv_inst" am: a50c91ad0d
am: c7b55d6d5d

Change-Id: I9a5bb45f4666554e87f34c1364a17ee0685b95ce

Merge "Add tracking of periodic_enable in adv_inst"
am: a50c91ad0d

Change-Id: Idc80f7c7bab0bc88b81176ad6a631415d20cdad5

Merge "Add tracking of periodic_enable in adv_inst"

Merge "gatt_api minor style fixes" am: c2a3b46b3a am: 60b221387d
am: c581376714

Change-Id: I1e6a9102998b07dcf54d019498588d64a630e2bb

Merge "Don't handle server respons after it was deallocated" am: 6d150a149d am: 93a67d5a1c
am: a96ae18050

Change-Id: Ie21ac2559bf263ce5091b5270396c969caa1d740

Merge "gatt_api minor style fixes" am: c2a3b46b3a
am: 60b221387d

Change-Id: Ie15a7e79b119d140383a0b1633b1d1b59f775190

Merge "Don't handle server respons after it was deallocated" am: 6d150a149d
am: 93a67d5a1c

Change-Id: If3054a415fa925c9899796132449ef5cbfb5f337

Merge "gatt_api minor style fixes"
am: c2a3b46b3a

Change-Id: If76bcc6faeecf8597f56b171535752d64f04e0a1

Merge "Don't handle server respons after it was deallocated"
am: 6d150a149d

Change-Id: I2a21c1b1585b986d9cb07f3c3beb40a45caed7e6

Merge "gatt_api minor style fixes"

Merge "Don't handle server respons after it was deallocated"

Merge "Ensure we don't send more packets than firmware allows" am: ec8ab16abe am: 33b2e123d8
am: 4fb92f0cfa

Change-Id: I2151deadbb5b59d8d0593ba13da1e5731fe089d9

Merge "Fix transport quota exhaustion" am: 0a0ee916fd am: 33d05d0dbd
am: ffded9bc6b

Change-Id: Ie790d032bfa2d15ccf91783528b6b2d0c647983d

Merge "Ensure we don't send more packets than firmware allows" am: ec8ab16abe
am: 33b2e123d8

Change-Id: Icf2812d79b81dcd39df2647841bc6498d5f589ee

Merge "Fix transport quota exhaustion" am: 0a0ee916fd
am: 33d05d0dbd

Change-Id: Iaeb66d5db775f05578054a8622e89f4cce42d2dd

Merge "Ensure we don't send more packets than firmware allows"
am: ec8ab16abe

Change-Id: I6084a7edf6a878b1294099ff76e300fee95bea1e

Merge "Fix transport quota exhaustion"
am: 0a0ee916fd

Change-Id: Id0cc56113fb8c883af4987ec0910c29c7f0a61d9

Merge "Ensure we don't send more packets than firmware allows"

Merge "Fix transport quota exhaustion"

gatt_api minor style fixes

Test: compilation test
Change-Id: I7656513b2999cec5aefbb5b9f75e10e9436d0412

Add tracking of periodic_enable in adv_inst

Track state of periodic advertisement similar to
extended advertisement. Make sure periodic adv
is disabled prior to calling remove adv set

Test: net_test_stack_multi_adv
Bug: 62306759
Change-Id: I677211a0bca40e7151c8c7ee138bac623888291d

Allow blocking AG SCO connects by policy am: acdc11cae5 am: 7d44d990a7
am: 5df0faaa75

Change-Id: I6e5008332c2cb83637bb1efc5de848b9ec9fbdf1

Allow blocking AG SCO connects by policy am: acdc11cae5
am: 7d44d990a7

Change-Id: I24c3f5380c00a66ee26337a2c471109f7bdcc084

Allow blocking AG SCO connects by policy
am: acdc11cae5

Change-Id: Ic468c3340f1f9b4b71f1267b096a372f9851c0af

Allow blocking AG SCO connects by policy

Needed for devices that support both HFP AG and HFP HF.

btsnoop logs show the sequence of events leading to failure:
(corroborated by bt logcat)

1. ACL connection with remote phone established on handle 0x01
2. ACL connection with remote headset established on handle 0x02
3. Incoming call in HF role from phone
4. Phone call locally published to telecom
5. Answer incoming call
6. SCO connection established with phone in HF role
7. AG role with headset picks up on call state change to answered
   via BTA_AG_IN_CALL_CONN_RES
8. Inside handler for BTA_AG_IN_CALL_CONN_RES, bta_ag_sco_open called
   to establish the SCO connection in the AG role with the headset,
   stomping on the previous (wanted) SCO connection with the phone.

Fix is to publish the SCO audio route policy to fluoride, so we can
stop 8 from happening.

Bug: 32958838
Test: manual:
      received incoming call in HF role, audio worked both ways;
      recieved incoming MT call, headset audio in AG role worked both ways
Change-Id: I12961598e8200bd1d5adb46ee8ec3a802114b80a

Ensure we don't send more packets than firmware allows

Make TX complete callback after we've sent the packet
to the lower layers and adjusted our in flight packet
count.

Bug: 26763700
Test: manual
Change-Id: Iaba14a11c1583da2a72769ebd096e0baedf857c3

Fix transport quota exhaustion

BREDR and BLE use round-robin for link quota.

If you have exhausted one transport's quota,
the break will cause traffic for all transports
to cease.

Change to a continue to prevent this.

Change-Id: I7ca739dfbf02acd2e30527c16ca616a7bad70c51
Test: manual
CR-Id: ALPS02303646

Fix GATT over BREDR not connecting am: e0035727e3
am: 3f8965abbc

Change-Id: Ie8ceb75882d743b19b90cb6f01f1bb6e1f799350

Don't handle server respons after it was deallocated

After a request is send to the GATT server, it might timeout, or get
cancelled, i.e. by client unregistering. If a response to such request
is later received, it should not be parsed. Currently such response is
parsed, and can call the response callback again, also on already
unregistered app.

Test: compilation test
Change-Id: Ia76770a693cd35d49973462c4d233d70f8a4e165

Fix GATT over BREDR not connecting am: e0035727e3
am: 808b26c982

Change-Id: I372135a8a4aec66c23f983347ae9af432d83c2e8

Fix GATT over BREDR not connecting
am: e0035727e3

Change-Id: I97ece2b97b1000bb9104757680e03063ee04c432

Merge changes If7a360bf,Ibc20c110 am: 206e9d8412 am: 6ce5e1b03e
am: b176b2fb31

Change-Id: Ia76747a5d5903b646f5a60eb862cd84770516e3a

Fix GATT over BREDR not connecting
am: e0035727e3

Change-Id: I5b149ca3735db7e951cf7d66c606cac60385646b

Merge changes If7a360bf,Ibc20c110 am: 206e9d8412
am: 6ce5e1b03e

Change-Id: I5e4f5f205ecb69478761bc3c25d8150cb51b85fc

Fix GATT over BREDR not connecting

For BREDR connections, the upper stack always sends the request with
transport type DUMO, and this should be treated as BREDR if the device
is not known as LE only.

Bug: 62387078
Test: pts GattOverBrEdrTest
Change-Id: I726252e85e78bff2c9585332d2748a0d92a9f6d3

Merge changes If7a360bf,Ibc20c110
am: 206e9d8412

Change-Id: I395a3aaee058bdf0d8f14082451d021ddcb9609c

Merge changes If7a360bf,Ibc20c110

* changes:
  gatt_cleanup_upon_disc refactor
  Simplify read/write start

Merge "Misc native service changes" am: 842574c91e am: 3f18651236
am: ccc61bd916

Change-Id: I310bb528003bab21da4eb2b5f7571e9bfd7fa85f

Merge "Misc native service changes" am: 842574c91e
am: 3f18651236

Change-Id: Icbc22e0fbc729a6bfff7cc38c35f97bee8c872bf

Merge "Misc native service changes"
am: 842574c91e

Change-Id: Iccf1fc5a6acd1c8b4a543658764ba12917bad8b8

Merge "Misc native service changes"

gatt_cleanup_upon_disc refactor

gatt_end_operation internally always call gatt_clcb_dealloc. Make sure
both those methods don't get called in gatt_cleanup_upon_disc. Also
minor style fixes.

Test: compilation test
Change-Id: If7a360bfd1ac2ccd7bb1565e8efe459333f8f025

Simplify read/write start

When read or write is started with insuficcient encryption, and the
encryption fails, it will either return immediate error
GATT_NO_RESOURCES, or the operation will be enqueued. In case the
enqueued operation fails again on the BTM_SetEncryption, no callback
will never be called, and no operation error will be returned. The
gatt_end_operation will never get called for the CLCB, and it will never
be freed, causing memory leak.

This patch fixes this by making sure that the call to
gatt_security_check_start always consumes the CLCB - either by executing
operation, queuing it, or returning error. This also ensures that if
BTM_SetEncryption fails, it will always be handled the same way - by
calling the callback.

Test: manual
Change-Id: Ibc20c1101cc7b5b6043e75df93b23a5b24b791e9

Add flag to bluetoothtbd to enable bluetooth on startup am: e8e6c6ba4b am: c745d07b92
am: ef6ec9c7f2

Change-Id: I8b3954f18a9f29e3e4ac01de271c99b6b22e4d63

Add flag to bluetoothtbd to enable bluetooth on startup am: e8e6c6ba4b
am: c745d07b92

Change-Id: Ia4f49546198d6dd2dab20128bb46012f0da71bdf

Add flag to bluetoothtbd to enable bluetooth on startup
am: e8e6c6ba4b

Change-Id: Ieec57f7a4a023c4f080abc48865357f0b213850b

Merge "Fix loops iteration" into oc-dr1-dev
am: 79ddc70569

Change-Id: Ib4f75dc8e4d60c552db69cfbc4c624fc9ad3859f

Merge "Fix loops iteration" into oc-dr1-dev

Misc native service changes

Bug: None
Test: Build bluetoothtbd
Change-Id: Ie81e023735799a726a5fdd62c443795beaf3eeac

Disable 48.0kHz sampling rate for AAC am: ad73569d62
am: ebf7dd1290

Change-Id: Iaa39440ae3bb9cd79bea7d96fa09cf9f99b2a78b

Disable 48.0kHz sampling rate for AAC am: ad73569d62
am: a2cea2cdf1

Change-Id: I1b80094a95c8142a0c1832785427054d155995df

Disable 48.0kHz sampling rate for AAC
am: ad73569d62

Change-Id: Ic295d9d8d09ba555372bd786b8045e309c80c390

Disable 48.0kHz sampling rate for AAC
am: ad73569d62

Change-Id: Ie55cf4ad1e3d43e3ce61230f9b6f411d75de8fe6

Add flag to bluetoothtbd to enable bluetooth on startup

Bluetooth startup takes a few seconds and it can be done in parallel
during the boot up process.

Bug: None
Test: Test on device.
Change-Id: Ia23bdd6907b6f24d1e3eacb011173acf64a583ed

Fix GATT over BREDR not connecting am: 3bd07c3618 am: c6045a60d7
am: a8be0c9f97

Change-Id: If78b891e37d32d5b956cb605061ab9190062dfb0

Fix loops iteration

The erase() already increment the iterator. If the loop additionally do
"it++", it will skip some elements, and might iterate over the end.

Test: compilation test
Change-Id: Ibd9c993d8e80d807f5d7bc920da36ee538477438

Fix GATT over BREDR not connecting am: 3bd07c3618
am: c6045a60d7

Change-Id: Ibc2b0acd48eb1b53538c35e8243466ba2fa3c332

Fix GATT over BREDR not connecting
am: 3bd07c3618

Change-Id: Iab8c43982daa098f4587ffd9fda282f499af1df4

Fix GATT over BREDR not connecting

For BREDR connections, the upper stack always sends the request with
transport type DUMO, and this should be treated as BREDR if the device
is not known as LE only.

Bug: 62387078
Test: pts GattOverBrEdrTest
Change-Id: I726252e85e78bff2c9585332d2748a0d92a9f6d3

Fix loops iteration am: bb95a1d622 am: 7bacf13459 am: b0e90eb7b7
am: 0278f5dd47

Change-Id: I1e03ea232eaa46127d1fd1c6a5e7eb48b0f6481b

Fix loops iteration am: bb95a1d622 am: 7bacf13459
am: b0e90eb7b7

Change-Id: Ie973c611b464e60bbc65d1be48cc745aa61e050f

Fix loops iteration am: bb95a1d622
am: 7bacf13459

Change-Id: I8e578684b46fe2faacda3b5f52c5f2de58e138cc

Fix loops iteration
am: bb95a1d622

Change-Id: If6b4c6bc5ff24d61fd4998c618445cb9742915cd

Disable 48.0kHz sampling rate for AAC

Some carkits have issues playing A2DP audio using AAC with 48.0kHz
sampling rate, while playing AAC with 44.1kHz sampling rate is fine.

Bug: 38443632
Bug: 62301376
Test: manual
Change-Id: I50de2db1d5386d41bfe361fbac99fc7ebbbd3640
(cherry picked from commit 3d268331ae32bd70f60984c9553db1c961073c42)

Fix loops iteration

The erase() already increment the iterator. If the loop additionally do
"it++", it will skip some elements, and might iterate over the end.

Test: compilation test
Change-Id: Ibd9c993d8e80d807f5d7bc920da36ee538477438

Disable 48.0kHz sampling rate for AAC am: 3d268331ae am: 56d227680a am: 57054ad14c
am: 02d32788cd

Change-Id: I403c229ca3b02ce706ae4ce4d0a0827af84a0987

Disable 48.0kHz sampling rate for AAC am: 3d268331ae am: 56d227680a
am: 57054ad14c

Change-Id: Id8005f86ee1aa88a64bf2454b6bae9ffb8053d2d

Disable 48.0kHz sampling rate for AAC am: 3d268331ae
am: 56d227680a

Change-Id: I50c312bc2f5c9b23cce1a0f01d600092860f1646

Disable 48.0kHz sampling rate for AAC
am: 3d268331ae

Change-Id: I8435c13619a686a5da20c063e172d100099db9e4

Disable 48.0kHz sampling rate for AAC

Some carkits have issues playing A2DP audio using AAC with 48.0kHz
sampling rate, while playing AAC with 44.1kHz sampling rate is fine.

Bug: 38443632
Bug: 62301376
Test: manual
Change-Id: I50de2db1d5386d41bfe361fbac99fc7ebbbd3640

Use std::unordered_set for keeping app_hold_link am: 3dda89880b am: f21ae6ed4f am: f13ffd521a
am: f987dd4bec

Change-Id: I2425fb1baf970e3d121e14e8294d45e2513ad47c

Use std::unordered_set for keeping app_hold_link am: 3dda89880b am: f21ae6ed4f
am: f13ffd521a

Change-Id: I51e28d5202184f3dafba5d22682497f649ff326f

Use std::unordered_set for keeping app_hold_link am: 3dda89880b
am: f21ae6ed4f

Change-Id: I1d06135b7488cfe3a9f8dd7f33ef71aec4abef86

Use std::unordered_set for keeping app_hold_link
am: 3dda89880b

Change-Id: I75b54eb119d655ede8331eb8105b01329a46a92e

Use std::unordered_set for keeping app_hold_link

Change-Id: Ia8a0c9a2264979c1ff89faa5ddf17a83fcb7c7ce

Get rid of clcb_idx in favour of pointers am: e4f42abab3 am: 5958c225ad am: 09273a05f4
am: 866740dbf9

Change-Id: I721e60b6f892cee0901540be505a75be6e3d6d0b

use std::queue for keeping tGATT_CMD_Q am: d1a9313471 am: 05bd66b651 am: 8b37d9d337
am: ad1e6a7c44

Change-Id: I0707f44414c56587e42ff41be058efc040c3e3ca

Get rid of clcb_idx in favour of pointers am: e4f42abab3 am: 5958c225ad
am: 09273a05f4

Change-Id: I332f5503adbc01a98ebc9554be2f6102770c00e7

use std::queue for keeping tGATT_CMD_Q am: d1a9313471 am: 05bd66b651
am: 8b37d9d337

Change-Id: I78e802eada40f4b5c29cb74fba580fed9b1c6ce7

Get rid of clcb_idx in favour of pointers am: e4f42abab3
am: 5958c225ad

Change-Id: Iad5b0a28e81f8a7eb6e332c22481ae49656e975e

use std::queue for keeping tGATT_CMD_Q am: d1a9313471
am: 05bd66b651

Change-Id: I2face2724f4c52c14769bd1c79a6783f8e4c2d47

Get rid of clcb_idx in favour of pointers
am: e4f42abab3

Change-Id: Ie15226cacea4121d0bb7ca9a9cec732d0170f4f1

Refactor tGATT_BG_CONN_DEV am: 7cfd8391f6 am: 77157460ba am: 7b55503837
am: 3491d92903

Change-Id: I1e69aa803d3e25080cd9e485e69eefe1918de025

use std::queue for keeping tGATT_CMD_Q
am: d1a9313471

Change-Id: I2708947e05fdf3835c23874af353349b203f11e4

Refactor tGATT_BG_CONN_DEV am: 7cfd8391f6 am: 77157460ba
am: 7b55503837

Change-Id: I3f1ef7c9daea0aee2769cc4386eb32b2d8c3e96f

Refactor tGATT_BG_CONN_DEV am: 7cfd8391f6
am: 77157460ba

Change-Id: I0d5bc72270c5d7b2cd88cb5e91ace28a07e2cc99