obexd/server: Fix leaking drivers list

The leak can be detected by using G_SLICE=always-malloc which will
produce the following trace using valgrind:
112 bytes in 7 blocks are definitely lost in loss record 123 of 167
   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 0x3B03C6344D: g_slice_alloc (in /usr/lib64/libglib-2.0.so.0.3600.3)
   by 0x3B03C647A5: g_slist_append (in /usr/lib64/libglib-2.0.so.0.3600.3)
   by 0x424DD3: obex_service_driver_list (service.c:76)
   by 0x42517F: obex_server_init (server.c:64)
   by 0x40D439: main (main.c:304)

unit: Remove not needed functions from test-sdp

SDP code no longer depends on adapter code.

core: Remove not needed sdp_init_services_list function

It is doing nothing now and can be removed.

adapter: Handle removing of SDP records

Make adapter in charge of updating SDP database. This allow to decouple
SDP of code used for notifying adapters about SDP database change.

adapter: Handle adding new SDP records

Make adapter in charge of updating SDP database. This allow to decouple
SDP of code used for notifying adapters about SDP database change.

adapter: Insert DeviceID record if setting DeviceID data to controller

DeviceID is special since its record is registered on start by
sdpd-server and always has handle 0x10000.

lib: Update company identifiers

core: Fix passing timeout when disabling discoverable mode

We should only pass a timeout value when enabling discoverable mode.
When disabling passing something non-zero would yield a "invalid
parameters" error.

emulator/bthost: Fix using correct address type for LE connections

emulator/bthost: Fix ACL handle packing

emulator/bthost: Track address type for connections

emulator/bthost: Use right CID for LE signaling

emulator/bthost: Pass full ACL connection context around

emulator: Add initial LE L2CAP signaling support to bthost

emulator: Add LE Connection Complete handling to bthost

emulator: Refactor bthost ACL connection tracking

emulator: Add skeleton for LE meta event handler in bthost

emulator: Add basic advertise enable support for bthost

tools/l2cap-tester: Fix using right hciemu type

emulator: Add basic support for LE connections

monitor: Add define for "Connection Failed To Be Established"

emulator: Remove redundant empty line

sdp: Check correct number of bytes received in recv

Instead of checking for error check that correct number of bytes received.

doc: Update supported A2DP version to 1.3

tools/l2cap-tester: Add invalid config request server test

tools/l2cap-tester: Add invalid CID server test case

tools/l2cap-tester: Fix sending right req code in tests

tools/l2cap-tester: Add expected response data to nval PDU test

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