adapter: Fix registering adapter with no address

adapter->bdaddr is set later in read_info_complete and current check
always returns false. Check against bdaddr received in command response
instead and fail if it is all zeros.

A2DP: Fix invalid write

Invalid write of size 8
   at 0x41F297: setconf_cfm (a2dp.c:567)
   by 0x42526B: session_cb (avdtp.c:3176)
   by 0x39B0847A54: g_main_context_dispatch (in /usr/lib64/libglib-2.0.so.0.3400.2)
   by 0x39B0847D87: ??? (in /usr/lib64/libglib-2.0.so.0.3400.2)
   by 0x39B0848181: g_main_loop_run (in /usr/lib64/libglib-2.0.so.0.3400.2)
   by 0x409C3E: main (main.c:583)
 Address 0x555fda8 is 40 bytes inside a block of size 88 free'd
   at 0x4A077A6: free (vg_replace_malloc.c:446)
   by 0x39B084D79E: g_free (in /usr/lib64/libglib-2.0.so.0.3400.2)
   by 0x41E217: setup_cb_free (a2dp.c:191)
   by 0x41E410: finalize_config (a2dp.c:234)
   by 0x41F296: setconf_cfm (a2dp.c:566)
   by 0x42526B: session_cb (avdtp.c:3176)
   by 0x39B0847A54: g_main_context_dispatch (in /usr/lib64/libglib-2.0.so.0.3400.2)
   by 0x39B0847D87: ??? (in /usr/lib64/libglib-2.0.so.0.3400.2)
   by 0x39B0848181: g_main_loop_run (in /usr/lib64/libglib-2.0.so.0.3400.2)
   by 0x409C3E: main (main.c:583)

core: Fix calling profile .connect multiple times

Sometimes profiles may complete the connection in different order
than expected so the code has to check if it was actually the
current pending profile to proceed to the next.

hostname: Fallback to static hostname if pretty hostname is not set

If pretty hostname is not set fallback to static hostname (if it is
set). If static or pretty hostname is not set appropriate properties
are empty strings not NULLs. This behaviour is recomended by hostnamed.

hostname: Fix setting adapter name to empty string

If pretty hostname is not set empty string is received. Don't update
adapters' names is such case.

adapter: Always set new default adapter if current one is removed

In case hci_get_route() failed mark first adapter on list as default.
This make sure default adapter is always set and that
btd_adapter_get_default will not return NULL if at least one adapter
is registered.

adapter: Mark adapter as default before probing drivers

Drivers may depends on adapter being default or not. This fix hostname
plugin setting default adapter name to 'foo #1' instead of 'foo' if
pretty hostname was received before probing adapter drivers.

adapter: Add is_default field to struct btd_adapter

Instead of global default_adapter_id variable use is_default field
to indicate if adapter is default one.

core: Remove unused code from device_browse_sdp()

The "search" parameter was always NULL, and therefore code depending on
it being non-NULL will never be reached.

lib: Check if SDP buffer has enough data on partial responses

Before manipulating data from previous partial responses, make sure the
buffer has enough data.

lib: Add range check for SDP_SVC_ATTR_RSP/SDP_SVC_SEARCH_ATTR_RSP

According to SDP spec, the byte count fields for these PDUs have a valid
range of 0x0002-0xFFFF.

lib: Fix buffer overflow when processing SDP response

rsp_count is either read or calculated from untrusted input, and
therefore needs to be checked before being used as offset. The "plen"
variable is appropriate because it is calculated as the sum of fixed and
variable length fields, excluding the continuation state field, which
has at least 1 byte for its own length field.

core: Use device_set_trusted() in set_trust()

core: Add a device_set_trusted() function

This will be useful for plugins that do their own device setup (e.g.
USB-based pairing).

neard: Add support for setting power state in RequestOOB reply

This allows neard to properly set Bluetooth carrier power state in
handover message.

adapter: Add btd_adapter_get_connectable function

neard: Check if adapter is powered in PushOOB

For PushOOB adapter needs to be powered as bluetoothd daemon is
expected to perform action on success.

adapter: Add btd_adapter_get_powered function

neard: Move pairable check from check_adapter

If device is already paired there is no need to fail with handover
if adapter is not pairable.

neard: Add ability to parse 'State' field

This contains hint for power state of remote device Bluetooth adapter.

neard: Refactor message processing

This refactor code for message processing for future feature addition.
nokia.com:bt and EIR processing is now separated from performing
actions based on received data.

neard: Adjust errors to latest API changes

neard Handover API was stripped of not really usefull error codes.

gdbus: Add g_dbus_proxy_set_removed_watch

tools: Fix hcidump parser of AVRCP GetItemAttributes command

tools: Fix hcidump parser of AVRCP media attributes

tools: Print UID of track changed event

Before 1.4 this was considered reserved but now it is being used for
UID of the current track.

gdbus: Fix missing PropertiesChanged signal

If D-Bus ObjectManager is not supported, InterfacesAdded signal
checking needs to be ignored otherwise PropertiesChanged signal
will never be sent.

tools: Track transport state if player status is not supported

This makes PlayPause method to work properly even if remote device
supports only AVRCP 1.0.

doc: Add missing device RSSI property

RSSI property was not documented in device API.

doc: Remove CancelConnect function from device API

This method is not implemented and Disconnect can be used instead.

doc: Fix device disconnect description

Document what Disconnect() function actually does.

doc: Clarify device connect description

Describe behaviour when some profiles are already connected. Clarify
that it returns success if at least one profile was connected.

transport: Fix inconsistent transport state

If a2dp_resume() fails, the transport state should not be modified. It
would otherwise enter an incosistent state where the transport will be
impossible to resume, since acquire() will see the transport in
TRANSPORT_STATE_REQUESTING state and will thus return
btd_error_not_authorized().

monitor: Add commands, events and features from CSA4

man: Remove misleading reference to main.conf man page

gdbus: Don't call property changed callback during client init

When the client uses ObjectManager to init properties, do not call
property changed callbacks. They should only be called once the proxy
added has been successfully signaled since the proxy itself provides
a full copy of available properties.

tools: Add mgmt-tester command line option to control power on wait

test: Add -p/--push option to map-client

This option can be used to push messages, it takes as a parameter a
file location which should contain the message in bmsg format.

obexd: Add PushMessage

Push message has been implemented similar to send file (OPP),
the message to send (in bMessage format) is read from a file.

test: Fix missing call to parser.parse_args() in test-proximity

shared: Fix use after free in read_watch_destroy

read_watch_destroy is called when received_data returns FALSE.
free mgmt in read_watch_destroy instead of received_data to avoid
use after free.

Invalid write of size 4
   at 0x8051604: read_watch_destroy (mgmt.c:271)
   by 0x48C7468E: g_source_callback_unref (gmain.c:1457)
   by 0x48C77287: g_main_context_dispatch (gmain.c:2723)
   by 0x48C774FF: g_main_context_iterate.isra.22 (gmain.c:3290)
   by 0x48C77962: g_main_loop_run (gmain.c:3484)
   by 0x805393E: tester_run (tester.c:784)
   by 0x804D1C7: main (mgmt-tester.c:2558)
 Address 0x4039b80 is 16 bytes inside a block of size 76 free'd
   at 0x4007F0F: free (vg_replace_malloc.c:446)
   by 0x48C7D44B: standard_free (gmem.c:98)
   by 0x48C7D607: g_free (gmem.c:252)
   by 0x8051BB6: received_data (mgmt.c:337)
   by 0x48CBA72E: g_io_unix_dispatch (giounix.c:167)
   by 0x48C7715A: g_main_context_dispatch (gmain.c:2715)
   by 0x48C774FF: g_main_context_iterate.isra.22 (gmain.c:3290)
   by 0x48C77962: g_main_loop_run (gmain.c:3484)
   by 0x805393E: tester_run (tester.c:784)
   by 0x804D1C7: main (mgmt-tester.c:2558)

monitor: Decode supported commands result

monitor: Fix wrong supported commands bit value

tools: Fix mgmt-tester LE expected supported settings

tools: Add basic mgmt_set_local_name EIR test case

core: Use more compact definitions for zero-tail byte arrays

tools: Add basic mgmt_set_local_name success test cases

tools: Generalize alternate mgmt event handling for mgmt-tester

audio: Fix uint64_t format specifier

Release 5.2

lib: Update list of company identifiers

AVRCP: Set addressed player as browsed player

This send SetBrowsedPlayer after getting the player details.

AVRCP: Split event handing to its own functions

AVRCP: Get player list if supported

If addressed player changed is supported get the list of players

AVRCP: Register to addressed player changed event if supported

This enables registering for addressed player changed event and handle
the interim response setting the player id and uid counter to the current
player.

AVRCP: Avoid repeating command due to changed event

Currently we are repeating the same command twice for notification, first
for changed and then latter for interim response.

To avoid this the code now just do it for interim responses, changed
responses just trigger another registration.

media-api: Add org.bluez.MediaFolder1

This interface adds support for browsing and searching in the player's
storage using AVRCP 1.4/1.5.

Some remarks about the design:

  - Exposing UIDCounter and UIDs was considered, but the spec seems to have
    missed to define the player's id persistency. There are also the fact that
    UIDCounter alone does not guarantee persistency across sessions and do not
    provide what exact items have changed, so in the end exposing these
    details will bring almost no value.
  - Indexing or caching the whole media library is not recommended, Bluetooth
    is too slow for that and even vendors such as Apple do not recommend doing
    it, so the only items keep in cache are the current listed ones.
  - Addressed vs Browsed player is done implicitly when accessed, this was done
    to simplify the API and avoid confusions between applications and players.

AVDTP: Fix memory leak

56 (16 direct, 40 indirect) bytes in 1 blocks are definitely lost in loss record 173 of 263
   at 0x4A0883C: malloc (vg_replace_malloc.c:270)
   by 0x4C8568E: g_malloc (in /usr/lib64/libglib-2.0.so.0.3400.2)
   by 0x4C9A7F1: g_slice_alloc (in /usr/lib64/libglib-2.0.so.0.3400.2)
   by 0x4C9BB92: g_slist_append (in /usr/lib64/libglib-2.0.so.0.3400.2)
   by 0x422072: send_req (avdtp.c:2569)
   by 0x422D0B: avdtp_discover (avdtp.c:3380)
   by 0x41D16C: source_setup_stream (source.c:307)
   by 0x41D1EF: source_connect (source.c:331)
   by 0x465411: connect_next (device.c:1073)
   by 0x466C92: device_profile_connected (device.c:1097)
   by 0x4629EB: ext_connect (profile.c:950)
   by 0x44AECB: connect_cb (btio.c:230)

core: Fix missing return statement in ext_svc_complete

device: Fix not marking svc_resolved when loaded from storage

When the device is loaded from storage, we should mark it as it
services were resolved.

core: Fix trying go stop passive scanning multiple times

core: Add debug log to stop_passive_scanning_complete

network: Fix user data handling on unregister

User data in unregister_server() is a network_adapter struct. Unregistering
using full UUID will now also work.

gdbus: Fix unpublished interface signal emission

If we haven't published an interface yet (i.e. it's in the data->added
list), we should just ignore any property changed indications as the
values for the properties will anyway be part of the InterfacesAdded
signal.

emulator: Fix BR/EDR-only and LE-only supported command checks

emulator: Add command line options for selecting controller type to btvirt

emulator: Use BR/EDR/LE vhci type as default for btvirt

tools: Add LE-only test case support to mgmt-tester

tools: Add BR/EDR-only test case support to mgmt-tester

tools: Fix mgmt-tester BR/EDR/LE test macro naming

emulator: Fix controller version for BR/EDR type

AVRCP: Get track duration using GetPlayStatus command

Some stacks, notably broadcom, don't send track duration together with
other metadata so it has to be set using GetPlayStatus.

A2DP: Fix not setting setup error to NULL

Once the error is informed to the callback it should be properly freed and
set to NULL otherwise a subsequent operation may end up reusing the same
error which might lead to crashes.

AVCTP: Fix memory leak

651 (280 direct, 371 indirect) bytes in 7 blocks are definitely lost in loss record 306 of 332
   at 0x4A06B2F: calloc (vg_replace_malloc.c:593)
   by 0x319724D706: g_malloc0 (in /usr/lib64/libglib-2.0.so.0.3400.2)
   by 0x418A54: avctp_send_browsing_req (avctp.c:1457)
   by 0x41A330: avrcp_handle_event (avrcp.c:1998)
   by 0x417810: session_cb (avctp.c:702)
   by 0x3197247A74: g_main_context_dispatch (in /usr/lib64/libglib-2.0.so.0.3400.2)
   by 0x3197247DA7: ??? (in /usr/lib64/libglib-2.0.so.0.3400.2)
   by 0x31972481A1: g_main_loop_run (in /usr/lib64/libglib-2.0.so.0.3400.2)
   by 0x409D6E: main (main.c:583)

AVCTP: Fix allways destroying browsing channel

This reverts commit 53d3fc3fad5152167458625ad2acf19070cdf26a that leads
to always destroy the browsing channel because the callback changes the
state.

To fix session_browsing_cb naw takes care of destroying the channel if it
disconnects.

tools: Fix mpris-player creating busname starting with digit

D-Bus does not allow a bus name starting with digits after . so the code
now prefix with bt_ if the device name starts with a digit.

tools: Use device Alias instead of Name in mpris-player

User may have set a different Name using Alias, Alias should always
be set.

hcidump: Fix parsing of SetBrowsedPlayer

hcidump: Fix parsing of GetCurrentPlayerApplicationSettingValue

hcidump: Distinct Control and Browsing AVCTP channels

This prints the respective channel of the trafic

tools: Update tester apps to use new hciemu API

shared: Provide hciemu type in hciemu_new()

emulator: Create separate BR/EDR, BR/EDR/LE and LE vhci types

emulator: Create separate server BR/EDR/LE and BR/EDR types

shared: Create BR/EDR/LE device type by default

emulator: Create separate BR/EDR/LE and BR/EDR device types

TODO: Remove completed passive scanning entry

core: Stop passive scanning when removing last item from connect_list

core: Refactor passive scanning stopping into its own function

core: Fix missing check for main_opts.reverse_sdp

core: Fix checking for correct async operation id

core: Move SDP record fetching to NewConnection sending function

core: Do authorization and service resolving in parallel

core: Force discovery timer to run immediately if a profile needs it

core: Fix SDP resolving for incoming external profile connections

This patch ensures that the reverse SDP procedure completes before we
accept incoming profiles for external profiles.

core: Add device API for waiting for SDP completion

core: Remove unnecessary assignment to device->svc_resolved

The device_svc_resolved() function already takes care of setting
device->svc_resolved to true.

TODO: Add item for improving stopping passive scanning

core: Fix updating discovery state when stopping passive scanning

core: Remove unnecessary adapter->passive_scanning variable

The adapter->discovery_list can function equally well as an indicator of
whether normal discovery or passive scanning is in progress. Normal
discovery always overrides passive scanning so if there's one or more
entries in discovery_list we must be performing normal discovery and not
passive scanning.