native crash while skipping songs with A2DP connected

this is caused by out of bound(invalid) index, pass to AVRC_MsgReq

bug:10787111
Change-Id: I5b06c646342d0d4e9202c6e22e1c1d67275efb43

Set default a2dp sbc encoding quality to high

* Increased bitpool to high quality setting (53). * Added rate quality
reduction to medium quality for basic rate connections * Enhanced
a2dp frame calculation ratio to smooth out frame distrubution * Removed
iop scaling patch.

Bug 8252054

Change-Id: I20725d0decfe5f820e1c03407889b6272e830aca

Merge "fix wrong PBAP profile SDP record which may cause some carkit interop problem." into klp-dev

Merge "a2dp media packet sent too slow, which cause packet overflow." into klp-dev

a2dp media packet sent too slow, which cause packet overflow.

the error message: btif_media_aa_prep_2_send congestion buf count 24
The reason is we increase all bluedroid thread related TX data path priority:
media, timer, BTU and HCI_WORKER, but we didn't increase the RX data path
priority: userial_read_thread. If the RX thread is blocked, it will also
affect TX, because the HCI have flow control, we need receive HCI Number of
 Completed Packets Events to continue send packet to the controller.
If we HCI complete Packets Event packets is blocked, the HCI flow control
will block the HCI packet trnasmission until we get HCI complete packets
event to know how many buffer is available in the controller.
I tested with a chrysler CIP/VP4 carkit which show this error message,
After I apply this fix, I can't reproduce this problem any more.
This look like good fix for this issue.

bug:8252054
Change-Id: Ic78b9abb69648c50aa63e2b3c8b90038018fb6b5

fix wrong PBAP profile SDP record which may cause some carkit interop problem.

bug:10488422
Change-Id: I8fb4a73a631c84dc812f44593743905f2d2ad108

Merge "Added lock in btu_xxx_timer api" into klp-dev

Merge "Changed MAP version from 1.0 to 1.1, since this is what was actually implemented" into klp-dev

Added lock in btu_xxx_timer api

Bug 10539101: Native crash in com.android.bluetooth while running monkey

Merge "change AVRCP profile version from 1.4 to 1.3 in SDP record." into klp-dev

change AVRCP profile version from 1.4 to 1.3 in SDP record.

bug:10728403
Change-Id: I741c8741173088bc91f43fbbcda8c0bd13a7d84d

Use product model as default bluetooth name if not defined

bug 7441329

Change-Id: I6afe54a8a0c4d1120e96eb808cf9fc4bf5b37fdb

Changed MAP version from 1.0 to 1.1, since this is what was actually implemented

Change-Id: If391121ca92f1025bff3a99941afa12da69c3b69
Bug: 10434562

Merge "Configure BTA DM to not force master role upon inquiry" into klp-dev

Configure BTA DM to not force master role upon inquiry

* Some headsets will request master role upon connection. When inquiry
is attempted during streaming this results in a small audio glitch
while attempting to switch back to master.

Bug: 8266826
Bug: 8569352

Change-Id: Iaca0bc0f8847e32bc2d40e33100f26ce19d79a28

Merge "Hid Host: Ensure Hid module is accepting events before processing" into klp-dev

Hid Host: Ensure Hid module is accepting events before processing

During shutdown sequence, Hid host module state machine was crashing
trying to process events in NULL state. Added the appropriate check.
bug 10429484
Change-Id: Ic8b8f61f2fa63027d3e473787cca6d4d221ec075

Incremented buffer size to accomodate default MTU

L2CAP Config request was not sent from Bluedroid as the MTU size was
higher than the max buffer size for default apdu tx/rx sizes.
Incremented the buffer size to accomodate this default MTU size. Also
included new datatypes and corresponding tx/rx apdus. This will set the
MTU size to that defined in datatype config rather than selecting
default value.
b/10565872
Change-Id: Id469c836581fb3d020bcb88c381421e0ef965073

Add support for AbsoluteVolume

Add support for AbsoluteVolume in BTIF
bug 9595622

Change-Id: I11b707f6ab50d31bd11e879b0ebe5ad5bac54ad9

Conflicts:
btif/src/btif_rc.c

Place handling of BTA_DM_DISC_BLE_RES_EVT in BLE feature ifdef

Without this fix, bluetooth fail to compile for non-BLE platform

Change-Id: Ida10c54ab841e0a2bcaeb389a9813dd846db8200

Merge "LE: Add support for the HID-over-GATT profile (1/3)" into klp-dev

LE: Add support for the HID-over-GATT profile (1/3)

bug:8330048
Change-Id: I5727161b0a87700487bee96cdffa8fd169034204

Add support for ff/rew

Change-Id: I017a7ddc9f7255d84ad8bf3284fefc1b7040900f
(cherry picked from commit 342bdf540d8c183d14ec5fb436b73cd979df3a8f)

Merge "LE: Add peripheral role support (2/4)" into klp-dev

LE: Add peripheral role support (2/4)

Initial stack support for the LE peripheral role.

Change-Id: I261d751b43b7020760bff345b472b6f60caa60be

Merge "LE: Do not generate an error if a device is already whitelisted" into klp-dev

Merge "LE: Fix for back-to-back GATT connection request issues" into klp-dev

Merge "LE: Memory leak when caching GATT service change" into klp-dev

Merge "LE: Check encryption state before starting encryption" into klp-dev

Merge "LE: Add instance ID to descriptors (2/4)" into klp-dev

LE: Do not generate an error if a device is already whitelisted

If a device is added to the whitelist that is already listed, no error
should be generated.

Change-Id: I81ad47eec47157d5ab2b5b25db3ed15bf873794e

LE: Fix for back-to-back GATT connection request issues

When two GATT client connections were sent back to back, the first
connection will not be processed due to an address comparison error.
This patch corrects the error.

Change-Id: I8eb97cd2a34a81da460f448dc71445b2898a5d32

LE: Memory leak when caching GATT service change

When a GATT connection is disconnected, a bonded device will be added
into service change pending list. But if the device is already in the
list, a duplicate record will be added and one buffer is lost.

This change ensures that devices are only added if they are not already
in the list.

Change-Id: Ied6872b145ed28bde8044444cb6c9cbee525943d

LE: Check encryption state before starting encryption

Previously, the btif layer kept track of when a link was encrypted using
an internal list to track encryptino callbacks. This change switches to
querying the stack for the actual encryption state, since the btif layer
may not receive all encryption related callbacks.

Bug: BLTH02382147
Change-Id: I426b87d77e9a18531c2d42733a7e736a73bb116f

Move bta_dm_remove_sec_dev_entry declaration out of BLE ifdef

The method applies to non-BLE cases.

Change-Id: I57508d09015e4ec4a74edf2d1fbc607fd692b402

LE: Add instance ID to descriptors (2/4)

If a remote devices offers multiple descriptors with the same UUID, the
instance ID is used to differentiate between them.

Change-Id: I1f3c284d714719fe9fb9cbb297c70a061c62a96d

LE: UPF 45 bug fixes

This change fixes the following issues:
- Second GATT-over-BR/EDR channel cannot be established when there
  already is an existing GATT-over-BR/EDR channel
- If encryption fails for an LE connection due to a missing key,
  the security state is not being cleared and blocks all further
  security processing
- When DM discovery of an LE Peripheral device fails with a
  connection timeout, no further discovery requests can be made
- GATT service discovery can get into endless loop when duplicate descriptor
  definitions are found on the remote device
- When GATT over BR/EDR fails, BTA does not give a connection
  callback to the application initiating the connection
- BR/EDR connection to remote platform does not generate API callbacks
- Stack crash discovered during UPF after remote disconnects
- The host is sending HCI disconnect to invalid HCI handle when
  SMP fails because of a connection timeout
- Possible race condition:
  If a disconnect is immediately followed by a connection complete,
  the connection complete cannot be processed in the BTA GATT state
  machine
- Write Complete event is not triggered for Prepare Write requests

Change-Id: I539cdedd68007818ff4f0d0213cee1c913f72d0f

Conflicts:
bta/gatt/bta_gatts_act.c

LE: Add NULL check for GATT client callback

Change-Id: I1087102d8a740969fb0a98b409b58230a3236aeb

LE: Fix crash on LE mouse reconnect

Sometimes, a GKI exception may be observed when re-connecting to an LE
mouse. This patch properly handles the encryption state change, avoiding
a possible race condition leading to the exception.

Change-Id: I4cf139c1ac22e71b598272fccebf645cd73fdf6a

LE: Limit minimum MTU size

Check requested MTU sizes to ensure a minimum MTU size. Otherwise remote
devices may request an MTU that's too small, preventing any further
communication with the remote device.

Change-Id: I3ba97cf6e6c464825778a8c8902f99f6662c3762

LE: Send correct device name during SMP pairing

During SMP pairing the device name is not properly copied to the upper
Java layers, causing a potentially wrong device name to be displayed to
the user.

Change-Id: Iab43871defc7df52b0cd9c64f52208879abb50f2

LE: Integrate power table for GATT

No power table exists for GATT, causing the device to never enter sniff
mode while using GATT over BR/EDR.

Change-Id: I0db539c1814a61af620f3dba560da64481291beb

LE: Add GATT disable functions

This patch adds required disable functions to the GATT sub-system to
properly unregister with the stack. Without the disable functions in
place, turning Bluetooth off with a GATT device connected may lead to
unexpected behaviour and cause GATT to fail on sub-sequent stack
restarts.

Change-Id: I7cb80e96109e2c09882991298d0487b506f5ffdd

LE: Fix GKI exception for 0 length values

When a remote device responds to a read request with a 0 length value, a
GKI exception will occur. This patch checks for 0 length values to avoid
the exception and return the correct status to the application
requesting the read.

Change-Id: Idce61c3d2be7072dafe429ab978763b0ee61aed2

LE: Only copy UUID for callback on success

When a GATT client application registers with the stack, the UUID is
always copied into the callback buffer, even if the registration failed.
This patch adds a check to make sure an invalid UUID is not passed back
to the application.

Change-Id: I53ddc7939c096a33a64b00a8902bedba00b61e7d

LE: Add NULL pointer check in deep copy function

Adds a NULL pointer check for the GATT read event deep copy function.

Change-Id: Ia8d31e851589612009e1038c11b7b113e5367c6d

LE: Initialize return status when registering client

Initialize status to BTA_GATT_NO_RESOURCES. If cl_rcb is full, the
wrong status could be returned to the client.

Change-Id: I45d147242c9f24c7636671ffca2f9429c175b8cf

LE: Deep copy buffers when transfering client context

Certain BTA client event types require a deep copy of the request
data buffers when transfering context. Shallow copy of the pointers
involved may cause a crash when overlapping read and write requests
are received.

Change-Id: Id59b034d7fdd28087695a0688caff66b4cef4a4d

Fixed issue that failed to update rfc credit to peer when host can not handle the incoming packets fast enough

Bug 10233699: Bluetooth sockets block indefinitely on read(...)

Bluetooth MAP profile - sms and mms support initial check-in

bug:10116530

Change-Id: I3bff487fdc0ee1256afa8d704a2cfa788081208c

Support for 128-bit UUID service discovery

Support the discovery of custom 128-bit UUIDs. This is mainly needed for
GATT service discovery to find custom services.

Bug: 10116729
Change-Id: I8669111dfb8806f5632b7ea8ad02f0e38dded66f

Fix check for BTIF_HL_CCH_OP_MDEP_FILTERING in btif_hl_upstreams_evt

Change-Id: If080fad6ba4b42944dc5ee8bf1d8f0586dbd05f8

Merge "Add debug menu to enable btsnoop"

am 4a5fc696: (-s ours) Reconcile with jb-mr2-zeroday-release - do not merge

* commit '4a5fc6964313a606abd89268b8a53036b14b4ad7':

Reconcile with jb-mr2-zeroday-release - do not merge

Change-Id: Ib88a72e7473f38fd079a12658875dec71352c19a

merge in jb-mr2-zeroday-release history after reset to jb-mr2-dev

am 2e1880d4: am 98657a36: LE: Release transmit buffer credits

* commit '2e1880d46eeb8fe1a53ac9e387a07ac802491ad7':
  LE: Release transmit buffer credits

am 98657a36: LE: Release transmit buffer credits

* commit '98657a365c7230b84a005c63ce5bcf036d69f6eb':
  LE: Release transmit buffer credits

LE: Release transmit buffer credits

When an ACL link goes down, some controllers do not send a "number of
completed packets" update for the last ACL packets sent from the host to
the controller. This could result in incorrect L2CAP transmit window
calculations and no transmit credit to be available to send data to the
controller.
This patch restores transmit buffer creedits regardless of the
controller behaviour.
bug 9474159

Change-Id: I2540131034271a0269fa50698f370bceb7747e2d

am 2f6542da: (-s ours) Reconcile with jb-mr2-zeroday-release - do not merge

* commit '2f6542daa3362e99bf0d09777b3c9913cb0fd85c':

Reconcile with jb-mr2-zeroday-release - do not merge

Change-Id: I6e7a1cf99db622b61d2e35c1bc27eece9ec9dcc7

merge in jb-mr2-zeroday-release history after reset to jb-mr2-dev

am 72912851: (-s ours) Reconcile with jb-mr2-release - do not merge

* commit '72912851ec38b96570ab60d19c175db1c73d53b5':
  Do not abort bonding if link disconnects during SDP after Auth cmplt

am 29a69a11: am ec7e2c8e: Do not abort bonding if link disconnects during SDP after Auth cmplt

* commit '29a69a1144fc42d3742bacb9acee89a612cbf8b3':
  Do not abort bonding if link disconnects during SDP after Auth cmplt

Reconcile with jb-mr2-release - do not merge

Change-Id: I3582cf75047d93865faeb93e56a8dd10c6f83f8e

Do not abort bonding if link disconnects during SDP after Auth cmplt

If ACL link gets dropped during service search after bonding,
re-attempt service search instead of setting bond state to BOND_NONE
Bug 8611134

Change-Id: I77672d9f4feab634b3795a41e0d4e58903b5d275

am ec7e2c8e: Do not abort bonding if link disconnects during SDP after Auth cmplt

* commit 'ec7e2c8e06396101c6b26187995db7c105ed3c3f':
  Do not abort bonding if link disconnects during SDP after Auth cmplt

Do not abort bonding if link disconnects during SDP after Auth cmplt

If ACL link gets dropped during service search after bonding,
re-attempt service search instead of setting bond state to BOND_NONE
Bug 8611134

Change-Id: I77672d9f4feab634b3795a41e0d4e58903b5d275

Removed COD check for unbond and UNPAIRED_EVT

PC supporting HID Device would not have Peripheral as
its MAJOR COD. So COD check for major peripheral device was removed
in case of UNBOND and while receiving UNPAIRED_EVT. Additonal checking
is done when virtual unplug evt is received to remove bonding only for
HID Devic with MAJOR COD as Peripheral or if its a locally initiated
VUP.

Change-Id: I5343919463e48babc9cd03e05ff92a3f93f6d26d

Add debug menu to enable btsnoop

bug: 8059358
Change-Id: I232f6d47e7fb7e9c4d71c119394fce3e72cd02fc

am ebfa4720: (-s ours) Reconcile with jb-mr2-release - do not merge

* commit 'ebfa47201fcac5d4f5772ac49bef186f8fc27f89':

Reconcile with jb-mr2-release - do not merge

Change-Id: Ib7391931437918b090222401b2ef7c3c26f0b667

merge in jb-mr2-release history after reset to jb-mr2-dev

am 67f5817f: am 77d1cb6f: Fix for AV lockup on local and remote start collision

* commit '67f5817f1526faa64d3720c2c3d471117c520de8':
  Fix for AV lockup on local and remote start collision

am 77d1cb6f: Fix for AV lockup on local and remote start collision

* commit '77d1cb6ff904b48ee85c09a06c5e05dd0ed049b9':
  Fix for AV lockup on local and remote start collision

Fix for AV lockup on local and remote start collision

Bug: 7002859

Change-Id: I6170be864b44011b98e08d7e2599021a3eec70a3

am d970d9e5: (-s ours) Reconcile with jb-mr2-release - do not merge

* commit 'd970d9e56173111f9e016a5a0c9f5e532758eefb':

Reconcile with jb-mr2-release - do not merge

Change-Id: I0497c8b9687906dfa54125cd9298a2474f423e3e

merge in jb-mr2-release history after reset to jb-mr2-dev

am b5de2bb1: am 5ae72cf0: Media profile drop on ending the outgoing video call and music is heard locally

* commit 'b5de2bb12924525804f7569a1d6ac0af4e0d278d':
  Media profile drop on ending the outgoing video call and music is heard locally

am 5ae72cf0: Media profile drop on ending the outgoing video call and music is heard locally

* commit '5ae72cf049004ac3c0f2f2bc048d1638a3b13d55':
  Media profile drop on ending the outgoing video call and music is heard locally

Media profile drop on ending the outgoing video call and music is heard locally

receive AVDTP_Suspend response rejection which cause the A2DP disocnnected.
There is collision of AVDTP_Suspend request:
We send AVDTP_Suspend request and the headset send AVDTP_Suspend request also at the same time.
then We send AVDTP_Suspend response, but the headset send AVDTP_Suspend response rejection
with error code bad state after it receive the AVDTP_Suspend response from us.
we call bta_av_suspend_cfm twice in this case:
the first time we will call bta_av_suspend_cfm without error after receive AVDTP_Suspend request from headset.
the second time we will call bta_av_suspend_cfm with error bad state after receive AVDTP_Suspend response rejection from headset.
To fix this problem, we should ignore the AVDTP_Suspend response rejection from headset
after receive AVDTP_Suspend request from headset.
After we receive AVDTP_Suspend request from headset, the scb state will be changed to AVDT_SCB_OPEN_ST from AVDT_SCB_STREAM_ST.
but when we receive AVDTP_Suspend response rejection from headset, we should ignore it when we are in AVDT_SCB_OPEN_ST.
and also currently we already ignore AVDTP_Suspend response when we are in AVDT_SCB_OPEN_ST,
I don't know why we didn't do it for AVDTP Suspend rejection.

issue 8735903

Change-Id: I8db3ea6e7e4edfea41f255e7e29b8c7255cf3d96

am d49bbba5: (-s ours) Reconcile with jb-mr2-release - do not merge

* commit 'd49bbba51d8d6a3aaed65715b87d251e3d181728':
  Solve compilation problem when BLE/GATT/SMP features are false
  Audio glitches while playing songs over MOTO S305 headset

Reconcile with jb-mr2-release - do not merge

Change-Id: If30a4d2103f4851f0faa1a8cf5ccf685b18e0664

merge in jb-mr2-release history after reset to jb-mr2-dev

am 2fca6e54: am 48ebe2c6: Preload timeout and retry mechanism (1/3)

* commit '2fca6e548ff48988e63d3550d807b016192e0a77':
  Preload timeout and retry mechanism (1/3)

am 48ebe2c6: Preload timeout and retry mechanism (1/3)

* commit '48ebe2c6928d8fd4dc97c8adb138c3440714dc89':
  Preload timeout and retry mechanism (1/3)

Preload timeout and retry mechanism (1/3)

If the Preload process was stuck due to unknown hardware init failure,
a 8-second ENABLE_TIMEOUT timeout would be eventually expired in Java layer
at AdapterState and attempted to set BT state back to STATE_OFF.
However BluetoothManagerService did not handle this case accordingly and led
to state mis-matching between Java layer and BTIF at the end.

Add a timeout and retry mechanism to manage Preload process.
Clean up native space resource and explicitly report STATE_OFF back to Java
space when Preload did not go through.

bug  7566317

Change-Id: I99a225e524f2b951249618990a2d0a8a39b43ff3

am 7cc4a7e9: am ebef4763: Added available size check to fix zero byte data available issue

* commit '7cc4a7e9c97eba4990af3c4cecb0b950f7eca204':
  Added available size check to fix zero byte data available issue

am ebef4763: Added available size check to fix zero byte data available issue

* commit 'ebef4763b36b05370d13dfd27c4fcaa9095dbeff':
  Added available size check to fix zero byte data available issue

Added available size check to fix zero byte data available issue

Bug 8950703
      rfcomm connection failed in case zero byte data available

merge in jb-mr2-release history after reset to jb-mr2-dev

am 83ecfd88: am 5fd74f06: Stability fixes for a2dp hal control path

* commit '83ecfd882bec4f5761c4114c571b170c5a9376db':
  Stability fixes for a2dp hal control path

am 5fd74f06: Stability fixes for a2dp hal control path

* commit '5fd74f06caab4324d0ba69df86ad5170c072ec64':
  Stability fixes for a2dp hal control path

Stability fixes for a2dp hal control path

Ensure av statemachine is preventing a2dp hal to restart the stream
when not ready. This prevents ending up in a scenario where a2dp hal
locks up. Code cleanup.
bug 7002859

Change-Id: I3f3eeaab4ca185733b8b0042ed2cfa701f76a203

am e1405531: am e915116a: Merge "Solve compilation problem when BLE/GATT/SMP features are false" into jb-mr2-dev

* commit 'e1405531f637a3851371a2a2c624ff3db5e074fa':
  Solve compilation problem when BLE/GATT/SMP features are false

am e915116a: Merge "Solve compilation problem when BLE/GATT/SMP features are false" into jb-mr2-dev

* commit 'e915116a24142955edbc40a0e380b4828957c762':
  Solve compilation problem when BLE/GATT/SMP features are false

Solve compilation problem when BLE/GATT/SMP features are false

bug 8664724
Change-Id: I582846ccf4b8995f298d8db533488fceb7993ae3

am f71a3f66: am 67e4077c: New call-in/-back functions for Controller to do vendor-specific shutdown (1/2)

* commit 'f71a3f6695cf7f8c605c0ee5fb3623990c9d2e9b':
  New call-in/-back functions for Controller to do vendor-specific shutdown (1/2)

Merge "Solve compilation problem when BLE/GATT/SMP features are false" into jb-mr2-dev