Fix a NULL-pointer access before check in connect_int am: 9b2e40df05
am: 73af061500

Change-Id: I596eaaa5f8e1919e75f4332534f480057465cb54

Fix a NULL-pointer access before check in connect_int
am: 9b2e40df05

Change-Id: I3217f206ef56d73dbac01b19eeb123b6cfb68870

Fix a NULL-pointer access before check in connect_int

This fixes a potential NULL-pointer access in case cb is returned NULL

Test: mm -j 8
Change-Id: Iaf7aa0ba736b5cb161811726b4aaea7f1ab7e5b3

Merge "Reduced the log verbosity level for aptX/aptX-HD data packets" am: 5256f1feea
am: 76efc23846

Change-Id: I38cf5bdaff222dbd7328f04bf82da3c5cc056ab6

Don't allow GATT and GAP service override am: 8a06ff5ce2
am: 3b672538ba

Change-Id: I235d976c6e1e1ab80fdc3a457f5ea776281a309a

Merge "Reduced the log verbosity level for aptX/aptX-HD data packets"
am: 5256f1feea

Change-Id: I762b7e65001740fec55515f7121f5217c4dbb455

Don't allow GATT and GAP service override
am: 8a06ff5ce2

Change-Id: I9eaff8a251269b3f073845c80284fefde35822b4

Merge "Reduced the log verbosity level for aptX/aptX-HD data packets"

Fix NPE when discovering invalid GATT datatbase am: 8957fdca4f
am: 1637dc7990

Change-Id: I1e9710d05b0276106c0fd19604db2b9db5cde257

Fix NPE when discovering invalid GATT datatbase
am: 8957fdca4f

Change-Id: I1e196651928314494a696e5a3ef126272dd3208a

Don't allow GATT and GAP service override

Those services are automatically added by the stack, and overriding them
results in invalid GATT database being created.

Bug: 34280184
Test: Manual test
Change-Id: Ie6a09e9d606e40066bef6f6a2c934eacd17bf53b

Fix NPE when discovering invalid GATT datatbase

When doing a discovery of invalid GATT database, where two services are
added with same start handle, characteristics are added to the first
one, where descriptors are added to the later. This cause NPE when
adding descriptor.

Bug: 34280184
Test: manual test
Change-Id: I27619098df33a641cb089b82cf4d4ffd3c6e6aea

Reduced the log verbosity level for aptX/aptX-HD data packets

Also, fixed a typo in a log message.

Test: A2DP streaming to headset, and capturing "adb logcat" output
Change-Id: I525d78e3ebde334857261f0cbd256a7f6a946e96

Fix for Extended Scan parameter length am: 0c86aad3f5
am: a41493f386

Change-Id: Id57ec583ff0a29a5dfdaf2aee449fd7e6d5414c6

Fix for Extended Scan parameter length
am: 0c86aad3f5

Change-Id: I561461bd6b119912ef53dfbca923f2aac9b3fad7

Fix for Extended Scan parameter length

With BT 5.0 extended advertisement scan parameter length
is configured with correct value.

Test: BT scan from settings menu
Change-Id: I749099282a9f2cf190d4f9576649aefe88564809
Signed-off-by: Reddy Praveen <praveen.reddy@intel.com>

Removed the feeding_init call from tA2DP_ENCODER_INTERFACE am: f8e2670d12
am: a7d87c4c12

Change-Id: I4b28e24b3fad9248028bae23038a12d8c4b67ce8

Removed the feeding_init call from tA2DP_ENCODER_INTERFACE
am: f8e2670d12

Change-Id: I716d1c077b089345c08ca76ccf2823ac1ff6d8e3

Removed the feeding_init call from tA2DP_ENCODER_INTERFACE

The feeding_init call is not used anymore - it has been superseded
by the newer A2dpCodecs.setCodecAudioConfig() mechanism.

Also, moved aptx_init_framing_params() and aptx_hd_init_framing_params()
calls to the corresponding encoder_update functions.

Test: A2DP streaming to Bluetooth headsets
Change-Id: I52f42b4701d04fe2ddb8f115bb9cd57fe38e2b38

Integrated A2DP LDAC support with the LDAC encoder library am: d556b752bd
am: 95c78b6741

Change-Id: I7c1f807c2ede174d4682e2acf6ced396b1b19356

Integrated A2DP LDAC support with the LDAC encoder library
am: d556b752bd

Change-Id: Ib0d6db87f7f30ac7544bf2e5764b7dda0d1a24f9

Integrated A2DP LDAC support with the LDAC encoder library

* Add compile-time dependency on the libldacBT_enc module
  in external/libldac
* Use the <ldacBT.h> header and remove local typedefs and defines

Also, fix an initialization issue of the LDAC handler
inside file a2dp_vendor_ldac_encoder.cc

Bug: 30958229
Test: A2DP streaming to LDAC headset
Change-Id: I3757230c9901509daab36f2b2018faa05490171d

Merge "Serialize stack shutdown, state change callback and cleanup" into cw-f-dev
am: 718b632f2b

Change-Id: I59546b88705ca7a9748bc3a41ad0a382caada0fe

Serialize stack shutdown, state change callback and cleanup
am: 352b8e89ea

Change-Id: Ie1262e4b3604a5e04910201d330419873a656f2f

Merge "Serialize stack shutdown, state change callback and cleanup" into cw-f-dev

Merge "Add "connecting" and "disconnecting" states to HID Device." am: 35ae4adf21
am: 0d5910c685

Change-Id: Ibd5437b69871d1dbf5159152fa488892523510a6

Merge "Add "connecting" and "disconnecting" states to HID Device."
am: 35ae4adf21

Change-Id: I5eef753ca30b57848ebeaeb3b73aa134cb6b6970

Merge "Add "connecting" and "disconnecting" states to HID Device."

resolve merge conflicts of 316589aa to nyc-mr1-dev-plus-aosp

Change-Id: I5758549f4c2e1f26d02ff087e240698b13343bbe

Remove position dependent lookup tables in AT command parser
am: 8d3cf5988a

Change-Id: I0a690f1538c49fbed743ccf324d240dbb8c927c8

Batch scan simplification am: eab3651db4
am: 58dde764e0

Change-Id: Idffdb211a7084a21789f1c2bbad4c634809be842

LE filters without custom data types am: 2ffd8da9af
am: 4d227fc750

Change-Id: Ib7f7de32290dd8efc4b97185324011135ec53229

Get rid of BTA_DmBleCfgFilterCondition am: ad6afdf96c
am: fbb8d006a6

Change-Id: I1b4b2e071a0101ed7db0eecd7fc9c457a6093f2d

Simplify LE scan filter memory management am: ae2aabef09
am: fad449ee33

Change-Id: I447b69cd3bc0ee2ad9e4d33473831b9ed2013ae8

Remove reduntant re-schedule am: 2c1522e2b8
am: cbaeb50923

Change-Id: I62f949769fd07a9adabe3ba2b83ab3d2b12ae1d5

Use callbacks for LE filter functions (BTM layer) am: 1febda9706
am: 8cf65e4d0e

Change-Id: I24828a1130fc54d0d72845751b5725b72db5c30c

Batch scan simplification
am: eab3651db4

Change-Id: Iab508f656ceaaa726edff3e04945146a3d4762d4

LE filters without custom data types
am: 2ffd8da9af

Change-Id: I787c473f2c5839674fbf0ab3c646f210d15ab717

Get rid of BTA_DmBleCfgFilterCondition
am: ad6afdf96c

Change-Id: I9d5464bc0199759ef83cac4a2b7b6a94635053a4

Simplify LE scan filter memory management
am: ae2aabef09

Change-Id: I654160519dd7b03fa11d58a6ce6e5dbc24dd5d88

Remove reduntant re-schedule
am: 2c1522e2b8

Change-Id: Id7a3ed5b5fe1460e14450616bad422b39f8cbc9d

Use callbacks for LE filter functions (BTM layer)
am: 1febda9706

Change-Id: I218fb9df79b2b4df4d8a1774273a951c63042772

Batch scan simplification

Bug: 30622771
Test: sl4a UniqueFilteringTest BleOpportunisticScanTest
Change-Id: I0e6ca8ed3d7d98057277a5187e684246f410d978

LE filters without custom data types

Bug: 30622771
Test: sl4a FilteringTest
Change-Id: Ib4edfa40038ca59c37ae1dfc763f0cac563177d7

Get rid of BTA_DmBleCfgFilterCondition

Bug: 30622771
Test: sl4a FilteringTest
Change-Id: I93b4f1275b9437a20cf1429d184dbefa79451986

Simplify LE scan filter memory management

Bug: 30622771
Test: sl4a FilteringTest
Change-Id: I58f3c335e84b4d7ee578c321109125084826efa0

Remove reduntant re-schedule

Calls to BleScannerInterface are already made using jni thread.
Also all this calls do is scheduling to BTA thread, so remove
unnecessary call to do_in_jni_thread

Bug: 30622771
Test: sl4a FilteringTest
Change-Id: I2c799dde720f9bc694fecacd6d3959b96c1b9ba8

Use callbacks for LE filter functions (BTM layer)

This patch use callbacks, instead of custom event queue when scheduling
LE filter VSC.

Bug: 30622771
Test: sl4a FilteringTest
Change-Id: I64ac4392883c647d1e500c257c31a8d3e740a0a8

Fix bad connectable advertising constant am: 35675d0e73
am: 4c8198cfd9

Change-Id: Idb7596057acbad7fb30e1719261cfe17e29440c0

Zero is a valid advertiser id am: daa8a99913
am: 17a68bf7df

Change-Id: I90e4ce2f7e0005de2d0bd5541883f3587f2a96dc

Remove redundant dependency on libpower am: 4302060551
am: 51a1abd14d

Change-Id: Id809a19186f9f31964fa4d66f19535413b1fd3bf

Fix bad connectable advertising constant
am: 35675d0e73

Change-Id: I02605b0df8e6d46db0ca2f81e6c5e172337614a7

Zero is a valid advertiser id
am: daa8a99913

Change-Id: I05b1d504381932959877a0c856b1bb7ef82292a1

Remove redundant dependency on libpower
am: 4302060551

Change-Id: Ie78d7e79d9f1b9043eceae84865b4eb5274fd4db

Fix bad connectable advertising constant

Test: start connectable advertising using bluetoothtbd
Change-Id: Ie7e42570748d43a27bc7691378aee85ab1565a05

Zero is a valid advertiser id

Test: start advertising using bluetooth-cli
Change-Id: Iead31c5e73a70850c6a209b6bbbeaeb792202608

Remove redundant dependency on libpower

* libpower dependency was introduced in change
  I1e8d49b087eff6301373e1e90e8d868f15847c34
* It is never used in any part of the system/bt stack and therefore
  should be removed from the shared library dependency list

Bug: 34079981
Test: code compilation, no user visible effect
Change-Id: Ic8e3ad0c369839fb9ced446695c72fdf0ffad1ca

Fix Heart Rate Service example am: e3051fec06
am: fb326faf46

Change-Id: I8367ba5e2238fddd5636a3b0ae7c896db9a0f1ff

Fix Heart Rate Service example
am: e3051fec06

Change-Id: I8ff34d31fe85899a0657ce437f4bf614a24528c8

Add "connecting" and "disconnecting" states to HID Device.

Currently the upper java layers have full support for all four
connection states, yet the lower stack only reports about
"connected" and "disconnected".
This patch adds sending "connecting" and "disconnecting" messages
when bta_hd_connect_act and bta_hd_disconnect_act finish
successfully.

Test: build, run, logcat
Change-Id: I00f9f414daa3854198ff9eee19e9c98a2e18f603

Fix Heart Rate Service example

Name must be appended at end, not beginning.

Test: run bt-example-hr-server --advertise
Change-Id: I70909e0bfc94521c396908d0dcb78cf4439a7e63

Remove unnecessary cutils dependency am: 19e3d703b2
am: c1f7859ee3

Change-Id: Iea809f705c041d384b514df146f510e21c185a0a

Remove unnecessary cutils dependency
am: 19e3d703b2

Change-Id: Id0b5878395183458449cb0c89b515659b5e86133

Update Linux BUILD.gn files am: a9743d8101
am: 5c4642ab77

Change-Id: I5e38f3e901431873e9d3ed19a5ffebd868850e4e

Update Linux BUILD.gn files
am: a9743d8101

Change-Id: I846fa35f6f6ec7ccdc242d77b2f7e5ce41e46f9b

Remove unnecessary cutils dependency

Test: build with ninja and mma
Change-Id: I31688da8a554ed71b580c30a0e4c000ea4903d70

Update Linux BUILD.gn files

Test: build with ninja
Change-Id: I4993dae7f76479da9bfad70aae88cc51fd7127d5

Renamed the aptX and aptX-HD shared libraries am: d43a901c57
am: ae84776457

Change-Id: I98d6d9a9d362f59dc7b1b18359d8e338c57e7f6e

Renamed the aptX and aptX-HD shared libraries
am: d43a901c57

Change-Id: I0bccfb379fb0719e9500c44d44378cfab23037ea

Renamed the aptX and aptX-HD shared libraries

* The names of the expected aptX and aptX-HD shared libraries are
  updated to reflect better their purpose:
  - aptX: libaptX.so -> libaptX_encoder.so
  - aptX-HD: libaptXHD.so -> libaptXHD_encoder.so

Also:
* Add a missing upcall with the codec info the first time the codec
  is selected internally
* Minor cleanup

Bug: 30958229
Test: A2DP streaming to aptX/aptX-HD headsets
Change-Id: Ib2bfc83823190fb7a057353eb47cd71c5b4d5a3a

Merge "Integration of the LDAC codec for A2DP source" am: 690c0478b6
am: 6077931f05

Change-Id: I552143138e0799f7da423710eb858a25241bd286

Merge "Integration of the LDAC codec for A2DP source"
am: 690c0478b6

Change-Id: I3ef8f12d26b0832bc8c6da03275ad1416de86e7c

Merge "Integration of the LDAC codec for A2DP source"

Merge "Fix A2DP metrics session duration" am: 644f61e760
am: 08802b44b7

Change-Id: Ia58a579368458a246a422a4a47b8524f32355dab

Merge "Fix A2DP metrics session duration"
am: 644f61e760

Change-Id: I675bf805857d7e37b522d29912e633154071bf8d

Merge "Fix A2DP metrics session duration"

Integration of the LDAC codec for A2DP source

The codec can be used if the encoding shared library is installed
on the device:
 libldacBT_enc.so

Also, increased the maximum number of SEPs AVDT_NUM_SEPS from 3
to 6 to accomodate Sink devices that support more than 3 codecs.

Test: A2DP streaming to LDAC headsets
Bug: 30958229
Change-Id: I114c3e6be2ebe1ac8aece62a3146637d37cd938e

Merge "Add LeakyBondedQueue to libosi" am: 797be14aba
am: b2b62f4d88

Change-Id: I0655e7e70dabf08557559938028708a30c57d3ca

Merge "Add LeakyBondedQueue to libosi"
am: 797be14aba

Change-Id: Ib260e88f8dc8be1f86d200f8ab4f0690c4b4aad9

Merge "Add LeakyBondedQueue to libosi"

Merge "Integration of the aptX and aptX-HD codecs for A2DP source" am: bcb80655e3
am: dc5afd7359

Change-Id: I1bd7ef5a5fffba6e4bb3fa3c321fbbb766185677

Merge "Integration of the aptX and aptX-HD codecs for A2DP source"
am: bcb80655e3

Change-Id: Ifd232e09aebc80ffb09fc195429e18787851e6aa

Merge "Integration of the aptX and aptX-HD codecs for A2DP source"

Serialize stack shutdown, state change callback and cleanup

Use Case: Bluetooth On/Off scenario, User switch

Steps:
1. Turn ON Bluetooth
2. Switch user
Also observed on occurrence of enable/disable timeout during On/Off testing.

Failure: Crash while sending state change HAL callback.

Root Cause: In the event when stack cleanup is queued to stack manager
thread when stack shutdown is being processed, stack cleanup starts
execution as soon as stack shutdown is finished.
If the function event_signal_stack_down posted to btif thread at the
end of stack shutdown is executed after stack cleanup crash occurs.

Fix: Serialized the execution of stack shutdown, state change callback
execution and stack cleanup to happen in same order.

Test: code compilation.

Change-Id: Ic96205b5c304acb44eab53f4e2cb150726643bda
(cherry picked from commit 16e112b1861d8f1147325372608c905627e00e88)

scripts: modify change_types.sh to skip matching "<data type>_" am: 69b8f87943
am: aafd275968

Change-Id: I39369270c553abdd1881eb60c282e9afc2c54dfb

scripts: modify change_types.sh to skip matching "<data type>_"
am: 69b8f87943

Change-Id: I24fc6ef1b6484bfb9a3a6e671b9b5d3b76882d9c

Remove position dependent lookup tables in AT command parser

The various position dependent lookup tables in the AT command parser
were out of sync, causing invalid responses to the AT+CBPS command for
example.

This patch gets rid of positionally dependent enums for simple lookup
tables that correlate all the values for easier, less error prone
maintenance of the related tables.

This re-instates a previously reverted patch after fixing incorrect
field order in tBTA_AG_INDICATOR_MAP.

Change-Id: I7f8a052e78706c8c72c5102b38cfe9ce200ae0d9
Fixes: 29978908

Fix A2DP metrics session duration

* Fixed A2DP duration counting. It is now counting from music play start
  to music play end.
* Start logging a2dp connection as Bluetooth sessions. Currently, only
  A2DP connections are logged. Thus the bluetooth session length will be
  the total connection length and the length within A2DP session message
  will be the audio connection length.
* Add a audio_duration_millis field in A2DPSession to record audio duration
* Add bonded memory constraint for metrics entries
* Use a builder mechanism to only build metrics upon dumping
* Refactor metrics module into BluetoothMetricsLogger class
* Created unit test for BluetoothMetricsLogger

Bug: 33694310
Test: Code compilation, Unit test, BtFunhausMetricsTest
Change-Id: Iea2a997c4ea074687a5d50860e9229f0e1b82659

Add LeakyBondedQueue to libosi

* LeakyBondedQueue is a fixed size queue that leaks oldest item when
  reaching its capacity. This is useful in creating memory bonded data
  structure where freshness is more important than full coverage.
* The queue is protected by a simple mutex and is thread-safe, although
  improvements could be made to lock enqueue and dequeue separately, it
  is not implemented at this moment due to lack of demand
* The queue uses unique_ptr to automatically free its content when it is
  destructed
* Add several tests to verify its API and memory management
* This data structure will be firstly used in the metrics module

Bug: 33781460
Test: Code compilation, unit tests
Change-Id: I51cb73666ac58e4792d9cba0d6f16dad30a0ff39

Integration of the aptX and aptX-HD codecs for A2DP source

Each of the codecs can be used if the corresponding encoding
shared library is installed on the device:
 - aptX: libaptX.so
 - aptX-HD: libaptXHD.so

Test: A2DP streaming to aptX and aptX-HD headsets
Bug: 30958229
Change-Id: I1ca36da653721576f5a5b6bdac186b9ea2790c85

scripts: modify change_types.sh to skip matching "<data type>_"

Script erroronously modifies UINT8_TO_BE_STREAM() to
uint8_t_TO_BE_STREAM(). Modidy the script to prevent that.

Test: Manual; Run the script on system/nfc
Change-Id: I9e130b8eb79e400282233fad7da206b4bb1301af
Signed-off-by: Ruchi Kandoi <kandoiruchi@google.com>

Fix an inappropriate sscanf return value judgment am: 0e42408b91
am: 764c271c70

Change-Id: I101045cbd010df0d1db24fa1c74949860dd1bc8c

Fix an inappropriate sscanf return value judgment
am: 0e42408b91

Change-Id: Ic3d9b940b9470590da8b7f403066757d0e6e2f91

Fix an inappropriate sscanf return value judgment

This fixes an inappropriate sscanf return value judgment that
res check here is not needed while res2 is the one that need to be checked

Test: mm -j 8
Change-Id: I3e332286c9434d5b9d01421f5f421039f84ad6c7

Make HID Device service independent from HID Host. am: 2a399b3745
am: 45a3078fe9

Change-Id: Idfec7365e8e340b5c1dc2a1f9fd39d7449f6448a

Make HID Device service independent from HID Host.
am: 2a399b3745

Change-Id: I21033a763e8103d35ff44a0a6d503935e504b12f

Make HID Device service independent from HID Host.

By default, enabling the HID Device service is only possible
during disabling the HID Host service. If, on the other hand,
the HID Host service was never initialized, then a request to
enable the HID Device service should just be executed
immediately.

Test: manual, on a device without the HID Host profile.
Change-Id: I2ab4b8aec413b66d57eb322d38660f1b4c48ba32

Merge "Allow HID Device to connect to a specific device" am: 7bd36a3588
am: eb3b067176

Change-Id: Ibe54b32b104d2deff108d84fd4e169951427df00

Merge "Allow HID Device to connect to a specific device"
am: 7bd36a3588

Change-Id: Ib9155090374c80102ee9874b4c692955ce1b60de

Merge "Allow HID Device to connect to a specific device"