Fix SIGBUS crash when copying data am: 3f06e14fdd
am: c4203a27fc

* commit 'c4203a27fc0efc6879ab7e8b8d005cb342fb66bb':
  Fix SIGBUS crash when copying data

Fix SIGBUS crash when copying data
am: 3f06e14fdd

* commit '3f06e14fddc0e674347ae28ea4a25218d4ce1d2a':
  Fix SIGBUS crash when copying data

Fix SIGBUS crash when copying data

We have the following memory alignment-related issue, that seems
to be architecture/compiler/memcpy(3) specific.

Within struct tBTIF_CONTEXT_SWITCH_CBACK, the beginning of the
zero-length array "char p_param[]" is not aligned (because of the
struct internals).
However, this p_param pointer is casted within function
btif_gattc_deep_copy() to the struct pointer (btif_adv_data_t *).
By definition, the memory pointed to by such pointer is suppose
to be aligned:

    btif_adv_data_t *dst = (btif_adv_data_t*) p_dest;

It seems that on some architectures/compilers the executed memcpy()
instructions are optimized for such memory alignment.
If the memory was not aligned, we get SIGBUS.

Apparently, just using (void *) casting for the memcpy() destination,
avoids using the optimized memory aligned instructions:
  memcpy((void *)dst, src, ...);

The solutions are twofold:
 * Make sure that "char p_param[]" within struct
   tBTIF_CONTEXT_SWITCH_CBACK is aligned. Otherwise, the casting
   to "(btif_adv_data_t*)" can be problematic.
 * Add (void *) casting to all memcpy() calls which might be
   referring to such mis-aligned memory.
   This is done by using the new macro maybe_non_aligned_memcpy()
   in all places that such casting might be needed.

Either solution is sufficient to prevent the crash as identified in
this particular case. We need to apply both solutions, to reduce the
chance of running again into a similar issue.

Bug: 25601669
Change-Id: I6c49645c00f10c594a5d1e53a9fac202c506657c

Indicate failed tests in run_unit_tests.sh am: 7ab59096c2
am: 08a960d795

* commit '08a960d7952900c419b8afa107af9d1e00e8b03f':
  Indicate failed tests in run_unit_tests.sh

Indicate failed tests in run_unit_tests.sh
am: 7ab59096c2

* commit '7ab59096c26fe1175258f4cf40674736884ceeb0':
  Indicate failed tests in run_unit_tests.sh

Merge "Retry HID Polling when EINTR is returned" am: 95e0c57578
am: 3d1d6595ec

* commit '3d1d6595ec8a44b6db3f89c8f81dd3125d8ba65f':
  Retry HID Polling when EINTR is returned

Indicate failed tests in run_unit_tests.sh

Previously "run_unit_tests.sh --all" did not indicate to the user if a
unit test other than the last one failed.

Change-Id: I9b64e4280612cd9af18acc17f245b8760bea793f

Merge "Retry HID Polling when EINTR is returned"
am: 95e0c57578

* commit '95e0c57578f1bb59b999195e08d7dfba3e4957e1':
  Retry HID Polling when EINTR is returned

Merge "Retry HID Polling when EINTR is returned"

Null-terminate hex string in uuid_new() am: addbe8f489
am: 5f2c3d86d2

* commit '5f2c3d86d25de232af9ff87e8504230996e0b7d8':
  Null-terminate hex string in uuid_new()

Null-terminate hex string in uuid_new()
am: addbe8f489

* commit 'addbe8f4895fdcac71c6ef2ea844612e3bb5938c':
  Null-terminate hex string in uuid_new()

Null-terminate hex string in uuid_new()

Change-Id: I28d8b9e6495992ab990e9bd618221373848957c8

Retry HID Polling when EINTR is returned

While I'm here
 - use __func__ consistently in btif_hh_poll_event_thread.
 - remove braces around a single line

BUG: 24647083
Change-Id: I80b99ff0d3ba6336f5ab7a517269fdf63df59b37
Signed-off-by: Myles Watson <mylesgw@google.com>

Fix net_test_hci am: b0f59256c6
am: 0005a705cc

* commit '0005a705cc80132c78fc9e69c4a59087325f22aa':
  Fix net_test_hci

Fix net_test_hci
am: b0f59256c6

* commit 'b0f59256c6b3c465bc6721394f5e193f711b8384':
  Fix net_test_hci

Fix net_test_hci

vendor->send_command is not send during init(), it's called as a result
to calling enable().

Change-Id: I498870a63057e3644c0b889cd5be046cb41db88d

Merge "service/doc: Add Binder GATT client API definition" am: 88e68c2cd6
am: 6b0dadb3a3

* commit '6b0dadb3a39c2c01843575eb9e0e5d38982ef39f':
  service/doc: Add Binder GATT client API definition

Fix an error loading audio.a2dp.default.so library am: a3aed3bd53
am: 4402e3a447

* commit '4402e3a44766d33d961294cfb6ed5aae1bc6f72b':
  Fix an error loading audio.a2dp.default.so library

Merge "service/doc: Add Binder GATT client API definition"
am: 88e68c2cd6

* commit '88e68c2cd6d28a21c2ac7bdf5c1019d4bef98890':
  service/doc: Add Binder GATT client API definition

Fix an error loading audio.a2dp.default.so library
am: a3aed3bd53

* commit 'a3aed3bd53e326ca79d8d01479d8737cd85729c5':
  Fix an error loading audio.a2dp.default.so library

Merge "service/doc: Add Binder GATT client API definition"

Fix an error loading audio.a2dp.default.so library

Fixes an issue where a symbol is being stripped from the compiled
library preventing it from being loaded.

Bug: 21743601
Change-Id: I60347e83ee85ec282c25ae76f3cf80567bcd19a2

Fix broken test caused by misordered callback arguments. am: 1b4c8b4ebb
am: f5dee186f1

* commit 'f5dee186f188331045eb6a49bb2e9c92c1c4bb58':
  Fix broken test caused by misordered callback arguments.

Fix broken test caused by misordered callback arguments.
am: 1b4c8b4ebb

* commit '1b4c8b4ebbc2f529225737055ba030175c94c9ac':
  Fix broken test caused by misordered callback arguments.

Fix broken test caused by misordered callback arguments.

Change-Id: I16976a2a680eab76af31090548c2efc53bebaa60

Treat empty/truncated GATT cache as error am: 1f656f15b1
am: 774defe11b

* commit '774defe11b4d58c8821f145a987b74e6ccfc8bd4':
  Treat empty/truncated GATT cache as error

Treat empty/truncated GATT cache as error
am: 1f656f15b1

* commit '1f656f15b146febdba8ed0ef0ee444a1e3bd9c0b':
  Treat empty/truncated GATT cache as error

Treat empty/truncated GATT cache as error

Bug: 23054535
Change-Id: I6f60ff556204dabca91ce9075d5802d75fd35857

service/doc: Add Binder GATT client API definition

Added AIDL-style docs for the GATT client Binder API.

Bug: 25128256
Change-Id: I03c4c5a371600b3946a6692a3d0c87f7e529d96a

Always update remote address type during LE scan am: 7b9fe757f5
am: 5c5d572459

* commit '5c5d572459b0e8a11b1bdb2c450bea752227511c':
  Always update remote address type during LE scan

Always update remote address type during LE scan
am: 7b9fe757f5

* commit '7b9fe757f5df8c529c506b1c41403f2f3f204f9e':
  Always update remote address type during LE scan

Always update remote address type during LE scan

This will ensure that devices which do not contain a device name in the
advertising record are stored with the correct address type to ensure
re-connection to the device succeeds.

Bug: 25499163
Change-Id: I85f18ce2de267844c5d5ed49beeb448b9b074d74

Removed libpower-related references. am: 45a94e198e
am: b8cde2c80d

* commit 'b8cde2c80dbf81c3e5f1aea00e9f6a25e38abc5e':
  Removed libpower-related references.

Removed libpower-related references.
am: 45a94e198e

* commit '45a94e198ec424c1f15d9735a605a6876fa6a493':
  Removed libpower-related references.

Removed libpower-related references.

Removed libpower-related references, because those are not
needed anymore. Now, the wakelock-related calls and references
are implemented within osi/src/alarm.c

Change-Id: Ifdb232b9724af216b9b414f8b65e85cd713c20d3

gn-build: Get net_test_bluetooth to compile am: 3e54b60e10
am: ee4744eed6

* commit 'ee4744eed62bc53f146174b247625f2e95fb9da1':
  gn-build: Get net_test_bluetooth to compile

gn-build: Get net_test_bluetooth to compile
am: 3e54b60e10

* commit '3e54b60e10093f48f1c4ea3414635bc942f9ecfe':
  gn-build: Get net_test_bluetooth to compile

gn-build: Get net_test_bluetooth to compile

This makes net_test_bluetooth compile.

Bug: 21584091

Change-Id: Id0c3d6bdcfd6f2c75c749e7f5a306645371fbb25

Fix bug in SDP 128-bit UUID lookup am: 526f3cc7e9
am: 1a1c20dd86

* commit '1a1c20dd86883db239cddb82ff7d9b96e213a2f0':
  Fix bug in SDP 128-bit UUID lookup

Do not mask out secure connections (SC) bit for BT <4.2 am: 97305504e5
am: 5d30292aaa

* commit '5d30292aaa8390e77aaeeaa026acede6476aeaae':
  Do not mask out secure connections (SC) bit for BT <4.2

Fix bug in SDP 128-bit UUID lookup
am: 526f3cc7e9

* commit '526f3cc7e93214f4f3e2c7c3c648710ff7251446':
  Fix bug in SDP 128-bit UUID lookup

Do not mask out secure connections (SC) bit for BT <4.2
am: 97305504e5

* commit '97305504e5771fdf76d16ca8583d5fc1680face8':
  Do not mask out secure connections (SC) bit for BT <4.2

Fix bug in SDP 128-bit UUID lookup

Also re-factored the surrounding code slightly to where it would have
prevented this type of bug in the first place...

Change-Id: I7b7ed4459c4f377d97b608bfe2e0c13aac31680e

Do not mask out secure connections (SC) bit for BT <4.2

Bug: 25467621
Change-Id: Ifdb52295fa5b8c5a39524a722dc044227c735240

Merge commit \'8b71f990de2b2ca9354726b56f0bd41bb9082ad3\' into HEAD
am: abb36da490

* commit 'abb36da49012a4b31a4e69754562509020118e11':
  Prevent integer overflows during GATT signing
  Reduce persistence on aquiring master role
  Raise alarm callback thread priority
  Fix 128-bit UUID byte order for GATT-over-BR/EDR
  Workaround for H4 HCI stream corruption during LE scans
  DO NOT MERGE Revert "am b3ca1b37: resolved conflicts for merge of a905eb62 to mnc-dr-dev-plus-aosp"
  DO NOT MERGE fix am error for 1dffda3b
  DO NOT MERGE Revert "DO NOT MERGE Use POSIX timer API for wake alarms instead of OSI callouts."
  Remove absolute volume control whitelist

Merge commit '8b71f990de2b2ca9354726b56f0bd41bb9082ad3' into HEAD

Merge "Rename net_test_bluedroid to net_test_bluetooth" am: c2ad136c3e
am: 028f32f35f

* commit '028f32f35fe62279e5e6ea7c9056b08109d8cb08':
  Rename net_test_bluedroid to net_test_bluetooth

Merge "Rename net_test_bluedroid to net_test_bluetooth"
am: c2ad136c3e

* commit 'c2ad136c3e19ca526356d7fd6fe44dcc1e7aa062':
  Rename net_test_bluedroid to net_test_bluetooth

Merge "Rename net_test_bluedroid to net_test_bluetooth"

Fix interop database unit test am: f2a7e6c4eb
am: da16e1c6a4

* commit 'da16e1c6a4cad8682cca7099c91c09d3cc3de12a':
  Fix interop database unit test

Fix interop database unit test
am: f2a7e6c4eb

* commit 'f2a7e6c4eba75d8c8a3103fe27bcd06f669d609e':
  Fix interop database unit test

Fix interop database unit test

Change-Id: Id2e45582b92e83804b55d4e8c918e32a6a2dab79

Merge "Serialize remote version query over LE" am: a0a3dacd68
am: 65ba9b572b

* commit '65ba9b572bbb2e765b956bfefa377f7ebe7709eb':
  Serialize remote version query over LE

Merge "Serialize remote version query over LE"
am: a0a3dacd68

* commit 'a0a3dacd682cc3e8c9ee4b5a196e06aeada6cc49':
  Serialize remote version query over LE

Merge "Serialize remote version query over LE"

Serialize remote version query over LE

Some devices (ex. Nexus Player remote) do not take well to having the
remote version queried while a remote feature request is also sent and
subsequently fail encryption.

This patch serializes the query of the remote version and requires both
remote features and the remote version to be read before calling a
connection established.

Bug: 25357767
Change-Id: Ie70eea241b47b81778d8e0df47f3a8f376316932

Remove unused HCI_PROTO_REVISION am: 879aeec727
am: e7a14f9a7f

* commit 'e7a14f9a7f10bf847e20401d296baa47857201ef':
  Remove unused HCI_PROTO_REVISION

Remove unused HCI_PROTO_REVISION
am: 879aeec727

* commit '879aeec72706fc7bae66277350ae9a49648a73e7':
  Remove unused HCI_PROTO_REVISION

Remove unused HCI_PROTO_REVISION

Change-Id: Icec52e4e63b14f2fe7a61cf5077e40c5e8baeff5

Rename net_test_bluedroid to net_test_bluetooth

Also remove other bluedroid references.

Change-Id: I349e48e50b0a0a45ca1267a765145299837601d3

Enable kernel wakelocks and timers am: 28bf007f71
am: 77680a2b16

* commit '77680a2b1622ccf356e97eb08f90d55023ea434c':
  Enable kernel wakelocks and timers

Enable kernel wakelocks and timers am: 28bf007f71
am: 77680a2b16

* commit '77680a2b1622ccf356e97eb08f90d55023ea434c':
  Enable kernel wakelocks and timers

Enable kernel wakelocks and timers
am: 28bf007f71

* commit '28bf007f7103ccd36d5fa2f87837947a27cc698d':
  Enable kernel wakelocks and timers

Enable kernel wakelocks and timers

Use kernel wakelocks now that they are available to us, and use thread
timers instead to eliminate bt_os_callouts usage.

Bug: 23390297
Change-Id: Ic9428457807eab5125d35eae3df6cd2a05a2a659

Check remote LMP version before enabling secure connections am: 27e239dbd3
am: a3f980ec88

* commit 'a3f980ec88bf8e0cac8e87c385b8df2e39847e99':
  Check remote LMP version before enabling secure connections

Check remote LMP version before enabling secure connections am: 27e239dbd3
am: a3f980ec88

* commit 'a3f980ec88bf8e0cac8e87c385b8df2e39847e99':
  Check remote LMP version before enabling secure connections

Check remote LMP version before enabling secure connections
am: 27e239dbd3

* commit '27e239dbd31dd9ee7fab5fb8558fbd2292cf7e5d':
  Check remote LMP version before enabling secure connections

Check remote LMP version before enabling secure connections

Request remote version information for LE links when connecting and
ensure LMP version is >= 8 (Bluetooth 4.2) before setting the LinkKey
bits in the pairing request.

This is to ensure older remote devices do not get confused by the
additional bit(s) in the pairing request.

Bug: 25277879
Change-Id: I25424e541fa5244d8f522dffc62cdedc24f4f35b

Prevent integer overflows during GATT signing am: b335ee9496
am: 0789953b15

* commit '0789953b15199fb1b1717d34543e81d58862735b':
  Prevent integer overflows during GATT signing

Prevent integer overflows during GATT signing
am: b335ee9496

* commit 'b335ee9496e671bf0ef0ab28a5990c9a024ac06b':
  Prevent integer overflows during GATT signing

Prevent integer overflows during GATT signing

Bug: 17901099
Change-Id: Ib61113b19c9c9798b9ea992fad0ce997dcc2e2f8

Merge "Raise alarm callback thread priority" into mnc-dr-dev am: 2c8802ab5f
am: 20cca2ff6b

* commit '20cca2ff6b7cb02d26d1e60bbe0303f14e7be3d4':
  Raise alarm callback thread priority

Reduce persistence on aquiring master role am: 59d9673187
am: 0be25b64e8

* commit '0be25b64e8447f450af9527373134d95255cdcd9':
  Reduce persistence on aquiring master role

Merge "Raise alarm callback thread priority" into mnc-dr-dev
am: 2c8802ab5f

* commit '2c8802ab5f4f73afe2a8b722d3b1bbd8e106fa24':
  Raise alarm callback thread priority

Reduce persistence on aquiring master role
am: 59d9673187

* commit '59d9673187a50c259b42e5a71482304ab850a809':
  Reduce persistence on aquiring master role

Merge "Raise alarm callback thread priority" into mnc-dr-dev

Reduce persistence on aquiring master role

Whenever a device has more than 1 ACL link active and transferring
data on one of its links via PAN, HH or JV (RFCOMM) the sys busy/idle
state toggles frequently. To avoid triggering role switches for each
of these events we filter this out and let the other SYS events handle
it in combination with other role policy management code. Ideally we
should revert the toggling to properly reflect the busy/idle state of
each profile but to limit risk of side effects at this stage we will
make this intermittent change.

This also affects audio streaming in certain cases.

Bug: 24570959, 25129209
Change-Id: I141e17ee069c82624e153fd8de5db90ae93724b9

Raise alarm callback thread priority

Since alarms [/callbacks] are used in the critical audio path, raising
the thread priority is required to ensure timely encoding/transmission
of audio packets and avoiding thread pre-emption.

This patch indiscriminatly raises the (single) callback thread priority,
affecting all alarm callbacks. In the future, thread priotiry should be
adjusted based on the alarms pending in the queue.

Bug: 24570959
Change-Id: I64928fd7325f1d6f76815a185dbccf1e01ae30d2

Workaround for H4 HCI stream corruption during LE scans am: bdd19e0873
am: 2c50605cb6  -s ours

* commit '2c50605cb64833e6763445590503994b9bb288f8':
  Workaround for H4 HCI stream corruption during LE scans

Workaround for H4 HCI stream corruption during LE scans am: bdd19e0873
am: 2c50605cb6

* commit '2c50605cb64833e6763445590503994b9bb288f8':
  Workaround for H4 HCI stream corruption during LE scans

Remove absolute volume control whitelist am: 58593a0273
am: e94a7d19bb

* commit 'e94a7d19bb41852d5eec825775ad49ecb1d03cc0':
  Remove absolute volume control whitelist

Remove absolute volume control whitelist am: 58593a0273
am: e94a7d19bb

* commit 'e94a7d19bb41852d5eec825775ad49ecb1d03cc0':
  Remove absolute volume control whitelist

Fix 128-bit UUID byte order for GATT-over-BR/EDR am: 61968383a6
am: f90c1f10cc

* commit 'f90c1f10cc10d5465106946bd81ec4cbbb46b376':
  Fix 128-bit UUID byte order for GATT-over-BR/EDR

Fix 128-bit UUID byte order for GATT-over-BR/EDR am: 61968383a6
am: f90c1f10cc

* commit 'f90c1f10cc10d5465106946bd81ec4cbbb46b376':
  Fix 128-bit UUID byte order for GATT-over-BR/EDR

Workaround for H4 HCI stream corruption during LE scans
am: bdd19e0873

* commit 'bdd19e0873854567bb6bd9bc8905a87a3a4744c4':
  Workaround for H4 HCI stream corruption during LE scans

Remove absolute volume control whitelist
am: 58593a0273

* commit '58593a02736ff9ac9bdccd0e3186602f17452cdf':
  Remove absolute volume control whitelist

Fix 128-bit UUID byte order for GATT-over-BR/EDR
am: 61968383a6

* commit '61968383a63be05a335e26bc53ef39d9af98ad72':
  Fix 128-bit UUID byte order for GATT-over-BR/EDR

Workaround for H4 HCI stream corruption during LE scans

Sometimes, during result-heavy LE scans, the UART byte stream can become
corrupted, leading to assertions caused by mis-interpreting the bytes
following the corruption.

This workaround looks for tell-tale signs of a BLE event and attempts to
skip the correct amount of bytes in the stream to re-synchronize onto a
packet boundary.

Bug: 23934838
Change-Id: Ifadaecf8077cb1defc7134c59c97302fca660f81

Remove absolute volume control whitelist

Remove the absolute volume control whitelist to enable absolute volume
control for all compatible devices.

Bug: 25081918
Change-Id: Iab0e4b4c4fd3dac0fefdf62ec80419dd64b56c0c

Fix 128-bit UUID byte order for GATT-over-BR/EDR

128-bit UUIDs for GATT services discovered using SDP are byte-ordered
incorrectly. This patch fixes both the SDP discovery as well as the SDP
record creation code.

Bug: 24344122
Change-Id: Iee4bf33dcbbc5ee0f2380b48330107232bd0401b

Fix 128-bit UUID byte order for GATT-over-BR/EDR am: deb1c7c5cf
am: 7424d92e36

* commit '7424d92e361dd1797f0f63ef7c17e3e40e3d18c7':
  Fix 128-bit UUID byte order for GATT-over-BR/EDR

Merge "Workaround for H4 HCI stream corruption during LE scans" into mnc-dr-dev am: 78870e0f38
am: 3d28925a71

* commit '3d28925a718166c114737a6e4d0142b74a4d70db':
  Workaround for H4 HCI stream corruption during LE scans

resolve merge conflicts of 15a5c3f852 to mnc-dr-dev-plus-aosp.

Change-Id: I072cf4d4fe0f4338647c806c064817ded06ee223

Fix 128-bit UUID byte order for GATT-over-BR/EDR
am: deb1c7c5cf

* commit 'deb1c7c5cfacba8ec2d896f6bb120bdc81314d9a':
  Fix 128-bit UUID byte order for GATT-over-BR/EDR

Fix 128-bit UUID byte order for GATT-over-BR/EDR

128-bit UUIDs for GATT services discovered using SDP are byte-ordered
incorrectly. This patch fixes both the SDP discovery as well as the SDP
record creation code.

Bug: 24344122
Change-Id: Iee4bf33dcbbc5ee0f2380b48330107232bd0401b

Merge "Workaround for H4 HCI stream corruption during LE scans" into mnc-dr-dev
am: 78870e0f38

* commit '78870e0f38658e389e22dfba5da3f239f01966d7':
  Workaround for H4 HCI stream corruption during LE scans

Merge "Workaround for H4 HCI stream corruption during LE scans" into mnc-dr-dev

Remove absolute volume control whitelist
am: 8289925079

* commit '8289925079c40ec91fa99d0ea6817f796a66681a':
  Remove absolute volume control whitelist

Workaround for H4 HCI stream corruption during LE scans

Sometimes, during result-heavy LE scans, the UART byte stream can become
corrupted, leading to assertions caused by mis-interpreting the bytes
following the corruption.

This workaround looks for tell-tale signs of a BLE event and attempts to
skip the correct amount of bytes in the stream to re-synchronize onto a
packet boundary.

Bug: 23934838
Change-Id: Ifadaecf8077cb1defc7134c59c97302fca660f81

doc: add supported features document am: ac43e88a0a
am: 2be3cd3f69

* commit '2be3cd3f69513aac3a0a199c486f680dda6ad74a':
  doc: add supported features document