Release 5.9

obexd/bluetooth: Fix memory leak

g_io_channel_unix_new creates a reference which is then passed to
obex_session_start which creates its on reference via g_io_channel_ref
leading to the following leak:

at 0x4A06409: malloc (in /usr/lib64/valgrind/vgpreload_memcheck-amd64-linux.so)
by 0x3B03C4D89E: g_malloc (in /usr/lib64/libglib-2.0.so.0.3600.3)
by 0x3B03C88224: g_io_channel_unix_new (in /usr/lib64/libglib-2.0.so.0.3600.3)
by 0x418967: profile_new_connection (bluetooth.c:148)
by 0x43D763: process_message.isra.5 (object.c:259)

obexd: Remove msg from MAP session structure

The D-Bus message for a pending method call is now stored in the
pending_request struct.

obexd: Use pending request in UpdateInbox

obexd: Use pending request in ListMessages

obexd: Use pending request in ListFolders

obexd: Use pending request in SetFolder

obexd: Add request struct to MAP

This adds a pending_request struct in order to store the D-Bus request
data.

The current version stores the received D-Bus message in the MAP session
struct. The stored message is overridden by intermediate D-Bus method
calls which can lead into a crash.

Trace:
  arguments to dbus_message_unref() were incorrect,
  assertion "!message->in_cache" failed in file dbus-message.c line 1618.

 0  0x00007ffff6a6a1c9 in raise () from /usr/lib/libc.so.6
 1  0x00007ffff6a6b5c8 in abort () from /usr/lib/libc.so.6
 2  0x00007ffff7313de5 in ?? () from /usr/lib/libdbus-1.so.3
 3  0x00007ffff730ab91 in ?? () from /usr/lib/libdbus-1.so.3
 4  0x000000000043721c in message_listing_cb (session=0x6a7d30,
    transfer=0x6a9450, err=0x0, user_data=0x6a9950) at obexd/client/map.c:1166
 5  0x000000000042f7af in session_terminate_transfer (session=0x6a7d30,
    transfer=0x6a9450, gerr=0x0) at obexd/client/session.c:830
 6  0x000000000042f83d in session_notify_complete (session=0x6a7d30,
    transfer=0x6a9450) at obexd/client/session.c:845
 7  0x000000000042f8dc in transfer_complete (transfer=0x6a9450, err=0x0,
    user_data=0x6a7d30) at obexd/client/session.c:865
 8  0x0000000000439ee7 in xfer_complete (obex=0x677250, err=0x0,
    user_data=0x6a9450) at obexd/client/transfer.c:577
 9  0x000000000043a05f in get_xfer_progress_first (obex=0x677250, err=0x0,
    rsp=0x678730, user_data=0x6a9450) at obexd/client/transfer.c:621
 10 0x0000000000413f08 in handle_response (obex=0x677250, err=0x0,
    rsp=0x678730) at gobex/gobex.c:949
 11 0x00000000004147db in incoming_data (io=0x6a8a00, cond=G_IO_IN,
    user_data=0x677250) at gobex/gobex.c:1192
 12 0x00007ffff702dda6 in g_main_context_dispatch ()
   from /usr/lib/libglib-2.0.so.0
 13 0x00007ffff702e0f8 in ?? () from /usr/lib/libglib-2.0.so.0
 14 0x00007ffff702e4fa in g_main_loop_run () from /usr/lib/libglib-2.0.so.0
 15 0x0000000000427ce8 in main (argc=1, argv=0x7fffffffdd48)
    at obexd/src/main.c:319

core: Remove legacy parsing of OUI database

sap: Keep reference to btd_adapter in struct sap_server

Path and adapter address can be obtained easily from btd_adapter
and there is no need to keep local copy.

core: Minor whitespace fix

gitignore: Add tools/btinfo

obexd: Use documented values in org.bluez.obex.Message1 Type property

This updates the values that are presented in the Type property to use
the values from the documentation ("email", "sms-gsm", "sms-cdma", "mms").
The existing code directly used the values as received in the messages
listing object ("EMAIL", "SMS_GSM", "SMS_CDMA", "MMS").

shared/hciemu: Fix sending a proper vhci create request for new kernels

gdbus/client: Use g_dbus_add_properties_watch to track properties

This make the handling much simpler and avoids duplicates of the same
match rule.

gdbus/client: Use g_dbus_add_signal_watch to track signals

This make the handling much simpler and avoids duplicates of the same
match rule.

gdbus/client: Use g_dbus_add_service_watch to track services

This make the handling much simpler and avoids duplicates of the same
match rule.

gdbus/watch: Fix crash when disconnecting from D-Bus

When disconnecting from D-Bus a message could be recieved with no sender:
Invalid read of size 1
   at 0x4A09EE1: strcmp (in /usr/lib64/valgrind/vgpreload_memcheck-amd64-linux.so)
   by 0x3B03C386B8: g_str_equal (in /usr/lib64/libglib-2.0.so.0.3600.3)
   by 0x4065D6: message_filter (watch.c:529)
   by 0x3B0700F9E5: dbus_connection_dispatch (in /usr/lib64/libdbus-1.so.3.7.4)
   by 0x4052E7: message_dispatch (mainloop.c:76)
   by 0x3B03C48962: ??? (in /usr/lib64/libglib-2.0.so.0.3600.3)
   by 0x3B03C47E05: g_main_context_dispatch (in /usr/lib64/libglib-2.0.so.0.3600.3)
   by 0x3B03C48157: ??? (in /usr/lib64/libglib-2.0.so.0.3600.3)
   by 0x3B03C48559: g_main_loop_run (in /usr/lib64/libglib-2.0.so.0.3600.3)
   by 0x4038C5: client_proxy_removed (test-gdbus-client.c:902)
   by 0x3B03C6B566: ??? (in /usr/lib64/libglib-2.0.so.0.3600.3)
   by 0x3B03C6B6E5: ??? (in /usr/lib64/libglib-2.0.so.0.3600.3)
 Address 0x0 is not stack'd, malloc'd or (recently) free'd

gdbus/watch: Fix aborting when removing D-Bus filter

In case of filter_data having a watch to a service name it may call
dbus_connection_remove_filter twice causing libdbus to abort:

process 24723: Attempt to remove filter function 0x4063e0 user data (nil), but no such filter has been added

To fix this the code will now only attempt to call
dbus_connection_remove_filter once in filter_data_free which is the
counterpart of filter_data_get where dbus_connection_add_filter is called.

gdbus/watch: Fix crash when g_dbus_remove_watch is called from connect callback

 at 0x40570C: update_service (watch.c:601)
 by 0x40584B: service_reply (watch.c:627)
 by 0x3B0700C511: ??? (in /usr/lib64/libdbus-1.so.3.7.4)
 by 0x3B0700F740: dbus_connection_dispatch (in /usr/lib64/libdbus-1.so.3.7.4)
 by 0x405167: message_dispatch (mainloop.c:76)
 by 0x3B03C48962: ??? (in /usr/lib64/libglib-2.0.so.0.3600.3)
 by 0x3B03C47E05: g_main_context_dispatch (in /usr/lib64/libglib-2.0.so.0.3600.3)
 by 0x3B03C48157: ??? (in /usr/lib64/libglib-2.0.so.0.3600.3)
 by 0x3B03C48559: g_main_loop_run (in /usr/lib64/libglib-2.0.so.0.3600.3)
Address 0x4c58a30 is 32 bytes inside a block of size 56 free'd
 at 0x4A074C4: free (in /usr/lib64/valgrind/vgpreload_memcheck-amd64-linux.so)
 by 0x3B03C4D9AE: g_free (in /usr/lib64/libglib-2.0.so.0.3600.3)
 by 0x406102: filter_data_remove_callback (watch.c:378)
 by 0x405FC0: g_dbus_remove_watch (watch.c:798)
 by 0x40A22B: g_dbus_client_unref (client.c:1227)
 by 0x40570B: update_service (watch.c:599)
 by 0x40584B: service_reply (watch.c:627)

unit: Add gdbus/client_force_disconnect

This tests if watches created using g_dbus_proxy_set_removed_watch works
when the service disconnects from the bus.

unit: Add gdbus/client_proxy_removed

This tests if watches created using g_dbus_proxy_set_removed_watch works
when the interface is removed.

monitor: Add missing include

Bionic library requires explicit include for data types used in ioctl
call. This fix following build error with bionic:

target  C: btmon <= external/bluetooth/bluez/monitor/display.c
external/bluetooth/bluez/monitor/display.c: In function 'num_columns':
external/bluetooth/bluez/monitor/display.c:59:18: error: storage size
    of 'ws' isn't known

profile: Remove support for Cancel method

doc: Remove Profile1.Cancel() method

There does not seem to be any actual need for this method.

The current implementation calls Cancel when a pending
NewConnection() or RequestDisconnection() method call fails.
Additionally it calls Cancel() when the remote client has disconnected.
In either case BlueZ calls shutdown on the file descriptor associated
with the agent and issues a Cancel() method call.

The closing of the file descriptor will trigger a HUP on the agent side,
which will trigger normal cleanup procedures.  The received Cancel()
method call is redundant in both cases.

doc: Label Profile1.Release() method as no reply

The current implementation does not expect a reply, nor would it make
sense for BlueZ to wait for one from the agent.  So explicitly label
this method as noreply.

avdtp: Remove unused STREAM_TIMEOUT define

avdtp: Remove not used includes

avdtp: Remove not used pending_auth from struct avdtp

shared: Check HCI packet type before sending it to btdev

codingstyle: Change __FUNCTION__ to __func__

The __func__ macro is part of the C99 standard whereas __FUNCTION__ is a
legacy gcc specific alias for it:

http://gcc.gnu.org/onlinedocs/gcc/Function-Names.html

Additionally, checkpatch.pl that's commonly used to verify coding style
also recommends to use __func__ instead of __FUNCTION__.

emulator: Add missing HCI commands to the supported commands response

emulator: Disallow SCO HCI commands on LE-only controllers

emulator: Disallow HCI_Write_Current_IAP_LAP for LE-only controllers

emulator: Add support for reading local out-of-band data

emulator: Add options to usage description

obexd/bluetooth: Fix error on register_profile

The error happens whenever RegisterProfile fails the interface is not
unregistered causing any subsequent call to register_profile function
to fail:

obexd[944]: D-Bus failed to register /org/bluez/obex/00005005_0000_1000_8000_0002ee000001
obexd[944]: bluetooth: Failed to register profile (null)

build: Check for clock_gettime with -lrt

obexd: Make use of g_dbus_send_message*

This replaces dbus_connection_send* with g_dbus_send_message* which do
not alter message order.

audio/media: Make use of g_dbus_send_message_with_reply

This replaces dbus_connection_send_with_reply with
g_dbus_send_message_with_reply which does not alter message order.

neard: Make use of g_dbus_send_message_with_reply

This replaces dbus_connection_send_with_reply with
g_dbus_send_message_with_reply which does not alter message order.

core: Make use of g_dbus_send_message_with_reply

This replaces dbus_connection_send_with_reply with
g_dbus_send_message_with_reply which does not alter message order.

gdbus: Fix emitting PropertiesChanged twice

This fixes double emission of PropertiesChanged introduced by flushing
changes, the flushing can happen during the pending processing so the
pending_prop flag needs to be updated in the beginning and the list of
properties can be freed before g_dbus_send_message as it is not required
anymore.

gdbus: Avoid calling dbus_connection_send*

dbus_connection_send* should not be called directly except by
g_dbus_send_message.

gdbus: Add g_dbus_send_message_with_reply

g_dbus_send_message_with_reply flushes pending signals before calling
dbus_connection_send_with_reply so it does not alter the message order

gdbus: Fix sending ObjectManager/Properties signals out of order

In some cases the order of the messages is altered when a message is
sent without processing the pending signals first, currently this affect
client_check_order unit test:

/gdbus/client_check_order: **
ERROR:unit/test-gdbus-client.c:795:property_check_order: assertion failed: (g_strcmp0(string, "value1") == 0)

As can be observed the value of the property is not yet updated because the
signal it is still pending, once this fix is applied the test pass:

/gdbus/client_check_order: OK

Note that the flushing only works when g_dbus_send_message is used so
places where dbus_connection_send and other variants are called directly
may still change the order.

unit: Add gdbus/client_check_order

build: Fix not rebuilding bluetoothd if gdbus changes

src_bluetoothd_DEPENDENCIES should list all local libraries including gdbus
otherwise it does not get relinked whenever gdbus changes.

sco-tester: Update ECONNABORTED to EOPNOTSUPP

Kernel interface has evolved in between.

monitor: Decode page scan and page scan period mode commands

monitor: Add definitions for page scan and page scan period mode

monitor: Add more details to Set Event Filter decoding

monitor: Add decoding for number of supported IAC command

monitor: Add definition for number of supported IAC command

monitor: Add decoding for link supervision timeout commands

monitor: Add definitions for link supervision timeout commands

monitor: Fix decoding of link policy settings command

emulator: Add support for LE test commands

emulator: Add emulation a few more HCI commands

monitor: Decode host buffer size and device under test commands

monitor: Decode inquiry scan activity commands

monitor: Decode security mode 3 auth and encrypt commands

monitor: Add some missing HCI command definitions

emulator: Add support for host buffer size and LE rand commands

monitor: Add definition for host buffer size command

hog: ignore UHID-setup events and document why

The UHID_{START,STOP,OPEN,CLOSE} events should be ignored by us to avoid
triggering the warn(). It is safe to do that. Add few comments that
explain why we don't have to handle these.

monitor: Fix minor white space issues in header

tools: Add example utility to demonstrate user channel usage

tools: Make parameter argument variables const

core: Fix emitting empty Device1.UUIDs property

In case there is a incoming connection but services has not been
resolved the UUID of the profile being connected is automatically added
using btd_device_add_uuid which does update uuids not eir_uuids which is
used if svc_resolved is false.

To fix this the code will now fallback to uuids field whenever eir_uuids is
empty so uuids added with btd_device_add_uuid previous to services being
resolved should appear in UUIDs property.

tools: Add command line options for btattach utility

build: Add test-driver to maintainer clean process

monitor: Add support for printing UUID-128 in ATT

monitor: Fix format string for UUID-128 print out

build: Check for clock_gettime and pthread_create

lib: Add definition for HCI_CHANNEL_USER

emulator: Filter for valid packet types from /dev/vhci

obexd: Fix disconnecting underline transport before processing operations

The transport connection should only be dropped once all references to
the session are released otherwise some pending operations may not be
processed and the session will not be freed.

obexd: Add property changed signals for 'org.bluez.obex.Message1'

This patch adds property changed signal emissions in case message properties
change on the server.

obexd: Add property Folder to MAP message

obexd: Add obc_session_get_folder function

This patch extends client session by the tracking of the current folder.
The current folder can be accessed by obc_session_get_current_folder.

This allows drivers to add a folder property to browsed objects so that
the application doesn't have to keep track of the folder an object
belongs to.

build: Remove all checks for USB library support

tools: Remove libusb dependency from hid2hci

tools: Fix a few USB handling issues within bccmd utility

build: Remove extra checks for libusb usage of bccmd

tools: Access USB device directly from bccmd without libusb

shared: Fix constants for extended btsnoop types

monitor: Fix minor typo in command description

monitor: Remove some confusing comments

monitor: Add support for decoding inquiry scan type commands

monitor: Add structures for inquiry scan type commands

monitor: Add support for AFH classification and assessment commands

emulator: Fix up AFH channel assessment structures

monitor: Add support for decoding Write Inquiry TX Power Level command

monitor: Add structure for writing inquiry tx power level

monitor: Add support for decoding HCI Read Local OOB Data command

monitor: Add structure for local out-of-band data details

monitor: Add support for decoding failed contact counter commands

monitor: Add structures for failed contact counters

monitor: Add support for decoding HCI Read Channel Map command