android: fix a build break

It's seems a typo introduced by the previous commit 10342fe.

Also fixes incorrect indents.

cm: android: update interfaces for CM 13.0 (6.0.1 r52)

android: fix building errors on android-6.0.1_r52

Fix all Android Bluetooth APIs mismatching.

Treat all warnings as errors to avoid overlooking such issues
in the future.

Android: Add interfaces for CM 13.0

Signed-off-by: Jaap Jan Meijer <jjmeijer88@gmail.com>

Android: Add CM_ANDROID_VERSION macro

Signed-off-by: Jaap Jan Meijer <jjmeijer88@gmail.com>

android: add dummy implementation for new functions

Some new functions were added since android-6.0.1_r31.
Add dummy implementation to avoid crashing the Bluetooth service.

Add btattach (the successor to hciattach)

android: Update device class info

Android-x86 devices are not phone.

Update signal socket structure to match Android 6.0

Android 6.0 introduced l2cap sockets to the bluetooth framework.
One of the changes modified the signal socket structure by
adding two unsigned short for l2cap use only.

android: fix building errors on Android 6.0

Change libhal to match the new HAL headers.

Signed-off-by: Chih-Wei Huang <cwhuang@linux.org.tw>

android: fix building errors of the 64-bit targets

Some functions prototypes are not declared implicitly for
the Android 6.0 64-bit targets that result in the errors:

  external/bluetooth/bluez/android/hal-ipc.c: In function 'notification_handler':
  external/bluetooth/bluez/android/hal-ipc.c:143:3: error: implicit declaration of function 'memset' [-Werror=implicit-function-declaration]
     memset(&msg, 0, sizeof(msg));
     ^
  external/bluetooth/bluez/android/log.c: In function 'init_logd':
  external/bluetooth/bluez/android/log.c:175:2: error: implicit declaration of function 'memset' [-Werror=implicit-function-declaration]
    memset(&addr, 0, sizeof(addr));
    ^

Signed-off-by: Chih-Wei Huang <cwhuang@linux.org.tw>

android: remove bluetooth-headers

The Android build system discourages from copying headers.
Besides, it's incorrect to do actions in the make shell function.

Signed-off-by: Chih-Wei Huang <cwhuang@linux.org.tw>

shared/utils: add the missing header <strings.h>

It defines the prototype of ffs() that fixes the building errors
of an Android 6.0 64-bit target.

Signed-off-by: Chih-Wei Huang <cwhuang@linux.org.tw>

android: tools: add brcm_patchram_plus

test/simple-endpoint: Fix ClearConfiguration signature

Release 5.40

build: Update library version

lib: Update company identifiers

audio/avrcp: Fix not handling Addressed Player Changed error

Some notification are completed in case the addressed player changes:

  'On completion of the Addressed Player Changed notification the TG
  shall complete all player specific notifications with AV/C C-Type
  REJECTED with error code Addressed Player Changed.'

Because reject only has the error code not the event it is necessary to
lookup by transaction to find out which event was completed thus the
transaction needs to be added to the avctp_rsp_cb callback.

monitor: Split public key into multiple lines

Printing the entire public key value on a single line requires some
160 characters which ends up wrapping on most terminals. Instead,
print the X and Y components separately, which is also consistent with
how SMP is decoded.

doc: btsnoop: Document the various BTSnoop opcodes

doc: btsnoop: Clarify drop counts.

profile: Fix accepting connections without an associated service

This fixes the flawed logic from the following commit:

commit b669604f43bbbf04ebaa66dd14ad9cfb005e561b
Author: Hsin-Yu Chao <hychao@chromium.org>
Date:   Fri Apr 24 10:36:09 2015 -0700

    src/profile: Set service to connecting in ext_connect

doc: Add BTSnoop/Monitor protocol documentation

shared/gatt-client: Fix sending confirmation

Confirmation shall be sent from topmost parent only.
Extra confirmation sent by clone confuses remote side.

> ACL Data RX: Handle 69 flags 0x02 dlen 27
      ATT: Handle Value Indication (0x1d) len 22
        Handle: 0x001f
          Data: 68656c6c6f2027302e302e302e303a3532353332
< ACL Data TX: Handle 69 flags 0x00 dlen 5
      ATT: Handle Value Confirmation (0x1e) len 0
< ACL Data TX: Handle 69 flags 0x00 dlen 5
      ATT: Handle Value Confirmation (0x1e) len 0

tools/gatt-service: Update to use new GATT API

This update use of ReadValue and WriteValue to include the options
introduced in the API.

client: Update to use new GATT API

This update use of ReadValue and WriteValue to include the options
introduced in the API.

test: Update GATT examples with the new API

doc/gatt-api: Add secure flags

This add secure-{read,write} which shall be used by servers that want
to restrict attribute access to secure connection only (BT_SECURITY_FIPS)

doc/gatt-api: Merge RegisterProfile with RegisterApplication

Since RegisterApplication makes use of ObjectManager it is also possible
to verify the existance of GattProfile objects unifying the API for both
services (GATT server) and profiles (GATT client).

doc/gatt-api: Add options dictionary to ReadValue/WriteValue

This adds the possibility to pass an offset to these operations, and
also in the server case to give the device object.

lib: Update company identifiers

tools: add more entities to company ID parser

This patch adds lower and uppercase u with umlaut.

device: Fix invalid memory access when removing a device

The current code uses disconnect_all() outside the disconnection timer
callback but doesn't remove the timer from the list of timeout sources,
making it possible (but tricky) to get into the situation that the
disconnect_all() callback gets called with a device that was already
removed/blocked.

Valgrind log:

bluetoothd[5207]: src/device.c:device_remove() Removing device /org/bluez/hci0/dev_00_1A_7D_DA_71_07
bluetoothd[5207]: src/service.c:change_state() 0x84f0d70: device 00:1A:7D:DA:71:07 profile gap-profile state changed: connecting -> disconnected (-103)
bluetoothd[5207]: src/device.c:device_profile_connected() gap-profile Software caused connection abort (103)
bluetoothd[5207]: src/service.c:change_state() 0x84f0d70: device 00:1A:7D:DA:71:07 profile gap-profile state changed: disconnected -> unavailable (0)
bluetoothd[5207]: profiles/gap/gas.c:gap_driver_remove() GAP profile remove (00:1A:7D:DA:71:07)
bluetoothd[5207]: src/service.c:btd_service_unref() 0x84f0d70: ref=0
bluetoothd[5207]: src/device.c:btd_device_unref() Freeing device /org/bluez/hci0/dev_00_1A_7D_DA_71_07
bluetoothd[5207]: src/gatt-client.c:unregister_service() Removing GATT service: /org/bluez/hci0/dev_00_1A_7D_DA_71_07/service0006
bluetoothd[5207]: src/gatt-client.c:unregister_characteristic() Removing GATT characteristic: /org/bluez/hci0/dev_00_1A_7D_DA_71_07/service0006/char0007
bluetoothd[5207]: src/gatt-client.c:unregister_descriptor() Removing GATT descriptor: /org/bluez/hci0/dev_00_1A_7D_DA_71_07/service0006/char0007/desc0009
bluetoothd[5207]: src/device.c:device_free() 0x8446200
bluetoothd[5207]: src/adapter.c:dev_disconnected() Device 00:1A:7D:DA:71:07 disconnected, reason 2
bluetoothd[5207]: src/adapter.c:bonding_attempt_complete() hci0 bdaddr 00:1A:7D:DA:71:07 type 1 status 0xe
bluetoothd[5207]: src/adapter.c:resume_discovery()
bluetoothd[5207]: src/adapter.c:trigger_start_discovery()
bluetoothd[5207]: src/adapter.c:cancel_passive_scanning()
==5207== Invalid read of size 1
==5207==    at 0x49E87C: disconnect_all (device.c:1218)
==5207==    by 0x50C5FA2: g_timeout_dispatch (in /usr/x86_64-pc-linux-gnu/lib/libglib-2.0.so.0.4800.0)
==5207==    by 0x50C5569: g_main_context_dispatch (in /usr/x86_64-pc-linux-gnu/lib/libglib-2.0.so.0.4800.0)
==5207==    by 0x50C58E7: g_main_context_iterate.isra.29 (in /usr/x86_64-pc-linux-gnu/lib/libglib-2.0.so.0.4800.0)
==5207==    by 0x50C5C01: g_main_loop_run (in /usr/x86_64-pc-linux-gnu/lib/libglib-2.0.so.0.4800.0)
==5207==    by 0x40CA68: main (main.c:687)
==5207==  Address 0x8446412 is 530 bytes inside a block of size 616 free'd
==5207==    at 0x4C2AEA0: free (vg_replace_malloc.c:530)
==5207==    by 0x4A2EA5: device_free (device.c:670)
==5207==    by 0x4C7439: remove_interface (object.c:667)
==5207==    by 0x4C8051: g_dbus_unregister_interface (object.c:1391)
==5207==    by 0x4AAADC: btd_device_unref (device.c:5998)
==5207==    by 0x4AAC85: device_remove (device.c:3951)
==5207==    by 0x48F541: btd_adapter_remove_device (adapter.c:1180)
==5207==    by 0x49136A: connect_failed_callback (adapter.c:7707)
==5207==    by 0x4CE139: notify_handler (mgmt.c:292)
==5207==    by 0x4CD723: queue_foreach (queue.c:220)
==5207==    by 0x4CF568: process_notify (mgmt.c:304)
==5207==    by 0x4CF568: can_read_data (mgmt.c:370)
==5207==    by 0x4E01EA: watch_callback (io-glib.c:170)
==5207==  Block was alloc'd at
==5207==    at 0x4C2BD20: calloc (vg_replace_malloc.c:711)
==5207==    by 0x4A2F7C: device_new (device.c:3535)
==5207==    by 0x4A4F14: device_create (device.c:3623)
==5207==    by 0x484A08: adapter_create_device (adapter.c:1115)
==5207==    by 0x48EBE2: update_found_devices (adapter.c:5473)
==5207==    by 0x48EBE2: device_found_callback (adapter.c:5647)
==5207==    by 0x4CE139: notify_handler (mgmt.c:292)
==5207==    by 0x4CD723: queue_foreach (queue.c:220)
==5207==    by 0x4CF568: process_notify (mgmt.c:304)
==5207==    by 0x4CF568: can_read_data (mgmt.c:370)
==5207==    by 0x4E01EA: watch_callback (io-glib.c:170)
==5207==    by 0x50C5569: g_main_context_dispatch (in /usr/x86_64-pc-linux-gnu/lib/libglib-2.0.so.0.4800.0)
==5207==    by 0x50C58E7: g_main_context_iterate.isra.29 (in /usr/x86_64-pc-linux-gnu/lib/libglib-2.0.so.0.4800.0)
==5207==    by 0x50C5C01: g_main_loop_run (in /usr/x86_64-pc-linux-gnu/lib/libglib-2.0.so.0.4800.0)
==5207==

tools/bluemoon: Add hardware variant for Intel Bluetooth 8265

This patch adds support for HW variant for Intel Bluetooth 8265
also known as Windstorm Peak (WsP).

gdbus: Fix the ordering of signals

Consider the following example:

/foo
  properties: "A", "B"

/bar
  properties: "C", "D"

If during a given mainloop iteration, property "A" of object '/foo' is
changed, then properties "C" and "D" of '/bar', lastly "B" of '/foo',
the current code will emit the PropertiesChanged signals in following
order: "A", "B", "C", "D".

This may confuse applications that have a dependency on the order of
those signals.

This fixes the ordering, so in the example, the order becomes:
"C", "D", "A", B". This is considered not to be a problem, as
applications may use the flag G_DBUS_PROPERTY_CHANGED_FLAG_FLUSH, so
property changed signals are emitted as soon as possible.

The solution is for each object, to reschedule the signals every time a
signal is emitted.

monitor: Pass drop count from TTY to BTSnoop file

shared/btsnoop: Add support for providing cumulative drops count

monitor: Add support for writing to BTSnoop file from TTY input

monitor: Introduce support for extended TTY header

Add support for packet drops and 32-bit timestamps.

doc/settings-storage: Add alternative format for descriptors

In case of some descriptors like CEP we must store the actual value since
extended properties are not part of the discovery and btd_gatt_client no
longer attempt to read it the value must be loaded in the db so
gatt_db_attribute_get_char_data is able to retrieve it.

core/device: Fix not storing GATT attributes

If services has been refreshed the attributes shall be stored so they can
be reloaded properly the next time.

core/device: Remove code related to GattServices

GattService has been removed so there shouldn't be any code related to it

core/gatt-client: Don't read extended properties

Extended properties are already read and stored in the database and can be
retrieved with gatt_db_attribute_get_char_data.

monitor: Disable monitor socket if a TTY is used

For now we don't have a way of resolving index conflicts between a TTY
and a monitor socket. Simply disable the monitor socket for now if a
TTY is used.

Make use of new tty_get_speed helper

shared: Add TTY speed helper

monitor: Update to new header format of TTY protocol

The updated TTY protocol content looks like:

uint16  data_len
uint16  opcode
uint8   flags
uint8   hdr_len
uint8   extra_hdr[hdr_len];
uint8   payload[data_len - 4 - hdr_len]

monitor: Permit monitor/mgmt socket errors if a TTY is used

One may want to run btmon without special privileges when using it in
TTY mode. Ignore any errors from control_tracing() in such a case.

monitor: Add error return value to control_tty()

monitor: Add --tty-speed command line switch

monitor: Fix processing left-over data

If there's enough data in the buffer after processing a packet we
should just continue looping and trying to parse it too.

monitor: Fix checking for sufficient data

client: Fix select-attribute behaviour

Recognize Eddystone Configuration Service UUIDs

lib: Add recently introduced SPI & I2C bus defines

monitor: Add --tty switch to help text

HACKING: Fix nit in instructions

audio/avrcp: Fix uninitialized variable

This fixes the following error:

profiles/audio/avrcp.c:2847:6: error: variable 'num_of_items' is used
      uninitialized whenever 'if' condition is true
      [-Werror,-Wsometimes-uninitialized]
        if (pdu->params[0] == AVRCP_STATUS_OUT_OF_BOUNDS)
            ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
profiles/audio/avrcp.c:2857:40: note: uninitialized use occurs here
        media_player_total_items_complete(mp, num_of_items);
                                              ^~~~~~~~~~~~
profiles/audio/avrcp.c:2847:2: note: remove the 'if' if its condition is always
      false
        if (pdu->params[0] == AVRCP_STATUS_OUT_OF_BOUNDS)
        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
profiles/audio/avrcp.c:2839:23: note: initialize the variable 'num_of_items' to
      silence this warning
        uint32_t num_of_items;
                             ^
                              = 0

gatt-client: Read extended props on service discovery

Now once extended props can be taken with characteristic data
we need to makes sure that after discovery correct value is in
database.

With this patch, value of all the extended properties descriptors
are read during discovery just after discover of descriptors

shared/gatt-server Check for ext. charact. prop. on reliable session

With this patch we make sure that reliable session is done on
characteristics which does support it.

shared/gatt-db: Extend gatt_db_attribute_get_char_data with ext. prop

This patch adds way to get extended properties from characteristic
extended property descriptor

test-gatt: Fix characteristic properties

This patch fixes characteristic property for those characteristics
which contains extended characteristic properties descriptor.

btgatt-server: Fix GATT device name properties

Since GATT device name characteristic has extended characteristic
property descriptor, that should be set in properties as well.
This patch fixes that.

monitor: Add support for TTY based data

Add support for receiving a monitor data stream over a TTY.

audio/media: Fix return NULL to .get_name

Player need to have a name set otherwise it may cause the following
crash:

Invalid read of size 1
   at 0x4826BD8: strlen (mc_replace_strmem.c:404)
   by 0x138289: avrcp_handle_get_folder_items (in /usr/libexec/bluetooth/bluetoothd)
   by 0x13A278: handle_browsing_pdu (in /usr/libexec/bluetooth/bluetoothd)
   by 0x1315F2: session_browsing_cb (in /usr/libexec/bluetooth/bluetoothd)
   by 0x48C69BB: g_io_unix_dispatch (in /usr/lib/libglib-2.0.so.0.3600.0)
   by 0x487FB2A: g_main_context_dispatch (in /usr/lib/libglib-2.0.so.0.3600.0)
   by 0x487FEB7: g_main_context_iterate.clone.5 (in /usr/lib/libglib-2.0.so.0.3600.0)
   by 0x48803AA: g_main_loop_run (in /usr/lib/libglib-2.0.so.0.3600.0)
   by 0x16D008: main (in /usr/libexec/bluetooth/bluetoothd)
 Address 0x0 is not stack'd, malloc'd or (recently) free'd

core/device: Fix breakage of Bluetooth pairing

This patch converts old propertis containing list of services available
on remote devices. Without that previously paired devices, i.e.
keyboards, will not work properly.

HACKING: Add necessary commands to configure git send-email

This is convenient for people not involved with projects using email to
as review process.

systemd: Enable ProtectHome and ProtectSystem options

These options protect from unintended access to the filesystem see
SYSTEMD.EXEC(5) for mode detail.

doc/gatt-api: Make Notifying property optional

If the characteristic does not support nofications nor indications the
Notifying property can be omitted as it will never going to be used.

tools: Change test-runner license to LGPL easy reuse

Release 5.39

lib: Update company identifiers

shared/gatt-server: Add support for long write

With this patch long write and nested long write reliable is supported.
GATT server is responsible now to do aggregation of prep write data
for long write session.
Note: We consider long write as the consequtive prepare writes with
continues offsets.

E.g. 1

prep_write: handle 1, offset 0, value_len 10
prep_write: handle 1, offset 10, value_len 10
prep_write: handle 2, offset 0, value_len 10
prep_write: handle 2, offset 10, value_len 10

Will result with following calles to app:

exec_write: handle 1: offset 0, value_len 20
exec_write: handle 2: offset 0, value_len 20

E.g. 2

prep_write: handle 1, offset 0, value_len 10
prep_write: handle 1, offset 2, value_len 5
prep_write: handle 2, offset 0, value_len 10
prep_write: handle 2, offset 4, value_len 5

Will result with following calles to app:

exec_write: handle 1: offset 0, value_len 10
exec_write: handle 1: offset 2, value_len 5
exec_write: handle 2: offset 0, value_len 10
exec_write: handle 2: offset 4, value_len 5

E.g. 3
prep_write: handle 1, offset 0, value_len 10
prep_write: handle 1, offset 5, value_len 5
prep_write: handle 1, offset 10, value_len 6

will result with following calles to app:

exec_write: handle 1, offset 0, value 10
exec_write: handle 1, offset 5, value 11

client: Fix code style problems

Only tabs shall be used for indentation.

input/hog: Fix crash if uhid is not enabled

If /dev/uhid is not present bt_hog_new_default() returns NULL.
This was resulting in NULL pointer dereference in attio_connected_cb.

audio: Fix double free

Because avdtp_unref can now cause a state change the stream reference
shall be dropped before a2dp_cancel is called otherwise the code may
attempt to unref once more using the same reference:

Invalid read of size 8
   at 0x41F33B: avdtp_set_state (avdtp.c:695)
   by 0x420CE7: connection_lost (avdtp.c:1118)
   by 0x4216C4: avdtp_unref (avdtp.c:1178)
   by 0x418098: source_disconnect (source.c:395)
   by 0x41D417: a2dp_source_disconnect (a2dp.c:2312)
   by 0x49A64B: btd_service_disconnect (service.c:273)
   by 0x49E120: dev_disconn_service (device.c:1325)
   by 0x50E6DAC: g_slist_foreach (in /usr/lib64/libglib-2.0.so.0.4600.2)
   by 0x4A3894: device_request_disconnect (device.c:1357)
   by 0x4A3A3C: dev_disconnect (device.c:1442)
   by 0x4C7F22: process_message.isra.3 (object.c:259)
   by 0x4C877C: generic_message (object.c:1071)
 Address 0x929b0c0 is 16 bytes inside a block of size 1,160 free'd
   at 0x4C29E00: free (vg_replace_malloc.c:530)
   by 0x50CE5ED: g_free (in /usr/lib64/libglib-2.0.so.0.4600.2)
   by 0x42163B: avdtp_free (avdtp.c:1101)
   by 0x42163B: avdtp_unref (avdtp.c:1182)
   by 0x417808: source_set_state (source.c:108)
   by 0x4178F9: avdtp_state_callback (source.c:122)
   by 0x41F386: avdtp_set_state (avdtp.c:698)
   by 0x420CE7: connection_lost (avdtp.c:1118)
   by 0x4216C4: avdtp_unref (avdtp.c:1178)
   by 0x418098: source_disconnect (source.c:395)
   by 0x41D417: a2dp_source_disconnect (a2dp.c:2312)
   by 0x49A64B: btd_service_disconnect (service.c:273)
   by 0x49E120: dev_disconn_service (device.c:1325)

audio/avdtp: Fix code style

build: Make old GATT plugins deprecated

This disables building plugins that are no longer supported by the core
since the transition to gatt-db. In the future these plugins will have to
be ported to use gatt-db or be removed if the profile can be implemented
using the GATT D-Bus APIs.

audio/avrcp: Fix registering to player events

If controller does not have a player skip registering to events which
requires a player.

audio/avdtp: Fix crash on outgoing connection failure

This fix double free if outgoing connection failed. This was due to
connection_lost() being called from avdtp_unref which could result
in another call to connection_lost when session ref is already 0.

Fix this in similar way pairing agent is handled: takes extra reference
before calling callbacks and unref it before exit. Then only unref is
suppose to free session.

connect error: Host is down (112)
profiles/audio/avdtp.c:connection_lost() Disconnected from 00:0C:8A:FB:D4:16
profiles/audio/a2dp.c:discover_cb() err 0xfff000240
profiles/audio/avdtp.c:avdtp_unref() 0x85a88f0: ref=1
src/service.c:change_state() 0x7f7c710: device 00:0C:8A:FB:D4:16 profile
    a2dp-sink state changed: connecting -> disconnected (-11)
src/device.c:device_profile_connected() a2dp-sink Resource temporarily
    unavailable (11)
src/device.c:device_profile_connected() returning response to :1.37
profiles/audio/a2dp.c:setup_unref() 0x85b0380: ref=0
profiles/audio/a2dp.c:setup_free() 0x85b0380
profiles/audio/avdtp.c:avdtp_unref() 0x85a88f0: ref=0
profiles/audio/avdtp.c:connection_lost() Disconnected from 00:0C:8A:FB:D4:16
profiles/audio/a2dp.c:discover_cb() err 0xfff000170
profiles/audio/sink.c:sink_set_state() State changed  /org/bluez/hci0/
    dev_00_0C_8A_FB_D4_16: SINK_STATE_CONNECTING -> SINK_STATE_DISCONNECTED
profiles/audio/a2dp.c:channel_remove() chan 0x85a8780
profiles/audio/avdtp.c:avdtp_free() 0x85a88f0
 Invalid free() / delete / delete[] / realloc()
    at 0x4C29CF0: free (vg_replace_malloc.c:530)
    by 0x50CE5ED: g_free (in /usr/lib64/libglib-2.0.so.0.4600.2)
    by 0x4177E3: finalize_discovery (avdtp.c:1039)
    by 0x41789A: connection_lost (avdtp.c:1114)
    by 0x41A7FD: avdtp_connect_cb (avdtp.c:2339)
    by 0x44CBFB: connect_cb (btio.c:232)
    by 0x50C8E39: g_main_context_dispatch (in /usr/lib64/libglib-2.0.so.0.4600.2)
    by 0x50C91CF: ??? (in /usr/lib64/libglib-2.0.so.0.4600.2)
    by 0x50C94F1: g_main_loop_run (in /usr/lib64/libglib-2.0.so.0.4600.2)
    by 0x40B7B7: main (main.c:687)
  Address 0x85b4c30 is 0 bytes inside a block of size 24 free'd
    at 0x4C29CF0: free (vg_replace_malloc.c:530)
    by 0x50CE5ED: g_free (in /usr/lib64/libglib-2.0.so.0.4600.2)
    by 0x4177E3: finalize_discovery (avdtp.c:1039)
    by 0x41789A: connection_lost (avdtp.c:1114)
    by 0x413EE2: setup_free (a2dp.c:163)
    by 0x413EE2: setup_unref (a2dp.c:178)
    by 0x413F5F: setup_cb_free (a2dp.c:201)
    by 0x41638D: finalize_discover (a2dp.c:346)
    by 0x41638D: discover_cb (a2dp.c:1855)
    by 0x4177DB: finalize_discovery (avdtp.c:1037)
    by 0x41789A: connection_lost (avdtp.c:1114)
    by 0x41A7FD: avdtp_connect_cb (avdtp.c:2339)
    by 0x44CBFB: connect_cb (btio.c:232)
    by 0x50C8E39: g_main_context_dispatch (in /usr/lib64/libglib-2.0.so.0.4600.2)
  Block was alloc'd at
    at 0x4C2A988: calloc (vg_replace_malloc.c:711)
    by 0x50CE530: g_malloc0 (in /usr/lib64/libglib-2.0.so.0.4600.2)
    by 0x4190FB: avdtp_discover (avdtp.c:3186)
    by 0x416C19: a2dp_discover (a2dp.c:1872)
    by 0x413642: sink_setup_stream (sink.c:265)
    by 0x4136C4: sink_connect (sink.c:294)
    by 0x470165: btd_service_connect (service.c:238)
    by 0x47583C: connect_next.isra.18 (device.c:1455)
    by 0x478500: connect_profiles (device.c:1710)
    by 0x48EC4A: process_message.isra.5 (object.c:259)
    by 0x53DD1A2: ??? (in /usr/lib64/libdbus-1.so.3.14.6)
    by 0x53CE733: dbus_connection_dispatch (in /usr/lib64/libdbus-1.so.3.14.6)

tools/btattach: Add command line switch for specifying the baudrate

tests/example-gatt-client: Fix using invalid property

This fixes the following trace caused by GattServices1 no longer
containing a property called Characteristics:

Traceback (most recent call last):
  File "./example-gatt-client", line 221, in <module>
    main()
  File "./example-gatt-client", line 207, in main
    if not process_hr_service(service_path):
  File "./example-gatt-client", line 167, in process_hr_service
    chrc_paths = service_props['Characteristics']
KeyError: 'Characteristics'

audio/a2dp: Simplify error handling

Don't expose avdtp_error in a2dp.h instead handle it internally
converting to posix error whenever necessary.

audio/avdtp: Fix passing wrong error code

finalize_discovery actually expect positive codes since it passes to
avdtp_error_init.

audio/avdtp: Only set disconnect timer if connected

If the session is not connected it makes no sense to enable the
disconnect timer.

audio/avrcp: Fix crash when connecting to controllers

Patch 70fdb77d0af137aa859f267da976f610cd9bbbd2 has actually caused the
following regression since no player is set:

Invalid read of size 8
  at 0x435B8E: avrcp_player_value_rsp (avrcp.c:2150)
  by 0x42FB83: control_response (avctp.c:831)
  by 0x42FB83: session_cb (avctp.c:996)
  by 0x50C8E39: g_main_context_dispatch (in /usr/lib64/libglib-2.0.so.0.4600.2)
  by 0x50C91CF: ??? (in /usr/lib64/libglib-2.0.so.0.4600.2)
  by 0x50C94F1: g_main_loop_run (in /usr/lib64/libglib-2.0.so.0.4600.2)
  by 0x40C258: main (main.c:687)
Address 0x58 is not stack'd, malloc'd or (recently) free'd

shared/gatt-server: Fix handle error on execute write

If there is an error during execute write we should drop all
outstanding prep_write data. This patch fix that.

unit/test-gatt: Fix long write test

This patch change characteristic we are writing too in the nested
long write reliable testcase.

Basically characteristic with handle 0x0072 already have characteristic
extended descriptor required for reliable session

unit/test-gatt: Extend large-db

This patch adds characteristic extended descriptor to characteristic
handle 0x0081. Also fixes testes using this.

We need this to make script pass when following patches start
to look into this descriptor on reliable write session

unit/test-gatt: Modify small-db

This patch adds Characteristic Extended descriptor to characteristic
with handle 0x0002 nad 0x0014.

Also fixes scripts using small-db.

We need this as next patches will start to look into this descriptor
on reliable write session.

unit/test-gatt: Fix long write testcases

Idea of long write is that each part of data is continuation
of previous one. There shall be not gaps in the offsets between.

If there are gaps in offset then we have reliable write rather than
long write

shared/gatt-client: Forward events from parent to clones

Some events such as ready and service changed shall be forward to clones
since the clones may have set callbacks as well.

shared/gatt-client: Make it possible to clone non-ready clients

This is necessary in order to be possible to send commands while the
discovery is in progress.

core/gatt-client: Fix crash on service changed

The following crash can be observed when a service is removed from the
remote device leading to remove a client but instead a different
pointer was given causing which may leave the notification queue with
invalid clients:

Invalid read of size 8
  at 0x4A80D1: remove_client (gatt-client.c:1311)
  by 0x4DA16E: queue_remove_all (queue.c:351)
  by 0x4A7DF3: unregister_characteristic (gatt-client.c:1331)
  by 0x4DA16E: queue_remove_all (queue.c:351)
  by 0x4A7D65: unregister_service (gatt-client.c:1447)
  by 0x4DA0D5: queue_remove_all (queue.c:336)
  by 0x4AB11C: btd_gatt_client_service_removed (gatt-client.c:1747)
  by 0x4AFEA3: gatt_service_removed (device.c:3448)
  by 0x4E8FD1: handle_notify (gatt-db.c:249)
  by 0x4D9DC3: queue_foreach (queue.c:220)
  by 0x4E9ECB: notify_service_changed (gatt-db.c:266)
  by 0x4E9F7C: gatt_db_service_destroy (gatt-db.c:277)
Address 0x8cf8620 is 0 bytes inside a block of size 32 free'd
  at 0x4C29E00: free (vg_replace_malloc.c:530)
  by 0x4A9405: register_notify (gatt-client.c:1669)
  by 0x4D9DC3: queue_foreach (queue.c:220)
  by 0x4AAFBB: btd_gatt_client_connected (gatt-client.c:1714)
  by 0x4B2DAA: gatt_client_init (device.c:4591)
  by 0x4B2DAA: device_attach_att (device.c:4705)
  by 0x4B4712: att_connect_cb (device.c:4742)
  by 0x47375B: connect_cb (btio.c:232)
  by 0x50C8E39: g_main_context_dispatch (in /usr/lib64/libglib-2.0.so.0.4600.2)
  by 0x50C91CF: ??? (in /usr/lib64/libglib-2.0.so.0.4600.2)
  by 0x50C94F1: g_main_loop_run (in /usr/lib64/libglib-2.0.so.0.4600.2)
  by 0x40C258: main (main.c:687)

tools: make parse_companies.pl more forgiving of weird HTML

Several company identifier lines do not end in a </td> but rather <br/>
followed by newline followed by </td>. This dirty hack is more forgiving
of HTML weirdnesses in the SIGs company identifiers page.

unit/test-crypto: Add test based on sample data from errata 4243

Release 5.38

monitor: Update vendor defined 16-bit UUID entries

lib: Update company identifiers