Use attribute handle instead of id in GATT cache

Currently we generate an id for each GATT attribute. Those ids
are used to differentiate between attributes that have the same
UUID. We need to get rid of those ids because it won't work with
partial GATT database updates due to "service change"
notifications. Instead, attribute handles, which are unique
across the whole GATT database will be used.

This is first patch in series to achieve that. It gets rid
of id generation, and just assigns handle in place of ids inside
of gatt cache. Upcoming patches will change the GATT cache API
and eventually get rid of "id" completly.

Bug: 27455533
Change-Id: I74c10872ecc223329b8ac7fef4085963bf272624

HID multi-device refactoring

We don't really support multiple HID services in same device.
We shouldn't use default value 0 for service id.
We shouldn't merge service id and characteristic id into one field,
as it's done not right right now.

Bug: 27455533
Change-Id: I46cc1b62bb2c7393e62f3b1a8e4650706954a066

SDP: Restrict attribute length to SDP_MAX_ATTR_LEN

Use case:
Bluetooth crash during the SNS testing.

Root Cause:
In sdpu_build_partial_attrib_entry there is no length check, it can
exceed the maximum allowable value and lead to a crash in memcpy.

Fix:
Restrict remaining attribute length to SDP_MAX_ATTR_LEN
in sdpu_build_partial_attrib_entry.

Change-Id: I5bde6868b82e50b3c09cf94277dabb4862f0a2a8

bluetooth-cli: add support for --exec
am: 5d80cb938d

* commit '5d80cb938d7fb3312870a047749cc839ba5bafca':
  bluetooth-cli: add support for --exec

bluetooth-cli: add support for --exec

Adds the --exec and -e arguments, which execute commands when the
cli starts.

This also adds support for chaining commands at the command line by
separating them with ';' characters.

Change-Id: Ic2bf8a908fbb7fb4cf775810754571fa96d75e87

GATT cache storage refactoring
am: c10b209f62

* commit 'c10b209f62a52d2a782d4d8c97734b6c142f2567':
  GATT cache storage refactoring

GATT cache storage refactoring

Currently when storing GATT cache into file, we save it piece
after piece, max 10 elements at once. This is not necessary,
we have enough space to just save all at once.
Also the logic doesn't need to be so complicated - no need
to state machine for such simple task.

Bug: 27455533
Change-Id: I82923dbedfb317835e45e13e0ea28151eb524d37

Ignore LCB entries that migth have been released while queued
am: fb2105f348

* commit 'fb2105f3488e48f2d908b13a33f47e34f6e3f78d':
  Ignore LCB entries that migth have been released while queued

Ignore LCB entries that migth have been released while queued

Function btm_send_pending_direct_conn() dequeues tL2C_LCB
entries from the conn_pending_q queue and calls the processing
function.
However, there is the possilility of a race condition
and releasing the entry by l2cu_release_lcb() before it is processed.
Now tL2C_LCB that are not in_use are ignored.

Bug: 27418154
Change-Id: I4b5fda605a60c8aebbf5587bb12dd74ac5d28f68

Update device type in NVRAM correctly during inquiry
am: ab7418f58c

* commit 'ab7418f58c84216d57b7c0d553f09f5748badd82':
  Update device type in NVRAM correctly during inquiry

Update device type in NVRAM correctly during inquiry

If the NVRAM device type for a device is not dual mode and the inquiry
result is received for BLE or BR/EDR transport, then this patch updates
the NVRAM device type to dual mode.

Change-Id: I688e7c5e3bd8d844288d095cb97c2e55ddf652c6

Default missing DevType and DevClass in NVRAM.
am: 3b6ef54cd2

* commit '3b6ef54cd262da6a41a16dafe361040740024717':
  Default missing DevType and DevClass in NVRAM.

GATT: Initiate L2CAP disconnect only when the channel is open
am: bf955cb7e5

* commit 'bf955cb7e50e1047452e741effa504c8aefc8a4f':
  GATT: Initiate L2CAP disconnect only when the channel is open

Default missing DevType and DevClass in NVRAM.

Missing DevType and DevClass in NVRAM were not properly set to a
default values previously.

GATT: Initiate L2CAP disconnect only when the channel is open

Use case:
Crash found during L2CAP disconnect from faulty BLE APP.

Steps:
Run SNS testing.

Failure:
Crash during L2CAP disc.

Root cause:
Faulty BLE application triggering GATT disconnection
which internally triggering L2CAP disconnection without checking
proper state of channel.

Fix:
- Initiate L2CAP disconnect only when the channel is open
- Validate LCB during disconnection from upper layers

Change-Id: Ic9d065f095feba659c861828732647e968c20db8

Remove btla-specific comments
am: 1be7ae1a08

* commit '1be7ae1a0890380b8a3daebcade0c1f55b04b167':
  Remove btla-specific comments

Remove btla-specific comments

Change-Id: I91a19b2cfbc96d7567e0e0914058619ea364c1d4

Correcting sniff behaviour in multi-connection scenario
am: 31d1a2b525

* commit '31d1a2b52507df33f13d19ed2349585dfa490542':
  Correcting sniff behaviour in multi-connection scenario

Correcting sniff behaviour in multi-connection scenario

Use case: When multiple profiles are connected on same link and if any one of the
profiles deregisters with powermanager, powermanager will initiate a sniff when it
removes the timer for one profile, even if a timeout did not trigger for other profiles.

Steps:
1. Turn on BT
2. Pair with Mi band (Wearable device)
3. Pair with carkit
4. Turn off BT
5. Turn on BT
HF and A2DP connection should successful after BT on to carkit.

Failure: HF connection fails.

Root Cause: powermanager initiates a sniff when it removes the timer for one profile
Even if a timeout did not trigger for other profiles.

Fix: When multiple profiles connect on the same link and if any one profile
deregisters with powermanager, pm should stop the timer for that profile
and restart the other profiles timers.

Change-Id: I8e215a9d868291976c70ff32b61145b4360a7f8f

Cleanup L2CAP socket properly on close
am: 9123096530

* commit '91230965304bc323a96ced0527d560aacb606415':
  Cleanup L2CAP socket properly on close

Cleanup L2CAP socket properly on close

Use case:
1. Sometimes pairing pop up occurs during Android beam
   transfer though it is not required.
2. No OPP Tx request over OBEX if we cancel during file transfer.

Precondition
1. WIFI ON and conneted with AP.

Steps:
1. Pair and connect from DUT to remote.
2. Go to gallery and send a file.
3. Accept same file and stop it from remote during transfer.
4. Repeat step 2-4 10 times.

Failure:
OPP Tx should not fail.

Root cause:
Close request was not properly handled from BT-IF layer.

Fix:
1. Handled the close request properly from BT-IF layer
   to close the socket variables and instruct the BTA layer
   close from there as well.
2. Clean up L2CAP socket properly on closure to free
   the entry registered with security manager to avoid
   leak in security manager records.

Change-Id: I4f5d6ac92317bcaee5f9285ba71d13eea84c737c

GAP Setting remove device pending status as FALSE
am: 9412bbaea5

* commit '9412bbaea51f577d903e060b8c8214fbd28b0b4f':
  GAP Setting remove device pending status as FALSE

GAP Setting remove device pending status as FALSE

Use case: DUT deletes Link key and failed to initiate
  authentication request after remote headset turned Off and ON

steps:
- Pair the DUT and HS. (Link Keys get established.)
- Now Switch OFF BT on DUT. (DUT still has the Link key of HS)
- Reset the paired devices memory in HS and keep HS in
  pairing mode. (HS loses DUT's link key)
- Now switch ON BT on DUT. (DUT starts reconnection with HS,
  but as HS lost DUT's key, PIN or KEY missing event will
  come to host)
- DUT reconnects to HS (Using newly established link keys)
- Now power OFF and ON the HS.
- HS will try reconnection with DUT.
- While there is an incoming connection from HS to DUT,
  host is giving negative link key reply always for the
  multiple link key requests from HS.
- HS sends disconnection with "Authentication failure"

Failure: DUT should not delete Link key and DUT should
  initiate Authenticatin request.

Root cause: pending status was not FALSE from security device
  DB, which will causing Authentication Failure.

Fix: Setting the remove device pending status to FALSE from
  security device DB, when the link key notification event
  comes. Basically it will avoid deleting the device from
  security device DB, Which is solving the reconnection
  initated from remote when we remove the link key at
  remote side.

Change-Id: Ic164d8d5b5c2e0b9cc5f04f993047fb0a8e5d9a9

Track TX empty event to manage sniff timer
am: 4e972d8c24

* commit '4e972d8c24ecfb808757ef5c81eb3877ebc93027':
  Track TX empty event to manage sniff timer

Track TX empty event to manage sniff timer

Use case:
OPP TX while inquiry is in progress.

Steps:
1. Send any file from DUT to remote using BT.
2. Accept file on remote.
3. During file transfer, start inquiry on DUT and
   observe behaviour.

Failure:
DUT sends sniff command in the middle of the transfer
though it is not required. eventually OPP Tx over
L2CAP file transfer fail.

Root Cause:
As per the current implementation, for tx it will
immediately starts idle timer without checking
for the completion status of tx.

Fix:
Handled the tx complete event from L2CAP properly
to start the idle timer to manage the sniff properly.

Change-Id: I298075590042e82a2f33837f6df6af0b2fd8179a

Read local supported codecs as part of boot sequence
am: f6dcf79f62

* commit 'f6dcf79f62880e716e127b19c70ea328e07c6051':
  Read local supported codecs as part of boot sequence

Read local supported codecs as part of boot sequence

Add support to read local supported codecs as part of
controller module initialization.

Also added an API from controller interface to get the
local supported codecs.

Change-Id: I37a4ab9e6a20ed057ca794dbdd4f99c2a8c65a6e

Added fd for monitoring when the channel is not congested
am: 7baa1e1792

* commit '7baa1e17924cdac702ab9d8146a966fbd26c8d50':
  Added fd for monitoring when the channel is not congested

Added fd for monitoring when the channel is not congested

Use case:
Skips and Media Audio Loss are Observed During PAN Connection

Steps:
1. Launched Local Music and Media Audio was in Headset.
2. Paired and Connected to other Phone to DUT Over BT Tethering.
3. While  Listening to Media Audio in DUT, Started Watching
   Youtube in other phone.
4. Observed Below Observations During Some 30mins of PAN Connection.

Failure:
Skips and Media Audio Loss are Observed During PAN Connection

Root Cause:
socket monitor thread keeps polling for PAN fd, eventually
it will make the BTU thread also busy, inturn NOCP processing
gets delayed.

Fix:
Add PAN fd for monitoring only when L2CAP channel related to
PAN profile is not congested. It will break loop by checking the
congestion status to put the fd for polling.

Change-Id: I2119c6c86d8e4798e516c512ad7aac7630b85a6a

Merge "Shorten local name in EIR data according to UTF encoding rules"
am: a9644b77b7

* commit 'a9644b77b7959fef0b58410230930317575d0668':
  Shorten local name in EIR data according to UTF encoding rules

Merge "Shorten local name in EIR data according to UTF encoding rules"

HID: Handle collisions during incoming and outgoing connections
am: 891723a428

* commit '891723a42851285ec7e6c2f616a405611789da6b':
  HID: Handle collisions during incoming and outgoing connections

HID: Handle collisions during incoming and outgoing connections

- This patch handles cases when incoming HID connection is in progress, and
  there is an outgoing connection attempt from upper layers. This was leading
  to a case when ougoing connection would fail as connection was already in
  progress, which triggered cleanup of connection states, which was not proper
  leading to issues in future connection attempts, as disconnection during
  ongoing connection was not handled properly in existing code.

- Reset the incoming flags during open failure to prevent any possible issues
  during further outgoing connections.

Change-Id: I6fb0c6156482d2da41ca0d2d280091d73c27f292

HID: Reset outgoing connection retry to 1
am: b7d4613b3c

* commit 'b7d4613b3c1fae825b4c7ba94907c75513feb1c7':
  HID: Reset outgoing connection retry to 1

HID: Reset outgoing connection retry to 1

Reduce the retry time from host to 1 from 3 to prevent collision scenarios
when remote also tries to connect which could lead to potential authentication
issues in SoC.

Change-Id: I3b5db4a4c4aa985cb9c0537499ce608f9e5d5522

HID: Remove bond for device in case Virtual unplug is ongoing
am: dc5272a39d

* commit 'dc5272a39d7915d0c921de2fb0727853fadda40a':
  HID: Remove bond for device in case Virtual unplug is ongoing

HID: Remove bond for device in case Virtual unplug is ongoing

Remove bond for remote device in case virtual unplug was initiated from
local device, and ACL got disconnected while processing the VUP request.
this can happen when aaplication is sending disconnect and remove bond
in quick succession. Without this fix it was observed that device was
getting disconnected, but was not getting unpaired even when user
tried to unpair from UI.

Change-Id: Ife1fc99ed59b13daf0bcf0dce3e0a59c7d9e87d7

Shorten local name in EIR data according to UTF encoding rules

Use case:
Test short name after setting the name with maximum length

Steps to reproduce:
1) Phone A: Rename bluetooth into a 60 character Chinese name
2) Phone B: Discover phone A.
3) The short name of phone A will display in phone B and the short
name cantains unrecognizable chars.

Root cause:
As Chinese characters are encoded in different
length in UTF, if shorten BT name in fixed length,
there will be unrecognized characters

Fix:
Shorten BT name according to UTF encoding rule and decrease
the length.

Change-Id: I5f1e49c1b5d87d5e220538c90e297c50ccc790ff

Use correct transport type for GATT client and server connections
am: d27af38849

* commit 'd27af388490dbe194035ad779096f98aef92d1a5':
  Use correct transport type for GATT client and server connections

Use correct transport type for GATT client and server connections

Use correct transport type for GATT client and server connect
API calls. This change uses the transport used while calling
the connect frameworks API instead of relying on the NVRAM to
find the transport on which the connect call needs to be sent
to the BT stack.

Change-Id: Idf5817b8980e848c6ebebf1e9a79dbce61d8cfc8

Update OCF values per 0.96 spec
am: ca58b09a91

* commit 'ca58b09a9129e334f116b7bbdfc0a3834afa299e':
  Update OCF values per 0.96 spec

Update OCF values per 0.96 spec

HCI command values are updated as per Google HCI
BLE requirements v0.96

Bug: 27069512
Change-Id: I30e0046c5d3198f0986e03baf697bad2be99c06e

OCF for LE Extended Set Scan Parameters Cmd is not correct
am: b67ada19a7

* commit 'b67ada19a7235552c802d3dccca011245379b6d5':
  OCF for LE Extended Set Scan Parameters Cmd is not correct

OCF for LE Extended Set Scan Parameters Cmd is not correct

OCF for LE Extended Set Scan Paramaters Cmd is defined as 0x15A
in documentation but it is defined as 0x160 in stack.
This results in command failing due to unknown hci command error.

Change-Id: Ia779164b7c03daa8053fb65e4fc8281235904031

Handle unknown codecs sent by HF
am: dccd759e2e

* commit 'dccd759e2ebb0c348d739e32599e50b419bcf7bc':
  Handle unknown codecs sent by HF

Handle unknown codecs sent by HF

Use case:
1. Connect to headset supporting CVSD, mSBC and other codecs.
2. Verify if headset sending AT+BAC with supported codecs.
3. Check if SCO is established with mSBC codec.

Failure:
SCO is established with CVSD codec.

Root cause:
When parsing AT+BAC command, if unknown codec is received,
we are falling back to CVSD although mSBC is supported in HF.

Fix:
AT+BAC parsing function modified to return correct codecs
supported by HF when it encounters unknown codecs in arguments
of AT+BAC.

Change-Id: I9ac7675c9d863e7362f85e3b7b8d9a42462bd5e9

Avoid double-free on SDP search failure
am: 07f612cbc0

* commit '07f612cbc08364a622e6e02cdbb039b82586d82c':
  Avoid double-free on SDP search failure

Avoid double-free on SDP search failure

Use case: BT transfer fails in Doze mode.

Pre-condition:
1. Device is in Doze mode.
2. Device is in Active state.

Steps to reproduce:
1. On test device and support device, turn on Bluetooth and keep test device in Bluetooth on screen.
2. Put device in Doze mode:
adb shell dumpsys battery unplug
adb shell dumpsys deviceidle step
3. From support device send pair request to test device.
4. Pair two devices.
5. Put test device in Doze mode again.
6. Share one picture from support device to test device by Bluetooth.
7. Accept Bluetooth file transferring request.

Failure: From DUT, BT transfer is failed.

Root cause: Free buffer is called twice for the SDP search failure case.

Fix: Added null check for p_sdp_db to avoid freeing same buffer
     again on SDP search failure.

Change-Id: Ic7776e842c52418c2b0fb6d6479233d01c709dc2

Remove bluedroidtest
am: cd0199a20b

* commit 'cd0199a20b48fcb191f3178a28a6a503e28b7685':
  Remove bluedroidtest

Remove bluedroidtest

Use bluetooth-cli/bluetoothtbd instead.

Bug: 25369253
Change-Id: I485919dce4bf3229733c85e6a33c754b9cbd62fc

Delete unused function declarations from sdp_api.h.
am: 5c35aa44a2

* commit '5c35aa44a2d92185d9eee51781f2522773607986':
  Delete unused function declarations from sdp_api.h.

Delete unused function declarations from sdp_api.h.

Change-Id: I8db02d557fdfc138f2ec8de2b2d45049175fd999

Avoid SMP process if another pairing is in progress
am: 0bb891dc54

* commit '0bb891dc54b64e6cc4f774f565c614dfaff05b3a':
  Avoid SMP process if another pairing is in progress

Remove MCB from mapping table when entry is released
am: 5170c746e5

* commit '5170c746e5b1ac033de0555760be96d68b139695':
  Remove MCB from mapping table when entry is released

Remove dead methods and callbacks
am: ee6b6d0965

* commit 'ee6b6d096541cc0f8333ad93cdbbf45e221396c4':
  Remove dead methods and callbacks

Avoid SMP process if another pairing is in progress

This change prevents the incoming pairing/security request
from a remote device when pairing with another remote has
been initiated by the host and has not yet begun.
The pairing request by the second remote will be rejected

Bug: 22525598
Change-Id: I676c85af39d5418c699161ad11eee5951f9203d7

Remove MCB from mapping table when entry is released

Explicitly remove MCB entry from the LCB mapping table when
the entry is released.

Bug: 27334916
Change-Id: I14e3f1f5eeffa33e733c4a116d0a70cd7c06753f

Remove dead methods and callbacks

Bug: 27455533
Change-Id: Ie5661577320810468dd635022027a1e1a2f70a3e

Simplify UUID handling in GATT cache
am: be18a60781

* commit 'be18a6078194547400fa9594a62d9d1bea5d7bea':
  Simplify UUID handling in GATT cache

Simplify UUID handling in GATT cache

Bug: 27455533
Change-Id: I653b563d3b3c233ffe31b0a35d396a66db363459

Increase the number of security slots for other new profiles
am: 9fed745a38

* commit '9fed745a38b1ff2c5d4d67ca06760da5b07b7f11':
  Increase the number of security slots for other new profiles

Correct the max SDU length used in BTIF layer
am: f19ca75c17

* commit 'f19ca75c17b8ee0c19261d946c5353c820238bc6':
  Correct the max SDU length used in BTIF layer

Increase the number of security slots for other new profiles

Use case: MAP Sanity Test fail in multi-profile scenario.

Steps:
1. Connect to Instance 1   - Email Sanity Test Case
2. Connect to Instance 0&1 - Email & SMS Sanity Test Case

Failure: MAP sanity fail in multi-profile scenario.

Root Cause: some profile coonnection may fail in multi
profile scenario due to non availability of security id.

Fix: Increased the security slots to 75 to accomadate new
profiles introduced.

Change-Id: Ic7c0268aea654cba2dee0c323ca0342e71ebc577

Correct the max SDU length used in BTIF layer

Use case: OPP Tx over L2CAP are failing due to invalid PUT packet from DUT.

Steps:
1. Make the Remote device setup with OPP server which supports MTU
   of more than 8k.
2. Send file from DUT to Remote device.
3. Transfer always fails.

Failure: OPP Tx over L2CAP file transfer fail. It will always fail
if remote supports more than 8076 as the MTU.

Root Cause: Max SDU of L2CAP layer will be communicated to the upper
layers, and same length of SDU being sent from upper layer
to BTIF layer, but because of change in the max SDU of
BTIF, it will receive less bytes than expected will lead
to form incomplete PUT request.

Fix: Correct the max SDU length used from BTIF layer to be in
sync with max SDU calculation from L2CAP layer.

Change-Id: Ie037c85e798bebf6a71c56488aae37455725028c

Merge "Reconstruct new packet to accomodate extra header length"
am: ab6a1335bf

* commit 'ab6a1335bf15d722b6fb420641b0b0ccc0f80d13':
  Reconstruct new packet to accomodate extra header length

Merge "Reconstruct new packet to accomodate extra header length"

Enable debug logs for bluetooth process threads, modules
am: 0fa3554630

* commit '0fa35546304901c62bafcb54420d23ace56a69c1':
  Enable debug logs for bluetooth process threads, modules

Reconstruct new packet to accomodate extra header length

Use case: Bluetooth process crash problem in the middle of RFCOMM
test cases due to buffer corruption detected while freeing it.

Root Cause: RFCOMM layer trying to reuse the same packet to send
to the lower layer but it will fail due to strick buffer
corruption checks.

Failure: Bluetooth process crash.

Fix:  created a new packet to accomadate new header
and copied the same payload to the new packet.

Change-Id: I5d12b76229d11480f7f37bbcd3561de97c80a0c1

Corrected the stream for error checking
am: a73cf49473

* commit 'a73cf4947355e38d2f44f2f43e87e32a6ad83fd5':
  Corrected the stream for error checking

Revert "DO NOT MERGE Proper construction of AVDTP general reject message"
am: ebc951060a  -s ours

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

Enable debug logs for bluetooth process threads, modules

Use case: Debug enhancement for bluetooth threads,
          modules

- Most of the bluetooth process threads,modules uses
  APIs provided from the OSI layer.

- This patch enables the debug logs to know when the
  thread, module is created and exited.

- This would be useful while debugging the ON/OFF,
  ANR issues.

Change-Id: I17f4f583d2c431725a8c44c586b29980b4bdab3f

Corrected the stream for error checking

Replace event_stream with acl_stream

Change-Id: I9ceb75ee05bad5d2ef1af819b3930216cc750507

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

Change-Id: Idfc960ce00662e8aacc26f3b3fdca9deca8a2aff

Peer support of Absolute Volume to be updated only if device supports A2DP

Peer support of absolute Volume feature should be updated to Audio
frameworks only for device which supports A2DP Sink and AVRCP Absolute
Volume TG roles together.
Audio manager need not be updated for AVRCP only peer devices as
that would cause sudden rise in ongoing stream volume when avrcp only
device connects, if it is set to some lower value with other device
connected for A2DP.

Change-Id: I75509d0b0fd1b06fa08390940a327dd2887d7562

Limit Max Bitpool value to SPEC recommended one

Preferred bipool for 44.1 KHz sampling rate for SBC
codec is considered as 53. Hence limit Max Bitpool
value to 53 in setconfig request to have it same as
the one DUT sets in getConfig response and in encoding
pcm packets.
few BMW series of CKs does not connect unless DUT
configures max bitpool in setconfig request to 53.

Change-Id: Ie7ca504c23f6bae8fda997cdeb3e0ff0733defb0

Allow Carkit to trigger play immediately after call

Removing check in stack where AVRCP Play and Pause commands
are being dropped if carkit initiates play or pause within
six seconds.
This addresses the scenario where JCI-Mazda CK pauses the
streaming by initiating Avrcp Pause right before making call
and can not resume the streaming by sending Avrcp Play right
after call ends.

Change-Id: Ibb04223de746f528b356e78bd65130d3fd526a44

Initialize sco codecs supported with CVSD

Usecase:
1. Connect headset/carkit supporting codec negotiation
   but does not send AT+BAC with available codecs.
2. Make a call and create SCO connection.
3. Transfer audio to AG from headset/carkit.
4. Transfer audio back to carkit.

Failure:
AG will not establish SCO connection with remote.

Rootcause:
We initialize codecs supported by remote as 0 and update
it with codecs received as part of AT+BAC. When AG sends
+BCS to start codec negotiation, it received AT+BCS=1 from
remote. Since 1 is not in the list of remote supported codecs
(0), AG will not create SCO connection in step#4.

Fix:
Initialize sco codecs supported by remote with mandatory
codec, CVSD.

Change-Id: Ibe0e1a7bd3e59e8618568d320acfbde88f3fc11e

Ignore empty optional parameters in CLCC response

Usecase:
Pair and connect with AG that sends the optional parameters
in CLCC response.

Failure:
When optional parameters in CLCC response from AG is empty,
HF client is not parsing the empty optional parameters.
User may not be see the phone number on UI properly on
hf client device.

Root cause:
AGs like iPhone sends optional parameters in the CLCC response.
HF client is returning NULL when it encounters empty optional parameters.

Fix:
Ignore empty optional parameters in CLCC Response from AG.

Change-Id: Ia90986ccc4c9c9f1a6fb3a914bc1df8b0a75ba0f

Clear remote BD address if SDP search fails

Use case:
1. Enable multi hf.
2. Pair and connect to a headset.
3. Turn off headset and try connecting to headset from AG.
4. After the connection fails, turn on headset.
5. Make sure headset initiates HF connection to AG.
6. Make MO/MT call, switch from headset to AG, vice versa using headset.

Failure:
When call is transferred from AG to headset, headset initiated
SCO connection request is rejected by AG.

Root cause:
For outgoing connection in step 3, SDP search fails. BD address of
remote headset is stored in scb[0]. BD address of headset with which
we tried to connect is not cleared in scb[0].

When incoming HF connection request is accepted in step 5, BD address
of headset is stored in scb[1]. Now, BD address of the same headset is
stored in both scb[0], scb[1].

When SCO connection request is received from headset, we get scb
information using BD address. However, since BD address in scb[0]
is not cleared, we end up selecting scb[0] for which there is no
headset connected. This makes us think that remote is initiating
SCO connection without SLC and we reject the SCO connection.

Fix:
Clear the BD address of remote when SDP search fails.

Change-Id: I21a0b83fcd47fed91e28fe27bdfd6d541b6a527c

Do not reset Sniff Subrating parameters on SCO link

Usecase:
1. Pair and connect to BT headset
2. Pair and connect to a watch
3. Pair and connect a LE device
4. Make a call and reset BT on phone
5. Verify devices connect back and SCO is established

Failure:
After BT reset, devices connect and SCO is up. After some time,
SCO and ACL link disconnects due to connection timeout.

Rootcause:
When Headset is connected during active call, HFP is connected
first. Due to active call, SCO is also established. When SCO is
established, we disable sniff subrating(SSR). When A2DP is
connected after SCO connection, it enables SSR. This enabling
of SSR is causing ACL disconnect.

Fix:
When there is new profile connection for the same device which
has SCO, we should not enable SSR. It should be enabled only
when there is no SCO.

Change-Id: I7815dba90851366ecc0523447c2394631c10ede1

Update the sniff policy when SCO is active

Use case:
1. Connect BT headset.
2. Make an outgoing call or receive a call on BT headset.
3. Check if sniff parameters are taking affect when SCO is active.

Failure:
Sniff parameters when SCO is active are not taking affect.

Root cause:
SCO link was not checked when updating power manager(PM)
about idle or busy states.

Fix:
During AG tx/rx activity, update PM based on SCO state.

Change-Id: I4c09406395998bacf13471a273402a60f3b77f89

Revert "Remove unused field p_cur_char"

This reverts commit 25e17fb1c7a894dec49fc8eddb2bbfb20a9eeb21.

Change-Id: I21936ae37c1e052ab53233b3465e336369a423df

Remove unused field p_cur_char

Change-Id: I956f966ddbb44506a5da8e1ab4ccb028bafa92de

Fix miscellaneous compilation errors

- Move variables within #defines to avoid unused errors.
- Place #define's correctly to avoid unused function errors.
- Correct parameter passed into a function.

Bug: 27336841
Change-Id: I7c2ab24b3bd3725d126c90483473af83b88687a2

Fix l2cu_initialize_fixed_ccb() to return only initialized entries

Update the l2cu_initialize_fixed_ccb() implementation so
it returns only entries that are properly initialized.
In some complex scenarios, the p_lcb->p_fixed_ccbs[] array
might contain an entry that has been released by l2cu_release_ccb(),
and such entry cannot be reused as-is.

Bug: 26927945
Change-Id: I42d42926cc6da6b8be662781545753c36b75968e

Fix Dell LE keyboard connection issue

Fixes a connection issue where Dell keyboards do not remain connected
immediately after pairing by removing the gatt_config_ccc_complete()
call in gatt_cl_op_cmpl_cback().

Bug: 27385897
Change-Id: Iee342975dce9fdcd5dcf8f9d459f74fe13d02531

Sync filesystem after saving a backup of the config file.

Bug: 27292025
Change-Id: Ied3f84157f491dcdf416de211ffbdb45b2abd529

Btsnooz doesn't find snooplog sometimes

In the bugreport in b/27274486 comment #11, the line indicating the
beginning of BTSNOOP LOGS was slightly indented thus causing btsnooz
to not find the beginning of the log.

Bug: 27419027
Change-Id: I3b88e6a3f6f2d6c4d3cbbadb0a93cbd7965f56c8

Reset queue pointer to NULL after fixed_queue_free()

Add missing reset of queue pointers to NULL affter
fixed_queue_free() calls.

Bug: 26742590
Change-Id: Ib9035d58c0f98a47ce2935e073c6f288ae03ac92

Ensure inquiry is always stopped before service discovery

Service discovery cannot be initiated when the dm search state machine
is not in idle state. This patch ensures that service discovery is
always cancelled regardless of the current state of inquiry.

Bug: 27368986
Change-Id: Ib83d4c8d389e50a1c65e52b343f4996455747836

Swapped memset params

Incorrect usage of memset

Change-Id: I7d96c560c4b462c2794b63eb66c7272b1280ba23

Fix broken formatting in README.md

Markdown formatter at:
https://android.googlesource.com/platform/system/bt/
interpret some things differently than other formatters.
This patch fixes badly formatted parts.

Change-Id: Ibe8a9c159d9d47396ef073f6555f06720bc3e661

Build on linux: final changes and instructions.

After this patch anyone should be able to build Fluoride on Linux.

Bug: 27324453
Change-Id: I7776be12591b8fed6d38fc0375019606d311ddb9

Move properties get/set code to osi

Currently we use Android-specific property_get and property_set
through the code. This patch adds osi_property_get and osi_property_set
that is encapsulating all external dependencies. This is step further
for errorless compilation on linux.

Bug: 27324453
Change-Id: I97c34344617a5855b683328bcd196a21a2fae6f9

Fix a crash triggered when firmware config takes too long

Fix a crash condition that is triggered when the firmware
configuration takes too long:
 1. The startup_timer expires, and it resets startup_future to NULL
 2. The delayed firmware_config_callback is received, and it
    tries to use the startup_future pointer (already set to NULL).

Bug: 27336555
Change-Id: I4b04ca08a32c947f6f1eaabec7c4b099f96aab59

Add missing file to BUILD.gn

This patch adds missing osi_module.c file. Without it Fluoride
on Linux compiles, but fails to run.

Bug: 27324453
Change-Id: I08eb6bc55149d1b70580643fdf131b197c027caf

Merge "Update build files for Fluoride on Linux"

Alarm Dump output cleanup

Before:
  Alarm : btm_ble.adv_raddr_timer (SINGLE)
    Action counts (sched/resched/exec/cancel)       : 1 / 0 / 0 / 0
    Deviation counts (overdue/premature)            : 0 / 0
    Time in ms (since creation/interval/remaining)  : 17824 / 900000 / 882176
    Callback execution time in ms (total/max/ave)   : 0 / 0 / 0
    Overdue scheduling time in ms (total/max/ave)   : 0 / 0 / 0
    Premature scheduling time in ms (total/max/ave)   : 0 / 0 / 0

After:
  Alarm : btm_ble.adv_raddr_timer (SINGLE)
    Action counts (sched/resched/exec/cancel       : 1 / 0 / 0 / 0
    Deviation counts (overdue/premature)           : 0 / 0
    Time in ms (since creation/interval/remaining) : 88268 / 900000 / 811732
    Callback execution time in ms (total/max/avg)  : 0 / 0 / 0
    Overdue scheduling time in ms (total/max/avg)  : 0 / 0 / 0
    Premature scheduling time in ms (total/max/avg): 0 / 0 / 0

Bug: 27294154
Change-Id: I0e318ef570a591626fbc6443feace1cc99c06baf

Merge "Clean A2DP dump output"

Remove extra newline before Btsnoop log summary

Bug: 27294154
Change-Id: Ice69835cdddfc01e2222a47b0ceeb30d243442f5