media: Enable parallel requests to endpoint

If endpoint is handling several devices, it should be able to handle
multiple requests, one per each.

media: Create multiple transports if needed

During endpoint registration one than one device might be connected.
Thus all matching devices should have one transport each.

media: Split media_endpoint_create

This function is starting to be too long and needs to be split.

After this patch, the resulting code should be exactly equivalent as the
previous implementation.

media: Support multiple transports per endpoint

Several transports may exist for each endpoint, for example if several
HFGW are connected. This should be exposed to the endpoint as one
transport each.

audio: Add multiple device search to manager

This method is useful to search for more than one device fulfulling
certain criteria.

audio: Fix gateway state check

Gateway should be considered active also if connecting or playing.

This could for example lead to manager_find_device() not returning a
device that is connecting, and thus the corresponding endpoint would
never be created in the Media API.

SAP: Fix not unregistering dummy interface on exit

Dummy interface is registered on plugin init and is never unregistered

mgmtops: Fix wrong parameter order for btd_event_ltk_notify()

The "master" parameter was in the wrong place.

audio: Reset hfp and hs handle before connection

When bluez initiates headset connection  hfp and hs handle is saved
each time when initiating connection. So at some point if hf service
is removed in remote headset then bluez connects to hs service. But
because of previously stored hfp handle bluez thinks that connection
made to hs service is hfp connection and waits for at commands.
Eventually bluez connection state will be incorrectly set.

Fix adaptername shutdown

An inotify watch descriptor isn't a file descriptor, so don't
treat it as such and remove it properly.

monitor: add version information

monitor: add help information

gatttool: Fix handle parsing in cmd_char_write()

errno is not set to 0 before call to strtoll() so cmd_char_write() can fail
even with proper data. Use strtohandle() helper function instead.

hciops: Fix event mask for Inquiry Result with RSSI

The right value is 0x02 and not 0x04 (which is "Read Remote Extended
Features Complete").

l2test: Add option to inform the address type

This patch adds 'V' option to inform the address type. Possible values
are: "bredr", "le_public", and "le_random".

Remove MGMT address type definition

This patch replaces the address type defined in management interface
header by the new address type constants defined in the bluetooth.h.

Remove addr_type_t definition from adapter.h

This patch replaces addr_type_t by the new Bluetooth address type
constants defined in bluetooth.h

Remove leftover field in remote_dev_info struct

Field "bdaddr_type" of the remote_dev_info structure is not being
referenced in the code. Field "type" is already being used to store
the Bluetooth address type.

Add address type for BLE bt_io_connect calls

This patch adds the address type option in bt_io_connect calls for BLE
devices. BR/EDR is the default value, and it is not mandatory to inform
it. For BLE devices, it is necessary to inform if the type is public or
random.

btio: Add address type in bt_io_connect

This patch adds a new BtIO option to allow setting the remote Bluetooth
address type for BLE connections. Allowed values for BT_IO_OPT_DEST_TYPE
option are: BDADDR_BREDR, BDADDR_LE_PUBLIC, and BDADDR_LE_RANDOM.

Add Bluetooth address type in sockaddr_l2

This patch adds the address type information to sockaddr_l2 structure,
allowing the userspace to inform the remote address type required for
LE Create Connection command.

core: Remove DiscoverSchedulerInterval option from main.conf

This option was designed to be the time where name resolving would happen
in between inquiries, but with the introdution of mgmt interface this is
no longer the case as it treats the discovery as both inquiry/scan + name
resolving and keeping it just to delay discovery rounds is probably
useless.

input: add error message when auth fails

core: Fix adapter mode inconsistency

The mode change in btd_adapter_start() doesn't check if the adapter is
discoverable, creating an inconsistency between the plugin and the core.

Using the management interface it is easy to reproduce the problem, by
starting BlueZ while the adapter is discoverable. After this, the state
cannot be changed.

hciops: map KeyboardDisplay IO Capability to DisplayYesNo

Since hciops is not used for LE devices, if an agent announces the
KeyboardDisplay IO Capability when creating a bonding, map it to
DisplayYesNo.

This matches the existing change in hciops_set_io_capability which
is only used in the RegisterAgent path.

audio: Fix AVDTP Reconfigure Reject message

The specification describes a proper reject response to
AVDTP_Reconfigure, so the previously used unknown command response is
not correct.

Makefile: Always add all test scripts to EXTRA_DIST

With commit e407e9ac6d8bfde31f95ed5d7e7bcecf0cfe48ed
test-attrib, test-proximity and test-thermometer are only packaged
when "./configure --enable-gatt && make dist" is called.

The dist target should be independant on the configure flags.

eir: Fix incorrect eir_has_data_type() parsing

Updating the "parsed" variable twice inside the for loop, leads to
incorrect parsing.

eir: Fix incorrect eir_length() parsing

Issue:
The COD value displayed via dbus during inquiry is wrong.
This is because of the incorrect return length of the eir_length(),
which leads to appending the COD at wrong location.

Analysis:
After appending the COD at the end of the eir data, we can see
there are some '00' present in the eir field length in the eir file.
XX:XX:XX:XX:XX:XX 07095359414D5043020A040B0312111F110C110E110311
0000000000000000000000040D000142

Fix:
Corrected the length calculation in eir_length(), which is determining,
which position the COD should append

sdp: Fix compilation warning due to data type mismatch

In certain cross compiling environment, if we build the code with
enabling the maintainer mode, the following error happens.

cc1: warnings being treated as errors
lib/sdp.c: In function 'sdp_process':
lib/sdp.c:4111:6: error: comparison between signed and unsigned integer
expressions
make[2]: *** [lib/sdp.lo] Error 1
make[1]: *** [all] Error 2

audio: Add missing debug message in authorization

adapter: Add warning if there is no authorization agent

Sometimes BlueZ Agent is disappear suddenly or not registered
because of agent problem. but Bluez just prints "Operation Not
permiited"

This will help debugging.

src: Add function for printing warnings

build: Remove glib-compat.h support

build: Require GLib 2.28 or later

gdbus: Remove unneeded NEED_DBUS_WATCH_GET_UNIX_FD check

build: Require D-Bus 1.4 or later

audio: Add SBC NULL check for GST AVDTP signaling

Due to RF condition or headset malfunctioning, gst audio plug-in could
not get SBC (mandatory codec) information.

audio: Fix default disabled values in audio.conf

Disabled features in audio.conf should be the same with source code
"audio/manager.c:struct enabled_interfaces enabled".

input: Disconnect SDP before connecting HID if HIDSDPDisable is set

If remote HID device has HIDSDPDisable attribute present and set to
true host should disconnect SDP channel before connecting HID control
channel. Such devices multiplex resources between SDP and HID L2CAP
channels and will reject connection to HID control channel if SDP
channel is already connected.

sdp-client: Add ability to clear cached session

This will be usefull for disconnecting SDP L2CAP channel before cached
session timeout fires.

Makefile.am: Fix indentation

Only these two lines were using spaces instead of tabs.

Add Syam's email address

doc: Fix typo

src: Fix newline before EOF

input: Fix newline before EOF

audio: Fix newline before EOF

network: Remove header declared but not defined

headset: Remove header declared but not defined

adapter: Remove headers declared but not defined

sap-u8500: Fix empty parameter list in function declaration

sdpd-request: Fix passing zero instead of NULL

agent: Fix double D-Bus message unref

Inside the g_dbus_send_message() we are already doing the
D-BUS message unref.

gateway: Fix D-Bus reply memory leak

Re-arrange the code to avoid the D-Bus memory leak. Earlier if rfcomm
is not connected, then there was a reply memory leak.

gateway: Fix D-Bus memory leak in audio

gstavdtpsink: Fix D-Bus memory leak in audio

agent: Free the agent request properly

In the case of error, Unref the req->msg which is allocated inside the
pincode_request_new()

avrcp: Fix the response ctype of SetPlayerApplicationSettingValue

Request ctype of "SetPlayerApplicationSettingValue" is CONTROL,
so response should be ACCEPTED instead of STABLE.

health: Fix memory leak

AVRCP: Add missing capability for TrackReachedEnd

Add missing capability for TrackReachedEnd.

audio: Fix memory leak when headset server probe() fails

If RFCOMM is disabled on the kernel, headset_server_probe() fails.
Relevant log messages:

 audio/manager.c:headset_server_probe() path /org/bluez/499/hci0
 src/adapter.c:btd_adapter_ref() 0x4bb4f78: ref=6
 audio/manager.c:audio_adapter_ref() 0x4ca3010: ref=1
 socket(STREAM, RFCOMM): Protocol not supported (93)
 audio/manager.c:audio_adapter_unref() 0x4ca3010: ref=0
 src/adapter.c:btd_adapter_unref() 0x4bb4f78: ref=5
 audio-headset: Operation not permitted (1)

The powered callback should only be registered if adapter driver probe
was successful. The callback unregister was moved to the beginning of
headset_server_remove() for consistency.

This fixes this memory leak:

==499== 8 bytes in 1 blocks are definitely lost in loss record 44 of 182
==499==    at 0x4826444: malloc (vg_replace_malloc.c:263)
==499==    by 0x4877243: g_malloc (gmem.c:132)
==499==    by 0x488D088: g_slice_alloc (gslice.c:836)
==499==    by 0x488E8A5: g_slist_append (gslist.c:230)
==499==    by 0x18AEEE: btd_adapter_register_powered_callback
(adapter.c:3416)
==499==    by 0x11AF61: headset_server_probe (manager.c:919)
==499==    by 0x18B67B: probe_driver (adapter.c:2033)
==499==    by 0x1908F5: adapter_init (adapter.c:2048)
==499==    by 0x189D20: btd_manager_register_adapter (manager.c:397)
==499==    by 0x1649AF: mgmt_cmd_complete (mgmtops.c:1075)
==499==    by 0x16665E: mgmt_event (mgmtops.c:1780)
==499==    by 0x48B2EFA: g_io_unix_dispatch (giounix.c:162)

simple-agent: add DisplayPinCode

doc: document DisplayPinCode

Display PIN generated by plugin

If a plugin pincode callback sets the display parameter to TRUE, send
the generated PIN to the agent for display using the new DisplayPinCode
agent method, including its fallback to RequestPinCode.

Add display parameter to plugin pincode callback

Pass a display parameter to the plugin pincode callback, a plugin
may set this to TRUE to indicate the PIN it generates should be
displayed on the screen for entry into the remote device.

Add AUTH_TYPE_NOTIFY_PASSKEY to device_request_authentication

This new authentication type accepts a pincode and calls the
DisplayPinCode agent method, a fallback is provided so that if the
method is not implemented the older RequestPinCode method is used
instead.

Due to this fallback, the agent_pincode_cb is used and calling
functions should send the pincode passed to the callback to the
adapter, which may differ from that generated.

agent: add DisplayPinCode method

In constrast to DisplayPasskey, this sends a UTF-8 string PIN code
to the agent; also we support a callback for the case where the
Agent doesn't implement this new method so we can fallback.

Pass passkey by pointer rather than by value

This allows alternate data of a different type to be passed to
device_request_authentication() for other notification types such
as those that require a PIN.

Rename AUTH_TYPE_NOTIFY to AUTH_TYPE_NOTIFY_PASSKEY

This makes room for additional notification types to be added.

fakehid: Remove define not referenced

fakehid: Remove unneeded header includes

mgmtops: Handle Start Discovery Complete in case of failure

This patch adds handling of start discovery complete event in cases
when discovery fails to get started.

mgmt: update mgmt control channel id

kernel side has updated its control channel id from 1 to 3.

DeviceInfo: Fix compilation with old glib

Add missing include glib-compat.h to deviceinfo.c. deviceinfo_free
is using g_slist_free_full() which is not available in old glib
resulting in compilation error.

deviceinfo/deviceinfo.c: In function ‘deviceinfo_free’:
deviceinfo/deviceinfo.c:66: error: implicit declaration of function ‘g_slist_free_full’

AVRCP: Fix response length for Subunit Info command

This affect PTS Test Case TC_TG_ICC_BV_02_I

AVRCP: Fix response length for Unit Info command

This affect PTS Test Case TC_TG_ICC_BV_01_I

DeviceInfo: Read PNP ID

Read the PNP ID characteristic of the DeviceInfo Service, and
store it inside the btd_device, for use by other profiles.

DeviceInfo: Discover Characteristics

Add logic to discover all characteristics and build a
characteristic list.

DeviceInfo: Add connection logic

Add connection logic to the Device Information Plugin. When the
driver is loaded, it will request a connection to the remote device
and release the connection request when destroyed.

Add DeviceInformation GATT Client

Add the DeviceInformation GATT Client plugin skeleton.

Fix spelling errors

Fix spelling errors found using the codespell tool
(https://github.com/lucasdemarchi/codespell).

Adapter: Fix Discovering state while Powering Off

This patch fixes the adapter discovering state while powering off.
Without this fix, BlueZ sends incorrect discovering state to upper
layers while switching off.

attrib: Fix D-Bus service disconnect watch removal

watcher_free() will be automatically called when removing the D-Bus
watch (it was set as the destroy callback when adding the watch),
therefore it is only necessary to remove the watch.

thermometer: Fix D-Bus service disconnect watch removal

destroy_watcher() will be automatically called when removing the D-Bus
watch (it was set as the destroy callback when adding the watch),
therefore it is only necessary to remove the watch.

gdbus: Use destroy callback for service watch

Even though service watches accepted a "destroy" callback, they were
being ignored. This fix properly pass them along so they are called when
the watch is removed.

attrib: Fix "make distcheck" build

Any new header files (att-database.h in this case) need to be added to
Makefile.am.

Fixes this "make distcheck" error:

  CC     attrib/bluetoothd-gatt-service.o
../attrib/gatt-service.c:35:26: fatal error: att-database.h: No such
file or directory
compilation terminated.
make[2]: *** [attrib/bluetoothd-gatt-service.o] Error 1
make[1]: *** [all] Error 2
make: *** [distcheck] Error 1

update Gustavo's email address

gattrib: Protect GAttrib when there is a pending write

attrib: Fix missing Discovery reply when disconnected

This patch fixes missing reply for DiscoverCharacteristics method on
Generic Attribute. If link is disconnected reply message is not sent.

proximity: reporter: implement D-Bus API

Use a device driver to track all GATT supporting devices and register
a D-Bus interface per remote device.

Implement the "GetProperties" method of the D-Bus interface by querying
the alert level of the remote device in the link-loss and
immediate-alert proximity profiles. The default alert level values for
non-connected devices are "none".

proximity: immediate alert: implement immediate alert server

The profile is implemented in immalert.[ch]. A GATT service is
registered with a write callback on the immediate alert level attribute.
This attribute is write-only and is maintained per remote device.

When a remote device write a raises or lowers the alert level,
an appropriate PropertyChanged signal is emitted. When the alert level
of a device is non-zero, a callback is registered on its disconnection.
When the callback is called, the alert level of the device is reset to
zero and an appropriate signal is emitted.

proximity: link loss: implement link loss server

The profile is implemented in linkloss.[ch]. A GATT service is
registered with read/write callbacks on the link-loss alert level
attribute. The alert level is maintained per device. It is returned
on read and updated on write.

When the alert level is non-zero, a callback is registered on the
disconnection of the remote device. If a device with non-zero alert
state is disconnected, an appropriate PropertyChanged signal is emitted
with the alert level previously set by the device. We avoid emitting
a signal when the disconnection was requested by us.

proximity: reporter: move definitions to .h and add util function

This allows us to re-use these definitions in GATT sub-profiles.

proximity: reporter: save global D-Bus connection

This connection will be used by reporter GATT sub-profiles.

att: add remote btd_device to ATT read/write callbacks

This allows us to identify the remote device that made the ATT
read/write.

adapter: Fix segfault when icon is uninitialized

If device type is LE and GAP Appearance characteristic does not exist,
the icon will not be initialized.

AVCTP: return not-implemented for unknown passthrough

Makefile: Add simple-player and test-nap to EXTRA_DIST

Add simple-player and test-nap to the EXTRA_DIST target.

eir: Read GAP appearance from advertising data

This was introduced on Core Specification Supplement (CSS) v1.

device: Appearance characteristic from GAP service

After searching for primary services, the appearance characteristic is
read from GAP service and saved in "appearance" file. Running
GetProperty in device interface, will show the respective icon based in
"appearance" value.