A2DP: Mark start flag if resume happen while in configured state

If SEP is in configured state that means OPEN is about to happen so just
mark start flag and send START once OPEN completes.

tools: Add support for registering external player

This adds --external/-e to register remote players found as local player

tools: Fix hcidump parser of AVRCP for SetBrowsedPlayer command

obexd: Handle absolute paths in obc_session_setpath

For absolute paths (that begin with '/'), obc_session_setpath gets called
twice to reset to the root folder.
This is caused by an empty first element in the folder list created by g_strsplit.

This solution sets the index to the folder array correctly and ignores
empty folder names. This fixes as well paths with double slashes.

Trace for 'SetFolder /telecom':

< ACL data: handle 21 flags 0x00 dlen 21
    L2CAP(d): cid 0x0040 len 17 [psm 3]
      RFCOMM(d): UIH: cr 1 dlci 32 pf 0 ilen 13 fcs 0xd8
        OBEX: SetPath cmd(f): len 13 flags 2 constants 0
        Connection ID (0xcb) = 17
        Name (0x01) = Unicode length 0
> HCI Event: Number of Completed Packets (0x13) plen 5
    handle 21 packets 1
> ACL data: handle 21 flags 0x02 dlen 11
    L2CAP(d): cid 0x0041 len 7 [psm 3]
      RFCOMM(d): UIH: cr 0 dlci 32 pf 0 ilen 3 fcs 0x2
        OBEX: SetPath rsp(f): status 200 len 3
< ACL data: handle 21 flags 0x00 dlen 21
    L2CAP(d): cid 0x0040 len 17 [psm 3]
      RFCOMM(d): UIH: cr 1 dlci 32 pf 0 ilen 13 fcs 0xd8
        OBEX: SetPath cmd(f): len 13 flags 2 constants 0
        Connection ID (0xcb) = 17
        Name (0x01) = Unicode length 0
> HCI Event: Number of Completed Packets (0x13) plen 5
    handle 21 packets 1
> ACL data: handle 21 flags 0x02 dlen 11
    L2CAP(d): cid 0x0041 len 7 [psm 3]
      RFCOMM(d): UIH: cr 0 dlci 32 pf 0 ilen 3 fcs 0x2
        OBEX: SetPath rsp(f): status 200 len 3
< ACL data: handle 21 flags 0x00 dlen 37
    L2CAP(d): cid 0x0040 len 33 [psm 3]
      RFCOMM(d): UIH: cr 1 dlci 32 pf 0 ilen 29 fcs 0xd8
        OBEX: SetPath cmd(f): len 29 flags 2 constants 0
        Connection ID (0xcb) = 17
        Name (0x01) = Unicode length 16
        0000: 00 74 00 65 00 6c 00 65  00 63 00 6f 00 6d 00 00  .t.e.l.e.c.o.m..
> HCI Event: Number of Completed Packets (0x13) plen 5
    handle 21 packets 1
> ACL data: handle 21 flags 0x02 dlen 11
    L2CAP(d): cid 0x0041 len 7 [psm 3]
      RFCOMM(d): UIH: cr 0 dlci 32 pf 0 ilen 3 fcs 0x2
        OBEX: SetPath rsp(f): status 200 len 3

obexd: Remove obc_session_setpath limit

obc_session_setpath is able to handle multiple folder changes at once.
This removes the limit, that was introduced because of PBAP's limitation
to 3 levels (75d32c6b3a3bddaad19b7880950f9a503afe652b).

Other profiles like MAP might have deeper folder hierarchies.

tools: btmgmt: Fix return value to return request id

Commit 947a7477d2a2ab6a61e5a7de44fd5b7cd2ab8d9c introduced the method
send_cmd which returned the index of the adapter instead of the
request id. As a result no mgmt command could be sent to hci0, as 0
marks an invalid request id.

build: Remove leftover from .gitignore

profiles/sap/sap.c used to be a symlink to a SAP driver. This symlink is
not created anymore (only the dummy driver is linked to bluetoothd at
the moment.)

tools: btmgmt: Add missing return after g_main_loop_quit()

This change keeps semantic of previous code that used to call exit()
directly.

avctp: Fix invalid file descriptor close

During avctp_confirm_cb(), if any error happens we set the session
state to AVCTP_STATE_DISCONNECTED, which inturn try to close fd 0.

tools: Fix compilation error with GINT_TO_POINTER

Fixes the following error:
tools/btmgmt.c: In function ‘index_rsp’:
tools/btmgmt.c:756:10: error: cast to pointer from integer of different size [-Werror=int-to-pointer-cast]
tools/btmgmt.c: In function ‘cmd_info’:
tools/btmgmt.c:791:9: error: cast to pointer from integer of different size [-Werror=int-to-pointer-cast]
cc1: all warnings being treated as errors
make[1]: *** [tools/btmgmt.o] Error 1
make: *** [all] Error 2

tools: btmgmt: Remove old unused mgmt command handling

tools: btmgmt: Convert User Confirm replies from mgmt_send_cmd to mgmt_reply

tools: btmgmt: Convert PIN replies from mgmt_send_cmd to mgmt_reply

tools: btmgmt: Convert confirm_name from mgmt_send_cmd to mgmt_send

tools: btmgmt: Convert from mgmt_send_cmd to mgmt_send

tools: btmgmt: Convert authentication request event monitoring to new API

tools: btmgmt: Convert device found event monitoring to new API

tools: btmgmt: Convert local name changed event monitoring to new API

tools: btmgmt: Convert auth failed event monitoring to new API

tools: btmgmt: Convert connection event monitoring to new API

tools: btmgmt: Convert new link key event monitoring to new API

tools: btmgmt: Prefer quitting mainloop over calling exit() directly

tools: btmgmt: Convert discovering event monitoring to new API

tools: btmgmt: Convert new settings event monitoring to new API

tools: btmgmt: Convert to using GLib main loop

tools: btmgmt: Convert index added/removed events monitoring to new API

tools: btmgmt: Convert controller error event monitoring to new API

tools: btmgmt: Add initial stubs for converting to new mgmt API

doc: Add short explanation for how to create mgmt sockets

unit: Add tests for sdp_get_server_ver()

unit: Add tests for sdp_get_profile_descs()

lib: Validate DTDs when parsing VersionNumberList

lib: Add comment to BluetoothProfileDescriptorList parsing workaround

Commits 0f5a5a9580084a3c4e0644ef5cd75689aeb5ff40 and
46b3a3d2d00bf70bc57ef0c9ad5542a2271e3350 introduced this workaround.

lib: Validate DTDs when parsing BluetoothProfileDescriptorList

The "seq->val.dataseq != NULL" check is also removed from the for()
statement because it should be done after verifying that the data
element is a sequence (inside the "if (SDP_IS_SEQ(...))" block.)

lib: Validate DTDs when parsing LanguageBaseAttributeIDList

Also check if the required number of entries is present.

lib: Add missing DTD validation in sdp_record_print()

unit: Add tests for sdp_get_lang_attr()

lib: Fix missing DTD validation while accessing SDP data elements

It is necessary to validate the sdp_data_t "dtd" field before accessing
the "val" union members, specially when handling SDP_SEQ*, SDP_ALT* and
SDP_STR* elements, otherwise remote devices can trigger memory
corruption by passing invalid data elements where others are expected.

lib: Add SDP_IS_ALT() macro

This macro will simplify internal SDP library code, and is similar to
the existing SDP_IS_SEQ() macro.

lib: Cleanup coding style in sdp_get_proto_descs()

lib: Reuse identical code in sdp_get_{add,}_access_protos()

sdp_get_access_protos() and sdp_get_add_access_protos() do almost
exactly the same thing, except for an additional statement for the
latter.

unit: Add initial SDP library unit tests

These tests will cover all public SDP library API not covered yet by the
tests in test-sdp.c (which focus on the SDP server).

AVRCP: Fix parsing of SetBrowsedPlayer response

Folder depth is actually the byte 13 not 14 of the response.

AVRCP: Add support for GetItemAttributes

GetItemAttributes should be used instead of GetElementAttributes if
browsing is supported.

AVRCP: Create folders for /Filesystem and /NowPlaying

If browsing is supported create objects representing /Filesystem and
/NowPlaying, these object implement MediaItem interface and can be
passed to ChangeFolder to change the scope.

AVRCP: Move features to avrcp.c

avrcp.c should be responsible for specifics of AVRCP not player.c which
is more high level abstraction.

media-api: Fix referencing to MediaLibrary instead of MediaFolder

MediaLibrary was replaced by MediaFolder so doesn't exist anymore.

AVRCP: Parse browsing and searching features bits

This parses browsing and searching features bits and set the respective
property.

AVRCP: Prefix folder name with /Filesystem

This separate the scopes of the folder as documented in doc/media-api.txt

neard: Use more appropriate log levels than info()

neard: Add fallback to legacy register if register failed

This will allow to work with neard 0.9 which doesn't support handover
agent register with carrier type.

neard: Updated neard handover registration agent API calls

neard RegisterHandoverAgent and UnregisterHandoverAgent APIs need an
extra parameter of carrier type(e.g. bluetooth).

neard: Update copyright information

neard: Use bool instead of gboolean for agent_register_postpone

There is no need to use gboolean as this flag is not used with any
glib function.

neard: Restrict method calls only to neard process

Disallow methods calls from processes other than registered to as
agent.

neard: Use service name and not boolean to track if registered to neard

gobex: Fix GPL version reference

These files (like everything else in the tree) should be GPL 2 or later
instead of GPL 2 only.

unit: Fix gobex GPL version reference

These files (like everything else in the tree) should be GPL 2 or later
instead of GPL 2 only.

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