Remove indirection in the controller reset sequence

Fix issue where last character of device name is missing

The device name stored on disk is correct but has a missing character
at the end when read off of disk. This happened because the returned
|length| parameter to |btif_config_get_str| is in number of *bytes*,
not in number of characters (i.e. must add one for NULL character).

Add a UUID module to btcore for operating on UUIDs.

This change only adds a single function, uuid_is_empty, but I expect
other UUID-related functions to find themselves here.

Introduce btcore, a low-level library to manipulate Bluetooth data types.

This change adds a single module to btcore: bdaddr. The bdaddr module
is reponsible for manipulating and working with Bluetooth addresses.

Fix digital transposition in saving binary config data

Was causing link keys to be saved/read incorrectly, causing reconnect on reboot to fail.

Eliminate unused arguments to btif_config and clarify variable names

Switch over from an XML based file format to .INI-style conf files

This change moves us to using a single file format for all configuration
throughout bluedroid. I've provided a transcoder that converts the
previous config file to the new format so settings will be preserved.

In a future revision of bluedroid, we should remove the transcoder and
drop our dependency on tinyxml2.

Add a method to check for available bytes on a socket

The available bytes function shouldn't normally be required (and
if it is, the code design is probably bad and should be rethought) but
is introduced as a stop-gap until we can redesign the BTA code that
expects a priori knowledge of the number of bytes that can be read from
the socket.

This change also relaxes the requirement that on socket_register that
at least one of read_cb or write_cb must be specified.

Make osi_strdup report allocation size correctly

Move HCI over to the osi allocators, and fix the tests accordingly

Finish allocation tracking for the rest of osi

And found a memory leak in config, so it's doing its job! :)

Remove unused #define RPC_INCLUDED

Fix timer allocations in the test harness, so bdtest runs again

Correct comments and some redundant statements.

Remove must-always-be-on #define HCISU_H4_INCLUDED

Add strdup to tracked allocations

Add internal constructors to hash map and list

This makes allocation tracker safe and ensures the hash map
and associated lists for the allocation tracker don't show
up in the allocation tracker's stats.

Clean up scattered naive hash functions

Move some of osi to use the allocation interfaces

Also changes some tests to ensure all memory freed.

Add a method to pass file descriptors and one to create a socket from an fd.

Rewrite the SCO socket implementation to be more robust.

This code presents a socket interface for SCO connections. Like the
previous implementation, it does not support sending/receiving data
over the sockets; rather, it only provides a mechanism to establish
and tear down connections. It is assumed that all SCO data routing is
done out-of-band over PCM/I2S.

A SCO socket is represented internally by a file descriptor and SCO
handle. The file descriptor is one end of a socketpair (the other
end is sent to the Java process) and the SCO handle represents a
server (listening) or client (connecting) socket owned by the BTM
layer of the stack.

The most notable implementation detail is that, unlike typical
sockets, listening SCO sockets get "upgraded" when a connection is
established. In other words, a SCO handle for a listening socket
turns into a handle for a connected socket. To present a BSD-style
API to Java processes, we play a trick: when a listening socket gets
upgraded, we create a new listening SCO socket and reassociate the
SCO handle for the two sockets. The reassociation allows the Java
process to continue listening for new incoming SCO connections on
the same file descriptor.

Move the disconnect_immediately logic so that it's only in effect for data sockets

Clear connected & disconnect_immediately flags when intializing SCO sockets

Fix a race condition where closing a SCO connection that was not yet fully open would silently fail.

Clean up closed SCO listen sockets

Fix locking in btsock_sco_signaled

Refactor SCO code to allow closing

SCO socket support in bluedroid

Enforce GKI API buffer usage

Also add another API GKI_queue_length(BUFFER_Q *)

Refactor RFCOMM code; also fixes a random-disconnect / performance bug.

The majority of this change is just style cleanup and eliminating
unnecessary code. There was a bug in the code where, if an entire
data buffer gets successfully written to the app, the rest of the
queue is not drained and the connection may get dropped. This bug
came about due to an early function exit from a switch statement
instead of breaking out of the switch.

Implement memory allocation tracker

Provides an interface to track memory allocations, and assert
bad things like double frees don't happen.

Add common hash functions so we don't have to redefine everywhere

Remove unnecessary layer of indirection

Add back hash_map_test to ositests

Somewhere along the way it got lost

Global counter API for bluedroid

Added encapulsated iterator to hashmap

Also added back hash_map_test which seemed
to have been dropped somewhere along the way.

Modify eager reader to support multi-byte reads

H4: -1 -> INVALID_FD; make uart_fd invalid on close

Implement an HCI HAL for MCT

Unit tested. Needs verification on actual device (i.e. HDP profile).

Scope the lock_count variable in the alarm test harness

Unspecial-case the zero queue in the data dispatcher

Since hash map supports zero keys now this is no longer necessary.

Refactor HCI layer

Refactor the HCI layer for better testability.
A future CL will add a HAL implementation for multi channel transport.

Since btsnoop and vendor are modified, it's not easy to incrementally introduce these changes.

Add eager reader

Add stub testing macros

Refactor alarm tests to use a test harness

This improves the test writing experience for tests that rely on alarm code.

Add some stream skip macros for code clarity

Add peek + dequeue registratation to fixed queue

Add a join function to thread

Make hash map support NULL or 0 keys

Add an allocator interface + default malloc allocator

Switch to an epoll-based reactor implementation.

epoll is a much nicer interface that very closely matches the
reactor interface. It's also thread-safe which makes it a more
suitable choice for bluedroid. As a result of this change,
reactor_register and reactor_unregister are both thread-safe without
introducing any synchronization in user-space.

Fix build after enabling assertions: variable reactor is no longer defined.

Add a function to check if a string represents a bdaddr

Const-correctness for str2bd.

Add a method to create a config object without loading from disk.

Add thread_is_self function

This is useful to compare the current thread against another one.

Add a tool to inject HCI commands into bluedroid.

This tool is very primitive in a number of ways. First, each new
HCI command needs to be carefully crafted. Second, it is entirely
stateless and operates in "fire-and-forget" mode. In other words,
there's no "read-modify-write" capability yet. That being said,
this tool will grow along with the infrastructure in bluedroid.

btpan thread signal function workaround

btsock does not provide a reliable teardown
procedure.  Stale file descriptors may continue
to actively invoke signal callbacks from the
btsock thread.  Extra logic to handle gracefully.

Extend add/subtract operations to atomics

Going to be used for counters

Add 64 bit atomic types

Increase thread queue size above default

The UART enqueues work requests in addition to
the actual work onto the rx_q.  Many small byte
packets received by the UART cause many work requests
which may fill the work queue causing deadlock.

Add fixed queue size option to thread creation

Expose fixed queue capacity

Fix reactor to use prepend, fix reactor tests

Fix socket to use safe reactor registration

Implement a data dispatcher.

This will be useful at layer boundaries.

Hide thread's reactor

Add functions on fixed_queue to check empty state

Replace magic number used instead of MAX_RFC_CHANNEL number.

osi: Support for atomic data types

Fix build: include osi.h from semaphore to get definition of INVALID_FD

Define INVALID_FD to improve readability and reduce likelihood of errors

Allow iterating over sections in a config file and writing config to disk.

These two additional operations will allow us to replace the existing
XML-based config files and switch to a single file format.

Switch over duplicated timer_create routines to new alarm code.

Add a mechanism for debug tools to inject HCI commands into the HCI stream.

Expose thread's reactor so we can register new fds with it.

This change allows a thread's reactor loop to manage additional
file descriptors, e.g. sockets and queues, entirely within that
thread's context.

hci: Renaming of callback functions

Changing name to conform to *_cb convention already
in place.  Further, current names will be re-used
for osi buffer resource management.

hash_map header uses standard integral types so it must include stdint.h.

Wake up reactor whenever a new object is registered/unregistered.

Before this change, if the reactor was blocked waiting for an event
and a new object was registered, it would not wait for events on the
new object until it woke up on its own. This change forces the reactor
to re-evaluate the file descriptor set whenever the object list is
modified.

Introduce a socket class

This socket implementation is limited to TCP server sockets and
provides no mechanism to connect to a remote host. It's primarily
intended to be used to interface with debugging tools. It's designed
to work seamlessly with the reactor (event-driven) model.

Remove some unused code in L2CAP

osi: hash map implementation

Fix gatt_main.c compile error due to log macro change

rfcomm: Check for null pointers on DLC request

The function to get the port control block from the DLCI
explicitly mentions it will return NULL if there is no
active rfcomm connection for the given DLCI.

Release GATT resources upon teardown

Add a system property for libbt-vendor.so initialization timeout

Use a system property, bluetooth.enable_timeout_ms, to specify in ms
how long to wait for libbt-vendor.so to initialize.

am 45085bf6: Merge "Ignore service UUID length if no UUID filter is specified" into lmp-mr1-dev

* commit '45085bf6ed93ea9398047c21bc4b94c8516f9daa':
  Ignore service UUID length if no UUID filter is specified

Merge "Ignore service UUID length if no UUID filter is specified" into lmp-mr1-dev

am 114c9ce4: Merge "Increase SDP buffer from 750 bytes to 4096 bytes" into lmp-mr1-dev

* commit '114c9ce459e9d9e40d55586da1d18d0d65c45c8d':
  Increase SDP buffer from 750 bytes to 4096 bytes

Merge "Increase SDP buffer from 750 bytes to 4096 bytes" into lmp-mr1-dev

Ignore service UUID length if no UUID filter is specified

Bug: 19016326
Change-Id: I6156fee1d6079fec7879f47b6120befe2c4d35cd

am dfb3b2f2: Add flag indicating encrypted connections to btif_dm_get_connection_state()

* commit 'dfb3b2f2595dc16fe7f576a4b0c2f5e73db0327f':
  Add flag indicating encrypted connections to btif_dm_get_connection_state()

Add flag indicating encrypted connections to btif_dm_get_connection_state()

Add a check to btif_dm_get_connection_state() to see if the currently
active connection is encrypted.

Bug: 19186961
Change-Id: I27bcc6e296d272ffd17c0bbe97631ffa4df57ccd

am d6f2fa93: (-s ours) am d6419cd6: am 0a8c803a: am 7f34538a: (-s ours) am 65aec3a3: am 6d429d9d: (-s ours) am 3efb9e2e: DO NOT MERGE Change pairing_cb to assume temporary pairing by default

* commit 'd6f2fa93052c1d9bdf6035ace6b32345013516db':
  DO NOT MERGE Change pairing_cb to assume temporary pairing by default

am e791f732: (-s ours) am ae9d174b: am 408dec19: am 977acabd: (-s ours) am eebc8ab9: DO NOT MERGE Change pairing_cb to assume temporary pairing by default

* commit 'e791f73202a2832b4d20dbc1ecd4b520adbb75d3':
  DO NOT MERGE Change pairing_cb to assume temporary pairing by default

am 0ad6cfde: (-s ours) am ce5ae4ab: am a708936d: DO NOT MERGE Change pairing_cb to assume temporary pairing by default

* commit '0ad6cfdee8857a7bc54d4c3eb98bf1cdc82dc9a7':
  DO NOT MERGE Change pairing_cb to assume temporary pairing by default

am d6419cd6: am 0a8c803a: am 7f34538a: (-s ours) am 65aec3a3: am 6d429d9d: (-s ours) am 3efb9e2e: DO NOT MERGE Change pairing_cb to assume temporary pairing by default

* commit 'd6419cd6309bd20a2ab095c783bb6ad1f208bf4f':
  DO NOT MERGE Change pairing_cb to assume temporary pairing by default

am 0a8c803a: am 7f34538a: (-s ours) am 65aec3a3: am 6d429d9d: (-s ours) am 3efb9e2e: DO NOT MERGE Change pairing_cb to assume temporary pairing by default

* commit '0a8c803a48fe5afa71d1f8660539012e9b1f9642':
  DO NOT MERGE Change pairing_cb to assume temporary pairing by default

am 7f34538a: (-s ours) am 65aec3a3: am 6d429d9d: (-s ours) am 3efb9e2e: DO NOT MERGE Change pairing_cb to assume temporary pairing by default

* commit '7f34538ac0e447a88d1f54ff819c28b3574f0aa4':
  DO NOT MERGE Change pairing_cb to assume temporary pairing by default

am 65aec3a3: am 6d429d9d: (-s ours) am 3efb9e2e: DO NOT MERGE Change pairing_cb to assume temporary pairing by default

* commit '65aec3a3f62c004d7073470f18330007b7ecf230':
  DO NOT MERGE Change pairing_cb to assume temporary pairing by default