Fix duplicate characteristics

The Discover() operation (from DBus API) was registering the same
characteristics after running several times. This can be checked using
test-attrib.

mgmt: Add support for connected/disconnected events

Keep connection handle information inside adapter_ops

There's no point to export connection handle details to the core daemon
since it will not be available in the management interface. Instead
track the handles inside hciops (managment interface will do this inside
the kernel).

mgmt: add initial support for link key handling

Remove unused storage functions

Use an unsigned integer for representing the PIN length

The value 0 is not allowed by the HCI spec so it can be used to denote
"not applicable" or "unknown". Also, the management interface will use
an unsigned integer for the PIN length so mirroring that in the core
daemon code makes sense. This patch also takes care of handling
previously stored -1 values and converts them to 0 whenever they are
detected.

Release 4.86

Add READ_BLOB_REQUEST support to attribute server

Fix Handle range if Pri/Sec Service is Empty

Update a2dpsink to use new Acquire API

Update NREC callback code to originally intended v2 state

Add proper tracking mechanism to NREC

NREC may change during the connections so it has to be tracked in order
to signal changes to applications.

Add Routing property to MediaTransport interface

This should indicate to the endpoint what routing the transport is using

Remove IMTU and OMTU properties and return its values on Acquire reply

This should make Acquire blocking friendly since the client no longer has
to call GetProperties to discover how much it can write/read when using
the acquired file descriptor.

Move common code from Discover all Characteristics to GATT library

The attribute client (attrib/client.c) and gatttool share similar code
to parse the PDU coming from server. This commit moves this common code
to attrib/gatt.c, and simplifies the callbacks implemented by the
clients. The client callbacks are now called just once and get a GSList
of characteristics, instead of the raw PDU.

Rename gatt_primary_t to more generic name

The gatt_primary_t typedef was renamed to gatt_cb_t because it will be
used for primary and characteristic callbacks.

gdbus: Remove root node 'name' attribute in introspection

generate_introspection_xml generates the root <node> tags with a
'name' attribute. This seems to be a valid attribute but it is not
consistent with the way the D-Bus daemon generates empty nodes.

For example if we register "/foo/bar", D-Bus daemon will generate for
"/foo" a introspection which looks like this:

<!DOCTYPE node PUBLIC "-//freedesktop//DTD D-BUS Object Introspection 1.0//EN"
"http://www.freedesktop.org/standards/dbus/1.0/introspect.dtd">
<node>
  <node name="bar"/>
</node>

and generate_introspection_xml generates for "/foo/bar":

<!DOCTYPE node PUBLIC "-//freedesktop//DTD D-BUS Object Introspection 1.0//EN"
"http://www.freedesktop.org/standards/dbus/1.0/introspect.dtd">
<node name="/foo/bar">
</node>

Just don't add the 'name' attribute to the root node.  The GLib
binding for D-Bus does it the same way.

gdbus: invaldate_parent_data: walk the whole path down

Assume there is only one object registerd at "/". If we add a new
object at "/foo/bar" the introspection of "/" has to be updated. A new
node has to be added at "/".

invalidate_parent_data stops invaldating the whole path because the
boolean return value of dbus_connection_get_object_path_data is used
wrong.

If we get a TRUE just go on down in the path, if FALSE is return
dbus_connection_get_object_path_data has run out of memory.

Fix use of deprecated glib on tests

Fix use of deprecated glib on core

Fix use of deprecated glib on hciops plugin

Fix use of deprecated glib on serial plugin

Fix use of deprecated glib on network plugin

Fix use of deprecated glib on input plugin

Fix use of deprecated glib on audio plugin

Use -DG_DISABLE_DEPRECATED compiler define

Update Class of Device handling to the latest management interface

This patch updates the class of device handling code to match the latest
management interface. A notable feature is that UUID to service class
mapping policy remains in user space with the help of the svc_hint
parameter of the add_uuid command. This is to make it easy to update the
mapping table when new profiles come along.

Revert "Send an Invalid PDU Size Error Response for Service Search Req"

This reverts commit 3094ec7008735c80d35bd255ffc41e7bea2d3a3c.

Commit 3094ec7008735c80d35bd255ffc41e7bea2d3a3c was supposed to fix
TP/SERVER/SS/BI-01-C but it turns out it doesn't even affect the outcome
of that test case (the code path is never traversed). Additionally the
commit breaks TP/SERVER/SS/BI-02-C so it should be reverted.

Fix memory leak when calling endpoint to select capabilities

When passing allocated memory to avdtp_service_cap_new it needs to be
freed since avdtp_service_cap_new copies the memory.

Fix possible memory leak on each a2dp stream setup

Setup capabilities were not freed when capabilities were automatically
generated and not just taken as it used to be with old unix socket
clients.

Fix possible memory leak when handling avdtp transport connection

When acting as initiator stream io already has a reference to io channel
to be able to cancel if necessary.

Fix memory leak when calling endpoint method

The pending call must be unrefed in those cases

Release 4.85

Fix event mask setting for a specific 1.2 adapter

One 1.2 Broadcom adapter doesn't respond to the event mask HCI command
at all unless the second byte of the mask is 0xff. Strictly speaking
this is an invalid value because bits 0x02 and 0x04 are reserved.
However, no other adapters seem to mind and using 0xff instead of 0xf9
makes this particular adapter work properly.

Fix event mask setting for 1.1 controllers

Controllers based on the core spec 1.1 support the event mask HCI
command but need a very stripped down default value for the mask. This
patch makes sure the command gets issued for 1.1 controllers with the
appropriate value.

telephony-ofono: add support for Three Way Calling feature

The Three Way Calling additionally support AT+CHLD values 0,3 and 4.

telephony-ofono: add support for Enhanced Call Control feature

The Enhanced Call Control feature is simply an extension of the current
AT+CHLD command:

 - Release Specified Call: AT+CHLD=1<idex>
 - Private Consultation Mode: AT+CHLD=1<idex>

The new arguments for this command include an index of a specific call as
indicated in the +CLCC response.

telephony-ofono: add proper support for AT+CHLD and AT+CKPD

telephony-ofono: add support for inband ringtone feature

Move interactive code of Discover Primary to gatt.c

Initial patch to move the shared code related to Discover All Primary
Services and Discover Primary Services by UUID to gatt.c.

Remove unused variable from browse_req structure

Remove unnecessary type cast

Move GATT service to attribute server core

GATT service (like GAP one) should be moved to the core attribute server
because there can be only one instance of it.

There was a "Attribute Opcodes Supported" characteristic inside the GATT
service in attrib/example.c which is not defined by the Core
specification and was removed before moving the code.

Move GAP service registration to src/attrib-server.c

GAP service shall be registered only once, so it makes sense to move it
to the core attribute server code.

The GAP "Device Name" characteristic was always "Example Device" on the
example server. This has been changed to use the Name attribute from
main.conf.

Refactor example server to use att_put_u16()

The original code was written before att_put_u16() was implemented.

Fix gatt_read_char() to support long Attrib Values

Fix gatt_read_char() to support long Attribute Values by recognizing
that results longer that 21 octets may include data beyond
what has been returned with the first read. Extra data is
obtained by issuing READ_BLOB requests until either a
result is returned shorter than 22 octets, or an error
is recieved indicating that no further data is available.
The API for this function has not changed.

Fix g_attrib_send() to include a new ID parameter

Overall purpose of change is to enable a GATT procedure to be
executed atomically, even if it requires multiple ATT
request/response transactions.

Fix g_attrib_send() to include an ID parameter, if the pkt to
be sent should be added to the Head of the pkt queue.  If the
ID is Zero, legacy functionality is maintained, and the pkt will
be added at the tail of the queuer, and a new ID will be generated,
and returned to the caller. If ID is non-zero, the pkt will be
added to the head of the queue, with the ID value requested, which
will also be returned to the caller.

Fix received_data() to not service the send queue until after the
received data has been processed by calling the cmd->func()
callback, to allow the callback to insert another pkt on the head
of the queue.

Fix all callers of g_attrib_send() to include new parameter.

telephony-ofono: fix not updating indicators when a call is removed

In some extreme cases such as modem removal ofono may not have a chance
to set the call to disconnected state so indicators would not be update
properly.

telephony-ofono: add handling for Modem.Interfaces property

This should make sure a modem is only used if a network is available

telephony-ofono: handle ofono appearing/disappearing from bus

telephony-ofono: simplify watches handling logic

telephony-ofono: fix not canceling pending calls on exit

telephony-ofono: make use of GetCalls method

telephony-ofono: make use of GetModems method

Ofono API has changed, modem list is no longer a property

telephony-ofono: fix not setting originating when status property changes

telephony-ofono: fix not being able to answer alerting/waiting calls

telephony-ofono: notify alerting calls when headset connects

TODO: Start Discover All Primary Services after SDP if necessary

TODO: Refactoring of gatt.c functions

TODO: Rename glib-helper file to a more convenient name

TODO: Remove characteristic discovery for non connectable devices

CreateDevice and CreatePairedDevice methods already check the event
type and AD Flags before to connect to discover all primary services.
Characteristic discovery will not be available since D-Bus method
is not exposed to Broadcaster devices.

TODO: Setup advertising parameters and data

Change security level on demand when reading characteristic descriptor

Change security level on demand when reading characteristic value

If a characteristic requires a higher security level, change it on
demand and re-send the GATT Charateristic Value Read. Request will not
be sent until the SMP negotiation finishes. This change doesn't affect
GATT over BR/EDR, since encryption is mandatory for BR/EDR.

mgmt: Fix opcode endianess in commands

mgmt: Add preliminary UUID handling

This patch adds support for the add_uuid and remove_uuid management
commands. The kernel uses them to track the list of UUIDs and
automatically set the service class bits as well as the EIR value
accordingly.

gdbus: Update copyright information

mgmt: Implement set_pairable

This patch implements support for the set_pairable managment command.
Due to the async nature of it a new btd_adapter_pairable_changed
function is added to the core daemon.

Refactor handling of UUIDs with respect to adapter_ops

The management interface will have simple add_uuid and remove_uuid
methods so it makes sense to have the adapter_ops interface reflect
that. This patch removes the services_updated callback and replaces it
with add_uuid and remove_uuid callbacks.

Rename mgmt_update_mode to mgmt_update_powered for clarity

mgmt: Use a common structure for powered, discoverable and connectable

The events, commands and responses to these modes are represented by
identical management messages. By having a unified struct for all of
them quite a lot of code can simplified and reused.

mgmt: Clean up scan mode evaluation code

Update mgmt-api.txt to match latest implementation

mgmt: Implement set_connectable

mgmt: Implement set_discoverable

mgmt: Implement set_powered

Keep found LE device list to use when creating a device

Found devices list should be used to identify device types when
CreateDevice or CreatePairedDevice are called. During the device
creation the selected device will be classified as LE device if it
belongs to the device found list and if the Flags AD type matches the
GAP requirements. BR/EDR devices don't need to be tracked since the
default remote device type is Basic Rate.

Add LE read supported states for hciconfig

Displays the states and state combinations that the link layer
supports. Usage: hciconfig hcix lestates

Simplify btd_event_conn_complete logic

Don't force device creation for pairing failure events

A pairing failure could be caused by switching Bluetooth off or a
disconnect forced by cancelling the pairing attempt so it doesn't make
sense to re-create a device object in these cases.

Remove unneeded set_connectable adapter_ops callback

Release 4.84

Fix leak of EIR data if RSSI does not change

If RSSI value does not change, memory used by parsed EIR data would leak
because it would not be assigned to the remote_dev_info structure.

Fix crash due to misplaced parameter in emit_device_found() call

To avoid related mistakes, dev->uuid_count was shortened to uuid_count
and kept on the same line.

Rename btd_adapter_get_state to btd_adapter_get_mode

There's already an adapter_get_state function so to avoid confusion it's
better to have a different name.

Remove unused variable

Fix adapter state when InitiallyPowered=false and RememberPowered=false

This patch reintroduces the adapter->initialized flag since it's
actually needed to make sure we only apply the InitiallyPowered setting
for the very first time when initializing an adapter.

Remove outdated comments

Remove ancient NAME_SENT name resolution status

The NAME_SENT status was introduced on commit
d6a16516a9f6deae8342f00e8186b02d0019a1e1, when there was a
"RemoteNameUpdate" D-Bus signal. Nowadays, there is no such signal, and
the device name (if any) is always sent on "DeviceFound" signal.

Fix printing D-Bus errors when headset record could not be found

When connection is started via headset_config_stream there is no D-Bus
message to reply to.

Add debug print for tracking adapter_mode_changed calls

Fix closing of SDP server sockets

The existing code relied on set_close_on_unref which doesn't work as
long as the io watch callbacks hold on to their own reference. To fix
this it's better to track the watch id's instead of the IO channels and
do g_source_remove instead of g_io_channel_unref.

This isn't a particularly critical fd leak since these are created once
on bluetoothd startup and closed on exit, however they do help clean up
the valgrind fd-leak report.

Release 4.83

Add Broadcaster property in DeviceFound signal

Broadcaster property is required to distinguish the device role. If the
remote is sending an advertising event, two possible roles are possible:
Peripheral or Broadcaster.

This change is required to pass on qualification tests which require
filtering Broadcasting devices during General Discovery Procedure.

Change CreatePairedDevice to support LE devices

CreatePairedDevice implements now the same behaviour of CreateDevice,
triggering Discover All Primary Services when needed. SMP negotiation
starts when the link is established. LE capable kernel is required to
test this method properly.

Limitation: For dual mode devices, Discover All Primary Services is not
being executed after SDP search if GATT record is found.

Fix spurious DeviceDisappeared signals after name resolution

update_oor_devices() was being called twice, first after a inquiry, then
after name resolution. This caused the just found devices to be always
reported as "disappeared" after name resolution.

Change power_on & power_off adapter_ops callbacks to set_powered

The current idea is to have a single set_powered management command
which for the switching on case would leave the controller in a
non-connectable, non-pairable and non-discoverable state.

Add a "services" command to test-device

This command adds a way to retrieve the Services property that each
device has.

Add GetProperties method the Service Interface

For now this interface just includes the path that each characteristic
is registered at.

Add support for adding services to the Services property

We need to fill the devices property as each service path gets
registered in the DBus system bus.