media: Add gateway_get_state() to internal API

Expose the state of the gateway interface in the internal API.

media: Add a2dp_sep_is_playing() to internal API

Add this function to expose whether the local SEP is streaming or not.

media: Fix accesstype comparison

Replace the string representation of the accesstype with a conventional
binary representation. This makes the code simpler and more efficient.

This also fixes a minor bug in the Release() D-Bus method, where the
string comparison was used to see whether the owner should be removed. A
client acquiring with "rw" and releasing with "wr" would lead to the
inconsistent state of having a released transport with an owner with no
accesstype. Partial releases can also get affected by this bug since the
released character (partial accesstype) got replaced by a whitespace.

Additionally, this approach is more robust in case new flags are added
in the future.

input: Remove unnecessary struct input_conn

There's no need to maintain the capability of multiple connections per
device, so just merge the connections list into the main input_device
struct.

input: Remove fakhid functionality

The HSP code conflicts with a real HSP implementation and the PS3
support should be done through the kernel.

profile: Add asynchronous completion functions for connect/disconnect

core: Move profile code into profile.{c,h}

avctp: Fix reply for message with invalid PID

Reply for control message with invalid (not registered for reception
of messages) PID should not have message information field present.

This was affecting AVCTP qualification test case TP/NFR/BI-01-C.

network: Fix minor whitespace issue

core: Remove pattern matching when probing services

This code was only needed for the Serial interface for matching all
RFCOMM-based records, and now that this interface is gone the code is
also no-longer needed.

fakehid: Add support for Logitech Harmony Adapter for PS3

This emulates a Sony BD Remote for the Logitech Harmony series of
universal remotes.

fakehid: Disconnect from PS3 remote after 10 mins

After 10 minutes, disconnect the PS3 BD Remote to avoid draining its
battery. This is consistent with its behaviour on the PS3.

Original patch by Ruslan N. Marchenko <rufferson@gmail.com>

input: Add helper function to request disconnect

fakehid: Use the same constant as declared

ps3remote_keymap[] uses 0xff as the max value, so should we.

test: Add compact mode to test-discovery

core: Add a list of LE devices to connect

This commit creates a per-adapter list of LE devices to connect when a
advertising from them is seen during a scan.

core: Add compare function for bdaddr in a struct btd_device

This is a utility function similar to device_address_cmp but comparing
bdaddr instead of the string representing the address. This way is
possible to avoid allocating two buffers to temporarily hold the
strings, two sprintf() calls to generate the strings from the bdaddr
arrays, and a string comparison, substituting all of it for one memcmp()
call.

mgmt: Print error message when start_discovery fails

If we fail to communicate with the MGMT socket is better to print the
error message on the mgmtops plugin, where it really happened, instead
of leaving this job to its users.

hcitool: Add LE scan based on White List

This patch extends the LE scanning command allowing the user to inform
an option to enable LE scanning detecting advertises sent from devices
in the White List Only. This option can be used combined with others.

gatt: Fix reading Service Changed CCC

Find Information Request is only sent when there are no more attributes
following Service Changed Value (incorrect) and thus will never find CCC.

mgmt: Use bt_get_* helper functions to access unaligned memory

eir: Use bt_get_* helper functions to access unaligned memory

avrcp: Fix compilation errors due to unaligned memory access

This fix following build errors on ARM.

  CC     audio/bluetoothd-avrcp.o
audio/avrcp.c: In function avrcp_handle_get_element_attributes:
audio/avrcp.c:667:25: error: cast increases required alignment of
target type [-Werror=cast-align]
audio/avrcp.c:690:20: error: cast increases required alignment of
target type [-Werror=cast-align]
cc1: all warnings being treated as errors
make[1]: *** [audio/bluetoothd-avrcp.o] Error 1
make: *** [all] Error 2

hciemu: Fix build errors due to unaligned memory access

This fix following build errors on ARM.

  CC     test/hciemu.o
test/hciemu.c: In function num_completed_pkts:
test/hciemu.c:429:4: error: cast increases required alignment of target
 type [-Werror=cast-align]
test/hciemu.c:430:4: error: cast increases required alignment of target
type [-Werror=cast-align]
cc1: all warnings being treated as errors
make[1]: *** [test/hciemu.o] Error 1
make: *** [all] Error 2

sdpd-service: Fix build errors due to unaligned memory access

This fix number of build errors on ARM similar to one below.

  CC     src/bluetooth-sdpd-service.o
src/sdpd-service.c: In function service_remove_req:
src/sdpd-service.c:517:20: error: cast increases required alignment of
target type [-Werror=cast-align]
src/sdpd-service.c:517:20: error: cast increases required alignment of
target type [-Werror=cast-align]
src/sdpd-service.c:536:2: error: cast increases required alignment of
target type [-Werror=cast-align]
src/sdpd-service.c:536:2: error: cast increases required alignment of
target type [-Werror=cast-align]
cc1: all warnings being treated as errors

sdpd-request: Fix build errors due to unaligned memory access

This fix number of build errors on ARM similar to one below.

  CC     src/bluetooth-sdpd-request.o
src/sdpd-request.c: In function extra_des:
src/sdpd-request.c:181:5: error: cast increases required alignment
    of targettype [-Werror=cast-align]

adaptername: Refactor handle_inotify_cb

Refactor handle_inotify_cb to avoid unaligned memory access.
Instead of reading whole events buffer and cast to event struct when
iterating over it, read data directly to struct inotify_event one
event at time.

This fix following build error on ARM.

  CC     plugins/bluetoothd-adaptername.o
plugins/adaptername.c: In function handle_inotify_cb:
plugins/adaptername.c:244:34: error: cast increases required alignment
of target type [-Werror=cast-align]
cc1: all warnings being treated as errors
make[1]: *** [plugins/bluetoothd-adaptername.o] Error 1
make: *** [all] Error 2

sap: Fix compilation errors due to unaligned memory access

This fix following build errors on ARM.

  CC     profiles/sap/bluetoothd-server.o
profiles/sap/server.c: In function connect_req:
profiles/sap/server.c:317:8: error: cast increases required alignment
of targettype [-Werror=cast-align]
profiles/sap/server.c: In function sap_connect_rsp:
profiles/sap/server.c:676:16: error: cast increases required alignment
of target type [-Werror=cast-align]
cc1: all warnings being treated as errors
make[1]: *** [profiles/sap/bluetoothd-server.o] Error 1
make: *** [all] Error 2

scotest: Fix compilation errors due to unaligned memory access

This fix following build errors on ARM.

  CC     test/scotest.o
test/scotest.c: In function send_mode:
test/scotest.c:272:4: error: cast increases required alignment of
target type [-Werror=cast-align]
test/scotest.c:273:4: error: cast increases required alignment of
target type [-Werror=cast-align]
cc1: all warnings being treated as errors
make[1]: *** [test/scotest.o] Error 1
make: *** [all] Error 2

rctest: Fix compilation errors due to unaligned memory access

This fix following build errors on ARM.

  CC     test/rctest.o
test/rctest.c: In function do_send:
test/rctest.c:539:4: error: cast increases required alignment of target
type [-Werror=cast-align]
test/rctest.c:540:4: error: cast increases required alignment of target
type [-Werror=cast-align]
cc1: all warnings being treated as errors
make[1]: *** [test/rctest.o] Error 1

l2test: Fix compilation errors due to unaligned memory access

This fix following build errors on ARM.

  CC     test/l2test.o
test/l2test.c: In function recv_mode:
test/l2test.c:826:9: error: cast increases required alignment of target
type [-Werror=cast-align]
test/l2test.c:834:8: error: cast increases required alignment of target
type [-Werror=cast-align]
test/l2test.c: In function do_send:
test/l2test.c:893:4: error: cast increases required alignment of target
type [-Werror=cast-align]
test/l2test.c:894:4: error: cast increases required alignment of target
type [-Werror=cast-align]
cc1: all warnings being treated as errors
make[1]: *** [test/l2test.o] Error 1

sdp: Use bt_get_be* helpers instead of bt_get_unaligned

This fix number of compilation errors on ARM similar to one below.

lib/sdp.c: In function 'sdp_uuid_extract':
lib/sdp.c:1019:27: error: cast increases required alignment
    of target type [-Werror=cast-align]
lib/sdp.c:1019:27: error: cast increases required alignment
    of target type [-Werror=cast-align]
lib/sdp.c:1026:27: error: cast increases required alignment
    of target type [-Werror=cast-align]
lib/sdp.c:1026:27: error: cast increases required alignment
    of target type [-Werror=cast-align]

sdp: Fix compilation errors due to unaligned memory access

This fix number of build errors on ARM similar to one below.

lib/sdp.c: In function 'sdp_set_seq_len':
lib/sdp.c:625:3: error: cast increases required alignment of target
 type [-Werror=cast-align]
lib/sdp.c:625:3: error: cast increases required alignment of target
type [-Werror=cast-align]
lib/sdp.c:631:3: error: cast increases required alignment of target
type [-Werror=cast-align]
lib/sdp.c:631:3: error: cast increases required alignment of target
type [-Werror=cast-align]

lib: Add unaligned put helper functions

These functions are similar to bt_get_* functions.

mgmt: Implement support for Passkey Notify event

core: Refactor authentication handling

simple-agent: Update to match correct DisplayPasskey API

mgmt-api: Add event for passkey notification

agent: Add missing parameter to DisplayPasskey

agent-api: DisplayPasskey: D-Bus doesn't have a uint8 type

audio: Handle error in gateway_request_stream()

gateway_request_stream() should check if the call to get_records() has
succeeded, and fail otherwise.

audio: Fix gateway in connecting state forever

If bt_search_service() fails the state should be left unchanged.
Otherwise the gateway state is set forever to GATEWAY_STATE_CONNECTING.

This issue can be easily reproduced if a connection attempt is done
very soon after startup.

audio: Remove unused function

gateway_config_stream() is now unnecessary after the removal of the unix
socket support in commit 1d9d0527cfb6d96a976ede56bd43a2bc16bb5f21.

audio: Fix gateway probe always failing

Commit 813b674bce1aab009e7f2d14d1825f603330563d introduced this trivial
bug which causes the failure of the gateway profile probe.

core: Fix calling profile remove callbacks

The device_remove callback for profiles applies to the entire profile so
it should only be used if no matching UUIDs are left or the device
object itself is removed.

gatt: Translate Characteristic names

Translate Characteristic UUID's to name. This list was taken from
the Bluetooth SIG developer site.

input: Use bt_uuid_strcmp where is makes sense

core: Use bt_uuid_strcmp where it makes sense

gatt: Fix whitespace in UUID definitions

thermometer: Fix indentation

network: Fix bnep setup failure handle bug

This patch initializes rsp varialbe again for the remain failure
procedure.

core: replace NULL by dbus invalid arguments

This patch replaces NULL with dbus invalid arguments return if dbus
method call gets invalid arguments from the peer. This will avoid dbus
peer coommander's infinite wait.

audio: Remove unused enum values

audio: remove redundant Media API option

BlueZ supports only Media API. So option handler's are useless.

proximity: Remove unnecessary check for GATT enabled

Proximity plugin won't be loaded if GATT is disabled so there's no point
in checking this once more when adapter is probed.

proximity: Simplify DBusConnection object usage

DBusConnection object can be obtained at any time by
get_dbus_connection() call so there is no need to pass and store it
across profile code. Also there's no need to ref/unref this object as by
design it will live for entire lifetime of plugin.

proximity: Remove unused definitions

proximity: Change adapter driver function names

The probe and remove suffixes better describe what these functions
actually do.

Add profile abstraction (replaces btd_device drivers)

This patch removes the btd_device_driver concept and replaces it with
btd_profile. The new construct also contains the necessary parts for
adapter drivers, so btd_adapter_driver is only needed for non-profile
related functionality (most of which is in plugins/*).

The main purpose of this new construct is to facilitate a centralized
connection creation mechanism for profiles, ultimately enabling the
addition of a Device.Connect() method instead of requiring a UI to know
to call e.g. Input.Connect or Audio.Connect.

This feature will also be extended to externally implemented profiles
once the internal implementation gets more stable, such as OBEX (obexd)
and HFP (oFono). The new D-Bus interface will also partially restore
functionality which was previously available through the Serial
interface.

uuid: Add string-format UUID comparison helper function

core: Add D-Bus disconnect function for easier detection

TODO: Add entry for converting storage format for BlueZ 5

rctest: fix typo

recevied -> received

btio: Remove BT_IO_L2ERTM

There's no need to have a separate type for ERTM. The mode parameter
takes care of most needs and if necessary a "reliable" parameter can be
added later which will map to SOCK_STREAM usage.

Update code base to use the new BtIO API

btio: Make BtIOType private

Since the socket type can be inferred from the socket itself or from the
PSM/Channel/CID/etc parameters provided to listen() and connect() it
doesn't make sense to expose this in the BtIO API.

btio: Remove unneeded L2CAP raw socket support

Now that pairing is done through mgmt this support is no-longer needed.

sdp: Fix error reporting in sdp_service_search_attr_req

This function reports error code via errno not return value.

sdp: Fix redundant NULL checks in convert_raw_attr_to_xml_func

Param data can't be NULL and was already dereferenced before check.

TODO: Add a few more items for BlueZ 5

core: Fix not updating storage when mode changes

With mgmt interface bluetoothd is no longer requesting mode change for
things like DiscoverableTimeout as the kernel will do that automatically
when the timer expires.

Fix trivial coding style issues on pointer declarations and casting

Avoid using C++ style pointer declarations like "char* ptr", as most
BlueZ code uses "char *ptr".

input: Add Nordic Semiconductor copyright

core: Fix memory leak

When ignoring a LE device that is not in a discoverable state 'alias'
is leaking.

input: Fix signedness issue on POSIX error code usage

By convention, error variables containing POSIX error code (usually
called "err") are negative. This commit fixes a couple of places where
positive values are expected.

network: Fix warning when removing connection timeout handler

The source should only be removed once as it can cause the following
warning:

GLib-CRITICAL **: g_source_remove: assertion `tag > 0' failed

network: Remove sessions from server on DBus call Unregister

network: Kill underlying session's connection before freeing it

network: Release session's interface from bridge when unregistering

network: Add helper function to remove an interface from a bridge

network: Keep track of session's interface name in server

gatt: GAP and GATT services are mandatory

This patch fixes potential segmentation fault if the remote doesn't
support GAP and GATT services.

gatt: Don't raise error if declaration was found

This patch avoids raising errors to the upper layer if at least one
characteristic declaration has been found. According to the Core SPEC
Characteristic declarations shall be read without authentication or
authorization.

gatt: Return not found if characteritics is empty

This patch changes the behaviour of Discover All Characteristics and
Discover Characteristics by UUID returning <<Attribute Not Found>> when
Characteristic list that matches the defined criteria is empty. This
scenario can happen when the GATT client avoids extra iteraction with
the remote attribute server if the last received handle(or handle + 1)
is equal to the end handle of the primary service.

att: Replace ATT magic number

This patch replaces the magic number used in Find Information Response
Format field by a proper declared constant.

att: Add Find Info Response Format constants

This patch adds constants declarations for Find Information Response
Format field. The format means that information data constains a list
of 1 or more handles with their 16-bit or 128-bit Bluetooth UUIDs

gatt: Add MTU exchange procedure

GATT Exchange MTU is a procedure defined by the Generic Attribute
Profile, it is not related to the remote GATT primary service. However,
gatt plugin is the most suitable place to manage this procedure.

core: Rename services_changed to uuids_changed

This patch renames the services_changed function to uuids_changed to
avoid misinterpretation. "Service Changed" expression is used to refer
to a GATT operation used to notify clients that a given attribute range
in the GATT server is not valid anymore.

gatt: Primary service interval may not change

This patch address the "Service Changed" scenario on which the start
and end Primary Service attribute handle interval doesn't change.

core: Add updating GATT services

This patch adds the functions to manage Profiles added, and removed
after a Discover All Primary Services procedure.

core: Rename update_services

This patch renames update_services functions to avoid clashing with GATT
based services.

gatt: Start Discover Services when handle changes

This patch starts the Discover All Primary services when a Service
Changed indication is received or the Service Changed characteristic
value is read.

gatt: Confirm the indication received

This patch adds the ATT command reply for the Service Changed
indication value received.

gatt: Enable indication for Service Changed

This patch enables indication of the Service Changed characteristic
setting the indication bit in the Client Characteristic Configuration
descriptor.

gatt: Add Service Changed CCC discovery

This patch adds the Client Characteristic discovery of the Service
Changed Changed characteristic.

gatt: Add Service Changed read

This patch implements the Service Changed GATT charateristic value read.
Force reading the value is necessary while Service Changed value handle
is not stored. Indication can be lost since it is the first data sent
from the GATT server when the connection is established, and the client
may not be filtering the ATT indications.

gatt: Map Appearance value 0x0000 to "unknown"

According to Bluetooth SIG Assigned numbers, Appearance characteristic
value 0x0000 is designated to "unknown".

core: Add storing Appearance

This patch stores the Appearance characteristic value read from the
remote GAP service.

core: PropertyChanged signal for Icon/Appearance

This patch emits the PropertyChanged signal in the Device hierarchy when
the remote device Appearance characteristic is read. In general
Appearance is a static value, if the device doesn't expose the
Appearance value in the advertising data, the GAP plugin should read the
value using GATT read procedure.

core: PropertyChanged signal for Icon/class

This patch emits the PropertyChanged signal in the Device hierarchy when
the remote device class is updated.