service: use Parcelable instead of helpers
am: 9bceebb

* commit '9bceebb4a441ef7f3c5d9ad2e9b84ec7e1974007':
  service: use Parcelable instead of helpers

Tell the build system this is a host module.
am: df515a4

* commit 'df515a46dd4fe27040c39d35b14b3e071319156d':
  Tell the build system this is a host module.

service: use Parcelable instead of helpers

In order to user .aidl in the future, all types will have to be
parcelable. We don't add Parcelable to base types, as they might
be also used with other transports than binder in future.

Change-Id: I88e6961e69ac0e5d04bd4b6fe3cd7d79bd6f73ba

Tell the build system this is a host module.
am: 06b3a57

* commit '06b3a5741afa443966ec1268241c98b8549f8c44':
  Tell the build system this is a host module.

Tell the build system this is a host module.

We need to tell the build system this is a host module before calling
local-generated-sources-dir.

Change-Id: Ie36ad938c0c7124865148f20fcb70d5187f158b8
(cherry-pick from commit df515a46dd4fe27040c39d35b14b3e071319156d)

Tell the build system this is a host module.

We need to tell the build system this is a host module before calling
local-generated-sources-dir.

Change-Id: Ie36ad938c0c7124865148f20fcb70d5187f158b8

HID: Fix to show mouse pointer movements on UI
am: 5e8b724

* commit '5e8b7245ae98436b34ecce6dc7b45d7f7dba74d7':
  HID: Fix to show mouse pointer movements on UI

Initiate SCO after Mode change event
am: 8847ff1

* commit '8847ff137b311a37c57366ac8d138b605397dce1':
  Initiate SCO after Mode change event

Avoid sending SCO disconnect when power mode is processing
am: 9ab2a9d

* commit '9ab2a9d97f4e0c4b3cd76416e3d6ed03b610ec6f':
  Avoid sending SCO disconnect when power mode is processing

HID: Fix to show mouse pointer movements on UI

As per the UHID implementation, UHID_OPEN & UHID_CLOSE
can be ignored, but it is useful for power-management.
Since these events are handled on hid host, ready_for_data
needs to be set to TRUE. Previously the hid reports were not
sent to kernel UHID driver leading to no mouse pointer movement
on UI of DUT.

Change-Id: I708f5be4240f07826c6eefc5404af3e42f80cfa4

Initiate SCO after Mode change event

Use case:
1. Connect a headset to AG.
2. Make an outgoing call using headset.

Failure:
SCO connection request is rejected by headset.

Root cause:
AG initiated SCO connection when mode change is pending
on headset. Remote is coming out of sniff and rejected
SCO connection request.

Fix:
AG initiates SCO connection request once AG receives mode
change event. It resolves a corner case where remote failed
to accept SCO connection when initiated in between unsniff
LMP request. Issue is seen with WEP301 headset.

Change-Id: I4ea5d073630320b30f90e6141f4daf25943cca0b

Avoid sending SCO disconnect when power mode is processing

Use case:
1. Pair and connect with Mercedes car kit.
2. Answer incoming call by phone or car kit.
3. Switch audio between car kit and phone.
4. Repease 1-3 steps.

Failure:
Car kit is going into bad state and not accepting SCO disconnect.

Root cause:
Phone has sent SCO disconnect command to car kit when
power mode command(sniff/unsniff) is in the processing
on the ACL where SCO is active. Car kit didn't accept
remove SCO request command and gone into bad state.

Fix:
AG should send sco disconnect once power mode change is completed
to avoid this failure.

Change-Id: I7eba162a8fd9615795009ab26005535a3e5c7e86

Process complete AT cmd response.
am: 8c10b05

* commit '8c10b05eda5f1676668cb1504e976f4437851032':
  Process complete AT cmd response.

Abort parsing in case of format error
am: e77ecc5

* commit 'e77ecc51ba21ebcaba74fb8c890bffba14d652bb':
  Abort parsing in case of format error

Process complete AT cmd response.

Usecase:
1. Enable hf client role and connect with AG.
2. Receive an incoming call and accept from hf client.
3. End call from hf client.
4. Repeat the steps few times.

Failure:
UI is not updated with proper call status although
call is active.

Root cause:
When OK is received as part of AT command response,
application is not updated for it. This caused response
state mismatch in application and hence queue was not
cleared properly.

Fix:
Parse complete AT command response and update application
accordingly.

Change-Id: I8cb2362369b209903700e87330ed2a3e3052b188

Abort parsing in case of format error

Usecase:
1. Enable hf client and connect to AG.
2. Send +COPS with length more than allowed 16 characters.

Failure:
Crash might be seen while parsing +COPS response from AG.

Root cause:
sscanf used for parsing the AT commands does not update
the characters scanned when there is format error.

Fix:
Abort the processing in case of format error.

Change-Id: Id3ca80712df3d0251cf9ffb04d9c766be809ae84

Move to idle when remote disconnects during connection.
am: 4d10d7f

* commit '4d10d7f5a0bb9444867b2f77b3e92725a851f8ac':
  Move to idle when remote disconnects during connection.

Allocate buffer for AV SDP in Stack
am: a986d61

* commit 'a986d610aa91222b3387497705aa12ccebe3835b':
  Allocate buffer for AV SDP in Stack

Move to idle when remote disconnects during connection.

Chrysler Uconnect Touch initiates disconnect for a2dp on
l2cap just after discovery on AVDTP channel, stack does not
send any message to BTIF layer for disconnection, this
causes btif getting stuck in opening state and later no stable
connection update is sent to app. Since it is intermediate
state all subsequent connection trials fails.

Send disconnect event from incoming state to closing state
and then closing state will send proper msg to btif layer. Add
closing msg handling in btif layer so that we can update app
for connection disconnect when initiated from remote and move to
proper stable state for accepting/initiating connection.

Change-Id: I011d9ffe9d721235217320c1e5281f70328c7f13

Do not initiate SDP for incoming AVDTP connection
am: c90c0d4

* commit 'c90c0d48e2adfaf42908177ea7eeab85787d84fc':
  Do not initiate SDP for incoming AVDTP connection

Allocate buffer for AV SDP in Stack

Remove buffer allocation in BTA while going for SDP before triggering
A2DP connection. This strategy is followed to address memory corruption
issues due to freeing of BTA buffer before SDP RSP call back is
triggered and access the freed buffer from there, leads to crash when
running Codemonicom tests.

The change allocates and frees buffer in stack.

Change-Id: If6d60d2aa60d19ce5120dd38c2f365d70943db81

Process AV HAL start only when previous instance is cleaned up
am: 59e97c2

* commit '59e97c20314b51cf0c2854e4d2aa889e7137207e':
  Process AV HAL start only when previous instance is cleaned up

Restrict DUT supported SBC configuration in GET_CAP response
am: d565837

* commit 'd56583705e63a8a508b908d2a0c62003834accae':
  Restrict DUT supported SBC configuration in GET_CAP response

Do not initiate SDP for incoming AVDTP connection

There are few instances when remote initiates L2CAP
channel for AVDTP signaling but keeps quite after that,
and later on timeout DUT goes for rest of the signalling
procedure but does sdp prior to the same.Few peer
devices deletes a2dp sink sdp record when there is an
ongoing connection, as a result sdp request fails and
DUT fails to proceed with signalling procedure and
connection setup fails.

This patch makes sure that DUT does not go for redundant
sdp request for AVDTP, provided remote has already
established L2cap connection for the same. SDP is done
explicitly only incase of outgoing AV connection.

This issue is seen with Nokia Es headset.

Change-Id: Id44368687d5b3f97bf0258f8d4ff75f185db23af

Process AV HAL start only when previous instance is cleaned up

Change-Id: Iee232dacd3b79e6e94d26c73fb3e563fddced75c

Restrict DUT supported SBC configuration in GET_CAP response

DUT shows all possible SBC configurations as supported
as part of GET_CAP response and also allows AV SINK to
opt for any of these values from that set as part of
SET_CONFIG request, but later opts for only recommended
figures while streaming so that leads to mismatch in
configuration between the config values SINK sets during
SET_CONFIG and config values DUT chooses for streaming
and hence it causes PTS (version 6.4 onwards) certification
failure for below A2DP and AVDTP cases:

A2DP:
TC_SRC_SET_BV_04_I
TC_SRC_SET_BV_06_I

AVDTP:
TC_ACP_SRC_SIG_SMG_BV_18_C
TC_ACP_SRC_SIG_SMG_BV_20_C
TC_ACP_SRC_SIG_SMG_BV_22_C

Restricting DUT supported SBC configurations to reflect only
the configuration DUT opts for during streaming makes sure
remote also opts for those config values during SET_CONFIG
and later there is no mismatch.

Change-Id: I13dbe94e0ca52708fc62c6db7513f7985237c18c

Use handles to identify GATT attributes (1/4)
am: 26f8828

* commit '26f8828cdb5f02b0b01bafaccd4ad0b0ba9d57ab':
  Use handles to identify GATT attributes (1/4)

HID service discovery refactor
am: 0673013  -s ours

* commit '06730134eb2d99cc1abb0e648bbff19641adc1ab':
  HID service discovery refactor

Restructure GATTC
am: 36b6707  -s ours

* commit '36b67072266468e4b4becc76c0ddbff6154bb12c':
  Restructure GATTC

Merge changes from topic \'bug-27455533-to-nyc-dev-merge-1\' into nyc-dev
am: 1af9063  -s ours

* commit '1af906353999b8f25ddd1030803292a4c84087f6':
  Remove bta_gattc_id2handle
  Use attribute handle instead of id in GATT cache
  HID multi-device refactoring
  GATT cache storage refactoring
  Remove dead methods and callbacks
  Simplify UUID handling in GATT cache

Use handles to identify GATT attributes (1/4)

Bug: 27778668
Change-Id: Id7454450c1ce2164001a1f9ca0b91f18d84e3fd6

HID service discovery refactor

This patch changes how HID is initialized.
Up till now, HID discovery was intricately chained,
to make sure two operations are not scheduled at same time.
From now on, all discovery is done and all operations are
scheduled for execution right away. There is separate set
of functions making sure all operations will be executed
without collisions.

This patch also removes BTA_HH_SCPP_INST_DEF, which was
used as hardcoded service id.

Bug: 27698756
Change-Id: I30a7388e129844c5e4727b2ed2b7c05867cc73a8

Restructure GATTC

This patch changes how GATTC is structured. Up till now, it
contained tBTA_GATTC_CACHE and tBTA_GATTC_CACHE_ATTR. Those
structures were so ugly, someone hide them inside internal
header, to make sure noone would ever access them from outside
GATTC. They are now replaced with:
tBTA_GATTC_SERVICE,
tBTA_GATTC_CHARACTERISTIC,
tBTA_GATTC_DESCRIPTOR and
tBTA_GATTC_INCLUDED_SVC.

Those looks much better, and were made globally avaliable.
Thanks to way they're build, we no longer need set of access
methods, which were also very ugly:
BTA_GATTC_GetFirstChar
BTA_GATTC_GetNextChar
BTA_GATTC_GetFirstIncludedService
etc.

This patch breaks HID, DO NOT SUBMIT without HID refactor.

Bug: 27455533
Change-Id: Ic42cfff175e0cc1a0d8e1a1216e2b4b756cbf77d

Merge changes from topic 'bug-27455533-to-nyc-dev-merge-1' into nyc-dev

* changes:
  Remove bta_gattc_id2handle
  Use attribute handle instead of id in GATT cache
  HID multi-device refactoring
  GATT cache storage refactoring
  Remove dead methods and callbacks
  Simplify UUID handling in GATT cache

Use handles to identify GATT attributes (1/4)
am: 9dc9985

* commit '9dc998503590163068845827d951fc2f05a17bda':
  Use handles to identify GATT attributes (1/4)

Use handles to identify GATT attributes (1/4)

Bug: 27778668
Change-Id: Id7454450c1ce2164001a1f9ca0b91f18d84e3fd6

Cleanup after p_mcb if L2CA_ConnectReq() failed
am: d212571

* commit 'd2125712aa0988434d7809e8497f49525a9eae9b':
  Cleanup after p_mcb if L2CA_ConnectReq() failed

Add missing initialization of queue btm_cb.sec_pending_q
am: 16c97b9

* commit '16c97b9a1732558868901f4315f4f0b3850be2a1':
  Add missing initialization of queue btm_cb.sec_pending_q

Cleanup after p_mcb if L2CA_ConnectReq() failed

If L2CA_ConnectReq() failed, make sure that
we cleanup the lcid cache state after the
affected p_mcb.

Also, extra check and a log message inside function
rfc_check_send_cmd().

Bug: 27334916
Change-Id: Ib2950d12ce456d74355f4bcc0e3c4d87603f8f91

Add missing initialization of queue btm_cb.sec_pending_q
am: 6aef394

* commit '6aef394995e2c99bb29be05fa63298ea56ea3f60':
  Add missing initialization of queue btm_cb.sec_pending_q

Add missing initialization of queue btm_cb.sec_pending_q

Bug: 27763878
Change-Id: I93001a679e22b744c3feafda1997e5f7e707db4e

Log error code on wakelock acquisition/release failure
am: b71bdd6

* commit 'b71bdd6f5bef9514eb358c4279dca57ae8c1df70':
  Log error code on wakelock acquisition/release failure

Log error code on wakelock acquisition/release failure

Bug: 27721443
Change-Id: I8ac9e738d916bfdcf125e33c0e09f90cc8d26d55

Cleanup after p_mcb if L2CA_ConnectReq() failed
am: 757047a

* commit '757047a147582d6c5be22f0fb5148005f345fc9b':
  Cleanup after p_mcb if L2CA_ConnectReq() failed

Add missing initialization of queue btm_cb.sec_pending_q

Bug: 27763878
Change-Id: I93001a679e22b744c3feafda1997e5f7e707db4e

Cleanup after p_mcb if L2CA_ConnectReq() failed

If L2CA_ConnectReq() failed, make sure that
we cleanup the lcid cache state after the
affected p_mcb.

Also, extra check and a log message inside function
rfc_check_send_cmd().

Bug: 27334916
Change-Id: Ib2950d12ce456d74355f4bcc0e3c4d87603f8f91

Don\'t clear the btsnoop log ringbuffer on compress.
am: b35c234

* commit 'b35c23483633826617d2312be805df93aeb9d8ca':
  Don't clear the btsnoop log ringbuffer on compress.

Add config source to dumpsys output
am: 63ddbe8

* commit '63ddbe8813aac242071dffc0bd413ba60e6dd7bf':
  Add config source to dumpsys output

Don't clear the btsnoop log ringbuffer on compress.

Change-Id: I0bf4279104e87b151553eba913491d934f3cde41

Add config source to dumpsys output

This logs where the config file was loaded from in the Bluetooth
Manager dumpsys output.

Bug: 27354612
Change-Id: I50d4aaa0be4f4d1d890580b03742713f4345c80f

Add config source to dumpsys output
am: 190cf46

* commit '190cf46ff6937bceff146aec026cf9f53c40e84d':
  Add config source to dumpsys output

Add config source to dumpsys output

This logs where the config file was loaded from in the Bluetooth
Manager dumpsys output.

Bug: 27354612
Change-Id: I50d4aaa0be4f4d1d890580b03742713f4345c80f

Do not leak eventfd\'s for RFCOMM/AVDT/L2CAP
am: 072bd75

* commit '072bd753e6ac2a011421cd7e4a2e622ee7d052fc':
  Do not leak eventfd's for RFCOMM/AVDT/L2CAP

Do not leak eventfd\'s for RFCOMM/AVDT/L2CAP
am: 9ccca2b

* commit '9ccca2b5937ec42409d8b6354884f1f4b6d524f7':
  Do not leak eventfd's for RFCOMM/AVDT/L2CAP

Do not leak eventfd's for RFCOMM/AVDT/L2CAP

Various users of fixed_queue's do not free the lists properly after
using them. As a result two event fd's leak for each fixed_queue that
is not free'd, as fixed_queue contains two semaphores.

This can lead to run-away file descriptor allocation and an eventual
hard limit leading to various undefined behaviours.

Bug: 27731859
Change-Id: If9ce64b711083bf474d217aa537220065722ad3a

Do not leak eventfd's for RFCOMM/AVDT/L2CAP

Various users of fixed_queue's do not free the lists properly after
using them. As a result two event fd's leak for each fixed_queue that
is not free'd, as fixed_queue contains two semaphores.

This can lead to run-away file descriptor allocation and an eventual
hard limit leading to various undefined behaviours.

Bug: 27731859
Change-Id: If9ce64b711083bf474d217aa537220065722ad3a

Log error code on wakelock acquisition/release failure
am: f4f3b60

* commit 'f4f3b602395c8961fce5937f8e97a57db276b978':
  Log error code on wakelock acquisition/release failure

Log error code on wakelock acquisition/release failure

Bug: 27721443
Change-Id: I8ac9e738d916bfdcf125e33c0e09f90cc8d26d55

Merge "Use audio_session_t consistently" into nyc-dev
am: 1b1bcbb

* commit '1b1bcbb3b0bc812410a3ee4f798570d40e0463bc':
  Use audio_session_t consistently

DO NOT MERGE Check size of pin before replying
am: fcc536a

* commit 'fcc536af189dfe534e5e7c6b352ee114de71b3a1':
  DO NOT MERGE Check size of pin before replying

Merge "Use audio_session_t consistently" into nyc-dev

Merge "Wait for UHID interface on boot"
am: 1302ca7

* commit '1302ca714a1fc812c392ab841253b5731e3c110b':
  Wait for UHID interface on boot

Merge "Wait for UHID interface on boot"

Don\'t send AT+BCC if local device doesn\'t support codec negotiation.
am: e79eaef

* commit 'e79eaef05e0778917dcc77d95292370d5aa4491e':
  Don't send AT+BCC if local device doesn't support codec negotiation.

Don't send AT+BCC if local device doesn't support codec negotiation.

The current code only checked if the peer supported codec negotiation
before sending AT+BCC. But if codec negotation is disabled on the
local device, we should definitely not send AT+BCC.

Bug: 27213013
Change-Id: Id372a32a61a3d96d5c093bc50594ef0b8c2eff2e

HID service discovery refactor
am: e8187e5

* commit 'e8187e565382ae9afb52a7992e9a6ab1e3108999':
  HID service discovery refactor

Restructure GATTC
am: 5204fd7

* commit '5204fd71b3ca799942c45a36548faeba4d89477d':
  Restructure GATTC

HID service discovery refactor

This patch changes how HID is initialized.
Up till now, HID discovery was intricately chained,
to make sure two operations are not scheduled at same time.
From now on, all discovery is done and all operations are
scheduled for execution right away. There is separate set
of functions making sure all operations will be executed
without collisions.

This patch also removes BTA_HH_SCPP_INST_DEF, which was
used as hardcoded service id.

Bug: 27698756
Change-Id: I30a7388e129844c5e4727b2ed2b7c05867cc73a8

Restructure GATTC

This patch changes how GATTC is structured. Up till now, it
contained tBTA_GATTC_CACHE and tBTA_GATTC_CACHE_ATTR. Those
structures were so ugly, someone hide them inside internal
header, to make sure noone would ever access them from outside
GATTC. They are now replaced with:
tBTA_GATTC_SERVICE,
tBTA_GATTC_CHARACTERISTIC,
tBTA_GATTC_DESCRIPTOR and
tBTA_GATTC_INCLUDED_SVC.

Those looks much better, and were made globally avaliable.
Thanks to way they're build, we no longer need set of access
methods, which were also very ugly:
BTA_GATTC_GetFirstChar
BTA_GATTC_GetNextChar
BTA_GATTC_GetFirstIncludedService
etc.

This patch breaks HID, DO NOT SUBMIT without HID refactor.

Bug: 27455533
Change-Id: Ic42cfff175e0cc1a0d8e1a1216e2b4b756cbf77d

Add LE L2CAP Connection Oriented Channel
am: 6721232

* commit '6721232129f137ab024d9b95fc1094a714bc4c01':
  Add LE L2CAP Connection Oriented Channel

Start listener thread only after successful open call
am: 95b9bc0

* commit '95b9bc0e533545a14cf12877c72e0d96ab4370fd':
  Start listener thread only after successful open call

DO NOT MERGE Check size of pin before replying

If a malicious client set a pin that was too long it would overflow
the pin code memory.

Bug: 27411268
Change-Id: I9197ac6fdaa92a4799dacb6364e04671a39450cc

Add LE L2CAP Connection Oriented Channel

This implementation for LE L2CAP Connection Oriented Channel (a Bluetooth
Core Spec 4.1 feature) has been tested on the following formal and informal
IOP events conducted by BT SIG Sport and Fitness Working Group.

* Informal IOP – Tokyo 2014
* UPF 49 – Berlin 2014
* Formal IOP – Tokyo 2014
* Formal IOP – Santa Clara 2015

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

Start listener thread only after successful open call

Use case: Stress tests ON\OFF with different concurrent
scenarios

Failure: Crash in btsnoop_net_listen thread

Root cause: While starting and shutting down the module
we are calling same function update_logging(). While at
the start it is creating listen thread and throwing error
and exiting without closing the listen thread. While
shutdown of module it "should_log" matches with "is_logging"
and it considers cleanup is not required and returns,
leaving Behind the listen thread open. In the next BT ON,
another listen thread is created and causes the crash.

Fix: Started listener thread after prerequisite operations
are successful.

Change-Id: I92ef293d9b061d0ea65a0c41b506870152114bb0

Wait for UHID interface on boot

Poll and sleep to wait for UHID interface on boot to avoid losing
initial input event after boot.

Bug: 27442119
Change-Id: Ie2141c94624363385951c8316ad854d6ae2e82ec

Allocate sufficient memory for FCS
am: f6a64de84f

* commit 'f6a64de84ff3045d6b4e5a9de44da947f597fbb7':
  Allocate sufficient memory for FCS

Allocate sufficient memory for FCS

Use Case: Hit LeakCanary detection during HDP

Failure: Crash while running LeakCanary detection.

Root cause: Did not have sufficient memory for data frame
which lead to crash.

Fix: Allocating sufficient memory for the data frame
which includes FCS in the header field.

Change-Id: I3bb862f29ecc854423796dc89820ba21c5db6050

osi: add osi_rand()
am: 093f013ab8

* commit '093f013ab883ed2ffeef875b17fd940dd92af659':
  osi: add osi_rand()

osi: add osi_rand()

Provide a replacement for rand() which has a better random source.

Change-Id: I38a8c74d86b89ec160b43b0f648f53b463be89bb

Removed unused function bta_sys_vs_hdl()
am: bea3130893

* commit 'bea31308930f20aac1f7422f262e509e9f51f084':
  Removed unused function bta_sys_vs_hdl()

Removed unused function bta_sys_vs_hdl()

Change-Id: Iaf199d418aafc0b518df2f7febfa1c9e29f1fa42

Removed duplicate (re)define of BTA_DM_SDP_DB_SIZE
am: f958ee655d

* commit 'f958ee655d67b78018a56a593b6e669ca543991e':
  Removed duplicate (re)define of BTA_DM_SDP_DB_SIZE

Removed duplicate (re)define of BTA_DM_SDP_DB_SIZE

Removed conditional duplicate re-define of BTA_DM_SDP_DB_SIZE.
BTA_DM_SDP_DB_SIZE has been already (conditionally) defined to 8000
in file include/bt_target.h

The conditional re-define of BTA_DM_SDP_DB_SIZE to a smaller
value (250) in file bta/dm/bta_dm_int.h should be removed, because
it is confusing and error-prone.

Change-Id: Id10a45562c7c57fd0a9131ef50b7b7a5c2801ec5

Free allocation tracker hash map entries to reduce memory pressure
am: 2bea757aba

* commit '2bea757aba0d0d84865d7d0d675d8bcf31f90b22':
  Free allocation tracker hash map entries to reduce memory pressure

Free allocation tracker hash map entries to reduce memory pressure
am: bb9148eda2

* commit 'bb9148eda25d7d39399fd837d4b13be9b4ecccb4':
  Free allocation tracker hash map entries to reduce memory pressure

Free allocation tracker hash map entries to reduce memory pressure

Use case:
A2DP playback becomes choppy after long hours of testing.

Steps:
1. installed 3rd party app that performs recurring LE scans.
2. Leave phone to run overnight.
3. After 24h+, connect a Bluetooth headset and music is very choppy.

Root cause:
The memory allocation tracker has support for double free check
where it stores all allocations in hash map but it never get
removed to track double free issue, A2DP memory allocation is
taking more time and leading to choppyness.

Fix:
Free the hash map entry to avoid memory leak from
allocation tracker.

Bug: 27621407
Change-Id: If57bd0b89c1d9b99c4f066c537a9e0fa49bc093f

Free allocation tracker hash map entries to reduce memory pressure

Use case:
A2DP playback becomes choppy after long hours of testing.

Steps:
1. installed 3rd party app that performs recurring LE scans.
2. Leave phone to run overnight.
3. After 24h+, connect a Bluetooth headset and music is very choppy.

Root cause:
The memory allocation tracker has support for double free check
where it stores all allocations in hash map but it never get
removed to track double free issue, A2DP memory allocation is
taking more time and leading to choppyness.

Fix:
Free the hash map entry to avoid memory leak from
allocation tracker.

Bug: 27621407
Change-Id: If57bd0b89c1d9b99c4f066c537a9e0fa49bc093f

Add pair event device class and fix timestamp
am: fecf606348

* commit 'fecf606348a0751e0be30e376c9817195afe68ca':
  Add pair event device class and fix timestamp

Add pair event device class and fix timestamp

Added a function to get the device class to use with pair events.
Also update check_cod_hid as it was only being used with COD_HID_MAJOR.
Also fixed an issue with pair event timestamps where they were
using the wrong clock.

Bug: 27499938
Change-Id: I0805d7a3f82b28e7d7e73d80818948843f51a82b

Merge "Add pair event device class and fix timestamp"
am: 00ada513e7

* commit '00ada513e7a034ac372557de5ef8097fc99a9109':
  Add pair event device class and fix timestamp

Merge "Add pair event device class and fix timestamp"

Add pair event device class and fix timestamp

Added a function to get the device class to use with pair events.
Also update check_cod_hid as it was only being used with COD_HID_MAJOR.
Also fixed an issue with pair event timestamps where they were
using the wrong clock.

Bug: 27499938
Change-Id: I0805d7a3f82b28e7d7e73d80818948843f51a82b

Fix startup error when no config file provided
am: e919eda232

* commit 'e919eda23208efd6c8d94e27d973632551412b95':
  Fix startup error when no config file provided

Fix startup error when no config file provided

If bluetoothtbd was started without config file
present, it would die on asserts about missing config.
From now it'll proceed with default settings.

Change-Id: I46c35fb2ed71efa49430206da13a19ca7df44a64

Merge "Linux vendor driver linkage fixes"
am: a15f218c9c

* commit 'a15f218c9c3164d943bbf32b463d6a9d866e76bd':
  Linux vendor driver linkage fixes

Merge "Linux vendor driver linkage fixes"

Linux vendor driver linkage fixes

Linux driver depends on osi, so it must be linked with it.
It also must export bt_vendor_interface_t, and named properly,
otherwise it is not being recognized as vendor library.

Bug: 27589244
Change-Id: Ib6817b11e4226f2e289ff0d521daca059ca9aca6

Use audio_session_t consistently

Bug: 27562099
Change-Id: I10780c8bfc84af568795a2c15557020fbe89a1c4

Merge "Fix build error caused by missing LOG_TAG"
am: a34a404e3d

* commit 'a34a404e3d59b7c25a64471b4502f37c2c75d4de':
  Fix build error caused by missing LOG_TAG

Merge "Add missing \'cd third_party\' step"
am: 8b1e8c3154

* commit '8b1e8c3154884451a4d71db4583334ba1c09729b':
  Add missing 'cd third_party' step