gdbus: Fix using NULL DBusPendingCall when disconnected from D-Bus

From D-Bus documentation for dbus_connection_send_with_reply():

"Warning: if the connection is disconnected or you try to send Unix file
descriptors on a connection that does not support them, the
DBusPendingCall will be set to NULL, so be careful with this."

Fix these errors when killing D-Bus daemon with the client still
running:

process 5712: arguments to dbus_pending_call_set_notify() were
incorrect, assertion "pending != NULL" failed in file
../../dbus/dbus-pending-call.c line 596.
This is normally a bug in some application using the D-Bus library.
process 5712: arguments to dbus_pending_call_unref() were incorrect,
assertion "pending != NULL" failed in file
../../dbus/dbus-pending-call.c line 572.
This is normally a bug in some application using the D-Bus library.

gdbus: Fix segfault when D-Bus daemon exits

Fix this crash if D-Bus exits while the client is still connected to it:

==5570== Invalid read of size 1
==5570==    at 0x402D28E: strcmp (in
/usr/lib/valgrind/vgpreload_memcheck-x86-linux.so)
==5570==    by 0x4070E22: g_str_equal (ghash.c:1704)
==5570==    by 0x8055F61: message_filter (client.c:1123)
==5570==    by 0x4141500: dbus_connection_dispatch (in
/lib/i386-linux-gnu/libdbus-1.so.3.5.8)
==5570==    by 0x80506F7: message_dispatch (mainloop.c:76)
==5570==    by 0x4081A7E: g_timeout_dispatch (gmain.c:3882)
==5570==    by 0x4080D85: g_main_context_dispatch (gmain.c:2539)
==5570==    by 0x4081124: g_main_context_iterate.isra.21 (gmain.c:3146)
==5570==    by 0x408156A: g_main_loop_run (gmain.c:3340)
==5570==    by 0x41BF4D2: (below main) (libc-start.c:226)
==5570==  Address 0x0 is not stack'd, malloc'd or (recently) free'd
==5570==
==5570==

core: Don't update a known long name with a short name.

update_found_devices() could attempt to update a device with new information
containing only a short verstion of the name. If the struct eir_data passed
has a short or NULL name, it will never update a previous name and will
also trigger a "HCI Command: Remote Name Request" if the previous name was
unknown.

profile: Error-cases for incoming connections

When an incoming connection attempt is received, if the corresponding
device or service is not found, handle the error-case.

profile: Fix remotely initiated connections without service

Remotely initiated connections need to be associated to a service
instance to avoid the following crash as reported by Vinicius Costa:

bluetoothd[14366]: src/profile.c:ext_confirm() incoming connect from A0:4E:04:F6:F5:05
bluetoothd[14366]: src/profile.c:ext_confirm() hfp_hf authorizing connection from A0:4E:04:F6:F5:05
bluetoothd[14366]: src/agent.c:agent_ref() 0x6a85e0: ref=2
bluetoothd[14366]: src/agent.c:agent_authorize_service() authorize service request was sent for /org/bluez/hci0/dev_A0_4E_04_F6_F5_05
bluetoothd[14366]: src/profile.c:ext_svc_complete() Services resolved for A0:4E:04:F6:F5:05
bluetoothd[14366]: src/profile.c:ext_svc_complete() Services resolved but still waiting for authorization
bluetoothd[14366]: src/profile.c:ext_auth() A0:4E:04:F6:F5:05 authorized to connect to hfp_hf
bluetoothd[14366]: src/agent.c:agent_unref() 0x6a85e0: ref=1
bluetoothd[14366]: src/profile.c:ext_connect() hfp_hf connected to A0:4E:04:F6:F5:05

Program received signal SIGSEGV, Segmentation fault.
btd_service_connecting_complete (service=0x0, err=err@entry=0) at src/service.c:315
315 if (service->state != BTD_SERVICE_STATE_DISCONNECTED &&

device: Add function to find a given service

Search within the list of services for a given remote UUID.

neard: Fix reading local OOB data when adapter is not powered

Read Local OOB data only if adapter is powered. This fix replying with
error instead of CPS set to inactive when adapter is not powered.

neard: Fix checking for unknown carrier power state

Message might not contain State field and CPS needs to be set to
unknown before processing message from neard. This fix processing
RequestOOB called without parameters.

audio: Remove left over audio_device_cb

media: Fix not canceling requests properly if endpoint exit

If endpoint exit while a request is pending, e.g. due to a crash, the
callback should be called with ret set to NULL otherwise the code that
has generated the request will not notice the request has been canceled
leaving the endpoint unusable.

network: Fix invalid read while removing device

Invalid read of size 8
   at 0x42F4B2: connection_unregister (connection.c:643)
   by 0x466CE6: service_shutdown (service.c:174)
   by 0x467B70: remove_service (device.c:963)
   by 0x32C3863CAC: g_slist_foreach (in /usr/lib64/libglib-2.0.so.0.3400.2)
   by 0x32C3863CCA: g_slist_free_full (in /usr/lib64/libglib-2.0.so.0.3400.2)
   by 0x46CABC: device_remove (device.c:2331)
   by 0x45E229: adapter_remove_device.constprop.44 (adapter.c:1049)
   by 0x45E31D: remove_device (adapter.c:2134)
   by 0x40CC80: process_message.isra.4 (object.c:258)
   by 0x32C641D9C4: ??? (in /usr/lib64/libdbus-1.so.3.7.2)
   by 0x32C640FC1F: dbus_connection_dispatch (in /usr/lib64/libdbus-1.so.3.7.2)
   by 0x40A807: message_dispatch (mainloop.c:76)
 Address 0x54e6e38 is 8 bytes inside a block of size 16 free'd
   at 0x4A077E6: free (vg_replace_malloc.c:446)
   by 0x32C384D79E: g_free (in /usr/lib64/libglib-2.0.so.0.3400.2)
   by 0x40DCA9: remove_interface (object.c:656)
   by 0x40DF09: g_dbus_unregister_interface (object.c:1413)
   by 0x466CE6: service_shutdown (service.c:174)
   by 0x467B70: remove_service (device.c:963)
   by 0x32C3863CAC: g_slist_foreach (in /usr/lib64/libglib-2.0.so.0.3400.2)
   by 0x32C3863CCA: g_slist_free_full (in /usr/lib64/libglib-2.0.so.0.3400.2)
   by 0x46CABC: device_remove (device.c:2331)
   by 0x45E229: adapter_remove_device.constprop.44 (adapter.c:1049)
   by 0x45E31D: remove_device (adapter.c:2134)
   by 0x40CC80: process_message.isra.4 (object.c:258)

audio: Remove device audio states

This is no longer useful without org.bluez.Audio interface

audio: Don't create an avctp session on avrcp disconnect

If a paired and connected audio device is disconnected, the
avrcp_disconnect() could create a new avctp session that will keep
a reference to the corresponding btd_device, preventing it to be
removed as explained below. This fix prevents avrcp_disconnect()
to create a new and disconnected avctp session when it doesn't
exists.

Calling org.bluez.Device1.Disconnect on an audio device like the
"Monster ClarityHD" will cause first a a2dp_sink_disconnect() call,
and then a sink_disconnect() call.
This will change the state of the existing avdtp session to
AVCTP_STATE_DISCONNECTED triggering a series of callback calls.
Among those, the avdtp_set_state() function will call the registered
avdtp_callbacks, including avdtp_state_callback() which in turns
updates the disconnected state using sink_set_state(). This function
will call the registered sink_callbacks, including device_sink_cb().

By this point, the device_sink_cb() will attempt a avrcp_disconnect()
over a session that was already disconnected before by the device's
diconnect_cb(). This new avrcp_disconnect() causes the avctp_get() to
create a new avctp session that holds a reference to the disconnecting
btd_device.

Steps to reproduce using bluetoothctl:
1. Pair and Connect a Monter ClarityHD audio device.
2. Play some music on it.
3. Disconnect the device.
4. Remove the device.
The "remove" command succeeds, but bluetoothd does not sends a removal
signal ([DEL] message) for that device.

audio: Remove AVDTP timer

The timer is no longer used after the introdution of btd_profile.

gdbus: Fix calling GetManagedObjects twice in a row

Calling g_dbus_client_new followed by g_dbus_client_set_proxy_handlers
cause two calls to GetManagedObjects in a row as GetNameOwner reply is
asyncronously it triggers the second call because the handlers have
been set by g_dbus_client_set_proxy_handlers.

unit: Get rid of gchar

Use plain char instead of gchar.

core: Get rid of gchar

Use plain char instead of gchar.

obexd: Get rid of gchar

Use plain char instead of gchar.

gobex: Get rid of gchar

Use plain char instead of gchar.

client: Get rid of gchar

Use plain char instead of gchar.

attrib: Get rid of gchar

Use plain char instead of gchar.

unit: Get rid of gint

Use plain int instead of gint. In glib gint is always a typedef to int,
so it's safe to use it even for callbacks with glib.

tools: Get rid of gint

Use plain int instead of gint. In glib gint is always a typedef to int,
so it's safe to use it even for callbacks with glib.

core: Get rid of gint

Use plain int instead of gint. In glib gint is always a typedef to int,
so it's safe to use it even for callbacks with glib.

thermometer: Get rid of gint

Use plain int instead of gint. In glib gint is always a typedef to int,
so it's safe to use it even for callbacks with glib.

scanparam: Get rid of gint

Use plain int instead of gint. In glib gint is always a typedef to int,
so it's safe to use it even for callbacks with glib.

input: Get rid of gint

Use plain int instead of gint. In glib gint is always a typedef to int,
so it's safe to use it even for callbacks with glib.

heartrate: Get rid of gint

Use plain int instead of gint. In glib gint is always a typedef to int,
so it's safe to use it even for callbacks with glib.

health: Get rid of gint

Use plain int instead of gint. In glib gint is always a typedef to int,
so it's safe to use it even for callbacks with glib.

gatt: Get rid of gint

Use plain int instead of gint. In glib gint is always a typedef to int,
so it's safe to use it even for callbacks with glib.

deviceinfo: Get rid of gint

Use plain int instead of gint. In glib gint is always a typedef to int,
so it's safe to use it even for callbacks with glib.

cyclingspeed: Get rid of gint

Use plain int instead of gint. In glib gint is always a typedef to int,
so it's safe to use it even for callbacks with glib.

audio: Get rid of gint

Use plain int instead of gint. In glib gint is always a typedef to int,
so it's safe to use it even for callbacks with glib.

plugins: Get rid of gint

Use plain int instead of gint. In glib gint is always a typedef to int,
so it's safe to use it even for callbacks with glib.

obexd: Get rid of gint

Use plain int instead of gint. In glib gint is always a typedef to int,
so it's safe to use it even for callbacks with glib.

gobex: Get rid of gint

Use plain int instead of gint. In glib gint is always a typedef to int,
so it's safe to use it even for callbacks with glib.

attrib: Get rid of gint

Use plain int instead of gint. In glib gint is always a typedef to int,
so it's safe to use it even for callbacks with glib.

emulator: Remove unneeded btdev_set_bdaddr function

lib: Fix list of company identifiers

Fix minor typos on company names.

shared: Remove static address assignment in hciemu

If we need multiple virtual devices, addresses should not be the same.
Inside the function btdev_create, a specific address is generated.

shared: Return real address of master dev in hciemu

Exports master btdev address in hciemu.

emulator: Add getter function for device address in btdev

This helper function returns bdaddr field from btdev.

attrib-server: Fix btd_device_get_storage_path failure handling

proximity: Fix btd_device_get_storage_path failure handling

gatt: Fix btd_device_get_storage_path failure handling

alert: Fix btd_device_get_storage_path failure handling

core: Refuse storage for private addressed devices

core: Fix storing info for private addressed devices

Devices with private addresses cannot persistently be identified by
their address. Therefore it doesn't make sense to store any information
about them as the information couldn't be looked up later once the
remote address has changed.

Once the kernel receives support for private address resolution we'll
start getting the public address of such devices to user space. However,
until then we just have to ignore such devices from a storage
perspective.

core: Remove useless attrib server confirm callback

profile: Use btd_service for connect/disconnect

Change the btd_profile connect/disconnect callbacks to receive a
btd_service pointer. This should make it possible to handle multiple
instances of the same profile in a specific device.

The patch strongly influences how the profiles should interact with the
core. The state transitions, previously reported using the device.h API
(device_profile_connected() and device_profile_disconnected()), have
now been replaced by the btd_service API.

The transitions will then be propagated to device.c by means of the
conventional state-changed callback mechanism.

service: Add callbacks to track state changes

Extend the btd_service API to support state observers.

service: Add error-code btd_service

The error-code represents the result of a connection/disconnection
procedure and can be useful when a state transition is detected.

tools: Fix parsing of PlayItem pdu in hcidump

monitor: Fix ATT Read By Type Response minimum length

input: Hold a reference to btd_service

Profile implementations are allowed to hold a reference to probed
services and make use of the service's userdata pointer, so update the
input profiles accordingly.

control: Bypass manager for connect state changes

Move the code to propagate the AVRCP->core state changes from manager.c
to control.c.

audio: Hold a reference to btd_service

Profile implementations are allowed to hold a reference to probed
services, so update the audio profiles accordingly.

network: Hold a reference to btd_service

Profile implementations are allowed to hold a reference to probed
services and make use of the service's userdata pointer, so update the
network profiles accordingly.

core: Add user_data to btd_service

Profile implementations can benefit from a user_data associated to a
service, which will typically be set during probe.

network: Add trivial helper function

Simplify the code and make it more readable by adding a trivial helper
function.

profile: Use btd_service for probing profiles

Change the profile probe mechanism in order to pass the btd_service
instance representing the remote service. This object is bound to a
btd_profile and a btd_device, thus replacing the previous parameters.

The probe callback is allowed to hold a reference to the btd_service
by means of btd_service_ref(), which should be unreferenced during
removal.

Keeping such a reference of the btd_service allows supporting multiple
instances of the same UUID, since the reference can serve as a handle
during the interactions between the profile implementation and the core.

device: Replace pending profile list with services

Use btd_service pointers to represent the list of services pending for
connection.

This list doesn't reference count the btd_service instances, since the
pending services should be a subset of the available services, and thus
they should already be referenced by device->services. This means
special care must be taken to make sure any removed service is also
removed from the pending list, as addressed in remove_service().

device: Find services instead of profiles

Trivially replace the former find_connectable_profile() with
find_connectable_service().

device: Replace connected_profiles with btd_service

The service state can provide enough information not to maintain such
a connected_profiles list. Therefore, avoid duplicated information and
remove the list.

core: Add a state enum to btd_service

Add a state to btd_service in order to distinguish which operations are
allowed in a certain point in time.

core: Add connect/disconnect API to btd_service

Make use of the btd_profile connect/disconnect callbacks directly within
service.c instead of doing it inside device.c.

core: Add probe API to btd_service

Make use of the btd_profile callbacks directly within service.c instead
of doing it inside device.c.

device: Use btd_service to represent profiles

Maintain a list of btd_service pointers instead of btd_profile pointers,
for services that have been probed for this device.

This list will not only reference count the btd_service instances but
also "own" them, in the sense of destroying them (make them completely
unavailable) when the service is removed from the list. This is needed
because the btd_service does not reference count the btd_device
instance.

core: Add btd_service to represent device services

Add a new object to centralize all data corresponding to a service
provided by a remote device, coupling a pair of btd_profile and
btd_device pointers.

network: Bypass manager during probe/remove

Use the internal device API to provide the btd_profile callback
functions, without boilerplate code in manager.c.

input: Remove useless forward declarations

input: Bypass manager during probe/remove

Use the internal device API to provide the btd_profile callback
functions, without boilerplate code in manager.c.

input: Remove unused UUID field

Trivially remove the field because it's actually not being used.

input: Move timeout setting to device.c

The timeout setting is global across devices and it's actually only used
inside device.c. Therefore, simplify the code by just passing it once.

input: Fix ignored profile shutdown

Unprobing the profile should never fail which means neither should
input_device_unregister(). This requires shutting everything down
regardless of possible pending connection requests.

network: Fail if multiple profile instances

Considerable parts codebase are assuming that there is one single
instance of each UUID and therefore return an error during probe instead
of silently ignoring duplicated entries.

profile: Remove UUID list from probe function

Each instance of btd_profile is associated to one single remote UUID
and therefore the list provided by the callback is redundant and can be
removed.

input: Simplify code since UUID is known

The UUID in this code path is fixed so trivially use it instead of
using the provided probe parameter.

control: Simplify UUID handling during probe

There is actually one single UUID which can be received during probe, so
simplify the code accordingly.

device: Fix returned error code

The second 'err' shadows the main local variable which is being used to
as return value of the function. Otherwise -ENOENT was always returned
in case of error.

doc: Fix optional properties not marked as such

Commit 826023de56896e83f35ae69a73d6a120a2f5b11e made some of the
device's properties optional but the documentation was not updated
accordingly. More specifically, the properties 'Name' and 'UUIDs' won't
be present if the information is not available.

monitor: Add an option to dump SCO traffic

attrib: Change match_event return type from gboolean

There is no reason to use gboolean as return type of match_event.

attrib: Change is_response return type from gboolean

There is no reason to use gboolean as return type of is_response.

attrib: Change command->sent type from gboolean

There is no reason to command->sent be gboolean.

attrib: Change GAttrib->stale type from gboolean

There is no reason to GAttrib->stale be gboolean.

attrib: Simplify boolean tests

Remove unnecessary TRUE/FALSE in boolean tests.

tools/mgmt-tester: Fix expect_len assignment to use right variable

tools/mgmt-tester: Add valid test case for LE-only discovery

tools/mgmt-tester: Update LE-only test case pre-conditions

Starting with Linux 3.10 single mode controllers will always have the LE
setting bit enabled in the current_settings mask.

tools: Fix avrcp_media_element_item_dump to print elements properly

core: Delay Paired property until SDP completes

To make it easier for clients to know when SDP has completed, and to
make the property consistent with the timing of the Device1.Pair()
method return it makes sense to delay the "Paired" property change until
SDP has completed.

input: Automatically attempt a reconnect when required.

The HID 1.1 spec requires a host to attempt a reconnection when a
HID device goes out of range and comes back. There is test (see
TP/HCE/BV-04-I [Host Initiated Re-connection]) to ensure that the
host initiates the connection when that happens.

This patch adds a reconnection timer for HID devices trying to
reconnect every 30s for a maximum of 3 minutes after the connection
to a HID device with a ReconnectMode of "host" or "any" is closed.

input: Fix memory leak for hidp_connadd_req.

If the struct input_device is destroyed while a hidp_connad_req
is pending, the input_device will be destroyed leaking the associated
memory.

==30790== 492 (168 direct, 324 indirect) bytes in 1 blocks are definitely lost in loss record 199 of 216
==30790==    at 0x4C29DB4: calloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==30790==    by 0x4E7FAE0: g_malloc0 (gmem.c:189)
==30790==    by 0x4378E4: hidp_add_connection (device.c:365)
==30790==    by 0x438055: input_device_connected (device.c:525)
==30790==    by 0x4380DA: interrupt_connect_cb (device.c:548)
==30790==    by 0x443508: connect_cb (btio.c:230)
==30790==    by 0x4E79D52: g_main_context_dispatch (gmain.c:2539)
==30790==    by 0x4E7A09F: g_main_context_iterate.isra.23 (gmain.c:3146)
==30790==    by 0x4E7A499: g_main_loop_run (gmain.c:3340)
==30790==    by 0x448CCC: main (main.c:583)

input: Convert gboolean to bool

Remove some usage of gboolean in favor of bool.

input: Implement the new ReconnectMode Input1 property.

The "Connectability" of a HID device, as defined by the HID spec,
governs the way the host may and should connect to a HID device or
expect a connection from it. In the comon case of mice and keyboards
the HIDNormallyConnectable is FALSE and HIDReconnectInitiate is TRUE,
since those devices only attempt a connection to the host when they
have some data to transfer. A connection attempt initiated from the
host after the device drops the connection (while still paired) will
result in a Page timeout.

This patch exposes a new property called "ReconnectMode" combining the
those two SDP attributes as shown in the Connectability section of the
HID spec (see section 5.4.2). The property can have one of the following
four values: "None", "Device", "Host", "Any", and is derived from the
SDP cached value on device creation even if the device is off.

input: Documentation for new Input1 interface

Adds documentation for a new Input1 interface explaining a new
"ReconnectMode" property that exposes the Connectability mode of
a HID device.

tools: Fix AVRCP parsing of GetFolderItems for folders and items

Length of name is 2 bytes not 1

emulator: Remove extended features from LE-only controllers

emulator: Move Event Mask Page 2 from BR/EDR to BR/EDR/LE