Delete unused function declarations from sdp_api.h.

Change-Id: I8db02d557fdfc138f2ec8de2b2d45049175fd999

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

Bug: 27455533
Change-Id: I653b563d3b3c233ffe31b0a35d396a66db363459

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"

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

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

Clean A2DP dump output

Before:
  TxQueue:
    Counts (enqueue/dequeue/readbuf)                        : 0 / 0 / 0
    Last update time ago in ms (enqueue/dequeue/readbuf)    : 0 / 0 / 0
    Frames per packet (total/max/ave)                       : 0 / 0 / 0
    Counts (flushed/dropped/dropouts)                       : 0 / 0 / 0
    Last update time ago in ms (flushed/dropped)            : 0 / 0
    Counts (underflow/underrun)                             : 0 / 0
    Bytes (underflow/underrun)                              : 0 / 0
    Last update time ago in ms (underflow/underrun)         : 0 / 0
    Enqueue deviation counts (overdue/premature)            : 0 / 0
    Enqueue overdue scheduling time in ms (total/max/ave)   : 0 / 0 / 0
    Enqueue premature scheduling time in ms (total/max/ave) : 0 / 0 / 0
    Dequeue deviation counts (overdue/premature)            : 0 / 0
    Dequeue overdue scheduling time in ms (total/max/ave)   : 0 / 0 / 0
    Dequeue premature scheduling time in ms (total/max/ave) : 0 / 0 / 0

After:
  TxQueue:
  Counts (enqueue/dequeue/readbuf)                        : 0 / 0 / 0
  Last update time ago in ms (enqueue/dequeue/readbuf)    : 0 / 0 / 0
  Frames per packet (total/max/ave)                       : 0 / 0 / 0
  Counts (flushed/dropped/dropouts)                       : 0 / 0 / 0
  Last update time ago in ms (flushed/dropped)            : 0 / 0
  Counts (underflow/underrun)                             : 0 / 0
  Bytes (underflow/underrun)                              : 0 / 0
  Last update time ago in ms (underflow/underrun)         : 0 / 0
  Enqueue deviation counts (overdue/premature)            : 0 / 0
  Enqueue overdue scheduling time in ms (total/max/ave)   : 0 / 0 / 0
  Enqueue premature scheduling time in ms (total/max/ave) : 0 / 0 / 0
  Dequeue deviation counts (overdue/premature)            : 0 / 0
  Dequeue overdue scheduling time in ms (total/max/ave)   : 0 / 0 / 0
  Dequeue premature scheduling time in ms (total/max/ave) : 0 / 0 / 0

Bug: 27294154
Change-Id: I2952ceed7b02db0d8497b67de0f57e4f9128e5bf

Update build files for Fluoride on Linux

uprev build/secondary/third_party/**/BUILD.gn"
Other BUILD.gn files were updated to match current state of project.

Change-Id: Ie895d25680d510f5e69e6f15553311f6671cdce3

Protect metrics API from race conditions

Bug: 27315491
Change-Id: If57f7c23c94890e5f1cf196d2ec7a2a667b48cec

Add Linux-specific version of metrics.

This patch adds empty metrics implementation for Linux in
order to fix build errors.

Bug: 27324453
Change-Id: I2f45db6147689bf295176b7ad30a613a7f5a8d9f

Make wakelock compile on Linux

This patch adds some missing header files that prevented wakelocks
from compiling on linux.

Bug: 27324453
Change-Id: Ifabf3350fe840524cf7e1ec7aa2825acb4399c81

Add missing log tag in config_clear()

Bug: 27324453
Change-Id: Icebf0273c39040ff6b1bb29dcc4d0a827e25b4a8

Revert "Pick proper addres type for unknown devices"

This breaks Nexus Player remotes. Need to only do this if the device is *not* known.

This reverts commit b7672f4a3c2a87fab24c81b243e1872443509254.

Change-Id: Iba0f25576432c8c8c399ee54abee3e91430784ef

Merge "Revert "Fix data reporting state when using HID device""

Revert "Fix data reporting state when using HID device"

This breaks HID completely...

This reverts commit 5f30560dbb80f0231fb9f351ab5b0180513dffe7.

Change-Id: I2c1994a366e193a73dbc661e5c6b03d87d7732f3

Get rid of counter

This patch removes counter. We were never using those values,
except for increasing them.

Bug: 27324326
Change-Id: I3122a36fa0435a36a8401792267cb5ebff5ab269

Merge "Revert "linux: Make logwrapper show log tag on Linux""

Revert "linux: Make logwrapper show log tag on Linux"

This reverts commit e4ae5c7522e6a77e1e0492afa3deed9fdc1569e8.

Change-Id: I8e0c890ee0960f986133939944c2c1362ac4f57c

Remove unused import

Change-Id: Ia259abc76b95d754fa31c2ccb96b6590348eb493

Merge "linux: Make logwrapper show log tag on Linux"

Add tool to extract and view snooz logs from a bugreport

Change-Id: I3893d094a1b1dd7853016d1f539f8998cdc84833

linux: Make logwrapper show log tag on Linux

Change-Id: I2219395f504952ae1d1371a439e58fef22b71184

service: Non-android OSs don't know how to Binder

Change-Id: I1eeabef6c6f065bbc309604345a0212fed7e7ef7

Clean up dump printout

Change-Id: I0b7e5a9dda30875c9fc877baffa0daac9184cd42

Removed unused defines from bt_target.h

Removed unused conditional defines from bt_target.h that
are not needed anymore.

Change-Id: Ic424f673332668b21db11d323b389f5a5f4fba81

Replace usage of HCI_GET_CMD_BUF() with osi_malloc()

* Remove HCI_GET_CMD_BUF() wrapper macro, and replace it with
  its default implementation: (BT_HDR *)osi_malloc(HCI_CMD_BUF_SIZE)

* Remove explicit checks for NULL when HCI_GET_CMD_BUF() was used.

Bug: 27048759
Change-Id: I7c71b408f40d58f6ebf3a310aa806daba3342f77

Instrument and fix calls to strncmp()

Bug: 27222508
Change-Id: I0075b4480aff52dcdc326b7bdb7154ca919ccee9

Write Protobuf in Base64

Change-Id: I3868365e7fd166517b13ff03a15d629271bd47b4

Fix ParcelHelpersTest.ScanResult unit test

Fix a failing unit test that was broken by some
earlier cleanup work.

Bug: 26879229
Bug: 27152031
Change-Id: I244d078a23a69c82fbc7cb4468649ab933131977

Add support for collecting A2DP Protobuf metrics

* Add new function metrics_a2dp_session() to
  connect A2DP metrics into Protobuf

* Add new function btif_update_a2dp_metrics()
  that is called when the metrics should be
  callected

* Add few more statistics to the BTIF media stats.

Change-Id: I5862ac82e55b3546dc1fbde0c78cb4683e40d19e

Merge "Additional logging for bond information"

Suspend properly when receiving events without response

In the BTU handler for messages from the controller, rn event may be
received that the stack does not have to respond to. In this case, the
stack should go to sleep.

This patch calls the provided function in the BTU to enter sleep.

Signed-off-by: Jaganath Kanakkassery <jaganathx.kanakkassery@intel.com>
Signed-off-by: Yingkai Cheng <yingkaix.cheng@intel.com>
Signed-off-by: Zhiquan Liu <zhiquan.liu@intel.com>
Change-Id: Ie7c3fcae418eb2480338509e86b7cc870177a965

Additional logging for bond information

Log the bond state, the funciton that updated the bond state, as well as the
device that was updated. Also added in functionality for protobuff logging.

Example output
Bond Events:
  Total Number of events: 8
  Time          BD_ADDR            Function             State
  00:01:12.353  ac:9e:17:07:58:d8  btif_dm_create_bond  BOND_STATE_NONE
  00:01:12.354  ac:9e:17:07:58:d8  bond_state_changed   BOND_STATE_BONDING
  00:01:13.999  ac:9e:17:07:58:d8  bond_state_changed   BOND_STATE_BONDING
  00:01:14.778  ac:9e:17:07:58:d8  bond_state_changed   BOND_STATE_BONDED
  00:01:14.827  ac:9e:17:07:58:d8  bond_state_changed   BOND_STATE_BONDED
  00:01:31.534  34:36:3b:c4:bf:99  btif_dm_create_bond  BOND_STATE_NONE
  00:01:31.535  34:36:3b:c4:bf:99  bond_state_changed   BOND_STATE_BONDING
  00:01:32.771  34:36:3b:c4:bf:99  bond_state_changed   BOND_STATE_BONDING

Change-Id: I01ab3a232300203704a356a17aa045e638f97463

Fix data reporting state when using HID device

Start and Stop are called on HID driver probe/unprobe.
Open/Close are called when driver is interesed/not-interested
in data reports, typically when user-space open/close the input
device. So we can have several open/close calls for one start/stop
cycle.

https://www.kernel.org/doc/Documentation/hid/hid-transport.txt

Problem is that current implementation disables data reporting on close
but never enables it on open. This patch fixes this issue to match the
expected behavior.

Change-Id: Id5d4627dba389338b06df60a6e1d993ac3e3e5ec
Signed-off-by: Loic Poulain <loic.poulain@intel.com>
Signed-off-by: Zhiquan Liu <zhiquan.liu@intel.com>

service: Use built-in byte vector parser

Change-Id: I72369c0b7678338fc8ccf520b59a95affdfd2994

Remove the need for -Wno-constant-logical-operand

Change-Id: Iea44eeed487df3236d9b482b7e5bbc92f7f46797

Remove Porsche car-kit pairing workaround

Bug: 26551752
Change-Id: I14c5e3fcda0849874c8a94e48aeb7d09585617e1

Fix compile error due to interop_feature_t not being public

Change-Id: I3c6d2d28d98098aea1854ebd2726fdaac9cc14e9

Add new HAL functions to fake Bluetooth interface

Change-Id: I5b2b5a495ea14fd743885c4061a6bcd3e7c39c23

Add ability to add interop entries dynamically (2/2)

Add ability and interface for adding dynamic entries to the interop
workaround database.

Bug: 26548845
Change-Id: Id886e4233fc1548727e79f1581cdc0c6f0738d59

Fix compilation warnings/errors on fugu

Fix compilation warnings/errors on fugu about unused
variables.

Change-Id: Ib68097122f27c631ab36b71a6815d4e1f3b60558

Fix C/C++ compiler generated warnings

 * Remove unused functions: -Wno-unused-function
 * Fix logical operations that are missing parentheses:
   -Wno-logical-not-parentheses, -Wno-parentheses
 * Add missing braces in initializers: -Wno-missing-braces
 * Remove unused variables: -Wno-unused-variable
 * Fix printf()-like formatting issues: -Wno-format
 * Avoid using unitialized variables:
   -Wno-maybe-uninitialized, -Wno-uninitialized
 * Fix a bug inside bta_gattc_get_gatt_db_impl()
   when the processed GATT DB attribute type is unknown.
 * Fix warnings about missing field initializers:
   -Wno-missing-field-initializers
 * Re-implement macro COMPILE_ASSERT(COND) to fix a compilation warning:
   -Wno-non-literal-null-conversion
 * Fix sign mismatch comparison warnings:
   -Wno-sign-compare
 * Fix warnings related to enum conversion:
   -Wno-enum-conversion
 * Fix warnings related to incompatible pointer types:
   -Wno-incompatible-pointer-types
 * Fix warnings related to enum conversion:
   -Wno-enum-conversion

Bug: 26879229
Change-Id: I522931fe156aeab23ae841051a9e25ceab00b1c2

service: Add services removed/added handlers

Change-Id: Ida3f232b4e77bfb95dec78ea4af7677c9c12b411

Cleanup C and C++ compiler flags

Cleanup the setting of C and C++ compiler flags:
 * (Almost) all compiler flags are set uniformly
   in the system/bt/Android.mk file.
 * Enable by default breaking the compilation if there is a
   compilation warning: -Werror
 * Enable most compilation warnings: -Wall -Wextra
 * Renamed Android.mk related flags:
   - bdroid_C_INCLUDES -> bluetooth_C_INCLUDES
   - bdroid_CFLAGS -> bluetooth_CFLAGS
 * Introduce variables for C-only and C++ only compiler:
   - bluetooth_CFLAGS: common C and C++ compiler flags
   - bluetooth_CONLYFLAGS: C only compiler flags
   - bluetooth_CPPFLAGS: C++ only compiler flags
 * Disable warnings for existing issues - to be removed as issues are
   resolved
 * Add a workaround for libchrome and -DNDEBUG usage.

Bug: 26879229
Change-Id: Ie7595965ca0c8ead0e95e983e76c327e7891b2c3

Pick proper addres type for unknown devices

Change-Id: I46641d0fe065b5e5a6620dc34c770429ac8dbe11

Fix discovery status handling

This typo was introduced in commit 48db2d25 3years ago.
This cause unnecesary purge of cache and rediscovery even though
discovery was successfull.

Change-Id: I828e148ad489af1e231d33d7807bbe0e46f445c5

Validates the PDU length of GATT Prepare Write attribute requests

Validates whether the PDU length of GATT Prepare Write
requests satisfy the minimum length requirements of
GATT write attribute request format.

Change-Id: Id18d32b050389ff8c71341bb738dd77225029acd

Create backup for bt_config.conf

Move current config file to a backup when a new config file is
saved. This backup can be used in case there is an error loading
the current config file.

Change-Id: Icc210c457dc71a72fc2e82ff72c91207b25deaf1

Removed function bte_load_ble_conf

The functionality provided by this function is not needed anymore.

Change-Id: I59696c7a7e1d7ddb095f3a450f58ce1db97927b6

Add handle range to BTA_GATTC_GetGattDb

Change-Id: Iaf0280853278a4f28dde5b952b646096cfb12d97

Removed checks for NULL returns after osi_calloc() / osi_malloc()

Removed explicit checks for NULL pointer returns after calls
to osi_calloc() and osi_malloc(), because those are not needed.
If the memory allocation fails, osi_calloc() and osi_malloc()
will trigger an assert.

Bug: 27048759
Change-Id: I2791eb2f69c08f991f8fcdef10e101a41568cd95

osi: Don't use -std=c99 to compile

Using -std=c99 to compile breaks when we are using C++ files, which
metrics uses for protobufs.

Bug: 27077250
Change-Id: Ie94e3ea5d8b219671329108e7795cdc02ecc38b4

Remove code related to legacy config file

Change-Id: Ia82d9bbffcc5df1ec4cddff28d812ca221505ab9

Add Privacy 1.0/1.1 check for BG connection

Check for privacy 1.0 and 1.1 controller and do not start
background connection, if RPA offloading is not supported,
since it will not connect after change of random address

Bug: 22102400
Change-Id: Ida618f8dc70e0a3860f76f856bb88de8b55b90d7

binder: fix possible stack corruption

The stack could be corrupted by crafting a IPC call in interesting ways
when a character buffer was passed.

This patch also removes code duplication where these would occur.

Bug: 26917241
Change-Id: Ib6c149a293abf01f31c69a94c8f6dd91d8a2fff2

osi: add metrics API

Add a metrics API, which supports creating events which are eventually
passed up to the clearcut logging to track pairings and other events.

Connect this to the dumpsys call when it is called.

Change-Id: Idcf75541fd18b0413cc843d6c7e23a5f08a634a5

Fix possible memory leak

Fix possible memory leak when allocating reused server cache control
block.

Change-Id: I3aab727e4f60e55530f49b0b05ccf2c891c72760

Fix auto disconnect right after connecting

Change-Id: Ia36cd898c21058b9a3ed56a0d300c3ea8384a5cf

Set GATT link inactivity timer to 1

GATT_LINK_IDLE_TIMEOUT_WHEN_NO_APP timer was zero. Due to this zero
inactivity timer after the authentication is complete the link is brought
down immediately. Set it to 1 instead.

Bug: 22100369
Change-Id: I6ba4f63b9e48759e4255f230d6bbf426eda31c6f

Add Developer menu entry to disable Bluetooth absolute volume

Bug: 27078729
Change-Id: I24ac605c6d02dc3fe3904c441ec416cb66b63168

Replaced osi_getbuf()/osi_freebuf() with osi_malloc()/osi_free()

Removed the alternative buffer allocation osi_getbuf() / osi_freebuf()
and use instead osi_malloc() / osi_free().

Correspondingly, replaced usage of osi_freebuf_and_reset()
with osi_free_and_reset().

Bug: 24914560
Change-Id: I7a9599ba7fa900321f087da684428133eb0ddd6b

Use dynamic memory for keeping GATT cache

Change-Id: I60674c47246d8fe2094fe78ce31b752fa2acb89e

Add missing "\" at end of line

Change-Id: I6b8a275769387cc731f4bf5ec9ec4d10700fc0ba

Removed function osi_get_buf_size()

Refactored code that uses function osi_get_buf_size(), and removed
the need for that function.

Bug: 24914560
Change-Id: I0d002635024a9703acb78f47735aafc957a2b761

Disable gnu-variable-sized-type-not-at-end warning

This warning is triggered unnecessary in too many places.
It spills through the build log and cause very useful warnings
to stay unnoticed.

Change-Id: Iee75327d05cbece4b9dc9c13005a9bfdf94c0d56

Merge "Remove BCM_STR* macros"

Add missing log tags

When building for Linux, missing log tags cause errors.

Change-Id: I0b3cda3333c059251b00a25bbfc7e764057286be