Convert unit tests from GLib to GNU Check

Add get_le/get_be helpers

Helpers to access LE / BE values. In different Bluetooth
protocols/profiles there is a mixture of LE / BE byte order.

Make whitespace usage consistent in mgmt-api.txt

mgmt: Rename Name Info command to match corresponding flag

mgmt: Allow specifying discovery type for Start Discovery

Add new command and event for LTK storage support

This documents the new command and event that are to be added to
the Management API to add support for persistent pairing of LE
devices.

Add device type to found and connected mgmt events

Add User Passkey commands and events to mgmt

Rename mgmt key commands and events

Add timeout to Set Discoverable mgmt command

Update controller info command with current plan

Fix whitespace for fast connectable mgmt command doc

Add (legacy) name resolution handling to mgmt API

Unregister players' events when disconnected

Fix leak of GTimer

Add missing TRACK-REACHED-START event

When track is changed, we automatically start to play it, therefore we
need to notify CT regarding this event.

Fix handling of dbus signals

We're previously leaking a D-Bus message and always returning that
the signal was not handled.

Add EIR length parameter to eir_parse()

LE uses shorter EIR (31 bytes) than HCI (240 bytes)

Add btohll / htobll macros

Handle 64 bit swap

Add definitions for L2CAP fixed channels

Simplify eir_parse_uuid* functions

Add unit/test-eir to .gitignore

Simplify eir_parse function

Fix MediaPlayer documentation

Fix leak of dbus message

Fix release all sessions on powering adapter off

When adapter is powered off, all sessions should be released without
changing mode, which is contrary to what session_remove does. This
prevents from such situation when powering off leads to new mode
derived from mode_sessions.

Currently, if there are multiple mode sessions requested by different
clients, powering adapter off results in powering immediately it on.

sbc: Reduce for-loop induced indentation in sbc_unpack_frame

sbc: overflow bugfix and audio decoding quality improvement

The "(((audio_sample << 1) | 1) << frame->scale_factor[ch][sb])"
part of expression
    "frame->sb_sample[blk][ch][sb] =
        (((audio_sample << 1) | 1) << frame->scale_factor[ch][sb]) /
        levels[ch][sb] - (1 << frame->scale_factor[ch][sb])"
in "sbc_unpack_frame" function can sometimes overflow 32-bit signed int.
This problem can be reproduced by first using bitpool 128 and encoding
some random noise data, and then feeding it to sbc decoder. The obvious
thing to do would be to change "audio_sample" variable type to uint32_t.

However the problem is a little bit more complicated. According
to the section "12.6.2 Scale Factors" of A2DP spec:
    scalefactor[ch][sb] = pow(2.0, (scale_factor[ch][sb] + 1))

And according to "12.6.4 Reconstruction of the Subband Samples":
    sb_sample[blk][ch][sb] = scalefactor[ch][sb] *
        ((audio_sample[blk][ch][sb]*2.0+1.0) / levels[ch][sb]-1.0);

Hence the current code for calculating "sb_sample[blk][ch][sb]" is
not quite correct, because it loses one least significant bit of
sample data and passes twice smaller sample values to the synthesis
filter (the filter also deviates from the spec to compensate this).
This all has quite a noticeable impact on audio quality. Moreover,
it makes sense to keep a few extra bits of precision here in order
to minimize rounding errors. So the proposed patch introduces a new
SBCDEC_FIXED_EXTRA_BITS constant and uses uint64_t data type
for intermediate calculations in order to safeguard against
overflows. This patch intentionally addresses only the quality
issue, but performance can be also improved later (like replacing
division with multiplication by reciprocal).

Test for the difference of sbc encoding/decoding roundtrip vs.
the original audio file for joint stereo, bitpool 128, 8 subbands
and http://media.xiph.org/sintel/sintel-master-st.flac sample
demonstrates some quality improvement:

=== before ===
    --- comparing original / sbc_encoder.exe + sbcdec ---
    stddev:    4.64 PSNR: 82.97 bytes:170495708/170496000
=== after ===
    --- comparing original / sbc_encoder.exe + sbcdec ---
    stddev:    1.95 PSNR: 90.50 bytes:170495708/170496000

Use color output for unit tests

Add basic unit tests framework for EIR parsing

Fix implicit declaration of 'g_slist_free_full'

audio/avrcp.c: In function 'avrcp_unregister':
audio/avrcp.c:1253: error: implicit declaration of function 'g_slist_free_full'

thermometer/thermometer.c: In function 'destroy_char':
thermometer/thermometer.c:79: error: implicit declaration of function 'g_slist_free_full'

Move SDP client functions from glib-helper.c to sdp-client.c

Add glib-compat.h to deal with issues in old GLib versions

AVRCP: Implement RequestContinuingResponse PDU

AVRCP: Implement AbortContinuingResponse PDU

AVRCP: respond with UINT32_MAX if duration is not available

Section 5.4.1 of AVRCP 1.3 spec says:

If TG does not support SongLength And SongPosition on TG, then TG shall
return 0xFFFFFFFF.

SongPosition is always available, but song length depends on user to
provied it.

AVRCP: implement TRACK-REACHED-START event

AVRCP: Use track's UID in event notification

When registering for TRACK_CHANGED event, CT expects the track's UID in
the INTERIM and CHANGED responses. According to AVRCP 1.3 spec, there
are only 2 possible values:
1) 0: there's a track selected
2) UINT32_MAX: there's no track selected

AVRCP 1.4 reserves the value UINT64_MAX for the second case. Since this
later value is the one used in certification process for best IOP
we return UINT64_MAX instead of the former.

This implementation allows to pass PTS test TP/NFY/BV-05-C.

hciops: fix discov_state when powering off an adapter

Simplify property variable usage in btd_adapter_stop

Fix updating Adapter.Discovering property when powering off

Fix input pending connect handling

iconn->pending_connect needs to be freed after sending the error reply

hciops: fix adapter initialization when commands are missed

Sometimes after e.g, suspend we miss some initialization commands from
the kernel, even though we had a workaround for this it doesn't really
work as expected since the adapter must be up in order to resend.

To fix this problem the resend logic is moved after DEVUP.

Simplify process_thermometer_char() logic

Fix thermometer Intermediate property changes

Set Intermediate property if intermediate temperature characteristic is
supported

Read measurement interval characteristic

Read temperature type characteristic

Get all characteristics in thermometer service

Add handler function to manage GATT indications

Add functions to manage attio callbacks

Unregister Health Thermometer Interface

Register Health Thermometer Interface

Get thermometer service range to load the driver.

hcitool: Fix EIR parsing

There were missing buffer size checks and wrong assumptions about the
EIR data length (core spec 4.0 defines it to be max 31 bytes, so the
name can be max 29 bytes).

Fix leaks and buffer overflows in EIR parsing

By calling g_utf8_validate and allocating eir->name inside the parsing
loop the code was exposing itself to buffer overflows and memory leaks.
This is because the check for incorrect length fields is only done after
exiting the loop (if (len > HCI_MAX_EIR_LENGTH)). By only setting a
pointer to the name and doing the processing after checking the length
validity both issues can be avoided.

Add support for parsing the remote name during LE Scan

Make title always available in metadata

Don't overwrite metadata when changing track

If we use the same hash table to set the new metadata, we have 2
undesired behaviors:

1) New track may contain fields from previous track if it didn't set all
the fields
2) If we fail on parsing the signal, we will still change some of the
fields

Fix +CLCC in maemo6 during CSD_CALL_STATUS_COMING

This fixes the same issue as in 8a6119ea1c685fc61419ee444a5967596f524c1c
but for CSD_CALL_STATUS_COMING. Call status change sequence in csd is
IDLE to COMING, COMING to PROCEEDING, and PROCEEDING to WAITING.

Returned 'state of the call' value after CSD_CALL_STATUS_COMING, as well
as after CSD_CALL_STATUS_PROCEEDING, in +CLCC shall be 5 (waiting, MT
call) instead of 4 (incoming, MT call) in maemo6 telephony driver for the
second incoming call.

Fix crash on power off in maemo6 after sending DTMF

This fixes regression introduced in
b48e610314fb466035b198f594c5b6a68126fcf1. Crash happens when powering
off adapter after sending DTMF tone.

Improve test/simple-player to allow user interaction

Allow user to trigger TrackChanged and PropertyChanged. Calls made by
remote side (coming from D-Bus) are printed to stderr while the ones to
interact with user go to stdout.

Refactor to share code

Refactor code when we are 'listing selected attributes' to share code
with 'listing all attributes'. This way we always keep the list of
attributes in a GList, and call player_get_media_attribute() in only one
place.

AVRCP: Only return available metadata attributes

When remote side requests all available metadata (i.e. it gives number
attributes equals to 0) do not return zero-length strings for
unavailable items. The only exception is title, that must be always present.

AVRCP: Check if len matches number of IDs

If number of attributes remote side provided is larger than the length
we read, we would read garbage from stack memory.

AVRCP: Return error for invalid metadata IDs

Add callback to list available metadata IDs

AVRCP: Do not list values for unsupported attributes

Move debug messages to their correspondent getters

Fix typo on doc

Add mpris-player to .gitignore

Add Szymon Janc to AUTHORS

Add Lucas De Marchi to AUTHORS

Remove leftover code of primary service storage

Primary services related to a given address is already being removed
in delete_device_service() function.

Remove leftover code of device type storage

Update control documentation to reflect the implementation

Many of the methods and signal don't make much sense since we now have
a dedicated interfaces that are Media and MediaPlayer.

Fix call status in +CLCC for maemo6 telephony

If list current calls is requested when there is second incoming call
and csd call status is CSD_CALL_STATUS_PROCEEDING, then returned 'state
of the call' value in +CLCC is 4 (incoming, MT call), which is incorrect.
Indication than proceeds from incoming to waiting state.

This patch sets the corresponding value to 5 (waiting, MT call) in
maemo6 telephony driver for the second call.

Fix compilation error on 32-bit systems

This fixes the following compilation error with GLib on 32-bit sytems:

audio/media.c: In function 'get_setting':
audio/media.c:1109:44: error: cast to pointer from integer of different size [-Werror=int-to-pointer-cast]
audio/media.c: In function 'set_setting':
audio/media.c:1132:41: error: cast to pointer from integer of different size [-Werror=int-to-pointer-cast]

Add mpris-player to test

It gather information from players implementing mpris spec and convert
to org.bluez.MediaPlayer spec.

Add support for cancelling a LE Scan with Control-C

Fix using the attribute struct for encoding ATT pdus

The enconding and decoding of ATT pdus should be kept as much
free of dependences from other parts of the code as possible, so
it can be used in many contexts.

In this case, for encoding and decoding notifications and indications
we did have to pass an instance of an attribute instead of direct
values.

Fix crash while registering player

Crash is caused by not removed freed player from the list:

Invalid read of size 8
   at 0x13E7B5: media_adapter_find_player (media.c:861)
   by 0x13FEBC: register_player (media.c:1561)
   by 0x120DFE: process_message (object.c:224)
   by 0x4F6E9A0: ??? (in /lib64/libdbus-1.so.3.5.6)
   by 0x4F6092F: dbus_connection_dispatch (in /lib64/libdbus-1.so.3.5.6)
   by 0x11F787: message_dispatch (mainloop.c:80)
   by 0x4C762CA: ??? (in /lib64/libglib-2.0.so.0.3000.0)
   by 0x4C74ADC: g_main_context_dispatch (in /lib64/libglib-2.0.so.0.3000.0)
   by 0x4C752D7: ??? (in /lib64/libglib-2.0.so.0.3000.0)
   by 0x4C75824: g_main_loop_run (in /lib64/libglib-2.0.so.0.3000.0)
   by 0x11EE4B: main (main.c:485)
 Address 0x642ef30 is 16 bytes inside a block of size 80 free'd
   at 0x4A0662E: free (vg_replace_malloc.c:366)
   by 0x4C7B7F2: g_free (in /lib64/libglib-2.0.so.0.3000.0)
   by 0x12D292: player_destroy (avrcp.c:1099)
   by 0x120C38: service_filter (watch.c:477)
   by 0x120950: message_filter (watch.c:527)
   by 0x4F608E5: dbus_connection_dispatch (in /lib64/libdbus-1.so.3.5.6)
   by 0x11F787: message_dispatch (mainloop.c:80)
   by 0x4C762CA: ??? (in /lib64/libglib-2.0.so.0.3000.0)
   by 0x4C74ADC: g_main_context_dispatch (in /lib64/libglib-2.0.so.0.3000.0)
   by 0x4C752D7: ??? (in /lib64/libglib-2.0.so.0.3000.0)
   by 0x4C75824: g_main_loop_run (in /lib64/libglib-2.0.so.0.3000.0)
   by 0x11EE4B: main (main.c:485)

Fix not being able to unregister player via UnregisterPlayer

AVRCP: do not allow CT to set a property not supported

If player never set a property, it means it doesn't support it. Doesn't
allow the remote side to set it and send a REJECTED response.

AVRCP: fix loop over number of application settings

Remove test-media-player

Remove MediaPlayer interface from control-api.txt

Add simple-player test script

AVRCP: move MediaPlayer to adapter object

This move the MediaPlayer registration to adapter object on Media
interface so we can track players properly.

Add automatic connect timeout config option

Add new option in main.conf allowing change the default automatic
connection timer. This value defines how many seconds re-connection
for bonded devices will be active when a platform/user event triggers
the automatic re-connections.

Enable re-connection if reason is Timeout

The re-connection timer must be added just when the disconnection reason
was "Connection timeout" for GATT based profiles.

Add an interval between connection attempts

For connection attempts to the same remote device, add 5 seconds
interval until the next connect attempt. At the moment, the behaviour
depends on if address is found in the advertising kernel cache only.

Passive scanning kernel patches are not upstream yet. LE scanning will
be executed in background during a short period of time until it finds
the address or EHOSTDOWN is returned to the connection attempt.

Set auto connect from maemo6 plugin

Inform the adapter that the system has been unlocked when the MCE
sends the tklock_mode_ind signal. This signal will trigger the
automatic re-connections for the devices selected by the profiles.
Profiles register ATTIO callbacks to indicate that ATT connection
is required.

Add display lock watcher on maemo6 plugin

Implements display status signal watcher in maemo6 plugin.

Disable automatic connections after 60 seconds

Fix repeated connection attempt to the same remote

Avoids queue dupplicated requests connects for the same device. Sources
can be triggered by auto connect or on demand connection request when a
given profile registers ATTIO connection callbacks.

Add checking if profiles are requesting connection

Before triggering automatic ATTIO connections driven by platform event,
the device needs to check if there is at least one entry in the ATTIO
connection callback, meaning that there is profile requesting connection
and the state is auto connection.

Add continuous connection attempt

If auto connect is enabled, queue another connection attempt until
the adapter sets auto connect to FALSE. The behaviour depends on if the
address is found in the advertising cache or not.

Add set auto connect in device

Skeleton of automatic connections driven by platform/user event.
Two layers will manage connections: Profiles can request on demand
connections registering ATTIO connection callbacks(one attempt) for
a given device and platform/user action can trigger automatic
connections for devices with ATTIO callbacks registered.

Remove timer to trigger connection attempts

Removes continuous timer for ATTIO connection attempts. A new approach
is being implemented managing scanning and connection attempts driven
by platform/user actions. eg: screen unlocked.

Fix crash when remote connects to AVRCP without having a record

Device object may exist but control wont be initialized causing the
following crash:

Invalid read of size 8
   at 0x12B510: state_changed (control.c:90)
   by 0x12BA20: avctp_set_state (avctp.c:367)
   by 0x12C0DC: avctp_confirm_cb (avctp.c:733)
   by 0x166481: server_cb (btio.c:200)
   by 0x4E75ADC: g_main_context_dispatch (in /lib64/libglib-2.0.so.0.3000.0)
   by 0x4E762D7: ??? (in /lib64/libglib-2.0.so.0.3000.0)
   by 0x4E76824: g_main_loop_run (in /lib64/libglib-2.0.so.0.3000.0)
   by 0x11ED19: main (main.c:473)
 Address 0x8 is not stack'd, malloc'd or (recently) free'd