monitor: Fix use of uninitialized variable

subevent code was never set in vendor_evt() resulting in printing
random stack data as subevent opcode in print_subevent().

tools/mgmt-tester: Add power on testcase with adv and privacy

This test verifies if kernel can properly enable advertising during
power on with privacy also enabled. This requires to generate new RPA
which will fail if not done after SMP is registered. As a consequence
advertising is not enabled in controller which means it cannot be also
disabled (controller will reject HCI command).

core: Fix replying with an error for connected profile

If the profile(s) is already connected don't reply with an error, if the
profile is in fact connecting already just wait until it completes.

tools: Add support for Get Advertising Size Information command

lib: Add data structures for Get Advertising Size Information command

doc: Mention the management version 1.11 information

tools/mgmt-tester: Fix minor typo

tools/mgmt-tester: Simplify command result log

lib/mgmt: Update string tables

tools/mgmt-tester: Take advantage of mgmt string helpers

build: Link mcaptest with -lrt

profiles/health/mcap.c uses clock_gettime(3) so it needs to be linked
with -lrt on some systems.

doc: Fix missing field for Get Advertising Size Information response

monitor: Add missing Intel Startup event

doc: Add description for Add Advertising Size Information command

AUTHORS: Update Szymon, Łukasz and Andrzej emails

Szymon, Łukasz and Andrzej are now working for CODECOUP.

test/test-discovery: Remove dead code

PropertyChanged is left over from BlueZ 4.x now it is PropertiesChanged.
However in this test it does not make sense to register for this signal
as StopDiscovery is never called from this test.

doc/adapter-api: Use correct signal name

emulator/hciemu: Fix return value

make throws out the following errors.

emulator/hciemu.c: In function ‘hciemu_get_master_scan_enable’:
emulator/hciemu.c:433:3: error: return makes integer from pointer without a cast [-Werror]
   return NULL;
   ^
emulator/hciemu.c: In function ‘hciemu_get_master_le_scan_enable’:
emulator/hciemu.c:441:3: error: return makes integer from pointer without a cast [-Werror]
   return NULL;
   ^
cc1: all warnings being treated as errors

Fixed by returning zero.

tools/mgmt-tester: Wait before checking BR/EDR scan state

Like the LE scan state, recent kernels moved to updating the BR/EDR
scan state for Add/Remove Device independent from the mgmt command
completion. We should therefore update the corresponding mgmt-tester
cases to wait a bit before making a test result conclusion.

emulator: Add API to get BR/EDR scan_enable value

emulator: Return raw LE scan_enable value

This is more future proof (in case other values besides 0x00/0x01 are
introduced) and consistent with a similar BR/EDR API that will be
added soon.

tools/l2cap-tester: Fix minor coding style issue (line lengths)

tools/hcidump: Print LDAC vendor codec name

tools/avinfo: Print content protection

tools/avinfo: Add partial support for Sony LDAC

Contents of Vendor Specific Value in codec information is unknown at
the moment so only raw value is printed.

monitor/rfcomm: Fix names formatting

monitor/avctp: Fix typos

input/hog: Fix not reseting queue to NULL after destroying it

This can cause crashes as the data pointed by devices if a new device is
probed it would most likely access invalid memory.

android/README: Fix typo

android/README: Update links to BfA on GitHub

Since code.google.com is now read-only, sample implementation of BfA is
moved to GitHub. README is updated with new links.

input/hog-lib: Print report value handle and type

This adds better debug logs when discovering report type.

input/hog: Make use of hog-lib

This changes the input plugin to use hog-lib so the same code is used in
both D-Bus daemon, Android and unit tests.

hog: Move code from android to profiles

This is a place holder until the code is ported to use shared API so it
can be shared by android and D-Bus daemon.

dis: Move code from android to profiles

This is a place holder until the code is ported to use shared API so it
can be shared by android and D-Bus daemon.

scpp: Move code from android to profiles

This is a place holder until the code is ported to use shared API so it
can be shared by android and D-Bus daemon.

bas: Move code from android to profiles

This is a place holder until the code is ported to use shared API so it
can be shared by android and D-Bus daemon.

android/hog: Port fixes to profiles HoG

This align the 2 copies of Hog since the plan is to merge android
version back to profiles and just reuse from there.

The fix for incomplete discover is modified so the reports are keep
while attempting to discover.

shared/att: Use smaller defines

Not every term of the command needs to be part of the define as it make
them awful long.

shared/att: Fix going over 80 columns

This fixes lines that are over 80 columns by renaming op_type to just
type so it doesn't turn into multi line either.

core/device: Fix not responding to Connect

If profile is removed while its connection is pending it may not respond
to Connect method and not cleaning up properly leaving dev->connect set
which will prevent any attempt to connect to the device again.

client: Fix printing selcted device prompt

client: Fix crash when exiting

When exiting the available proxies are destroy in the same order they are
added causing the following crash when there are attributes whose service
has already been removed:

Invalid read of size 8
   at 0x414AAD: g_dbus_proxy_get_path (client.c:525)
   by 0x40B948: characteristic_is_child (gatt.c:136)
   by 0x40C420: gatt_remove_characteristic (gatt.c:157)
   by 0x4067A7: proxy_removed (main.c:446)
   by 0x414A2E: proxy_free (client.c:439)
   by 0x4E7AF6C: g_list_foreach (in /usr/lib64/libglib-2.0.so.0.4400.1)
   by 0x4E7AF8A: g_list_free_full (in /usr/lib64/libglib-2.0.so.0.4400.1)
   by 0x415D54: g_dbus_client_unref (client.c:1310)
   by 0x40511B: main (main.c:2067)
 Address 0x5eb5450 is 16 bytes inside a block of size 80 free'd
   at 0x4C29D6A: free (in /usr/lib64/valgrind/vgpreload_memcheck-amd64-linux.so)
   by 0x4E8479E: g_free (in /usr/lib64/libglib-2.0.so.0.4400.1)
   by 0x4149D6: g_dbus_proxy_unref (client.c:517)
   by 0x414A8D: proxy_free (client.c:451)
   by 0x4E7AF6C: g_list_foreach (in /usr/lib64/libglib-2.0.so.0.4400.1)
   by 0x4E7AF8A: g_list_free_full (in /usr/lib64/libglib-2.0.so.0.4400.1)
   by 0x415D54: g_dbus_client_unref (client.c:1310)
   by 0x40511B: main (main.c:2067)

core/gatt-client: Fix warning when removing services

This fixes the following warnings caused by not resetting id to 0:

GLib-CRITICAL **: Source ID 145 was not found when attempting to remove it
GLib-CRITICAL **: Source ID 151 was not found when attempting to remove it
GLib-CRITICAL **: Source ID 155 was not found when attempting to remove it
GLib-CRITICAL **: Source ID 162 was not found when attempting to remove it

tools/l2cap-tester: Fix NULL-dereference for tests with no data

client: Make GATT listing more readable

tools/l2cap-tester: Minor whitespace fix

doc/test-coverage: Update l2cap-tester test count

tools/l2cap-tester: Add test case for direct advertising

android/log: Fix compilation error in AOSP

btd_debug() has new parameter introduced in dcfdc556.

tools/l2cap-tester: Fix expected LE CoC conn rsp parameters

emulator: Fix sending valid HCI_LE_Create_Connection parameters

monitor: Update UUID strings to latest versions

tools/l2cap-tester: Don't treat unexpected L2CAP PDUs as failures

core: Use daemon specific GLib logging handling for backtraces

core: Add function for logging with priority information

tools/mgmt-tester: Fix Remove Device tests for LE

The timing of Add/Remove device is different in recent kernels so that
if they are sent in rapid succession no HCI command will be sent at
all. Since the relevant thing is that LE scan is disabled after the
test add a timeout to verify this instead of waiting for a specific
HCI command.

doc/test-coverage: Update l2cap-tester test count

tools/l2cap-tester connect two sockets disconnect one test

This test open two sockets to same client, then close first one and make
sure second one connect successfully.

tools/l2cap-tester: Two socket connect test

This test tries to open two sockets to same address, to make sure
both would succeed.

tools/l2cap-tester: Disconnect during connect attempt test

This patch adds test that checks folowing:
1. Try to connect to existing BLE device that is not advertising.
2. Internally in kernel this adds this device to whitelist and enable scan.
3. Make the device advertise once. That would stop the scan, and trigger
   connect attempt to device.
4. Close socket. This should stop the connect attempt, by sending
   BT_HCI_CMD_LE_CREATE_CONN_CANCEL.

tools/l2cap-tester: add close socket test

This patch adds test that checks folowing:
1. Try to connect to non-existing remote BLE device using socket.
2. Internally in kernel this adds this device to whitelist and enable scan.
3. At this moment test would try to close socket.
4. That should cause scan to be stopped.

emulator: add support for checking le scan state

shared/gatt-client: Don't reset database on service changed

If the remote send a full reset, 0x0001 - 0xffff, just discovery the
attributes again as device may send this even though nothing has
changed.

shared/mgmt: Fix crash when removing index

Because queue entries are no longer protected by a reference it is
necessary to return the use of in_notify flag, etc, otherwise the
following crash can happen when removing an index:

Invalid read of size 8
   at 0x41AD6F: queue_foreach (queue.c:219)
   by 0x41CA6C: process_notify (mgmt.c:280)
   by 0x41CA6C: can_read_data (mgmt.c:338)
   by 0x422DCA: watch_callback (io-glib.c:170)
   by 0x4E7EA89: g_main_context_dispatch (in /usr/lib64/libglib-2.0.so.0.4400.1)
   by 0x4E7EE1F: ??? (in /usr/lib64/libglib-2.0.so.0.4400.1)
   by 0x4E7F141: g_main_loop_run (in /usr/lib64/libglib-2.0.so.0.4400.1)
   by 0x422A31: tester_run (tester.c:830)
   by 0x403013: main (l2cap-tester.c:1489)
 Address 0x5754b38 is 8 bytes inside a block of size 16 free'd
   at 0x4C29D6A: free (in /usr/lib64/valgrind/vgpreload_memcheck-amd64-linux.so)
   by 0x41AFCF: queue_remove_if (queue.c:302)
   by 0x41B0BA: queue_remove_all (queue.c:331)
   by 0x41C6A2: mgmt_unregister_index (mgmt.c:737)
   by 0x405033: index_removed_callback (l2cap-tester.c:162)
   by 0x41B751: notify_handler (mgmt.c:270)
   by 0x41AD83: queue_foreach (queue.c:220)
   by 0x41CA6C: process_notify (mgmt.c:280)
   by 0x41CA6C: can_read_data (mgmt.c:338)
   by 0x422DCA: watch_callback (io-glib.c:170)
   by 0x4E7EA89: g_main_context_dispatch (in /usr/lib64/libglib-2.0.so.0.4400.1)
   by 0x4E7EE1F: ??? (in /usr/lib64/libglib-2.0.so.0.4400.1)
   by 0x4E7F141: g_main_loop_run (in /usr/lib64/libglib-2.0.so.0.4400.1)

lib: Fix missing error handling when SDP record is NULL

tools/mgmt-tester: Fix invalid settings for power off test case

The test that's supposed to verify correct "power off" error response
shouldn't be powering on the adapter in the setup stage.

core: Fix broken #ifdef around backtrace support

core: Initialize the backtracer early on at start

tools/valgrind: Add suppression for __btd_log_init

valgrind seems to treat every bind to PF_BLUETOOTH as an RFCOMM socket
causing the following false positives:

Syscall param socketcall.bind(my_addr.rc_bdaddr) points to uninitialised byte(s)
   at 0x52728A7: bind (in /usr/lib64/libc-2.21.so)
   by 0x4045B1: logging_open (log.c:76)
   by 0x4045B1: __btd_log_init (log.c:298)
   by 0x402823: main (test-avctp.c:278)
 Address 0xffefffcb6 is on thread 1's stack
 in frame #1, created by __btd_log_init (log.c:290)
 Uninitialised value was created by a stack allocation
   at 0x404490: __btd_log_init (log.c:290)

Syscall param socketcall.bind(my_addr.rc_channel) points to uninitialised byte(s)
   at 0x52728A7: bind (in /usr/lib64/libc-2.21.so)
   by 0x4045B1: logging_open (log.c:76)
   by 0x4045B1: __btd_log_init (log.c:298)
   by 0x402823: main (test-avctp.c:278)
 Address 0xffefffcb8 is on thread 1's stack
 in frame #1, created by __btd_log_init (log.c:290)
 Uninitialised value was created by a stack allocation
   at 0x404490: __btd_log_init (log.c:290)

core/profile: Fix possible crash when registering profiles

Profiles under ext_profiles are local and should no be mixed with other
external profiles since its type is different which can may cause invalid
memory when accessing member of ext_profile struct.

shared/queue: Remove reference count from queue_entry

Since the queue itself has support for reference and queue_foreach
actually make use of it to prevent crashes when queue_destroy is
called from the callback.

Note: The intention of reference count was to protect the entry itself
in queue_foreach, it may still be possible to have a crash if the next
entry if freed but that should be considered an implementation bug.

obexd/session: Fix crash when disconnecting

If session owner disconnect from the bus while g_obex_connect is pending
it may lead to a crash since it is never canceled connected_cb may still
be called after callback_data is freed.

core: Add missing HAVE_CONFIG_H for shared GATT code

core: Use local assert function where possible

core: Add support for basic stack backtrace helper

core: Start SDP server only for BR/EDR or dual mode operation

monitor: Adjust colors for system note and user logging messages

monitor: Add support --priority option to getopt parsing

core: Use constant for logging ident string and its length

monitor: Use btsnoop priority constants for user logging

btsnoop: Add constants for Bluetooth monitor fields

btsnoop: Update file format identifiers for trace files

monitor: Allow using "debug" string for setting priority level

monitor: Add support for user logging priority selection

monitor: Do not abort user logging for invalid index messages

core: Use adapter->dev_id for adapter specific logging

core: Add logging functions for controller specific messages

monitor: Use cleaner decoding for system note and user logging packets

core: Add support for user logging with ident information

monitor: Add support for handling user logging with ident

monitor: Support obmission of label when printing packets

obexd: Use common logging function for daemon version

core: Use common logging function for daemon version

core: Add support for logging output to HCI_CHANNEL_LOGGING

lib: Add definition for HCI_CHANNEL_LOGGING

monitor: Add support for decoding user logging packets

monitor: Add support for system note packet type

shared/gatt-client: Fix use after free

This fixes the following crash which happens when a service changed
removes characteristics which have notification pending:

==42544== Invalid read of size 4
==42544==    at 0x4939FA: enable_ccc_callback (gatt-client.c:1163)
==42544==    by 0x490D37: handle_rsp (att.c:673)
==42544==    by 0x490D37: can_read_data (att.c:845)
==42544==    by 0x498ED4: watch_callback (io-glib.c:170)
==42544==    by 0x4E7EE39: g_main_context_dispatch (in /usr/lib64/libglib-2.0.so.0.4600.1)
==42544==    by 0x4E7F1CF: ??? (in /usr/lib64/libglib-2.0.so.0.4600.1)
==42544==    by 0x4E7F4F1: g_main_loop_run (in /usr/lib64/libglib-2.0.so.0.4600.1)
==42544==    by 0x40B12E: main (main.c:661)
==42544==  Address 0x7e4eaa8 is 8 bytes inside a block of size 32 free'd
==42544==    at 0x4C29D6A: free (in /usr/lib64/valgrind/vgpreload_memcheck-amd64-linux.so)
==42544==    by 0x48C96D: queue_remove_all (queue.c:360)
==42544==    by 0x491AA2: gatt_client_remove_notify_chrcs_in_range (gatt-client.c:331)
==42544==    by 0x491AA2: process_service_changed (gatt-client.c:1398)
==42544==    by 0x48C74D: queue_foreach (queue.c:239)
==42544==    by 0x4936B2: notify_cb (gatt-client.c:1614)
==42544==    by 0x490BC6: handle_notify (att.c:800)
==42544==    by 0x490BC6: can_read_data (att.c:881)
==42544==    by 0x498ED4: watch_callback (io-glib.c:170)
==42544==    by 0x4E7EE39: g_main_context_dispatch (in /usr/lib64/libglib-2.0.so.0.4600.1)
==42544==    by 0x4E7F1CF: ??? (in /usr/lib64/libglib-2.0.so.0.4600.1)
==42544==    by 0x4E7F4F1: g_main_loop_run (in /usr/lib64/libglib-2.0.so.0.4600.1)
==42544==    by 0x40B12E: main (main.c:661)

btsnoop: Identify pklg vendor diagnostic packets

doc/test-coverage: Update mgmt-tester & l2cap-tester test counts