Check the security level on a per request base

Check security level on attribute server on each request, and update the
"encrypted" flag.

Fix HFP response and hold (AT+BTRH) command response

Since no telephony driver supports this command at the moment the
correct behavior is to always return an error. Therefore, a new
"not supported" value is needed for the respective state variable.

Add btd_error_already_connected()

convert error to btd_error_not_supported()

convert error to btd_error_not_connected()

convert error to btd_error_busy()

convert errors to btd_error_does_not_exist()

convert errors to btd_error_not_authorized()

Fix coding styling issues in src/error.c

gdbus: explicitly compare pointers to NULL

This patch was generated by the following semantic patch
(http://coccinelle.lip6.fr/)

// <smpl>
@fix disable is_null,isnt_null1@
expression *E;
@@

- !E
+ E == NULL
// </smpl>

Fix bt_extract_eir_name to use GLib memory allocation

The string returned by bt_extract_eir_name is freed using g_free so
GLib's own allocators should be used instead of libc ones.

Revert use of new error function for Blocked

The error message says a lot about what the user need to do, I'm reverting
this change and will fix in the next patch series about the DBus error
handling.

Remove fixed item from TODO

Read by UUID is already supported by attribute server and gatttool.

Make device type evaluation more readable

Make bt_name2class static on glib-helper

glib-helper: fix coding style issues

Make EIR type an optional argument for bt_extract_eir_name

Check if the remote LE is connectable when creating a device

Before issue Discover All Primary Service the advertising event type
needs to be evaluated to avoid connection attempts to non-connectable
devices. For non-connectable devices, CreateDevice creates the device
instance however no Services/UUIDs will be exposed.

Add device type to identify LE, BR/EDR or dual mode devices

If EIR Flags field is sent in the advertising data, it can be used
to detect the operation mode. If the remote device is dual mode,
GAP operation mode defines that it shall follow the connectable
mode for BR/EDR and non-connectable mode for LE. This patch forces
service discovery protocol prior to Discover All Primary Services.

Remove automatic battery state change on attribute example

Legacy code implemented to test Indication and Notification

Add btd_error_not_authorized()

Add btd_error_does_not_exist()

Add btd_error_busy()

Add btd_error_not_available()

Add btd_error_in_progress()

Add btd_error_not_connected()

Add btd_error_not_supported()

Add btd_error_already_exists()

Create btd_error_invalid_args()

DBus error handling in BlueZ is a mess. This is the first patch to unify
all DBus error handling like in ConnMan and oFono. This unifies all
.InvalidArguments errors.

Fix writing to GAttrib socket without POLLOUT event

If the GIOChannel is in the buffered state (the default) the watch
function is called without receiving a POLLOUT from the socket. GLib
adds a G_IO_OUT condition just because there is space in the GIOChannel
internal buffer.

The solution is disabling the internal buffer, which in turn, makes the
call of g_io_channel_flush() useless.

Check authentication permissions on attribute server

Attributes may require encryption for certain operations. This commit
adds checks to the attribute server which verify whether the current
connection is encrypted (by checking the security level) and the
attribute being accessed requires authentication. If encryption
requirements are not satisfied, the "Insufficient Encryption" error is
returned by the server.

Note that currently there is no actual distinction between
authentication and encryption. Currently, if an attribute requires
authentication, it is only checked whether the link is encrypted. Core
Specification, as of version 4.0, does not specify an explicit
"encryption" permission (see Vol.  3, Part F, 3.2.5 "Attribute
Permissions").

Check attribute permissions in attribute server

The attribute server must verify if the operation (read/write) is
permitted before running the request, and send "Read Not Permitted" or
"Write Not Permitted" error response to client if appropriate.

Initial attribute permission implementation

Add enums for attribute read/write requirements, which may be "none",
"authentication required" or "authorization required". Additionally, a
"not permitted" requirement means that operation is not permitted,
allowing read-only or write-only attributes.

The attrib_db_add() API was changed to allow setting these requirements,
and the example server was changed to set requirements for its
attributes.

Fix deinitializing telephony backend when it wasn't initialized

Telephony driver is now deinitialized on headset adapter driver remove so
telephony_exit should not be called on plugin exit anymore.

Release 4.81

Fix not canceling pending calls on maemo6 telephony driver exit

This happens when the driver exit is called quickly after init due to
adapter power changes.

Use specific members in D-Bus match rules on telephony maemo6 driver

This reduces the amount of unnecessary wake-ups without breaking anything

Fix interface name of modem states on maemo6 telephony driver

Initialize adapter services list

In case service records have been added to bluetoothd before a new
adapter is registered, the records which are shared by all adapters
(indicated by having the address set to BDADDR_ANY) need to be added
to the services list of the new adapter. This patch adds a function
for this on adapter initialization.

The issue could be reproduced by running bluetoothd and obexd on a
PC and briefly removing the BT dongle. The service records from
obexd would not be present in the adapter's local list (which is
used to set the class of device).

Emit Connect signal for LE capable devices

Attrib server should truncate attribute value to pdu length

When the size of attribute value is greater than pdu size, it should be
truncated to the pdu length - 2

Don't trigger the discovering when registering the client

Discover All Primary Services should not be trigged by the attribute
client when a given device is registered. Discover services is now done
by the device entity.

Fix memory leak of sdp list in the attribute client

Return a D-Bus error if device_browse fails

If Discover All Primary Services or SDP search fails, the CreateDevice
caller will not receive a response. Error reproducible when a Discover
All Primary Services is sent over a LE link and the kernel doesn't
address properly the connections in the channel ID 4.

Check the device type when creating the device

For CreateDevice, if the remote device is LE capable, Discover All
Primary Services shall be issued instead of SDP queries. The logic
to define if the remote is LE capable shall be based on the result
of the interleaved discovery results. Meaning, the remote device
is classified as LE capable only if an advertising event from this
device was collected during the discovery.

Limitation: Advertising event data is not being considered yet to
infer the remote properties and CreateDevice needs to be called
during the discovery session.

Implement primary service search when creating a device

Discover primary services implemented inside the device entity to allow
proper integration of attribute plugin. Implements a single entry point
to the attribute plugin no matter the transport(BR/EDR or LE), the device
probe callback is called for both types.

Add a new function to discover all primary services without additional
calls to fetch the remaining primary services, sub-procedure iterations
is handled inside this function.

The next action are: clean the attribute client removing implicity service
and characteristics discovery, issue the Discover Primary Service based on
the remote properties and fetch the characteristic on demand.

Removed unused define

hcid header cleanup

Remove unused scan_mode variable

Use existing bit functions from hci.h

Remove unused includes

Remove unused define

Use address instead of object path in device logs

Improve hciops logging

Improve agent PIN length error log

Use hci_send_cmd instead of hci_send_req for authentication

In addition to blocking the mainloop hci_send_req changes the socket
filters which makes it unusable for the per-adapter HCI socket in
hciops.c. Using hci_send_cmd makes much more sense here.

Fix crash after simultaneous authentication requests

Previously simultaneous authentication requests to the same device caused
bluetoothd crash. Now if ongoing authentication occurs error is returned,
preventing from simultaneous requests to the same device.

sdpd header cleanup

Fix telephony maemo6 driver deinitialization

Remove all match rules and unregister its interface when telephony_exit
is called.

Fix telephony dummy driver

Make it unregister its interface on telephony_exit

Fix not deinitializing telephony driver when there is no adapter powered

This changes hook telephony driver de/initialization to powered state so
telephony drivers can now free their resources when there is no adapter
active in the system.

Extend discover characteristic by UUID in gatttool to fetch all values

If the number of characteristics returned exceeds the MTU size, it will
be needed to ask for more data until the handle range is entirely
searched.

hciattach: download configuration at user requested baud rate

This patch support downloading configuration for Atheros AR300x HCI UART
chip at user requested baud rate instead of the initial baud rate.

hciattach: make set_speed return error if any operation fails

This patch lets set_speed function changing UART baud rate to return an
error code if any one operation fails instead of returning only the last
operation's status.

Add a runtime option to set the BCSP communication rate

Fix typo in adapter documentation

Fix MDL and MCL reference counting in IO watchers

When a io_watcher is added to an MDL or an MCL channel, its reference
should be incremented because the watcher should keep its own reference
the the structure.

Also a destroy function is added in order to decrement the reference
once the watcher is removed.

Do not increment mdl reference in reconnections

Release 4.80

Make LE_LINK define private to hcitool.c for now

The value 0x03 isn't something that exists in the core spec, so it
shouldn't be used in the same API as SCO/ACL link types. Since there are
some experimental kernel patches that use it this patch makes it private
to hcitool.c.

Fix possible race condition when initializing adapters

Due to a possible kernel bug, sometimes the very first HCI commands that
the kernel sends might go unnoticed. Therefore check for this when one
of the last commands (read_local_name) that the kernel sends completes.

Fix not indicating disconnected state when connecting to avdtp

This cause parts of code such as audio/sink.c to keeps its internal state
as connecting and thus prevent any other connection attempt to succeed.

Fix not dropping avdtp session reference when cancelling stream setup

This will indicate to the session that there is no need to proceed with
stream configuration and that it should disconnect.

Fix possible crashes when attempting to connect avdtp

Since the connections can be cancelled while still not complete the
GIOChannel must be stored in order to be properly closed when freeing
avdtp session or stream and thus avoid any callbacks to be called after
that.

Add debug logs for state changes on sink

Fix reference count of a2dp stream setup

Each callback should have a reference to the setup and once the callback
is removed the setup should be unreferenced which wasn't always the case.

Fix abort not being send when the state is idle

The spec clearly suggested that abort can be send while on idle state:

"AVDTP_ABORT_CMD can be sent or received in IDLE state. In the event that
an AVDTP_ABORT_CMD is received in IDLE state, ACP or INT shall reply with
an AVDTP_ABORT_RSP, no state change is required."

In fact abort is the only way to cancel set_configuration and open
commands.

Fix not removing all timers when user request to disconnect audio

This timers can trigger reconnections which is not a desirable behavior

Cleanup a2dp_cancel code

If the id doesn't match any data we should not try to free anything since
obviously there is nothing to free.

Cleanup a2dp setup code

Fix minor typo

Avoid reallocations of services UUID array

The array of service UUIDs used by the DeviceFound signal contains the
same information from the GSList of services. Instead of reallocating
this array on each signal, store it on the remote_dev_info structure and
only reallocate it if there are new UUIDs.

Merge service UUIDs from different BR/EDR EIR data

For LE devices, service UUIDs present in advertising data are merged
into a single list. This change makes the same thing for BR/EDR devices.

Extract flags from advertising data

These flags will be used to decide if GATT service discovery will happen
over LE (for single mode devices) or BR/EDR (for dual mode and non-LE
devices).

Implement Characteristic Value Read using UUID in the gatttool

Sub-procedure used to read a Characteristic Value when the client
only knows the characteristic UUID and doesn't know the handle.
More than one handle and attribute value pair can be returned,
it is up to the user define the handles range based on the service
handles range.

Usage example:
$gatttool --char-read --uuid=2a00 -i hcix -b xx:xx:xx:xx:xx:xx

hciattach: send hci commands after hci socket bring up

This patch makes The Atheros AR300x specific initialization code
to send power management command over HCI socket after bringing up
the device.

Implement Discover Primary Service by Service UUID in the gatttool

Implement only the first interaction of the discovery procedure. If the
response doesn't fit in the MTU, "start" and "end" options can be used
to discover the handles ranges of the remaining primary service instances.
UUID16 and UUID128 are supported in the uuid option.

Usage example:
$gatttool -i hcix -b xx:xx:xx:xx:xx:xx --uuid=1801 --primary

Add an extra parameter in the discovery primary to specify the UUID

Extends discover primary function to perform discover by UUID. UUID
parameter defines which procedure will be executed: Discover All
Primary Services or Discover Primary Service by Service UUID.

Extend bt_string2uuid to convert hex strings to UUID16

Convert four or six(0x) digits length hexadecimal strings to UUID16.

Implement Find by Type Value Request in the atttribute server

GATT Discover Primary Service by Service UUID sub-procedure is based
on ATT Find By Type Value Request/Response.

Implement an extra verification for broken requests: "Ending Handle"
different than 0xFFFF. The Group End Handle may be greater than the
"Ending Handle" in the Find By Type Value Request. Forces the "Ending
Handle" in the response to 0xFFFF to avoid another request from the
clients. 0xFFFF means that the sub-procedure is complete.

Add Find By Type Value Response encoding/decoding functions

Find by type operation is used by Discover Primary Service by Service
UUID. Find By Type Value Response shall contain one or more group handles.

Add iwmmxt optimization for sbc for pxa series cpu

Add iwmmxt optimization for sbc for pxa series cpu.

Benchmarked on ARM PXA platform:
===  Before (4 bands) ====
$ time  ./sbcenc_orig  -s 4     long.au  > /dev/null
real    0m 2.44s
user    0m 2.39s
sys     0m 0.05s
===  After (4 bands) ====
$ time  ./sbcenc  -s 4     long.au  > /dev/null
real    0m 1.59s
user    0m 1.49s
sys     0m 0.10s

===  Before (8 bands) ====
$ time  ./sbcenc_orig   -s 8     long.au  > /dev/null
real    0m 4.05s
user    0m 3.98s
sys     0m 0.07s
===  After (8 bands) ====
$ time  ./sbcenc  -s 8     long.au  > /dev/null
real    0m 1.48s
user    0m 1.41s
sys     0m 0.06s

===  Before (a2dp usage) ====
$ time  ./sbcenc_orig   -b53 -s8 -j    long.au  > /dev/null
real    0m 4.51s
user    0m 4.41s
sys     0m 0.10s
===  After (a2dp usage) ====
$ time  ./sbcenc   -b53 -s8 -j    long.au  > /dev/null
real    0m 2.05s
user    0m 1.99s
sys     0m 0.06s

Fix C++ style comment

Check if the mcl insiede the device is correctly set before use it

Check if MCAP Instance is already released when a callbacks comes back.

Fixes to echo

Remove magic number to check ECHO MDEPs in HDP

Return proper response code if there is an error creating echo channel

Fix dereference to NULL pointers during data channels creation

Code refactorization in MCAP