Replace SendDTMF by StartDTMF/StopDTMF in maemo6

SendDTMF method call is replaced by StartDTMF/StopDTMF in maemo6 telephony
driver. StartDTMF/StopDTMF places DTMF tone in queue on modem side playing
it minimum required time. When using SendDTMF, modem ignores new tone
until currently playing one is not finished.

Downside of StartDTMF/StopDTMF combination is absence of audio feedback to
hands-free.

AVRCP: limit AVRCP packet size to the MTU of AV/C

AVRCP is an extension of AV/C spec which has a limit of 512 bytes. The
only place where it can exceed this value is in the response to
GetElementAttributes command.

Now we simply don't add the attributes that would overflow the available
buffer space.

Rename fast_connectable function in hciops

Rename fast_connectable function in hciops to set_fast_connectable
to make it consistent with rest of the file and with mgmt interface
counterpart.

Implement fast connectable mode for mgmt interface

Management interface implementation for fast connectable mode.

Fix test-proximity usage info

Minor style fix

Remove unneeded checks in gateway_is_connected

Check all states before allowing gw connection

Fix possible crash while removing a device

If adapter has more than one sdp session active find_by_bdaddr always
return the first one regardless if the device address match or not
which leads to the following crash:

Invalid read of size 4
   at 0x16CB31: search_cb (in /usr/sbin/bluetoothd)
   by 0x15C0CA: connect_watch (in /usr/sbin/bluetoothd)
   by 0x48CFD37: g_io_unix_dispatch (in /usr/lib/libglib-2.0.so.0.2800.6)
   by 0x488B4A0: g_main_context_dispatch (in /usr/lib/libglib-2.0.so.0.2800.6)
   by 0x488BCD0: g_main_context_iterate.clone.5 (in /usr/lib/libglib-2.0.so.0.2800.6)
   by 0x488C3B8: g_main_loop_run (in /usr/lib/libglib-2.0.so.0.2800.6)
   by 0x152D09: main (in /usr/sbin/bluetoothd)
  Address 0x4c8c600 is 16 bytes inside a block of size 48 free'd
   at 0x48234A9: free (in /usr/lib/valgrind/vgpreload_memcheck-x86-linux.so)
   by 0x48920F0: g_free (in /usr/lib/libglib-2.0.so.0.2800.6)
   by 0x16BDD3: browse_request_free (in /usr/sbin/bluetoothd)
   by 0x16C7A2: browse_request_cancel (in /usr/sbin/bluetoothd)

Reported by Alberto José Rodríguez Rodríguez <ajrodriguez@blulabs.es>

Add device blocking events to mgmt API documentation

Fix Device UUIDs property list being sent empty

When creating a new device, the UUIDs list is updated in the device
probing function. This patch moves the function call which emit the
signal after the device driver probing.

Fix the attrib server trying to handle responses

The attribute server should only try to handle ATT requests, responses
and events should be handled by the client.

Attrib server integration for outgoing connections

We need support for attaching a client connection to the attribute
server because the server that we are connected may want to do
something similar to a reverse service discovery, and we need to be
able to receive indications and notifications.

hciattach: fix serial speed setting for wl1271

override speed setting if firmware script issues serial settings command,
otherwise the value given in the firmware script will be overridden.

Example:

hciattach /dev/ttyO1 texas 115200

will fail, because /dev/ttyO1 will be opened with 115200 b/s, then
the firmware script will set the buadrate to 3000000 b/s, after UART init
hciattach.c will set the baudrate to 115200, so communication is broken.

The only correct way is to set both speeds:

hciattach -s 115200 /dev/ttyO1 texas 3000000

With this patch only initial speed must be specified. The former semantic
will be preserved in case the firmware script doesn't provide serial
settings action.

Tested with wl1271 and firmware TIInit_7.2.31.bts

Fix reference count issue in incoming connections

In case of incoming RFCOMM connection, manager calls
gateway_connect_rfcomm which already take reference to GIOChannel, hence
no need to take another one.

Set state to "connecting" on connection requested

Make sure that state changes to disconnected on errors (authorization
refused, disconnect, sdp) This change will become necessary when
integrating the Audio interface which rely on state change to confirm
that connection has started successfully.

Fix state to "playing" on SCO establishment

Fix asynchronously run request stream cb

Cancel pending callback if stream is canceled Asynchronously run
gateway_config_stream cb Remove occurences of sco_start_cb

Fix disconnect SCO at same time than RFCOMM

If RFCOMM disconnects, SCO should be disconnected too.

Fix freeing watches list in maemo6 telephony

This fixes regression introduced in 83064481481d23523e5cdd04.
List watches has to be freed instead of list pending.

Add Phone Alert Server skeleton plugin

Add GATT Time Server skeleton plugin

Fix characteristic property on GATT example plugin

Any GATT characteristics which have a Client Characteristic
Configuration must have "Notify" property set.

Refactor value assignments of bt_uuid_t variables

Prior to this commit, the assignments were made with memcpy(). This can
be unsafe and less readable, therefore it was replaced with code like:

<dst> = *src;

This also allows more compiler safety checks.

Fix memory allocation of struct attribute

On commit 6a6da5de107e192f62ce2ecdde96eae985181df0 the struct
attribute "data[0]" member was replaced with a dynamically allocated
"data" pointer. This commit fixes remaining places where the old
allocation scheme was still assumed.

Remove empty line before declaration

Fix remove emitting Paired=false when deleting device

When adapter is requested to remove device, device's property
Paired=false is emitted after the adapter notifies clients that
list Devices is empty. Some clients get confused.

Emitting property was introduced in clean-up patch
c92a4af85f480c16f3ff017783c821304c899ce5. Since such notification
does not have practical meaning, it is better to remove it.

Use medium security level when reconnecting

For GATT based profiles encryption is mandatory. Proximity Monitor
shall support LE Security Mode 1 and Security Level 2. Over basic rate,
switch the security level to medium for testing purpose only.

Fix invalid free when stopping adapter

This patch fix an "invalid free" error when the adapter is powered off
with an active discovery session. Error happens because session_remove
function removes the elements from the list also. Partial valgrind log:
Address 0x6012a00 is 0 bytes inside a block of size 16 free'd
  at 0x4C27DCC: free (vg_replace_malloc.c:366)
  by 0x4E927AC: g_slist_remove (in
  by 0x19F788: session_remove (adapter.c:689)
  by 0x19F82A: session_free (adapter.c:708)
  by 0x4E92CD6: g_slist_foreach (in
  by 0x4E92CFA: g_slist_free_full (in
  by 0x1A3ADD: btd_adapter_stop (adapter.c:2491)

headset: Fix missing check for non-existent pending message

Remove redundant empty line

AVDTP: Fix state if authorization fails

Fix agent fall-back mechanism

We can't change the agent for a request internally within agent.c since
externally code may depend on a specific agent being used so that
agent_cancel work properly. This patch exports the fall-back behavior
from agent.c into device.c.

AVDTP: Refactor audio_device_cancel_authorization usage

build: Use private copy of Bluetooth library for audio support

Fix crash when update service record with an invalid XML

If we pass an invalid xml to sdp_xml_parse_record(), then it returns
NULL. Further we are passing the this NULL pointer to the
sdp_record_free(), which leads to invalid memory access.

Add valid range descriptor uuid

Fix magic numbers for local/remote name length

Remove unneeded headers included in storage.c

Remove not referenced function in storage.c

Remove create_file calls for read operations

This patch fixes some read only functions in storage.c removing
create_file function calls.

Remove storing device type

Device type doesn't need to be stored since GATT services information
exported through basic rate can be retrieved from the SDP records.
Device "type" is also a wrong expression to represent the operation
mode: over which transport GATT service is being exported.

Fix allocation of attribute values

Now that pointers to attribute are passed as reference to functions
we cannot have they change during run time, what g_try_realloc()
could do.

avrcp: get/set three-byte company-id

avrcp: fix handling of metadata item 0x7

Metadata field number 0x7 should be the total playing time of the track
(TrackDuration) in msec, not current position within track.

avrcp: use LAST element on media_info_id enum

avrcp: fix overwrite of number of attributes

The response of GetCurrentPlayerApplicationSettingValue expects the
first operand to be the number of attributes in response. Since we start
with len=0, we were overwriting this number with the value of the first
attribute.

Also use g_memdup instead of g_malloc + memcpy.

avrcp: fix missing error code

AVRCP: change debug to better reflect packets protocol/profile

serial: add Serial.ConnectFD()

It's similar to Serial.Connect() but returns the actual RFCOMM file
descriptor instead of creating a device in /dev

serial: fix DBus message reply

If reply is NULL we have a crash.

build: Link all tools with local copy of Bluetooth library

AVRCP: move handling of vendor dependent PDU from control.c to avrcp.c

This helps to isolate AVRCP PDU handling and MediaPlayer interface.

AVRCP: split AVCTP specific code from control.c

The code is moved to avctp.c to simplify control.c

AVRCP: rename avrcp_spec_avc_pdu to avrcp_header

Since old avrcp_header got renamed to avc_header now we can use
avrcp_header for AVRCP PDUs.

AVRCP: rename avrcp_header to avc_header

AVCTP carries AV/C packets/PDUs not AVRCP as avrcp_header suggests.

AVRCP: use a vtable to simplify PDU parsing/handling

This simplify a bit the handling by introducing common checks before
calling the handler callback, it is also much easier to add/remove
new PDUs in this way.

Add test for serial proxy and serial proxy manager

Return EALREADY if the adapter is already up/down

Serialize powering up/down in maemo6 plugin

This patch fixes wrong adapter state when using Management interface.
When bluetoothd starts, the adapter should be switched to the state
informed by Maemo6 MCE. In the current implementation, Maemo6 plugin
sends set powered command when the adapter is still powering up, the
last command sent fails(kernel returns EALREADY).

Fix adapter->up declaration

Minor code convention fix. This patch fix "up" variable declaration in
adapter structure. Convert "up" to gboolean making it compliant with
its usage.

Fix minor coding-style issue

Prevents sending Write Local Name when not needed

Fix unneeded Write Local Name command being sent when bluetoothd starts
or switching the adapter on. Using an array of chars it is not possible
to distinguish if there is a valid value to be set since empty is a
valid value.

Fix loop when setting adapter name

When management interface is enabled, name changed event comes when the
adapter is initialized as consequence of the Read Local Name. Use the
same function to set and handle event causes looping when bluetoothd
starts if the name stored in the controller is different from the name
provided by the adapter name plugin.

Splitting the adapter_update_local_name also fix the PropertyChanged
(for Name) signal being sent before AdapterAdded.

Remove not referenced function

Remove leftover function declaration

Remove unneeded header include

Remove forward declaration

Fix ATT disconnect callback not being called

GAttrib disconnect function needs to be set after discovering the
primary services if the connection callback list is not empty. After
creating a device the connection can stay up, for this scenario the
registered disconnection callbacks are not being called.

Write Link Loss Alert level if connected

When SetProperty gets called for LinkLossAlertLevel, the alert level
can be written in the remote is the link is up.

sbc: Use __asm__ keyword

There are two reasons for this change:

First: consistency. __asm__ was already used elsewhere in the files, so
using that throughout is cleaner.

Second: both asm and __asm__ are GCC-specific extensions, not defined in
the C standard. When compiling with --std=gnu99 both are recognized, but
when using --std=c99 only __asm__ is recognized to make it perfectly
clear that you're not using some standard C99 construct, but a
GCC-extension.

Add support for blacklisting events

Add support for mgmt interface events for device blocking/unblocking.
Kernel sends a mgmt event when a device has been blocked by another
management socket or with ioctl command (e.g. by hciconfig).

Parameter update_only is added to device_block/unblock functions in
device.c to avoid code copying. When update_only is true, blocking
command is not sent to kernel, but only device status is updated and
dbus signal is sent.

Add definition for L2CAP Ext Window Size

This is needed e.g. by hcidump.

Fix fd usage when not connected

When the avctp channel is not connected, we call
g_io_channel_unix_get_fd() with a NULL pointer. Glib does not check the
pointer before dereferencing it, causing bluetoothd to segv.

Move the function call to the place it's actually needed, after the
safety checks.

Print an error message if g_dbus_setup_bus fails.

That is standard processing when dbus_error_is_set() returns true, according to
dbus/dbus/dbus-errors.c.
In this case of g_dbus_setup_bus(), it provides extra help for determing the
specific cause of the error.

Add Serial Proxy and Serial Proxy Manager doc

Off-by-one error in uses_rfcomm

Variable len could be assigned to PATH_MAX + 1 length (returned
 from readlink) and this variable is used as index to array link
of size only PATH_MAX + 1.

formfactor: Add missing comma

Missing comma in static array chassis_map definition between
"Blade" and "Blade Enclosure" field.

Typing error fixed.

Off-by-one(two) error in form factor detection

Indexing of chassis_map array has to be done like that:
    chassis_map[chassis_type * 2 - 1]

because if not, everything is shifted by one. When (e.g.) chassis_type
is 0x04 result should be "Low Profile Desktop" =>  "desktop" (not a
"Pizza Box" => "server"). Lets see the 2.6.1 document on:

http://www.dmtf.org/standards/smbios

build: Link bluetoothd with private copy of libbluetooth

audio: Update license for shared header files

The header files with constants and structures for audio specific
interaction with Pulseaudio are suppose to be under LGPL license.

For some odd reason a2dp-codecs.h ended up being under GPL license
which is against the intention of this being shared and re-used by
non-GPL programs. Fix this now to avoid any future confusion.

Implement media_transport for HFP_HS_UUID

The transport is used by pulseaudio to suspend and resume streams.

Introduce gateway locking mechanism

Set state to "connecting" when AG connects.

This state is used by media.c to trigger pulseaudio configuration.

Fix RFCOMM disconnect on suspend request

When pulseaudio release an audio stream, gateway.c disconnects
RFCOMM instead of disconnecting only SCO.

Update g_strcmp0 to strcasecmp

Use state change callback from media interface

When a HFPHS endpoint is created, media will start to watch for
HandsfreeGateway state changes. media call SetConfiguration upon
connection and ClearConfiguration upon disconnection.

Add state change callback to HandsfreeGateway

Media interface needs to monitor state changes of HandsfreeGateway.

Fix double free in error case in endpoint_reply

If SetConfiguration call fails, the headset or gateway for which
SetConfiguration is called is disconnected. This will free any
pending request, but this does not prevent the request to
terminate (endpoint_reply) and try to free itself once again.
Note that a copy of the freed pointer is tested which has not
been updated.

Disable Path Loss service in configuration file

Path Loss service requires kernel and userspace patches to work
properly. Keep it disabled while the implementation is not ready.

Change default Link Loss alert level to high

After creating a Proximity device, Link Loss alert level is always
written. Setting the default value to "high" enables alert without
requiring an application to change it later.

Add Trusted to DeviceFound signal

This makes it consistent with the documentation which says that this
signal is has the same parameters as GetProperties reply.

Fix remove temporary link key for No Bonding

This fixes regression caused by 33cdfcb0fabcfb5260953a13ae277b5abe9a322d.
Temporary link key should be removed after device is disconnected if
bonding information is not stored in file system.

Currently, the key is deleted only when paired device is removed, or
adapter is powered off. In case of No Bonding authentication, device is
not paired after disconnection, which results in link key left in memory.

On the other hand, device is not necessarily removed after disconnection
that also leaves unneeded temporary link key in memory. Therefore,
btd_adapter_remove_bonding is moved to device_set_paired(device, FALSE)
covering both cases.

build: Fix external plugin installation

Read local extended features at the initialization

Local extended features bitmask may change when Write LE Host Supported
Command is sent by the kernel. This patch adds reading of the local
extended features in the initialization sending a HCI command, extend
the ioctl HCIGETDEVINFO is not suitable due possible inconsistency
between kernel and userspace.

Remove EnableLE option from the user-space

Deprecated configure option after adding "enable_le" parameter in the
bluetooth module. Write LE Host Supported Command is now sent by the
kernel if "enable_le" is enabled and the controller supports LE.

Add support for listing all adapters using test-adapter

Fix get/set_hfp_active to match name conventions

These two functions are defined exclusively for headsets, so
follow established name conventions.

Increase timeout before initiating AVDTP connection

AVDTP_CONNECT_TIMEOUT controls the delay between HSP/HFP
connection establishment and AVDTP signal channel establishment.
The original value of 1 sec. is too short to avoid racing for AVDTP
connection establishment (eg., if the device continues to configure
the HFP service level connection with additional AT cmds).

Some devices have broken AVDTP implementations that just cannot
handle the race conditions that arise if both devices are attempting
stream establishment at the same time. However, these conditions arise
only when the remote device is the ACL initiator (and in practice, the
RFCOMM initiator as well).  This fix bumps out the timeout value only
when the headset has initiated the link.

plugin: Use weak reference for external debug statements