attrib-server: Allow zero length attribute update

attrib_db_update always fails when g_try_realloc returns NULL, not
taking into account that the length passed to g_try_realloc could be
zero. In this case, g_try_realloc frees the currently allocated memory
and returns NULL.
As a result, not only will attrib_db_update fail needlessly, a
use-after-free could occur as the attribute's length will still hold the
length of the freed buffer.

Fix this by only returning an error if the length is non-zero.

AVRCP: Fix not registering to VolumeChanged event again when notified

The spec says:

"A registered notification gets changed on receiving CHANGED event
notification. For a new notification additional NOTIFY command is
expected to be sent."

audio: Add Volume property to A2DP transport GetProperties

Now that volume is being handled by SetProperty it should also be
available in GetProperties.

AVRCP: Add support for sending SetAbsoluteVolume

Once the transport volume is changed update the remote volume by sending
SetAbsoluteVolume:

< AVCTP: Command : pt 0x00 transaction 9 pid 0x110e
    AV/C: Changed: address 0x48 opcode 0x00
      Subunit: Panel
      Opcode: Vendor Dependent
      Company ID: 0x001958
      AVRCP: SetAbsoluteVolume: pt Single len 0x0001
        Volume: 100.00% (127/127)
> AVCTP: Response : pt 0x00 transaction 9 pid 0x110e
    AV/C: Accepted: address 0x48 opcode 0x00
      Subunit: Panel
      Opcode: Vendor Dependent
      Company ID: 0x001958
      AVRCP: SetAbsoluteVolume: pt Single len 0x0001
        Volume: 100.00% (127/127)

audio: Fix signature type for transport Volume

Signature is now uint16 instead of byte

audio: Fix updating volume property for non-A2DP transports

Volume property is A2DP only

AVRCP: Fix initializing volume before checking PDU type

Response may be rejected or not implemented so the operand used to
initialize the variable may not even exist.

SAP: Remove not needed checks from disconnect_ind

disconnect_ind is an helper funtion and is always called with valid
connection and from valid state so there is no need to double check
that.

sap-dummy: Fix DBus connection leak if interface registration failed

If DBus interface registration failed dbus connection should be
unreferenced.

SAP: Remove one line of indentation in sap_io_destroy

SAP: Remove not needed src field from struct sap_server

src is only used in sap_server_register, no need to keep in memory for
SAP session duration.

SAP: Remove not needed includes

sap-dummy: Remove not needed includes

sap-dummy: Small refactor to sap_connect_req

This makes function easier to read and more similar to rest of code.

sap-dummy: Use constants when setting ongoing call status

This improves readbility of whan new ongoing call status is being set.

SAP: Fix some code style issue

SAP: Remove not needed NULL pointer check in check_msg

Buffer passed to check_msg() is always valid.

udev: remove deprecated function

This function has in the past returned "/sys" unconditionally. As of
udev-183 it is gone, so just replace it with the string.

storage: Rename appearance file to appearances

This patch renames the "appearance" file to "appearances" to be more
suitable with its content.

storage: Rename primary file to primaries

This patch renames the "primary" file to "primaries" to be more suitable
with its content.

storage: Rename characteristic to characteristics

This patch renames the "characteristic" file to "characteristics" to be
more suitable with its content.

storage: Store address type in "attributes" file

Addressing types can be either BR/EDR, BLE public or BLE random so the
entries in the "attributes" file did not contain enough information to
distinguish which addressing type it's supposed to be. Entries will now
contain both address number and address type as a single key in every
entry in the file.

core: Fix creating device from "longtermkeys" file

storage: Store address type in "longtermkeys" file

Addressing types can be either BR/EDR, BLE public or BLE random so the
entries in the "longtermkeys" file did not contain enough information to
distinguish which addressing type it's supposed to be. Entries will now
contain both address number and address type as a single key in every
entry in the file.

storage: Store address type in "ccc" file

Addressing types can be either BR/EDR, BLE public or BLE random so
the entries in the "ccc" file did not contain enough information to
distinguish which addressing type it's supposed to be. Entries will
now contain both address number and address type as a single key in
every entry in the file.

storage: Store address type in "appearance" file

BLE addressing types can be either public or random so the entries in
the "appearance" file did not contain enough information to distinguish
which addressing type it's supposed to be (LE public or LE random).

Appearance is an information related to BLE device only. Entries will
now contain both BLE address number and BLE address type as a single key
in every entry in the file.

storage: Store address type in "characteristics"

GATT can use BR/EDR or BLE as transport. Addressing types can be either
BR/EDR, BLE public or BLE random so the entries in the "characteristics"
file did not contain enough information to distinguish which addressing
type it's supposed to be. Entries will now contain both address number
and address type as a single key in every entry in the file.

AVRCP: Remove unneeded check for set_volume callback

media-api: Update documentation to include Volume property

AVRCP: Fix not setting audio device connected to player

While connecting device should be set to match the AVCTP session and when
disconnected reset it back to NULL.

AVCTP: Fix setting wrong transaction id expected for responses

The id were incremented after being set to the request so it is always
+1 of the actual transaction.

input: Remove unwanted codes from device

Here the variable iconn never be NULL, so no need to check it
against NULL.

audio: Remove unwanted code from manager

Here the variable adp never be NULL, so no need to check it
against NULL.

core: Remove unnecessary agent variable

adapter: Remove unwanted if check and code

Here the variable agent never be NULL, so no need to check it
against NULL.

avctp: Fix NULL check after dereference

Check for session != NULL has to be done before accessing session.

Remove redundant D-Bus error initilization in maemo6 plugin

agent: Remove redundant D-Bus error initilizations

media: Remove redundant D-Bus error initilization

storage: Use g_free() instead of free()

It is important to match g_*() memory allocation with g_free()

monitor: Fix memory leak

sdp/build: Build fails when SDP_DEBUG enabled

Enabling SDP_DEBUG prevents build to be done.

core: Fix removing device services from "primary" file

The BLE address type needs to be passed as parameter in
delete_device_service() function in order to remove a device from
"primary" file. This is why the entries in "primary" file will be using
the new storage format.

core: Fix creating device from "primary" file

This patch removes the hard-coded address type for the BLE device
created from the storage.

storage: Store BLE address type in "primary" file

BLE addressing types can be either public or random so the entries in
the "primary" file did not contain enough information to distinguish
which addressing type it's supposed to be (LE public or LE random).
Entries will now contain both BLE address number and BLE address type
as a single key in every entry in the file.

audio: Change the file mode back to 644

gdbus: Fix incorrectly discarded signals

Signals with no arguments were incorrectly filtered out due to the NULL
inequality check.

audio: Fix media transport creation for gateway endpoints

The device maybe connected to another adapter other than the endpoint's
so the adapters must be first check.

audio: Fix media transport creation

Prevents set_configuration() to be called on another adapter
than connecting one

avdtp: Fix incorrect gchar buffer allocation

The code was allocating an array of gchar pointers, where an array of
gchar is expected.

AVRCP: Add missing Player Setting feature in TG record

AVRCP: Refactor the code using constants

sdp: Remove unnecessary memset()

The memory referenced by "u" pointer is initialized right after the
memset() call.

AVRCP: Bump TG record to 1.4

The only mandatory feature for 1.4 is absolute volume control which is
now supported.

AVRCP: Subscribe for VolumeChanged Notification

If the remote device support version 1.4 or latter send register command
for VolumeChanged event:

< AVCTP: Command : pt 0x00 transaction 0 pid 0x110e
    AV/C: Notify: address 0x48 opcode 0x00
      Subunit: Panel
      Opcode: Vendor Dependent
      Company ID: 0x001958
      AVRCP: RegisterNotification: pt Single len 0x0005
        EventID: 0x0d (EVENT_VOLUME_CHANGED)
        Interval: 0x00000000 (0 seconds)
> AVCTP: Response : pt 0x00 transaction 0 pid 0x110e
    AV/C: Interim: address 0x48 opcode 0x00
      Subunit: Panel
      Opcode: Vendor Dependent
      Company ID: 0x001958
      AVRCP: RegisterNotification: pt Single len 0x0002
        EventID: 0x0d (EVENT_VOLUME_CHANGED)
        Volume: 100.00% (127/127)

AVRCP: Add define for VolumeChanged event

sdp: Fix removing servers own record

To avoid misbehaving plugins trying to remove handle 0 it's better to
directly check for this and return EINVAL.

adapter: do not check signature twice

Message signature is already checked in generic_message(), so there's no
need to check again in the callback.

gdbus: do not check signature twice

Message signature is already checked in generic_message(), so there's no
need to check again in the callback.

gdbus: add Method.NoReply annotation in introspection

gdbus: add Deprecated annotation in introspection

gdbus: remove signature and reply from tables

Do not set signature and reply in GDBus tables

Use GDBUS_* macros, so signature and reply fields are not set in each
method/signal.

gdbus: loop over args to check message signature

gdbus: use GDBusArgInfo to generate introspection

By using GDBusArgInfo in methods and signals, the introspection
generation is much simpler and we can add each argument name.

Convert GDBus methods to use macro helpers

With these macro helpers we can separate in/out arguments and use their
own vector.

gdbus: add and use helpers for table declarations

gdbus: add argument info to methods and signals

Constify GDBus signal tables

Constify signal tables with the following command:

    find . -name '*.[ch]' -exec \
             sed -i 's/\(GDBusSignalTable .* =\)/const \1/g' {} \;

Constify GDBus method tables

Constify method tables with the following command:

find . -name '*.[ch]' -exec \
     sed -i 's/\(GDBusMethodTable .* =\)/const \1/g' {} \;

gdbus: Constify introspection method table

gdbus: do not call memset for terminating NUL

gdbus: return if method signature is malformed

Add dbus service file that references the systemd unit

This allows bluez to be bus-activated.

hciemu: Fix gcc 4.7 compilation error

This patch fixes the following compilation error with gcc 4.7:

  CC     test/hciemu.o
test/hciemu.c: In function 'getbdaddrbyname':
test/hciemu.c:1115:3: error: dereferencing type-punned pointer will break strict-aliasing rules [-Werror=strict-aliasing]

doc: Update possible values of Agent capabilities

adapter: Remove dead code from set_property()

build: Do not set CFLAGS/LDFLAGS directly

Set a separate variable for adding warning flags, optimization, etc.
Build systems are not supposed to change CFLAGS and LDFLAGS, these are
user variables.

Doing so we guarantee CFLAGS and LDFLAGS from environment is appended
to the flags used during build. One useful use-case is to temporarily
disable -Werror when using --enable-maintainer-mode, without completely
loosing the warning flags and other parameters in CFLAGS (like -fPIC).

Without this patch, fiddling with CFLAGS/LDFLAGS after configure may
result in errors like below:

/usr/bin/ld: tools/rfcomm.o: relocation R_X86_64_32 against `.bss' can
not be used when making a shared object; recompile with -fPIC
tools/rfcomm.o: could not read symbols: Bad value
collect2: error: ld returned 1 exit status
make[1]: *** [tools/rfcomm] Error 1
make: *** [all] Error 2

Reference: http://www.gnu.org/software/automake/manual/html_node/Flag-Variables-Ordering.html

build: Fix typo in AM_CFLAGS

textfile: fix missing call to munmap in wite_key

In some cases it was possible that munmap is not called corresponding
to mmap, which might result in future read or writes to fail.

lib: Add definitions for AMP Assoc data

Definitions are needed for decoding A2MP Assoc Rsp.

core: Remove unused variable

"srcaddr" variable is not used for anything else in
btd_event_remote_name() function. So, remove it for now.

hciattach: Add support for Intel Bluetooth device

This patch enables the Intel Bluetooth device (UART sku) over the H4
protocol.  It is responsible for bring up the device into known state by
configuring the baudrate and applying the patches, if required.

eir: Keep parsed eir_data on error

Do not discard already parsed eir_data on eir data length error.
Data corruption may occur, but this does not mean that we need
to ignore already parsed valid eir data.

monitor: remove extra black line

hidd: Fix indentation

audio: Fix tab misuse

deviceinfo: Fix indentation and tab misuse

monitor: Fix tab misuse

sap: Fix tab misuse

device: Fix tab misuse

event: Fix tab misuse

tools: Fix tab misuse

lib: Fix tab misuse

csr_bcsp: fix link establishment

The current link-establishment implementation times out if a packet is
not received after the link is active (i.e. reception of CONF_RESP).

Fix by making sure that the link establishment loop in open is
terminated when ubcsp sets the UBCSP_PACKET_SENT flag (on reception of
CONF_RESP) rather than UBCSP_PACKET_RECEIVED (on reception of any later
packages).

udev: remove udev activation of the blutooth daemon

Udev is not meant to spawn long-running processes, and as of the next
udev version such processes will be killed.

If using systemd, device activation can be used to get the same effect.

On Arch we have been disabling this rule for a long time, even though we
don't use systemd, as it was causing more problems than it was worth.

systemd: don't order against syslog.target

This was only needed in old versions of systemd. All messages are logged
by default to the journal now, no ordering required.

input: increase security level of the connection

gatttool: Add option to specify LE address type

This patch makes possible to specify LE address type.

After advertising cache was removed from kernel we should always specify
address type for LE link when calling bt_io_connect() as otherwise random
will always be used.

LE address type can be specified either by 'addr-type' or 't' command line
parameter or as additional parameter to 'connect' command in interactive mode.

Possible values are 'public' (default) and 'random'.

media: Enable parallel requests to endpoint

If endpoint is handling several devices, it should be able to handle
multiple requests, one per each.