core: Handle existing start discovery gracefully

In case a discovery already exists, keep using it if is has the same
type and only if not, stop it and start a new one.

core: Simplify discovery start and stop handling

To make the discovery start, stop and restart logic even simple, just
trigger start of discovery when client gets registered. It is almost
impossible to handle all corner cases, so just trigger another restart
of the discovery with a larger timeout in case of an error.

core: Fix multiple issues with discovery handling

This patch is the first attempt in fixing the discovery issues for
dual-mode BR/EDR + LE controllers. It creates a clean chain for
start/stop discovery client handling and also for the internal
handling of suspend, resume and restart.

However this patch removes the LE connection handling. And that needs
to be put back with a separate patch.

tools: Use proper test data in mgmt_unblock_device test

Copy and paste error. Although both data values are same use proper
one for clarity.

core: Use discovery found list instead of nested struct

core: Remove pending_mode session variable

core: Remove unneeded session reference counting

core: Remove session_req_type variable and parameter

core: Rename a few functions and parameters

core: Remove unused got_reply session variable

AVRCP: Fix rejecting RegisterNotification if no transport exists

If no transport exists set the volume as 0, the real volume will be set
when a transport becomes available.

core: Add workaround for device pairing kernel bug

Some kernels reply to device pairing command with bogus command status
instead of command complete event if adapter was not powered.

Command status event doesn't provide remote device address and type
needed to properly complete bonding request.

Pass possibly missing data as user_data to mgmt_send and use it in
pair_device_complete function if needed.

tools: Add basic invalid params test case for mgmt_unblock_device

tools: Add basic invalid params test case for mgmt_block_device

tools: Add basic mgmt_disconnect invalid params test case

tools: Add basic mgmt_unpair_device test cases

tools: Add mgmt_pair_device test with invalid address type

tools: Add mgmt test case for pair device while powered off

This tests if kernel is responding with proper command complete event.

tools: Add another invalid params mgmt_load_long_term_keys test

tools: Add more mgmt_load_ltks invalid params tests

obexd: Add parameter Status to GetMessageListing response

The parameter Status indicates the status of reception of the message.
It can be used to differentiate between messages with a reception status
of "completed", "fractioned" and "notification".

This parameter got lost when obexd was refactored to use of D-Bus properties.
The documentation and the parsing code for it was still there.

obexd: Add parameter AttachmentSize to GetMessageListing response

The parameter AttachmentSize indicates if the message contains
any attachment and their overall size (in bytes).

obexd: Add parameter Text to GetMessageListing response

The Text flag indicates if the message contains any textual content
or has binary content only.

obexd: Move parse_size function in map.c

This reorders msg_parsers and moves the function parse_size
to match the order in the MAP specification.

obexd: Add parameter SubjectLength to map_list_messages

This parameter allows the client to request a maximum length of the
parameter "subject" in the messages listing.

unit: Add gdbus/client_get_uint64_property

unit: Add gdbus/client_get_array_property

unit: Add gdbus/client_get_boolean_property

unit: Add gdbus/client_get_string_property

unit: Add gdbus/client_get_dict_property

tools: Add basic failure test case for mgmt_load_long_term_keys

tools: Add basic success test case for mgmt_load_long_term_keys

tools: Add another invalid params test for mgmt_load_link_keys

doc: Add note about Debug_Keys parameter for Load Link Keys Command

tools: Add basic invalid parameters test case for mgmt_load_link_keys

tools: Add initial success test cases for mgmt_load_link_keys

tools: Add temporary workaround for mgmt_set_powered kernel bug

shared: Add separate timed out test result

tools: Use a timeout for all mgmt test cases

tools: Fix EIR reference data to account for TX power in newer kernels

tools: Add mgmt test case for multiple UUID-16 values

shared: Fix missing removal of test timeout for success/failure

tools: Add basic add_uuid UUID-32 test case for mgmt-tester

tools: Add basic add_uuid UUID-16 test case for mgmt-tester

unit: Add advertising data from MIO Alpha heartrate watch for testing

tools: Define power off CoD test with a timeout

tools: Add support for defining mgmt tests with a timeout

shared: Add timeout support for test cases

tools: Do more generic tracking of unmet test conditions in mgmt-tester

tools: Fix test for unset settings in mgmt-tester

tools: Add power off test for testing correct class of dev

tools: Fix waiting for expected events in mgmt-tester

tools: Add class of device change monitoring support to mgmt-tester

tools: Add basic power off test to mgmt-tester

tools: Add mgmt-tester support for testing for unset settings

unit: Add tests for SDP integer Data Elements

SDP_DATA_NIL does not have a value, but its sdp_data_t val field is
zeroed by memset(), so just treat it as UINT8 with zero value to
simplify checking (and not require a separate macro.)

SDP_BOOL is handled by SDP library as a INT8.

SDP_UINT128/SDP_INT128 are just byte arrays converted to host order, so
use memcmp() to compare them (converting from host to network order
first.)

unit: Add {TEXT,URL}_STR{8,16} tests for sdp_extract_attr()

These tests are for valid data. Other tests might be added later to
check for error paths, but will require separate macros.

{TEXT,URL}_STR32 cannot be tested with the same macros because
sdp_extract_attr() does not support them. They will be tested
separately with other SDP library functions.

As example of failure output, if commit
504a0cf46ad89cab8005ce9cffb22e41048f6a30 is reverted for testing, the
STR16 test will fail with:

ERROR:unit/test-sdp.c:776:test_sdp_de_attr: assertion failed
(test->input_size == size): (7 == 11)

tools: Add HCI command checking for mgmt_set_dev_class test

core: Fix checking for device object creation failures

core: Log device interface registration failures

monitor: Differentiate between E0 and AES-CCM encryption

tools: Add some basic mgmt_set_dev_class test cases

tools: Add some basic mgmt_start_discovery failure test cases

AVRCP: Fix sending volume changed when not registered

Notifications should only be sent if remote device has registered using
RegisterNotification.

AVRCP: Check version before sending volume changed event/SetAbsoluteVolume

Volume change event and SetAbsoluteVolume command are only available
starting from 1.4, so we should not attempt to send these commands for
versions before that.

audio: Bump AVRCP CT version to 1.5

avrcp: Handle SetAbsoluteVolume command

transport: Update volume passing only audio_device

avrcp: Notify remote of volume changes

avrcp: Handle RegisterNotification for volume

This commit answers a NOTIFY command to register for nofications of
EVENT_VOLUME_CHANGED with an INTERIM response containing the current
absolute volume value.

transport: Get volume passing only audio_device

transport: Keep a list o all existent transports

transport: Get volume from transport

avrcp: Add EVENT_VOLUME_CHANGED to CT capabilities

transport: Store the value set through the 'Volume' property

transport: Initialize the "Volume" property

core: Fix handling mgmt socket write failures

If a write to the socket fails we should properly notify the command
callback and remove the command from the queue.

core: Move valid discovery type check to mgmt_start_discovery

core: Fix connect list usage for non-LE adapters

core: Don't try to start discovery with invalid address type

core: Add more debug logs to start_discovery functions

core: Add debug log to mgmt_start_discovery()

tools: Fix map-client to use new print syntax

print now requires () on newer python:
  File "./map-client", line 101
    print err
            ^

obexd: Remove transferred file only if the GET operation failed

Since 9606375649e12f1b9f302bbb5bb8f87957387ddd xfer_complete() sets the
transfer status rather than the size. Adapt obc_transfer_free to check
for the completed status to avoid deletion of completed transfers.

obexd: Change behavior of parse_filter_read

Calls to ListMessages with filter 'Read' set to true should
request the MSE to send read messages only.

The old code requests the MSE to send unread messages only.
This behavior is not matching the other filters.

obexd: Use defines for values in parse_filter_read parse_filter_priority

obexd: Fix segfault in parse_filter_read and parse_filter_priority

Calls to ListMessages with filter 'Read' or 'Priority' caused a segfault
in parse_filter_read / parse_filter_priority. The functions read
D-Bus boolean values (uint32) into uint8.

0  0x00007ffff730332d in ?? () from /usr/lib/libdbus-1.so.3
1  0x00007ffff7304219 in dbus_message_iter_next () from /usr/lib/libdbus-1.so.3
2  0x000000000043ef0f in parse_message_filters (
    apparam=<error reading variable: Cannot access memory at address 0x7ffffeffff08>,
    iter=<error reading variable: Cannot access memory at address 0x7ffffeffff00>)
    at obexd/client/map.c:1246

obexd: Fix FILTER_ALL in MAP client to set 16 bit

The MAP specification defines ParameterMask as a bitmask of 32 bit / 4 bytes.
For the lower 16 bit the specification defines parameters, the higher 16 bit
remain reserved for future use. Therefore FILTER_ALL is set to 0x0000FFFF.
(Reserved bits have to be set to 0)

In addition this fixes the issue that ListFilterFields didn't show all fields.

obexd: Fix infinite loop in ListMessages with filter "Types"

Calls to ListMessages with filter 'Types' make obexd hang in an infinite loop.
This is caused by a missing dbus_message_iter_next in parse_filter_type.

0  0x00007ffff7304ca7 in dbus_message_iter_get_basic ()
   from /usr/lib/libdbus-1.so.3
1  0x0000000000434fba in parse_filter_type (iter=0x7fffffffd7d0, apparam=
    0x6987f0) at obexd/client/map.c:1086
2  parse_message_filters (iter=0x7fffffffd730, apparam=0x6987f0)
    at obexd/client/map.c:1222
3  map_list_messages (connection=<optimized out>, message=0x669ae0, user_data=
    0x698a60) at obexd/client/map.c:1273
4  0x00000000004109a1 in process_message (connection=0x662b20,
    message=<optimized out>, iface_user_data=<optimized out>,
    method=<optimized out>, method=<optimized out>) at gdbus/object.c:285
5  0x00007ffff7308e15 in ?? () from /usr/lib/libdbus-1.so.3
6  0x00007ffff72fb070 in dbus_connection_dispatch ()
   from /usr/lib/libdbus-1.so.3
7  0x000000000040e3d8 in message_dispatch (data=0x662b20)
    at gdbus/mainloop.c:76
8  0x00007ffff703d3cb in ?? () from /usr/lib/libglib-2.0.so.0
9  0x00007ffff703c845 in g_main_context_dispatch ()
   from /usr/lib/libglib-2.0.so.0
10 0x00007ffff703cb78 in ?? () from /usr/lib/libglib-2.0.so.0
11 0x00007ffff703cf72 in g_main_loop_run () from /usr/lib/libglib-2.0.so.0
12 0x000000000040df82 in main (argc=1, argv=0x7fffffffdd88)
    at obexd/src/main.c:323

obexd: Add property exist functions to map client

This fixes crashes in MAP client when the server does not send optional properties.

0  0x00007ffff6a792c5 in raise () from /usr/lib/libc.so.6
1  0x00007ffff6a7a748 in abort () from /usr/lib/libc.so.6
2  0x00007ffff731c145 in ?? () from /usr/lib/libdbus-1.so.3
3  0x00007ffff7312a25 in ?? () from /usr/lib/libdbus-1.so.3
4  0x00007ffff73050d6 in dbus_message_iter_append_basic () from /usr/lib/libdbus-1.so.3
5  0x0000000000433cc5 in get_replyto (property=<optimized out>, iter=<optimized out>,
    data=<optimized out>) at obexd/client/map.c:484
6  0x00000000004103b6 in append_property (p=p@entry=0x6594c0 <map_msg_properties+192>,
    dict=dict@entry=0x7fffffffd8e0, iface=0x6a3720) at gdbus/object.c:547
7  0x0000000000410472 in append_properties (data=data@entry=0x6a3720, iter=iter@entry=
    0x7fffffffd960) at gdbus/object.c:576
8  0x00000000004104d1 in append_interface (data=0x6a3720, user_data=0x7fffffffda40)
    at gdbus/object.c:591
9  0x00007ffff7058a4d in g_slist_foreach () from /usr/lib/libglib-2.0.so.0
10 0x0000000000411d05 in emit_interfaces_added (data=0x6a2ff0) at gdbus/object.c:623
11 process_changes (user_data=0x6a2ff0) at gdbus/object.c:1006
12 0x00007ffff703c845 in g_main_context_dispatch () from /usr/lib/libglib-2.0.so.0
13 0x00007ffff703cb78 in ?? () from /usr/lib/libglib-2.0.so.0
14 0x00007ffff703cf72 in g_main_loop_run () from /usr/lib/libglib-2.0.so.0
15 0x000000000040df82 in main (argc=1, argv=0x7fffffffdd88) at obexd/src/main.c:323

obexd: Handle empty path name in SetPath

If the empty path is given, an empty name should be sent via OBEX.
Currently the name field is not set at all and later checks which
depend on data->index will access invalid memory regions as g_strsplit
returns NULL when an empty string is given.

0  0x000000000041a181 in g_obex_setpath (obex=obex@entry=0x662eb0, path=
    0x20 <Address 0x20 out of bounds>, func=func@entry=0x42d300 <setpath_cb>,
    user_data=user_data@entry=0x668f10, err=err@entry=0x7fffffffda08)
    at gobex/gobex.c:1397
1  0x000000000042d395 in setpath_cb (obex=0x662eb0, err=0x0, rsp=<optimized out>,
    user_data=0x668f10) at obexd/client/session.c:902
2  0x0000000000418e54 in handle_response (obex=obex@entry=0x662eb0, err=err@entry=0x0,
    rsp=rsp@entry=0x668f40) at gobex/gobex.c:948
3  0x0000000000419d7a in incoming_data (io=<optimized out>, cond=<optimized out>,
    user_data=0x662eb0) at gobex/gobex.c:1191
4  0x00007ffff703c845 in g_main_context_dispatch () from /usr/lib/libglib-2.0.so.0
5  0x00007ffff703cb78 in ?? () from /usr/lib/libglib-2.0.so.0
6  0x00007ffff703cf72 in g_main_loop_run () from /usr/lib/libglib-2.0.so.0
7  0x000000000040def2 in main (argc=1, argv=0x7fffffffdd88) at obexd/src/main.c:323

doc: Fix trivial typo in adapter-api

monitor: Add decoding for minor class for wearable major class

core: Rename adapter->disc_sessions to discov_sessions for clarity

monitor: Add GATT-based Service UUIDs

monitor: Add UUID values for Characteristic Descriptors

monitor: Add UUID values for Characteristic Descriptions

monitor: Fix the singular entry vs plural entries

monitor: Decode the minor class field for major class phone

monitor: Decode 16-bit and 32-bit UUID values from EIR data