audio: Fix interface names

In some places, the old names of the interfaces were still being used.
So we update them to use the new names.

media: Fix some wrong interface names

Trivially fix some wrong interface names introduced by commit
3aa15b56dca27a73f6bc34467ba0ce4c85ee2d9a.

TODO: Update to current status

media: Add version suffix to all media interfaces

Append the version number to all D-Bus interface names in the Media API.

media: Trivial style fix

No whitespace should exist before parenthesis close.

media: Add MediaTransport.TryAcquire()

Split the Acquire() method in two parts so that the optional acquires,
formerly represented as a "?" flag in the accesstype parameter of
Acquire(), are now implemented in TryAcquire().

media: Remove transport owner list

Transports now have zero or one owners, so there is no need to maintain
a list any more.

media: Remove internal transport locks

The internal transport_lock_t is not needed any more since transports
are now always acquired with read and write permissions.

media: Remove transport access type from D-Bus API

There is no known use-case making use of these access types and
therefore the Media API can be simplified.

From now on, the transport will always be acquired with read and write
access rights.

l2test: Add support to test auto select PSM

This patch enable us to test the auto select PSM by passing
PSM value as 0.

Ex: l2test -d -P 0
l2test[2585]: Waiting for connection on psm 4099 ...

hciconfig: Remove putkey command

With new storage architecture legacy storage "linkkeys"
file doesn't exist anymore.

Use entire include path for gobex.h

Reorder btio.h includes

Let "#include <btio/btio.h>" together with other includes using '<'
rather than '"'.

Use the entire include path for btio.h

Use the entire include path for gdbus.h

build-sys: Add missing force arg to symlink creation

If we don't force the symlink creation the build fails when the link
already exists. This happens if the rule to create the link is triggered
again because the file it points to changed. This is the case when doing
incremental builds and we modified the file.

$ make
$ touch lib/cmtp.h
$ make
  GEN      lib/bluetooth/cmtp.h
ln: failed to create symbolic link ‘lib/bluetooth/cmtp.h’: File exists
make: *** [lib/bluetooth/cmtp.h] Error 1
make: *** Waiting for unfinished jobs....

test: Fix re-declaration of "encrypt" symbol

test/l2test.c:106:12: error: ‘encrypt’ redeclared as different kind of symbol
In file included from test/l2test.c:30:0:
/usr/include/unistd.h:1147:13: note: previous declaration of ‘encrypt’ was here

Ensure config.h is included by using CPPFLAGS

Instead of trying to include config.h in each file over the tree and
possibly forgetting to include it, give a "-include config.h" argument
to the compiler so it's guaranteed that a) it will be included for all
source files and b) it will be the first header included.

gdbus/ directory is left out, since it would break other projects using
it.

build-sys: Define _GNU_SOURCE in config.h

Instead of defining _GNU_SOURCE in each source file (and potentially
forgetting in some), tell the build system we use extensions and let it
define _GNU_SOURCE in config.h.

dbus: Rename to org.bluez.Adapter1

Trivially add the numbering suffix to org.bluez.Adapter according to
the proposal for BlueZ 5.

dbus: Rename to org.bluez.Device1

Trivially add the numbering suffix to org.bluez.Device according to
the proposal for BlueZ 5.

adapter: Remove FindDevice method from D-Bus API

ObjectManager.GetManagedObjects() returns all devices and their
corresponding properties to any interested client. The device address is
included in the property dictionary and therefore having such a
FindDevice method is an unnecessary duplication.

test: Avoid using Adapter.FindDevice()

The method is now deprecated and thus the replacement utility library
should be used in the test scripts.

adapter: Remove redundant Devices property

The ObjectManager interface already reports the list of devices, so the
the property can be entirely removed.

adapter: Remove DeviceCreated/DeviceRemoved signals

The Adapter interface already reports changes in the device list in form
of property changes, so there is no need to keep these two signals.

test: Add helper function to find devices

Add a helper function to the utility library as an alternative to the
convenience method Adapter.FindDevice() in the D-Bus API.

test: Use ObjectManager instead of Devices property

Use the objects and interfaces reported by the ObjectManager in order
to list the available devices per adapter.

dbus: Remove org.bluez.Manager

After the decision to drop the "default-adapter" feature in the D-Bus
API, the org.bluez.Manager interface can be entirely dropped since it
has been replaced by the ObjectManager.

test: Use ObjectManager instead of Adapters property

Use the objects and interfaces reported by the ObjectManager in order
to list the available adapters.

test: Update test-manager script to ObjectManager

Make the script compatible with the new API provided by the
ObjectManager interface.

test: Update monitor script to ObjectManager

Make the script compatible with the new API provided by the
ObjectManager and Properties interfaces.

test: Avoid using DefaultAdapter()

Call the utility library -which uses ObjectManager API- to avoid using
Manager.FindAdapter() and Manager.DefaultAdapter().

test: Add utility library for python scripts

Several convenience functions/features will be removed from BlueZ's
D-Bus API, and therefore a utility library is required to avoid
boilerplate code in the test scripts.

cups: Remove unnecessary code

The adapter is always known so the code to handle the NULL case is not
needed.

adapter: Always write address type when converting to new storage

When address type is not present in legacy storage assume BDADDR_BREDR
type and store it.

This also fix following compilation error on ARM:

  CC     src/bluetoothd-adapter.o
src/adapter.c: In function ‘convert_entry’:
src/adapter.c:2710:2: error: comparison is always true due to limited
range of data type [-Werror=type-limits]
cc1: all warnings being treated as errors
make[1]: *** [src/bluetoothd-adapter.o] Error 1

audio: Remove not used auto connect flag

This flag is not used anymore because HFP is moved away from bluez
It was used for Automatic HFP+A2DP connection with incomming connection.

audio: Remove left over of HFP removal

build: Include bluetooth lib folder for obexd

Because obexd is included in bluez repo, obexd should refer to local
lib folder before system's one.

audio: Remove left over of HFP removal

cyclingspeed: Fix various coding style issues

test: Enable speed and cadence calculation in test-cyclingspeed

This patch allows to enable instantenous speed and cadence calculation
based on measurement received from remote device.

To enable calculations additional parameter "--circumference <value>"
needs to be used when calling script which sets wheel circumference in mm.

Both speed and cadence are enabled at the same time. Speed is calculated
in km/h and cadence in rpm.

test: Add cyclingspeed test script

build: Add CSCP API document to EXTRA_DIST

doc: Rename cycling API to cyclingspeed

doc: Remove Get-/SetProperties from CSC API document

cyclingspeed plugin uses DBus.Properties instead of custom methods.

core: Add CyclingSpeedWatcher interface to default policy

cyclingspeed: Add support for Set Cumulative Value

cyclingspeed: Add support for Update Sensor Location

cyclingspeed: Add support for Request Supported Sensor Locations

cyclingspeed: Add stub to use SC Control Point

This patch implements common functions to use SC Control Point.
Individual procedures will be implemented in subsequent patches.

cyclingspeed: Add DBus.Properties for org.bluez.CyclingSpeed interface

cyclingspeed: Process measurement notifications

cyclingspeed: Add support to enable measurement notifications

cyclingspeed: Add CyclingSpeedManager interface

This patch registers org.bluez.CyclingSpeedManager interface for each
adapter to allow registration and deregistration of measurement watchers.

cyclingspeed: Read Sensor Location characteristic value

cyclingspeed: Read CSC Feature characteristic value

cyclingspeed: Discover characteristics CCC

cyclingspeed: Discover CSCS characteristics

cyclingspeed: Add attio callbacks

cyclingspeed: Add CSC profile plugin skeleton

This patch adds stub profile driver plugin for CSC profile.

build: Compile obexd and obex-client

obexd: Fix using BtIOType

obexd: Fix using BtIOType

obexd: Add get_next_header() in message listing

In case of message listing we need to send application
parameters in response and also it is required for sending
response to message listing size request.

obexd: Reset ap_sent variable

For multiple subsequent request for message listing
we need to reset this variable to FALSE, so that each
time the get next header function is executed properly.

obexd: Fix sending absolute path

As per OBEX spec the NAME header should not contain absolute paths

obexd: Fix returning empty if messages was already listed

Once a message was already listed and inserted on the cache it could
not be listed again as the code was using the wrong key to lookup for
found messages then once we try to create the message again it fails
as the object already exists.

obexd: Fix freeing apparam data on PBAP module

Invalid read of size 8
   at 0x40EC04: g_obex_apparam_free (gobex-apparam.c:362)
   by 0x41A66A: obc_transfer_free (transfer.c:272)
   by 0x413221: pending_request_free (session.c:163)
   by 0x413659: session_terminate_transfer (session.c:745)
   by 0x41A53E: xfer_complete (transfer.c:518)
   by 0x41B5D7: get_xfer_progress_first (transfer.c:562)
   by 0x409750: handle_response (gobex.c:948)
   by 0x40A609: incoming_data (gobex.c:1191)
   by 0x371D047824: g_main_context_dispatch (in /usr/lib64/libglib-2.0.so.0.3200.4)
   by 0x371D047B57: ??? (in /usr/lib64/libglib-2.0.so.0.3200.4)
   by 0x371D047F51: g_main_loop_run (in /usr/lib64/libglib-2.0.so.0.3200.4)
   by 0x40542F: main (main.c:175)
 Address 0x4f64510 is 0 bytes inside a block of size 8 free'd
   at 0x4A079AE: free (vg_replace_malloc.c:427)
   by 0x371D04D50E: g_free (in /usr/lib64/libglib-2.0.so.0.3200.4)
   by 0x416060: phonebook_size_callback (pbap.c:266)
   by 0x413651: session_terminate_transfer (session.c:743)
   by 0x41A53E: xfer_complete (transfer.c:518)
   by 0x41B5D7: get_xfer_progress_first (transfer.c:562)
   by 0x409750: handle_response (gobex.c:948)
   by 0x40A609: incoming_data (gobex.c:1191)
   by 0x371D047824: g_main_context_dispatch (in /usr/lib64/libglib-2.0.so.0.3200.4)
   by 0x371D047B57: ??? (in /usr/lib64/libglib-2.0.so.0.3200.4)
   by 0x371D047F51: g_main_loop_run (in /usr/lib64/libglib-2.0.so.0.3200.4)
   by 0x40542F: main (main.c:175)

obexd: Fix not using port when it is set by the client

If the client set a channel/port the code would just ignore and attempt
to discover the service anyway.

obexd: Add implementation for UpdateInbox

obexd: Fix build when using --with-messages=tracker

Commit 15b15167fb2dd82e2189cc1955def1818a7bc6a6 broke the build:

plugins/messages.c:324:42: error: unknown type name ‘messages_update_inbox_cb’
plugins/messages.c:331:5: error: unknown type name ‘messages_set_read_cb’
plugins/messages.c:337:6: error: unknown type name ‘messages_set_delete_cb’

obexd: Add Message.SetProperty and Message.GetProperties implementation.

obexd: Update the file offset to the beginning after writing to the file

When the transfer file is opened in O_RDWR mode, just after the contents are
written to the file, the file offset has to be set to the beginning of the
file. If not subsequent read fails. This patch fixes this.

obexd: Make obc_transfer_put to accept NULL as filename

In that case contents should be valid so the modules can use
obc_transfer_put to create temporary files.

obexd: Avoid extra copies while passing apparam to transfer

By passing directly the reference to GObexApparam it is no longer
necessary to use intermediate buffers to pass data around.

obexd: Fix not sending parameters to get message in map module

Attachment and charset are mandatory, so Message.Get now takes an
additional boolean parameter which the user application should set
if it wants the attachments to be downloaded, charset is always set
to UTF8.

obexd: Add MessageAccess.ListFilterFields

obexd: Rename MessageAccess method GetMessagesListing to ListMessages

In addition to that add missing parsing of the filters

obexd: Rename MessageAccess method GetFolderListing to ListFolders

In addition to that add missing parsing of the filters.

obexd: Use larger timeout first packet in transfers

This align the client timeout for GET operations with g_obex_get_req_pkt
which does use 60 seconds to cover for extra interaction during the first
packet.

obexd: Unify callbacks for operations that only report status

This removes duplicated code.

obexd: Add set delete status function

This function will inform MAP server to modify
the delete status of a given message.

obexd: Add set read status function

This function will inform MAP server to modify the
read status of a given message.

obexd: Add support for MessageStatus

Add driver to support x-bt/messageStatus mimetype

obexd: Move common code to pull_phonebook

obexd: Remove deprecated methods from PhonebookAccess

SetFormat, SetOrder, SetFilter and GetFilter methods are no longer
necessary as other methods now take them as parameters to avoid round
trips.

obexd: Add filters to PhonebookAccess.List and PhonebookAccess.Search

This avoid D-Bus round trips and is more aligned with what has been
proposed for MessageAccess interface.

obexd: Add filters to PhonebookAccess.Pull

This avoid D-Bus round trips and is more aligned with what has been
proposed for MessageAccess interface.

obexd: Add filters to Phonebook.PullAll

This avoid D-Bus round trips and is more aligned with what has been
proposed for MessageAccess interface.

obexd: Fix crash on map module

gboolean is expected to hold 0/1. But it is holding int return from
strcasecmp which crashes DBusMessage at:

_dbus_return_val_if_fail (*bool_p == 0 || *bool_p == 1, FALSE);

Trace:
0  0x00007ffff7328d95 in __GI_raise (sig=6) at raise.c:64
1  0x00007ffff732a2ab in __GI_abort () at abort.c:93
2  0x00007ffff78d0655 in _dbus_abort () at dbus-sysdeps.c:94
3  0x00007ffff78c75f1 in _dbus_warn_check_failed at dbus-internals.c:289
4  0x00007ffff78ba28b in dbus_message_iter_append_basic at dbus-message.c:2538
5  0x00000000004201c3 in append_variant at client/dbus.c:44
6  0x000000000042024e in obex_dbus_dict_append at client/dbus.c:65
7  0x000000000041dcc9 in parse_read at client/map.c:423
8  0x000000000041dfa7 in msg_element at client/map.c:518
9  0x00007ffff7b323b9 in emit_start_element at gmarkup.c:986
10 0x00007ffff7b33b44 in g_markup_parse_context_parse at gmarkup.c:1323
11 0x000000000041e1ad in message_listing_cb at client/map.c:586
12 0x000000000041744c in session_terminate_transfer client/session.c:743
13 0x00000000004174d7 in session_notify_complete at client/session.c:758
14 0x000000000041755a in transfer_complete at client/session.c:778
15 0x000000000041f57b in xfer_complete at client/transfer.c:521
16 0x000000000040efdf in transfer_complete at gobex/gobex-transfer.c:102
17 0x000000000040f418 in transfer_response at gobex/gobex-transfer.c:221
18 0x000000000040b320 in handle_response at gobex/gobex.c:948
19 0x000000000040bbc1 in incoming_data at gobex/gobex.c:1191
20 0x00007ffff7b2f94a in g_main_dispatch (context=0x62f130) at gmain.c:2515
21 g_main_context_dispatch (context=0x62f130) at gmain.c:3052
22 0x00007ffff7b2fd10 in g_main_context_iterate at gmain.c:3123
23 g_main_context_iterate at gmain.c:3060
24 0x00007ffff7b3010a in g_main_loop_run (loop=0x62e1b0) at gmain.c:3317
25 0x000000000041527d in main at client/main.c:175

obexd: Remove redundant include errno.h

obexd: Remove redundant include sys/stat.h

obexd: Add subject length parameter in GetMessageListing

Add subject length parameter to allow MAP server to
create subject as per MAP client length preference.

obexd: Add offset parameter decription

Add offset parameter description for
messages_get_messages_listing() function header

obexd: Fix invalid oflag check

Change logic to check for O_WRONLY, as oflag is
created by an OR operation of O_WRONLY, O_CREAT and
O_TRUNC.
Message update is a PUT function, this check returns
bad response if GET is received.

obexd: Remove map_ap.c

It is no longer needed as MAP plugin is now using GObexApparam API

obexd: Make use of GObexApparam API

obexd: Make use of GObexApparam API

obexd: Port PBAP module to use GObexApparam

obexd: Fix pbap_select using absolute path with known locations

pbap_select has to use absolute path with known location to support
repeatable pbap_select calls. In other way the second call fails.