emulator: cmd_status() using send_packet() for run hooks easily

Now like cmd_command(), cmd_status() directly uses send_packet() instead
of send_event(), consequently cmd_status() must build the hci packet
without help of send_event(). With this change the events sent by
default_cmd() no more use send_event(), who is a good place to run hooks
for BTDEV_HOOK_POST_EVT. And the functions cmd_command() and
cmd_status() can run hooks for BTDEV_HOOK_POST_CMD.

emulator: Split default_cmd() for better command and event handler

Now except for command complete event and command status event, all
other are handled by default_cmd_completion(). With this we can easily
add hooks in some points.

emulator: Add support for delete a hook

emulator: Export command and event hooks for hciemu user

emulator: Add hook only if there aren't other with same type and opcode

emulator: Add support for pre/post command/event hooks

Now who use emulator can add hooks to manipulate hci packets before
and after process commands and send events.

attrib: Reduce scope of variable declarations in char_discovered_cb()

This makes it easy to identify where variables are being used.

attrib: Allow dec_read_resp() to just check if PDU is valid

If a NULL "value" parameter is given, only check whether PDU is valid,
without copying attribute value to a buffer.

attrib: Add encode/decode execute write support

Add functions for encoding/decoding Execute Write Request PDUs.

attrib: Add encode/decode prepare write support

Add functions for encoding/decoding Prepare Write Request PDUs.

attrib: Remove unused parameter from enc_write_resp()

attrib: Fix coding style for various enc_*/dec_* functions

Fix coding style for enc_read_blob(), enc_read_blob_resp(),
dec_read_resp(), enc_error_resp(), enc_find_info_req(),
enc_find_info_resp() and enc_prep_write().

attrib: Simplify prepare_write()

All parameters can be retrieved from struct write_long_data fields,
therefore there is no need to pass them separately.

attrib: Fix coding style in prepare_write_cb()

attrib: Do not use Write Command in gatt_write_char()

Previously, if no callback was given to gatt_write_char(), it was
assumed that a "Write Without Response" (which uses Write Command)
should be used instead of Write Request.

This "shortcut" is unnecessary (there is gatt_write_cmd() for the
situations where Write Without Response is required) and just duplicates
code.

This commit also fixes the few places where gatt_write_cmd() should be
used.

attrib: Rename gatt_find_info() to gatt_discover_char_desc()

The old name referred to the underlying ATT PDU (Find Information
Request and Find Information Response) and not to the actual GATT
procedure (Discover All Characteristic Descriptors).

gatttool: Remove unused code

Since commit 6a3a89774c6738df4d5b77636c4ce747b7497ae9 the code
removed here has no sense.

audio/player: Fix attempting to set the same setting value

This cause unnecessary commands to be sent over the air so instead the
code should just return success as nothing should be changed.

audio/media: Fix not updating position

The status may not be 'playing' as MPRIS documents it as 'Playing'
instead, to make this less prone to typos like this the code now
uses strcasecmp.

audio/AVRCP: Fix registration of VOLUME_CHANGED

First check with GetCapabilities if the event is really support before
registering.

audio/AVRCP: Rework role detection

This makes AVRCP agnostic to any roles so the same session can be used
both for controller and target role.

cyclingspeed: Fix MultipleLocationsSupported property name usage

MultipleLocationsSupported is the correct property name, as implemented
in profiles/cyclingspeed/cyclingspeed.c.

mailmap: Add Martin due to one commit with incomplete name

attrib: Fix interactive gatttool segfault

Passing the special character single quote:

$gatttool -I
[   ][                 ][LE]> '
Segmentation fault (core dumped)

Checking g_shell_parse_argv fail status (return FALSE on failure), works
fine.

build: Build a simpler version of libgdbus-internal

build-sys: Don't build gdbus several times

We are currently compiling gdbus 3 times: one for tests, one for
bluetoothd and another for obexd.

$ ls gdbus/*watch.o
gdbus/bluetoothd-watch.o  gdbus/obexd-watch.o  gdbus/watch.o

Instead of prepending $(gdbus_sources) to several _SOURCES variable, use
a convenience library.

build-sys: Rename lib suffix to -internal

libbluetooth-private -> libbluetooth-internal

lib: Remove duplicated HSP_HS_UUID definition

There is an identical definition on the same header file (next to
HSP_AG_UUID).

tools/bluetooth-player: Make show-item command print Player property

This makes show-item command to print Player property that indicates
which player the item belongs to.

doc/media-api: Update documentation of MediaItem1

This adds documentation of property Player

audio/player: Add Player property to MediaItem1

This proper refer to the player object path which the item belongs to

doc/media-api: Update documentation of MediaFolder1

This adds missing access types to properties NumberOfItems and Name

core: Fix default authorize value for external profiles

Previously the value of "authorize" was essentially ignored in the
settings table. This wasn't very noticeable since almost all profiles
use the default true value. Since the default value of "authorize" is
true it cannot be left out from the default settings table entries
(leaving it out would set it implicitly to false).

doc/media-api: Update documentation of MediaPlayer1

This adds documentation of properties Type and Subtype

input: Add debug logs for attio connection status in HoG code

input: Fix minor coding style issue in HoG code

core: Add debug log for attrib disconnection reason

core: Remove unnecessary btd_device reference count in browse_req

The browse request is "owned" by the btd_device structure so it doesn't
make sense to have browse_req keeping an extra reference for the
btd_device.

core: Use proper header for S_IRUSR/S_IWUSR

According to open() manual one should include <sys/stat.h>.
This fix build with bionic library:

target thumb C: libbluetoothd <= external/bluetooth/bluez/src/attrib-server.c
external/bluetooth/bluez/src/attrib-server.c: In function 'write_value':
external/bluetooth/bluez/src/attrib-server.c:922:26: error: 'S_IRUSR'
 undeclared (first use in this function)
external/bluetooth/bluez/src/attrib-server.c:922:26: note: each undeclared
identifier is reported only once for each function it appears in
external/bluetooth/bluez/src/attrib-server.c:922:36: error: 'S_IWUSR'
undeclared (first use in this function)

plugins/policy: Limit the number of retries to sink profile

The policy should only retry a limit number of times.

plugins/policy: Limit the number of retries to source profile

The policy should only retry a limit number of times.

audio/control: Enable initiate connection to CT role

Since the policy uses service API to initiate the connections this needs
to be enabled for both CT and TG roles.

audio/sink: Move stream retry logic to policy plugin

policy plugin is a better place to have such a logic as it is already
handling other connection policies related to such profiles.

audio/source: Move stream retry logic to policy plugin

policy plugin is a better place to have such a logic as it is already
handling other connection policies related to such profiles.

plugins/policy: Reword audio policy code in a simple plugin

This moves audio device policy to a plugin and removes any dependency of
audio plugin to it.

audio/main: Remove dependency on struct audio_device

This is part of the work necessary to completely remove
struct audio_device

audio/manager: Remove dependency on struct audio_device

This is part of the work necessary to completely remove
struct audio_device

audio/transport: Remove dependency on struct audio_device

This is part of the work necessary to completely remove
struct audio_device

audio/media: Remove dependency on struct audio_device

This is part of the work necessary to completely remove
struct audio_device

audio/source: Remove dependency on struct audio_device

This is part of the work necessary to completely remove
struct audio_device

audio/sink: Remove dependency on struct audio_device

This is part of the work necessary to completely remove
struct audio_device

audio/A2DP: Remove dependency on struct audio_device

This is part of the work necessary to completely remove
struct audio_device

audio/control: Remove dependency on struct audio_device

This is part of the work necessary to completely remove
struct audio_device

audio/AVRCP: Remove dependency on struct audio_device

This is part of the work necessary to completely remove
struct audio_device

audio/AVDTP: Remove dependency on struct audio_device

This is part of the work necessary to completely remove
struct audio_device

audio/AVCTP: Remove dependency on struct audio_device

This is part of the work necessary to completely remove
struct audio_device

audio/control: Reduce dependency on struct audio_device

This is part of the work necessary to completely remove
struct audio_device

audio/source: Reduce dependency on struct audio_device

This is part of the work necessary to completely remove
struct audio_device

audio/sink: Reduce dependency on struct audio_device

This is part of the work necessary to completely remove
struct audio_device

audio/control: Use service user_data for private data

This remove the need of forward declaration of struct control and prepare
for a complete removal of struct audio_device.

audio/source: Use service user_data for private data

This remove the need of forward declaration of struct source and prepare
for a complete removal of struct audio_device.

audio/sink: Use service user_data for private data

This remove the need of forward declaration of struct sink and prepare
for a complete removal of struct audio_device.

core: Clear temporary flag only when a real profile is connected

input: Fix rejecting connections from unknown devices

core: Fix incorrect update of dev->svc_refreshed

input: Fix service disconnected state transition

Without this patch the input service would remain in connected state
whenever a remotely initiated disconnection happens.

core: Remove unnecessary device_resolve_svc function

input: Add some extra debug statements to clarify connection flow

core: Fix returning correct D-Bus reply when refreshing services

core: Clear device->attrib pointer before calling g_attrib_cancel_all

The g_attrib_cancel_all function may call functions such as
find_included_cb that should at this point no longer have access to the
GAttrib instance and try to do operations with it. Therefore it is
safest to ensure that device->attrib is no longer available in these
functions.

core: Remove redundant includes from sdpd-*.c

core: Refresh SDP services if no matching profiles were found

core: Simplify code flow in search_cb

core: Implement passive SDP discovery after connecting known services

core: Remove unnecessary secure paramter from device_browse_primary

core: Remove unused reverse parameter from device_browse_sdp

core: Set low priority for SDP data

In the case that there are other channels over the same ACL we want SDP
transactions to interfere as little as possible.

core: Use g_dbus_send_reply over dbus_message_new_method_return

core: Remove profile removal detection

This code has been nonfunctional since 5.0 and it seems no-one cares for
it, so just remove it.

core: Fix incorrectly restarting service discovery after pairing

If services are already discovered through a Device1.Connect() call we
should not retry discovering them when Device1.Pair() is called.

core/device: Don't call btd_service_disconnect on device_remove

device_remove is called to cleanup/free the device objects so the
services should be disconnected before hand.

core/device: Fix crash while freeing services list

btd_service do alter its state on service_remove which can cause
plugins to attempt to access services list which may have freed some
services already.

To fix this the code now updates the list in place so the services are
first removed from services list before calling service_remove.

core/service: Rename service_shutdown to service_remove

This rename service_shutdown to service_remove to make it more similar to
other internal APIs such as device_remove which only do object
cleanup/free and do not have any disconnect logic.

lib: Add range check for SDP_SVC_ATTR_RSP/SDP_SVC_SEARCH_ATTR_RSP

This is an improved version of recently reverted commit 1796f00e8465.
Response size is verified against minimal allowed value only if it is
complete response. If response is partial it is allowed by spec that
it will be split in arbitrary manner.

Verified against Nokia BH217 on which original commit caused
regression.

obexd/client: Add Target property to Session interface

Target property is already documented property but it was not
implemented.

audio/source: Fix not notifying service about connection state

btd_service_connecting_complete should be called whenever the service
is connected otherwise the service state will not be consistent.

audio/sink: Fix not notifying service about connection state

btd_service_connecting_complete should be called whenever the service
is connected otherwise the service state will not be consistent.

emulator: Implement LE advertising report

When a virtual device starts a LE advertising, emulator searches for
other virtual devices that are in scan mode, in order to send adv data
to these devices.
Inverse goes when LE scan is enabled. Emulator searches virtual devices
that are in advertising mode and copy adv data to them.

emulator: Dummy LE set advertising parameters

Implement basic LE set adv parameters command.

emulator: Disallow LE set scan params command when scan is enabled

When LE scan is enabled, the command LE set scan parameters should
return a command disallowed in status of command complete event.

emulator: Store LE scan state of virtual controller

The LE command set scan enable can change the scan state of virtual
controller.

emulator: Implement basic LE set adv enable command

Store advertising state of virtual controller in btdev struct.

emulator: Store LE advertising data length in btdev

LE advertising data length will be used for emulating reports in other
virtual devices.

obexd: Re-add D-Bus session properties

This got lost when switching to org.freedesktop.DBus.Properties.

monitor: Print UUID-32 and UUID-128 strings

monitor: Add stub for UUID-128 to string conversion

Release 5.7

build: Update library version

obexd/MAP: Fix assigning pointer not value to 0

The fix introduced by 455fb131f08cddc9759c877e38ebff3b4bd975b2 would
leave err value uninitialized and set the pointer to 0 which doesn't
make any sense.

lib: Update company identifiers