Refactoring btif_sock_sdp.c for clarity

Revert "BT MAP: added support for email sharing over BT"

Another change being backed out until the megamerge is over.

This reverts commit c331807e3c3a0e15180a11325e1635ae5268aa39.

Revert "MAP: SDP corruption"

Backing out so our rewrite merges cleanly.

This reverts commit 54e24af853a77a057995ee4f234bae4d490b32e5.

Revert "Forgot device/Android.mk"

Broken build...

This reverts commit d935ef9446990c17412d2b0d5ce158df54377871.

Revert "Remove unused inquiry variables"

Still used by BTA API BTA_DmSetScanParam

This reverts commit d84f0c8c5eb67a45cb2e0993324d1dff8b499b1b.

Forgot device/Android.mk

Slipped by...

Remove always undefined BTM_EIR_UUID_LKUP_TBL

Remove always true definition BTM_EIR_CLIENT_INCLUDED

Remove unused inquiry variables

A2DP sink: reply with success on Start command.

We've observed a Nexus 5 sending back-to-back START requests. If we
accept the first request and move to the STREAMING state, we'll reject
subsequent requests. The phone, however, ignores the first response
(since it has already issued another START) and will not route audio
to Glass.

This is a speculative fix. If we're in the STREAMING state and we
get an incoming START request, accept the request instead of rejecting
it to work around a buggy implementation on the peer.

Bug: 18165461

Fix bdtool compile errors

Add Android.mk to tools/ directory.

Don't touch alarm's contents if it has already been freed.

The current code potentially writes values to the alarm struct after
it has been freed. This change makes sure we check that the struct
is owned by the current thread before touching its contents (read or
write).

Fix NULL pointer dereference in bta_pan_co_tx_path.

This happens when the PAN connection terminates but the stack is
still sending data back to the system. There's a timing component
to this bug; the stack doesn't crash 100% of the time since it
depends on whether the TX path notices the PAN connection was torn
down or not before calling bta_pan_co_tx_path.

Directory layout documentation

Document network ports used by bluedroid.

Document the set of log tags used by bluedroid.

Going forward, we should clean up the log tags to use a consistent
naming scheme. All log tags should be lowercase and should use
underscores instead of hyphens.

avdt: Make sure LCID field is filled out even on incoming connections.

Make bdtest report the total number of tests (sanity + suite)

There are currently 11 tests, but if one failed it would report 1/9 rather than 1/11.

Change bdtest text color gray to the default color

My command prompt outputs white text, so this change makes bdtest
restore the default color after running, instead of leaving it gray.

Add system tests for RFCOMM sockets.

Add a watchdog timer to test suite.

If a test case doesn't make forward progress after some amount
of time, we terminate the process and assume failure. It's important
for test automation that the test suite does not hang forever.

Only colorize test result output if stdout is attached to a tty.

This will make sure redirected output won't have escape sequences
while still leaving the output pretty when running tests by hand.

Make build more strict by adding more warning flags.

This change introduces -Wunused-but-set-variable to help catch
programming errors. It also undefines NDEBUG so assertions are
fatal and defines LOG_NDEBUG=1 to suppress LOGV.

Removing a duplicate call to socket_register.

First entry for bdtool

Comment said BTM_PM_SNIFF_SLOT_WORK_AROUND would be removed someday. Today is that day.

Looks like this has been around for a while, and isn't/shouldn't be used.
restore_pkt_types then becomes obsolete, so removed.

Make hci/ depend on bdroid_CFLAGS

Fix BLE GAP handling after callback refactoring

BLE supported states was moved to controller during the callback
re-factoring CL, but in the commotion I forgot to remove the
supported states variable from devcb.

This meant I didn't realize GAP was looking at the wrong place
for BLE supported states. This CL fixes that.

Remove unused function BTM_ChangeLinkKey & friends

Not used in the codebase. Removed it along with
all its related code.

According to the spec the change link key event is only
sent in response to the change link key command, so
it should be completely safe to remove that logic.

Remove excessive log message from btu_task.

Remove some remaining references to OBX

Android implements the Object Push Profile in java
against RFCOMM sockets with its own implementation of
OBX there.

OBX in Bluedroid isn't compiled, and is missing header files required
to compile it anyway. Removed a number of OBX related headers and updated
a few references where they were used out of place.

Change-Id: Ifaecfb6f85b1dca9055932886af484b6dc107fc1

Remove a bunch of always-false macros

Referenced missing header files, so they wouldn't even
compile if set to true.

Remove definitions for DUN

Missing header files, etc so it wouldn't compile anyway.

Remove always false TCS_INCLUDED macro & friends

Referenced non existent headers, so would not compile if set to true.

Remove remaining references to GOEP

FALSE by bt_target. Header files never used. Missing header files.

Remove unused header utfc.h

Remove a bunch of unused functions and definitions from HCIC

Left the BLE stuff alone for now.

Remove unused macros LMP_TEST and BTISE

Disambiguate event mask setting for bluetooth DUT mode

The nuances of addressing a string literal are tricky, so it
looks like we're using a char ** when in reality it's a pointer to
the array containing the literal, which converts just fine.

Removing the address-of makes it easier to see correctness.

Remove unused code in GAP

Left the BLE stuff for now, since some of it appears to
be used at least right now.

Don't call null callback in vendor lib

Don't require a vendor library command transmit to provide a callback.
If the vendor library provides a null callback just ignore it.

Fix accidental hardcoding of 2+ extended feature page numbers

Happened to be ok on devices with 1 or 2 feature pages, but for
devices with 3 or more feature pages it was always loading the
page at index 1 for pages past the page at index 0, causing
an infinite loop, causing bluetooth enable to time out.

Move properties data primitive into btcore

This is a valuable data structure that could be
used widely within the stack.  It's used by
bdtest and bdtool, and could be used by any
other application built on top of the stack.

Add comments to the device features and event mask structs

Remove L2CAP_CORRUPT_ERTM_PKTS macro

In general, test code and production code should not coexist in the same file.

Refactor bta_jv functions

Remove unused L2CAP and SDP code to reduce distraction from unused code
path.

In general, if we need any of this behavior in the future, it's best
to talk directly to the apis instead of through another layer.

Remove duplicate local bdaddr management functions

Refactor btm_devctl reset sequence

* Controller bring up on blockable thread now
* Removed some duplicate and commands during controller bring up
* The code to make commands for controller bring up is smaller and better

Prune BTM_ReadLocalVersion

It was called once, but the version info read back was
never used. The status returned was used, but that was
implicitly getting whether the device was up or not so
replaced with a call to BTM_IsDeviceUp.

Remove <BT 2.0 workarounds

Prune BTM_SetSecurityMode and GAP_SetSecurityMode

BTM_SetSecurityMode was only used from GAP_SetSecurityMode,
and GAP_SetSecurityMode was never used.

Add temporary callbacked wrapper for module startup

When all is said and done with the module conversions, all lifecycle functions
will run on the lifecycle managment thread, which can be blocked while waiting
for futures.

This CL means new modules can depend on that fact during start_up, alllowing
newly converted modules to be spliced into the existing startup callback madness
without having to sacrifice clean implementation.

We can add other callbacked lifecycle function variants as necessary.

Removing unnecessary BTAPI #defines.

Add a futured version of the HCI transmit_command

This will allow the controller restart sequence to be serialized into a single function.

Remove some < BT 2.1 code & repeated init

Simple pairing is already enabled and the BLE event mask
is already set by this point.

Remove BTM_SetDeviceClass call in devctl

BTM_SetDeviceClass is called later on in bta_dm_sys_hw_cback
during BTA_SYS_HW_ON_EVT using the real device class. No need
to set an "init" device class as far as I know.

Remove duplicate L2CAP_NUM_FIXED_CHNLS

Remove unused hci_cmd_cb in btu

An earlier change obviated the need for it, but I
forgot to remove it at that time.

Modulizification of bt_utils

Modulizification of btif_config

Remove BTTRC_INCLUDED

It used functions that didn't exist, so there's no way it
would compile if it was defined to true.

Also removes some defines that would nop without it.

Modulizification of GKI

Modulizification of the HCI layer

Shutdown is managed by the stack manager now, so we don't need to lock
around it. The fetching of the device address is a bit hacky right now,
until it becomes part of its own module.

Add paired cleanup routine for logmsg

Move module_management_stop to clean_up

Module management isn't done when the stack is shut_down (bluetooth disabled)

Refactor btsnoop and stack config into modules

Moves stack config out of the combined bte_config, and into
its own module.

Makes btsnoop more self sufficient and removes uneccessary
levels of indirection.

Refactor logging slightly into a (temporary) module to disassociate
from the direct calls from config. Eliminates some useless stuff in
the module as well.

First pass at implementing modules

This first step creates the notion of a module and corresponding
lifecycle functions, with helpers to simplify working with them.

Once everything is converted over to this module format, then we
can make the stack manager automagically find the correct order for
init/start_up/shut_down/clean_up

Unregister BTU queues upon shutdown

This ensures that the next run doesn't try to access memory
that is no longer valid.

Remove HCI access macros

And some other minor bt_target cleanup.

Remove GKI task code

Replace bt media GKI task with thread_t

Remove btu_task and replace with bt_workqueue reactor

Logging cleanup

Remove unused definitions from bt_target.h

Eliminate memory leaks and unsafe callbacks in BTA and BTE.

Alarms were being added to a hash map but were not freed during
teardown. This results in both memory leaks and potential access
of torn-down data (in case the still-armed alarm fires after
teardown completes).

Add string hashing function using djb2

Disassociate hash algorithm with hash function name

This will make it easier to select the correct one for the job, and also
make it possible to substitute a better one in the future if need be without
having to change the function name.

kunth is duplicated, once for integer hashing and once for shallow pointer hashing, to
make usage more readable.

Rewerite btif_sendmsg in terms of thread_post

Fix alarm setting for newer deadlines

If an alarm is being set and is the newest deadline
among a set of existing alarms, it would be appended
after the first existing alarm rather than prepended
in front.

This in turn meant the root alarm wasn't getting
rescheduled correctly, especially if the old
deadline was rather distant in the future.

(This was causing the bluetooth disable sequence to
fail intermittently.)

Post adapter_state_changed events to the JNI workqueue

Otherwise, JNI rejects the callbacks. Fix for b/17655336

Don't send adapter change until stack manager updates state

Since the callback occurred before the stack manager updated its
state to say the stack was fully loaded, there was a chance that
the adapter name change would be attempted before the state was
updated.

This would cause the adapter property change function to return
a "not ready" status.

Eliminate memory leak in stack initialization.

A heap allocated semaphore was not freed after use.

Fix bug where some BTU timer callbacks weren't propagated in a timely manner.

This change sends an event to the BTU task after enqueuing a timer
callback into its timer queue so that the BTU task thread wakes up
and dispatches the callback ASAP. Otherwise, the callback will be
dispatched at a random time or sometimes not at all.

All bdtest cases pass after this change.

Fix timeout values for BTA system timers.

bta_sys_start_timer assumes a timeout value in GKI ticks but the
callers are passing in a timeout value in milliseconds. This change
updates the implementation to use units of milliseconds.

Add back timer list entry in_use field

I thought this field was used for internal
bookkeeping, but it turns the higher layer
protocols use it to condition if a timer
should be started or stopped.

Enable assertions.

Remove btif enable state management

It's no longer necessary with the stack manager.

With these changes I don't think we'll be able to handle clean_up
as a result of an enable timeout coming down from Java.
But I think I'm ok with that.

At that point we're already hosed because of something else
which should be fixed. And if we wanted to handle clean_up during
start_up we'd violate the mutually exclusive guarantees the stack
management thread makes about init/start_up/shut_down/clean_up.

Make stack manager functions not finish until action complete

It's a bit hacky at the moment, but this incremental update means the
stack manager booleans should be correct now and we can start removing
code that relies on the btif states.

s/async_result/future/g and also improve the future API

Freeing now happens implicitly upon await, and immediate
futures will allow a uniform usage pattern for calls
that are async and calls that are not.

Make semaphore_free idempotent

Passing NULL before would lead to a segfault.

Move all stack init/start_up/shut_down/clean_up on a single known thread

Execution happens on the same thread, but the next step will be to
ensure each of the functions do not complete until start_up/shut_down/clean_up
complete. That will ensure each is mutually exclusive.

async_result abstraction for signaling async return values

This will be useful when signaling success/fail during individual
asynchronous steps of stack start up and shut down.

Fix packet fragmenter so it doesn't request acl sizes unless it's actually an ACL packet

During the startup process, commands are sent before the ACL sizes are fetched.
With the change that we now assert the acl sizes are fetched when you request
them from the controller, the assertion was being triggered during that phase.

When I was rebasing the waiting changes and also making the assertion change,
I should have been more careful to test this. mea culpa

Also simplifies that code path just a bit.

Shuffle HCI layer internal functions into a more logical ordering

No actual code additions/deletions, other than additional forward declarations

Refactor bte_main into more of a shell.

Moves all HCI initialization into hci_layer. Removes superfluous exposed
functionality on the HCI layer (like turning the chip on/off and logging)

Also reorganizes some of hci_layer to group related functions together.

Add non-repeating timer wrapper for alarm

Refactor the hci layer to use it.

The timer wrapper allows you to specify the duration, etc at construction time
rather than at start time, making some usage patterns of alarm more convenient.

HCI layer cleanup

- x_interface_t started to get annoyingly verbose, changed to x_t
- buffer_allocator is standalone now, no longer part of bte_main
- new controller_t to handle controller start up and hold controller state
- new hci_packet_factory to make packets, and hci_packet_parser
  to read information out of packets.

Remove must-always-be-true BTU_BTA_INCLUDED

Change-Id: I7ddfb9d611b7f4a2787e0f4bec4a090cd9efc202

Add a check for a NULL root element from tinyxml2's XMLDocument.

This shouldn't really be necessary since a parsed document
should have at least a root element. In practice, however, tinyxml2
sometimes returns success on LoadFile() and NULL on RootElement().

Bug: 17504829