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

audio/player: Fix crash when current item UID is set to 0

Apparently some stacks set the current UID to 0 when paused/stopped
which causes the following:

bluetoothd[23185]: profiles/audio/player.c:media_player_set_playlist_item() 0
bluetoothd[23185]: profiles/audio/player.c:media_folder_create_item() (null) type audio uid 0
process 23185: arguments to dbus_message_iter_append_basic() were incorrect, assertion "_dbus_check_is_valid_path (*string_p)" failed in file dbus-message.c line 2531.
This is normally a bug in some application using the D-Bus library.
  D-Bus not built with -rdynamic so unable to print a backtrace

UID 0 is not a valid UID according to the spec so the code should not
attempt to create any object to represent it.

Revert "lib: Add range check for SDP_SVC_ATTR_RSP/SDP_SVC_SEARCH_ATTR_RSP"

This reverts commit 1796f00e846561af80679efba4d7c36c78710fb6.

This patch causes a regression with the Nokia BH217 headset. A correct
patch must take into account fragmented responses.

core: Fix not adding UUID found during pairing

This is a regression introduce by de63199f0c89fce76180edd40359fde4b13c54c1
where the list of UUID found is always empty thus causing no driver to be
probed.

obexd/MAP: Fix crash when receiving an event report

Conditional jump or move depends on uninitialised value(s)
   at 0x42C1AF: obex_put_stream_start (obex.c:869)
   by 0x428D1A: mns_put (mns.c:148)
   by 0x42B521: cmd_put (obex.c:982)
   by 0x419FB5: incoming_data (gobex.c:1022)
   by 0x3F31A47A54: g_main_context_dispatch (in /usr/lib64/libglib-2.0.so.0.3400.2)
   by 0x3F31A47D87: ??? (in /usr/lib64/libglib-2.0.so.0.3400.2)
   by 0x3F31A48181: g_main_loop_run (in /usr/lib64/libglib-2.0.so.0.3400.2)
   by 0x40DEE2: main (main.c:319)
 Uninitialised value was created by a stack allocation
   at 0x42C160: obex_put_stream_start (obex.c:862)

obexd: Notify registered notification handlers in MNS

obexd: Register notification handler in the MAP client

build: Add support for building MAP notification dispatcher

obexd: Add MAP notification dispatching

The MAP specification allows to reuse one MNS server instance for all
local MAS client instances. This dispatching of event reports to the
correct MAS client instance is done by the MAS instance id and the
device address.

The dispatcher component allows MAS client instances to register a
notification handler. Events reports are forwarded by the MNS server using
map_dispatch_event.

obexd: Add function to get the destination from the obc_session

core: Add mode option to external profile default configuration

Some external profiles, FTP and OPP, require a different L2CAP mode such
as ERTM to interoperate properly.

input: Use btd_service userdata pointer

Avoid maintaining an internal list of probed input_device instances by
making use of btd_service's userdata pointer.

scanparam: Use btd_service userdata pointer

Avoid maintaining an internal list of probed struct scan instances by
making use of btd_service's userdata pointer.

deviceinfo: Use btd_service userdata pointer

Avoid maintaining an internal list of probed deviceinfo instances by
making use of btd_service's userdata pointer.

network: Remove find_connection()

The userdata pointer in btd_service provides the necessary information
to find which service should be connected. This makes possible to remove
the restriction of having one single UUID instance per profile.

network: Fix missing NULL check for given UUID

The code dereferences a NULL pointer if find_connection() doesn't find
an existing connection, which will be the case if the input UUID is
invalid or not supported.

network: Remove duplicated search

Commit 0624791ea6e917d6c9ecb8e7e6e5a1327199448d seems to accidentally
have introduced this duplicated search as part of a non-trivial revert.

client: prettify help table

cmd + options have a maximum length of 25. Align the description
after this value. In order not to reach 80 chars so easily change the
first \t to 2 spaces, like is done in udev, kmod, systemd, etc.

client: Add description for default-agent command

adapter: Remove never used parameter

adapter_remove_device() is always called with remove_storage set to
TRUE, so remove it.

gitignore: Add missing tools

lib: Cleanup SDP logs to not include double new lines

SDPERR and SDPDBG already add new line to prints.

Release 5.6

build: Update library version

obexd: Fix crash when retrieving multiple MAP event reports

The remote address used for dispatching the MAP notifications is
initialized when the MNS is connected. Therefore it needs to be freed
when the session is destroyed and not after receiving an event report.

Trace:

  0  0x00007ffff6a6a1c9 in raise () from /usr/lib/libc.so.6
  1  0x00007ffff6a6b5c8 in abort () from /usr/lib/libc.so.6
  2  0x00007ffff6aa8037 in __libc_message () from /usr/lib/libc.so.6
  3  0x00007ffff6aad8ae in malloc_printerr () from /usr/lib/libc.so.6
  4  0x00007ffff6aae587 in _int_free () from /usr/lib/libc.so.6
  5  0x00000000004273b0 in event_report_close (obj=0x69a5b0)
     at obexd/client/mns.c:295
  6  0x0000000000429549 in os_reset_session (os=0x69c210)
     at obexd/src/obex.c:199
  7  0x000000000041bec6 in transfer_complete (transfer=0x69a9d0, err=0x0)
     at gobex/gobex-transfer.c:103
  8  0x000000000041c20c in transfer_put_req (obex=0x69b470,
     req=<optimized out>, user_data=0x69a9d0) at
       gobex/gobex-transfer.c:407
  9  0x000000000041988d in handle_request (req=0x69f3d0, obex=0x69b470)
     at gobex/gobex.c:1022
  10 incoming_data (io=<optimized out>, cond=<optimized out>,
     user_data=0x69b470) at gobex/gobex.c:1194
  11 0x00007ffff702de46 in g_main_context_dispatch ()
     from /usr/lib/libglib-2.0.so.0
  12 0x00007ffff702e198 in ?? () from /usr/lib/libglib-2.0.so.0
  13 0x00007ffff702e59a in g_main_loop_run () from
       /usr/lib/libglib-2.0.so.0
  14 0x000000000040dead in main (argc=1, argv=0x7fffffffddc8)
     at obexd/src/main.c:319

obexd: Use correct callback in obc_session_mkdir

The function obc_session_mkdir needs to use file_op_complete as
callback.

  0  0x00007ffff72f7553 in ?? () from /usr/lib/libdbus-1.so.3
  1  0x00007ffff72f7dff in ?? () from /usr/lib/libdbus-1.so.3
  2  0x00007ffff72fef9a in dbus_message_get_sender ()
     from /usr/lib/libdbus-1.so.3
  3  0x00007ffff72feff9 in dbus_message_new_method_return ()
     from /usr/lib/libdbus-1.so.3
  4  0x000000000043c93f in async_cb (session=0x6a9d30, transfer=0x0,
       err=0x0,
     user_data=0x675660) at obexd/client/ftp.c:65
  5  0x0000000000438c7c in async_cb (obex=0x6aa980, err=0x0,
       rsp=0x67a690,
     user_data=0x67ced0) at obexd/client/session.c:1035
  6  0x000000000041cbcc in handle_response (obex=0x6aa980, err=0x0,
     rsp=0x67a690) at gobex/gobex.c:949
  7  0x000000000041d49f in incoming_data (io=0x67d0f0, cond=G_IO_IN,
     user_data=0x6aa980) at gobex/gobex.c:1192
  8  0x00007ffff702de46 in g_main_context_dispatch ()
     from /usr/lib/libglib-2.0.so.0
  9  0x00007ffff702e198 in ?? () from /usr/lib/libglib-2.0.so.0
  10 0x00007ffff702e59a in g_main_loop_run () from
       /usr/lib/libglib-2.0.so.0
  11 0x0000000000430a09 in main (argc=1, argv=0x7fffffffddc8)
     at obexd/src/main.c:319

obexd: Fix file_op_complete callback

The file_op_complete callback added for the OBEX session command queuing
is called with a file_data parameter and not with a pending_request.

This fixes a crash when calling one of the file commands (like delete).

  0  0x0000000000438cd6 in file_op_complete (session=0x6a9d30,
       transfer=0x0,
     err=0x0, user_data=0x6762e0) at obexd/client/session.c:1054
  1  0x0000000000438c64 in async_cb (obex=0x6aa980, err=0x0,
       rsp=0x67a710,
     user_data=0x6ac2c0) at obexd/client/session.c:1035
  2  0x000000000041cbcc in handle_response (obex=0x6aa980, err=0x0,
     rsp=0x67a710) at gobex/gobex.c:949
  3  0x000000000041d49f in incoming_data (io=0x67d0f0, cond=G_IO_IN,
     user_data=0x6aa980) at gobex/gobex.c:1192
  4  0x00007ffff702de46 in g_main_context_dispatch ()
     from /usr/lib/libglib-2.0.so.0
  5  0x00007ffff702e198 in ?? () from /usr/lib/libglib-2.0.so.0
  6  0x00007ffff702e59a in g_main_loop_run () from
       /usr/lib/libglib-2.0.so.0
  7  0x00000000004309f0 in main (argc=1, argv=0x7fffffffddc8)
     at obexd/src/main.c:319

obexd: Call setpath_complete with the correct parameter

The function setpath_complete has to be called with a pending_request as
user_data. In one possible error case, the current code calls it
incorrectly with setpath_data.

obexd: Fix handling error cases in setpath

This adds a setpath_op_complete callback, that unpacks the user data and
finally calls the user callback. The callback is now used for success and error
cases.

The previous implementation was using setpath_complete which did not
work for error cases, because it was called with incorrect user data.
This was leading to a crash, that can be reproduced by disconnecting PBAP
after trying to select a non existing phone book.

  0  setpath_complete (session=0x66bd90, transfer=0x0, err=0x69b370,
     user_data=0x69a810) at obexd/client/session.c:912
  1  0x000000000042d100 in obc_session_shutdown (session=0x66bd90)
     at obexd/client/session.c:537
  2  0x000000000040f227 in service_filter (connection=0x664b20,
     message=<optimized out>, user_data=0x66bed0) at gdbus/watch.c:486
  3  0x000000000040f49b in message_filter (connection=0x664b20,
     message=0x66ba30, user_data=<optimized out>) at gdbus/watch.c:554
  4  0x00007ffff72f40a6 in dbus_connection_dispatch ()
     from /usr/lib/libdbus-1.so.3
  5  0x000000000040e148 in message_dispatch (data=0x664b20)
     at gdbus/mainloop.c:76
  6  0x00007ffff702e9a3 in ?? () from /usr/lib/libglib-2.0.so.0
  7  0x00007ffff702de46 in g_main_context_dispatch ()
     from /usr/lib/libglib-2.0.so.0
  8  0x00007ffff702e198 in ?? () from /usr/lib/libglib-2.0.so.0
  9  0x00007ffff702e59a in g_main_loop_run () from
     /usr/lib/libglib-2.0.so.0
  10 0x000000000040dd72 in main (argc=1, argv=0x7fffffffddc8)
     at obexd/src/main.c:319

audio/media: Fix setting player settings

The value has to be converted to MPRIS setting otherwise the player won't
recognize it and will probably discard the change.

audio/AVRCP: Fix invalid response to RegisterNotification

The response to RegisterNotification for event settings changed was
not setting the initial length properly which cause the code to send
malformed/invalid PDUs.

audio/media: Fix notifying settings changed incorrectly

We should notify only the setting that has changed not all of them.

core: Fix crash when a duplicated record is found

 Invalid read of size 8
    at 0x470101: update_bredr_services (device.c:2784)
    by 0x470591: browse_cb (device.c:2975)
    by 0x458B0E: search_completed_cb (sdp-client.c:186)
    by 0x47C154: sdp_process (sdp.c:4343)
    by 0x458954: search_process_cb (sdp-client.c:205)
    by 0x3F31A47A54: g_main_context_dispatch (in /usr/lib64/libglib-2.0.so.0.3400.2)
    by 0x3F31A47D87: ??? (in /usr/lib64/libglib-2.0.so.0.3400.2)
    by 0x3F31A48181: g_main_loop_run (in /usr/lib64/libglib-2.0.so.0.3400.2)
    by 0x40A265: main (main.c:595)
  Address 0x0 is not stack'd, malloc'd or (recently) free'd

audio/player: Create item on media_player_set_playlist_item

This makes sure that even if /NowPlaying folder has not been listed the
current item is available and its metadata is updated.

audio/player: Add implementation of MediaFolder.Search

audio/AVRCP: Add support for Search command

This adds support for Search command via player callback.

audio/player: Add implementation of MediaItem.AddToNowPlaying

audio/AVRCP: Add support for AddToNowPlaying command

This adds support for AddToNowPlaying command via player callback.

audio/player: Add implementation of MediaItem.Play

audio/AVRCP: Add support for PlayItem command

This adds support for PlayItem command via player callback.

audio/player: Add implementation of MediaFolder.ChangeFolder

audio/AVRCP: Add support for ChangePath command

This adds support for ChangePath command via player callback.

audio/player: Add support for MediaItem.Metadata property

audio/player: Add function media_item_set_playable

audio/player: Add support for setting current Item object

This introduces media_player_set_playlist_item which is used when the
current item UID is known, with this the application can tell what
is the current item in the playlist.

audio/player: Add implementation of MediaFolder.ListItems

audio/AVRCP: Add support for GetFolderItems command

This adds support for GetFolderItems command via player callback.

audio/AVRCP: Add browsed flag to player

This flag indicates if the player is browsed and are checked to avoid
setting the same player as browsed multiple times.

audio/player: Split item creation

This split item creation into two functions, media_player_create_folder
for folders and media_player_create_item for regular items.

tools/mpris-player: Fix crash when forwarding player message

The iterator to which the message should be copied to should be the copy
message not the original message.