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.

storage: Add storage to GAP characteristic

Add functions to read and write remote Appearance characteristic. The values
are saved in "appearance" file.

dbus-common: Map GAP Appearance characteristic to icon

Add function appearance_to_icon() to map values from Appearance
Characteristic to icon available.

Note: The Appearance Characteristic is composed of a category (10-bits)
and sub-categories (6-bits). These categories are defined at:
http://developer.bluetooth.org/gatt/characteristics/Pages/
CharacteristicViewer.aspx?u=org.bluetooth.characteristic.gap.appearance.xml

attrib: Attempt to remove attio callback on watcher exit

When registering a watcher, it is attempted to add an attio callback.
Therefore, when the watcher is unregistered or exits, this attio should
be removed if there are no other users.

This is already done when unregistering a watcher, it just missed when
it exits without unregistering.

attrib: Fix missing D-Bus message unref on DiscoverCharacteristics

This patch fixes a missing message unref when DiscoverCharacteristics on
Generic Attribute finishes.

attrib: Register only new characteristics found

This avoids duplicating registration for previously found
characteristics.

attrib: Move Characteristic Discovery reply to a new function

Minor cleanup patch moving the creation of the reply for Discover
All Characteristics call to a new local function in the Generic
Attribute.

attrib: Discover Characteristics if already connected

On Generic Attribute, Discover All Characteristics of a Service can be
started immediately if already connected, without registering a ATTIO
connection callback.

attrib: Centralize ATTIO connection management

On Generic Attribute, register only one ATTIO connection callback to
centralize the connection management.

Also make sure the connection reference is properly dropped only if
there is no pending operation.

Fix common spelling error in code and comments

Correct spelling is "incoming" not "incomming".

core: Fix device version in GetProperties

Fix how device version is added to the property dictionary.

Move common code to sdp to avoid duplication

LanguageBaseAttributeIDList is set from few places and each time static
function was copied and used for that. Move this funtion to sdp code
and use that instead of copying code.

Bump Device ID service version to 1.3

Device ID information in EIR are specified in DID 1.3 and this is
supported by BlueZ.

btmgmt: Add support for setting Device ID

mgmtops: Add support for setting Device ID

Add support for setting VID source in DeviceID from config file

This allows to set if VID source is Bluetooth SIG or USB. Assigner is
provided as string {bluetooth,usb} and fallback to usb if none is set
in config.

Set DeviceID when registering adapter not when sdp server is starting

Adapter ops are registered after sdp server is started. Thus no
adapters were registered yet when setting DID on sdp startup and DID
was never set.

Parse config DeviceID string on bluetoothd startup

Instead of parsing config DeviceID string on sdp server startup, parse
it in main while reading config file. This allow to store logical DID
values in main_opts (instead of raw config string) and make use of them
in bluetoothd code.

input: fix coding style

Fix line going over 80 columns.

gatttool: Update interactive prompt if connection is lost

In interactive mode, when connection is lost, the prompt used to remain
in "connected" state. This patch fixes that case, by always showing the
actual connection state.

event: Store Class of Device after connection

The Class of Device is usually sent on the EIR returned by the Device
Connected mgmt event. For hciops, the Class of Device is always NULL on
the connection complete event.

The stored Class of Device is used later for Class/Icon device
properties.

HDP: Omit MainChannel when first reliable channel is nil

Do not add MainChannel to property list when HDP first reliable channel
does not exist. Upon channel destruction, ChannelDeleted signal is already
emitted.

This also fixes a crash where MainChannel was filled with invalid path ""
when first reliable channel did not exist.

avdtp: fix audio source init check

sdp: Fix sdp_get_profile_descs for Mac Os X Lion

Mac OS X 10.7.3 publishes the following SDP record for HFP Gateway.
      record #2
          aid 0x0000 (SrvRecHndl)
             uint 0x10003
          aid 0x0001 (SrvClassIDList)
             < uuid-16 0x111f (Handsfree AG) uuid-16 0x1203 (Audio) >
          aid 0x0004 (ProtocolDescList)
             < < uuid-16 0x0100 (L2CAP) > <
             uuid-16 0x0003 (RFCOMM) uint 0x2 > >
          aid 0x0005 (BrwGrpList)
             < uuid-16 0x1002 (PubBrwsGrp) >
          aid 0x0006 (LangBaseAttrIDList)
             < uint 0x656e uint 0x6a uint 0x100 >
          aid 0x0009 (BTProfileDescList)
             < uuid-16 0x111e (Handsfree) uint 0x105 >
          aid 0x0100 (SrvName)
             str "Hands Free Audio Gateway"
          aid 0x0301 (SuppDataStoresList)
             uint 0x0
          aid 0x0311 (SuppFeatures)
             uint 0x0

BlueZ do not read correctly the version number of the profile.
The patch fixes this.

From my understanding, the SDP profile descriptor is a list. The for
loop in sdp_get_profile_desc iterates that list.  Until now, bluez
expected a collection of either uuid or lists with a uuid and a
version number. This can be represented as follow:
list=(uuid, uuid, (uuid, version))

The patch fixes the situation where the profile descriptor list
contains a uuid AND a version number both on the root level and not
enclosed in a sublist. If the next item after uuid is uint16, then
this is the version number. This can be represented as follow:
list=(uuid, uuid, version, uuid, (uuid, version))

Note that core v4.0 spec. states that the Profile Descriptor
list is a data sequence of profile descriptors. Each profile descriptor
is a data sequence whose first element is a UUID and second element is
a version number. This can be represented as follow:
list=((uuid, version), (uuid, version))

GATT: Replace start/end with att_range

Use att_range struct instead of individual start/end handles in
gatt_primary structure

GATT: Rename service and char structs

Rename the att_primary to gatt_primary and att_char to gatt_char.
Characteristic and Service do not exist in the ATT spec, only in
GATT.

GATT: Profile support for EnableGatt

Add support for the EnableGatt for all GATT profiles.

GATT: Rename AttributeServer switch

Rename the AttributeServer main.conf option to EnableGatt and
change its purpose to enable/disable all GATT related activity.

GATT: Remove individual config switches

Remove individual GATT plugin configuration switches and add a
new master gatt switch called --enable-gatt to enable/disable all
GATT related plugins at once.

Fix not marking a reverse SDP discovery as "reverse"

Some devices "hide" some of their records while connected to other
devices. Setting a service discovery as reverse makes BlueZ to not
remove the records that are "hidden".

This patch fixes a mistake that reverse parameter was changed to false
when it should be true.

Remove leftover code of main_opts operation

wiimote: Detect Wii Remotes by device name

We currently detect Wii Remotes by DID information. However, not all Wii
Remotes (especially 3rd party Wii Remotes) provide DID information.
Moreover, they provide no way of identifying them aside from their
device name. Hence, this adds an experimental name-detection for Wii
Remotes.

input: Remove unneeded alias

input: Remove unneeded Bluetooth Address arguments

attrib-server: Fix sending "not supported" for indications/notifications

Now that our Attribute Client is able to handle notifications and
indications we don't need to respond that they are not supported.

HFP gateway: fix failure on very first GSM connection

This patch fix the very first incoming connection from a GSM device
(playing the gateway role), when 'device->gateway' is NULL (when we
didn't perform a SDP browse request yet)

we add the service with 'btd_device_add_uuid(device->btd_dev,
remote_uuid)' but we provide HFP_HS_UUID as remote_uuid. Consequently,
the HFP headset service is activated instead the gateway service.

Revert "core: Add provision for terminating an ATT connection"

This reverts commit f89a77478af78d41c80ab7605662382b9e4e1c36.

This is not needed and actually introduces a bug. When the "Disconnect"
API of device is called device->attrib is unref-ed via a watch set on
G_IO_HUP. The channel is shutdown when the last reference is removed.

The code introduced here shuts down the channel and prevents the watch
from getting called. This means we leak a reference to device->attrib.
This can cause a number of bad things. For example, if the device is
temporary, it will never be freed, and we won't be able to pair to it
again.

device: don't auto-connect on disc-cb attio callback registration

If a device is already connected, don't auto-connect if we register
a disconnect-only attio callback. This will obviously fail.

sdp: Check type of sdp data before dereferencing

audio: fix io channel shutdown on error

In the unlikely case of service record allocation error, the io channel
should be properly shut down.

audio: fix missing io channel shutdown

Channel must be explicitly shut down because otherwise the reference
counter never reaches zero, due to the server installed by bt_io_listen.

audio: fix missing unref in case of error

audio_adapter_get() increases the reference counter of the adapter, so
it's necessary to decrement it in case of error.

avrcp: Add error code for rejected vendor command

Fix response for any vendor commands when there are no players
registered. According to the specification responses should be
REJECTED with error code.

"For error response PDU the response parameter is always the
error code independent of the response format defined for
ACCEPTED PDU response for the corresponding PDU command."
(source: section 4.7.9 of AVRCP 1.3 specification)

avrcp: Fix response ctype of AbortContinuingResponse

Request ctype of "AbortContinuingResponse" is CONTROL, so response
should be ACCEPTED instead of STABLE. This affect PTS Test Case for
TP/RCR/BV-04-C.

test-discovery: Fix printing non-ASCII characters

media: register disconnect watch at transport add

disconnect watch is not removed when the media owner is freed if
an error occurs while resuming in acquire (id == 0).

gateway: Fix crash if SCO connection fails

In some situations, a connect callback is created, but
this callback is not added to media_owner. Thus when the owner
is destroyed and at rfcomm disconnect, the callback is executed
with an invalid pointer.

Add emulator/btvirt to .gitignore

Release 4.99

Update library version

lib: Update company identifiers

lib: Add a2mp.h to lib_headers

lib: Add A2MP definitions

mgmtops: Use DBG not error for printing name in read_info_complete

This is debug not an error message.

mgmtops: Add debug print to update_settings

Print settings for easier tracking what has changed.

mgmtops: When settings discoverable also set connectable if not set yet

Otherwise command would be rejected e.g. when setting discoverable at
bluetoothd startup.

Remove not needed total variable in get_ltk_info

This variable is not really needed and was mostly dead assigned.

input: Set up uinput device again, if it was previously closed.

If you connect a PS3 controller to bluetoothd as an input device, then
take the batteries out of a PS3 controller, then put them back in and
push a few buttons, it will eventually cause an error which causes the
uinput socket to be closed. It will then re-connect to bluetoothd, but
the uinput socket fd will be -1, so it needs to be set up again after
the re-connect.

lib: Fix compile issue when using in C++

The compiler error is:
 /usr/include/bluetooth/bluetooth.h::131:9: error: invalid conversion from 'void*' to 'bt_get_le64(void*)::<anonymous struct>*'
 ...

The reason is that C++, in contrast to C, does not allow conversion of
void * to anything, and this code gets compiled as C++ when the app is
written in C++. The macro with the assignment itself is older, but only
recent Bluez starts to use it in inline functions, thus triggering the
problem.

This patch keeps the "struct __attribute__((packed))" magic and merely
changes the typecast so that it works in C and C++. Like the existing
macro this patch relies on support for typeof.

core: Fix calling stop_discovery through btd_adapter_stop

core: remove set_limited_discoverable from adapter_ops driver

Core should not longer need to take care of details of discoverable bits

core: Make adapter_ops->set_discoverable to take discoverable timeout

This enables the driver to implements its own handling of the timeout