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

monitor: Fix L2CAP connection response code decoding

The Connect Response, LE Connect Response and Create Channel Response
PDUs have to some extent similar, but still different response code
meanings. Add dedicated decoding functions for these depending on the
context.

tools/l2cap-tester: Add invalid LE SCID test case

core/service: Fix failing to connect external profile

When initiating a connection to a external profile btd_service_connect
will be called which will change the service state to
BTD_SERVICE_STATE_CONNECTING but then once the connection completes
service_accept is called but since it now checks the state it would
return -EALREADY to prevent driver accept to be called more than once.

tools/mgmt-tester: Remove unnecessary define

There's no need to have a define for opcode 0 since all of the structs
needing this are declared static and will have the value set to 0 if
not explicitly mentioned.

core: Remove dead code

This removes dead code due to memory allocation with new0 not being
able to fail.

android: Remove dead code

This removes dead code due to memory allocation with new0 not being
able to fail.

shared: Remove dead code

This removes dead code due to memory allocation with new0 not being
able to fail.

shared: Make new0 abort on failure

New is used to allocate small (typically much less than 1 page) chunks
and if such allocation fails system is most likely in state where
recovery is unlikely. Also by default Linux follows an optimistic memory
allocation strategy with OOM killer.

Aborting on allocationg failure allows to significantly simplify error
paths (which were most likely never tested anyway) and thus makes code
easier to understand.

btd_malloc name is used as malloc wrapper so that it can be exported
by bluetoothd and used also in external plugins.

Release 5.36

lib: Fix UTF-8 characters in company name

build: Update library version

lib: Update company identifiers

monitor: Add support for handling SCM_CREDENTIALS if present

tools/hciconfig: Fix hci_close_dev

It had been closed different file descriptor.
When has two or more interfaces, hciconfig -a exit after cmd_class
function.

unit: Fix gobex unit tests

/gobex/test_header_encode_connid test name was duplicated.
This results in Glib assert:
(unit/test-gobex-header:15356): GLib-ERROR **: duplicate test
    case path: /gobex/test_header_encode_connid

android/README: Update known Android issues section

This adds a patch which allows handling ssp passkey request.

core/device: Fix not claiming attributes of drivers

Driver loaded during init do not call add_gatt_service since the
attributes are only loaded during the first connection which means the
attributes need to be claimed on add_gatt_service but it was returning
if the driver was already probed causing the attribute to be exported
over D-Bus.

core/service: Fix calling driver .accept multiple times

service_accept shall not call driver .accept callback multiple times.

profiles/deviceinfo: rewrite deviceinfo profile

This patch include new version of deviceinfo profile. It is now
not triggering autoconnect. It's also using accept callback instead
of btd_device_add_attio_callback.

profiles/scanparam: rewrite scanparam profile

This patch rewrites scanparam profile. It will no longer trigger
autoconnect.

systemd: Check if bluetooth is supported in the kernel

When running a kernel without bluetooth support, bluetooth.service fails to
start with

    bluetoothd[1640]: Failed to access management interface
    bluetoothd[1640]: Adapter handling initialization failed
    systemd[1]: bluetooth.service: Main process exited, code=exited, status=1/FAILURE
    systemd[1]: Failed to start Bluetooth service.

This causes an unnecessary "degraded" state and more importantly breaks package
installation when the bluez package auto-starts the daemon.

Add a condition to only start the service if /sys/class/bluetooth exists.

https://launchpad.net/bugs/1506774

audio: Fix mtu input order

This patch fixes imtu,omtu input order in avdtp_stream_set_transport().

core/device: Fix build on some platforms

due to type promotions in C this code was failing on some platforms.

src/device.c:2016:8: error: format specifies type 'unsigned short' but the
      argument has type 'uint8_t' (aka 'unsigned char') [-Werror,-Wformat]
  properties, uuid_str);
  ^~~~~~~~~~
/build/falco/usr/include/bits/stdio2.h:39:7: note: expanded from macro 'sprintf'
                           __VA_ARGS__)
                           ^

gap/gas: Remove unused field

db_id is no longer necessary since gatt_db_register is no longer called.

gap/gas: Remove code handling service added/removed

The will take care of calling probe/remove when a matching service is
added removed.

core/device: Be consistent with function names

Rename device_remove_gatt_profile to device_remove_gatt_service.

core/device: Fix not calling accept callback

For GATT based services accept is used to indicate that there is a new
connection available so it needs to be called immediately once ATT
connects so the driver can register for notification, etc.

This also renames probe_gatt_profile to add_gatt_service which is what
in fact it was doing.

core: Fix double ATT connect() upon failure

If we fail connecting ATT we should not attempt to call
device_browse_gatt(). This would only trigger a new connect attempt
which would likely also fail.

monitor: Handle the Intel memory write and secure send correctly

monitor: Add more Intel specific debugging events

monitor: Check length when decoding extended LMP opcodes

monitor: Add detailed decoding support for various Intel commands

obexd: pbap: add headers correctly for size query

When client queries for the size of a phonebook we fall into a
indefinite loop as g_obex_apparam_encode always returns the same
number of items added to the buffer regardless how often it is
called. In former times where this code wasn't using GObexApparams
a array was reduced each time the headers where added and so we could
easily find out when we've added all headers. However today we need
to solve this a bit differently by also setting the firstpacket flag
when we receive the phonebook size result from the phonebook
implementation which then lets us correctly go through without
falling into a indefinite loop.

src/profile: Fix segmentation fault

The change that made the 'deviceinfo' profile "external" had an side
effect, now it is possible to have an external profile without 'owner'
and 'path' information. The fix considers that having an external
profile without that information is not an error.

Valgrind log:

bluetoothd[9974]: src/adapter.c:adapter_service_insert() /org/bluez/hci0
bluetoothd[9974]: src/adapter.c:add_uuid() sending add uuid command for index 0
bluetoothd[9974]: Endpoint registered: sender=:1.38 path=/MediaEndpoint/A2DPSink
bluetoothd[9974]: src/profile.c:register_profile() sender :1.38 path /Profile/HSPAGProfile
==9974== Invalid read of size 1
==9974==    at 0x65F21E0: __strcmp_sse2_unaligned (in /usr/x86_64-pc-linux-gnu/lib/libc-2.22.so)
==9974==    by 0x4E6C7E8: g_str_equal (in /usr/x86_64-pc-linux-gnu/lib/libglib-2.0.so.0.4600.1)
==9974==    by 0x467917: find_ext_profile (profile.c:745)
==9974==    by 0x469A8D: register_profile (profile.c:2373)
==9974==    by 0x4889C3: process_message.isra.5 (object.c:259)
==9974==    by 0x518E33E: _dbus_object_tree_dispatch_and_unlock (in /usr/x86_64-pc-linux-gnu/lib/libdbus-1.so.3.14.3)
==9974==    by 0x51805E3: dbus_connection_dispatch (in /usr/x86_64-pc-linux-gnu/lib/libdbus-1.so.3.14.3)
==9974==    by 0x4856DF: message_dispatch (mainloop.c:72)
==9974==    by 0x4E7C669: g_main_context_dispatch (in /usr/x86_64-pc-linux-gnu/lib/libglib-2.0.so.0.4600.1)
==9974==    by 0x4E7C9E7: g_main_context_iterate.isra.29 (in /usr/x86_64-pc-linux-gnu/lib/libglib-2.0.so.0.4600.1)
==9974==    by 0x4E7CD01: g_main_loop_run (in /usr/x86_64-pc-linux-gnu/lib/libglib-2.0.so.0.4600.1)
==9974==    by 0x40BABE: main (main.c:661)
==9974==  Address 0x0 is not stack'd, malloc'd or (recently) free'd
==9974==
==9974==
==9974== Process terminating with default action of signal 11 (SIGSEGV)
==9974==  Access not within mapped region at address 0x0
==9974==    at 0x65F21E0: __strcmp_sse2_unaligned (in /usr/x86_64-pc-linux-gnu/lib/libc-2.22.so)
==9974==    by 0x4E6C7E8: g_str_equal (in /usr/x86_64-pc-linux-gnu/lib/libglib-2.0.so.0.4600.1)
==9974==    by 0x467917: find_ext_profile (profile.c:745)
==9974==    by 0x469A8D: register_profile (profile.c:2373)
==9974==    by 0x4889C3: process_message.isra.5 (object.c:259)
==9974==    by 0x518E33E: _dbus_object_tree_dispatch_and_unlock (in /usr/x86_64-pc-linux-gnu/lib/libdbus-1.so.3.14.3)
==9974==    by 0x51805E3: dbus_connection_dispatch (in /usr/x86_64-pc-linux-gnu/lib/libdbus-1.so.3.14.3)
==9974==    by 0x4856DF: message_dispatch (mainloop.c:72)
==9974==    by 0x4E7C669: g_main_context_dispatch (in /usr/x86_64-pc-linux-gnu/lib/libglib-2.0.so.0.4600.1)
==9974==    by 0x4E7C9E7: g_main_context_iterate.isra.29 (in /usr/x86_64-pc-linux-gnu/lib/libglib-2.0.so.0.4600.1)
==9974==    by 0x4E7CD01: g_main_loop_run (in /usr/x86_64-pc-linux-gnu/lib/libglib-2.0.so.0.4600.1)
==9974==    by 0x40BABE: main (main.c:661)
==9974==  If you believe this happened as a result of a stack
==9974==  overflow in your program's main thread (unlikely but
==9974==  possible), you can try to increase the size of the
==9974==  main thread stack using the --main-stacksize= flag.
==9974==  The main thread stack size used in this run was 8388608.
==9974==

core/gatt: Fix logging invalid device address

This fix extra ':' at the end of device address if service interface
cannot be registered.

core/device: Fix crash when disconnecting

When device disconnect while GATT discovery is ongoing the following
crash can happen if the device is destroyed in case it is temporary:

Invalid read of size 4
   at 0x40A707C: g_slist_find_custom (in /usr/lib/libglib-2.0.so.0.4400.1)
   by 0x80B7382: gatt_service_removed (device.c:3422)
   by 0x80DF2FB: handle_notify (gatt-db.c:266)
   by 0x80D3B8E: queue_foreach (queue.c:251)
   by 0x80DF981: notify_service_changed (gatt-db.c:283)
   by 0x80DF981: gatt_db_service_destroy (gatt-db.c:294)
   by 0x80D3DFE: queue_remove_all (queue.c:387)
   by 0x80DFA4C: gatt_db_clear (gatt-db.c:438)
   by 0x80D908C: init_fail (gatt-client.c:1524)
   by 0x80D9A52: discovery_op_unref (gatt-client.c:419)
   by 0x80E18F2: destroy_mtu_op (gatt-helpers.c:495)
   by 0x80D7F6E: cancel_att_send_op (att.c:212)
   by 0x80D7F6E: bt_att_cancel (att.c:1238)
   by 0x80DB26A: bt_gatt_client_cancel_all (gatt-client.c:1962)
 Address 0x5536b58 is 0 bytes inside a block of size 8 free'd
   at 0x402B2C9: free (in /usr/lib/valgrind/vgpreload_memcheck-x86-linux.so)
   by 0x408CE4F: g_free (in /usr/lib/libglib-2.0.so.0.4400.1)
   by 0x40A6772: g_slice_free_chain_with_offset (in /usr/lib/libglib-2.0.so.0.4400.1)
   by 0x40A6B6B: g_slist_free (in /usr/lib/libglib-2.0.so.0.4400.1)
   by 0x40A7374: g_slist_free_full (in /usr/lib/libglib-2.0.so.0.4400.1)
   by 0x80B6CE0: device_free (device.c:624)
   by 0x80CFCCB: remove_interface (object.c:658)
   by 0x80D0208: g_dbus_unregister_interface (object.c:1382)
   by 0x80BC281: btd_device_unref (device.c:5942)
   by 0x80A9ACA: btd_adapter_remove_device (adapter.c:1165)
   by 0x80A9C52: adapter_remove_connection (adapter.c:5624)
   by 0x80AA77F: dev_disconnected (adapter.c:6584)

monitor: Fix firmware version numbering for Broadcom chips

monitor: Update handling for vendor commands and events

tools: Add p_ prefix to parser field helper functions

monitor: Print vendor specific string before vendor command

monitor: Don't print index label for HCI_DEV_NONE messages

monitor: Add support for handling Intel vendor commands

monitor: Add support for handling Broadcom vendor commands

monitor: Add the missing LL opcodes for Bluetooth 4.1 and 4.2 features

monitor: Update controller address from Read BD Addr command

btsnoop: Fix timestamp assignment for little-endian packet logger format

btsnoop: Add support for little-endian packet logger (pklg) format

deviceinfo: Enable external flag

This enable attributes to be accessed over D-Bus.

monitor/avctp: Decode player features

This decodes the bits related to browsing channel which is useful to
debug:

> ACL Data RX: Handle 256 flags 0x02 dlen 68
      Channel: 69 len 64 ctrl 0x0100 [PSM 27 mode 3] {chan 5}
      I-frame: Unsegmented TxSeq 0 ReqSeq 1
      AVCTP Browsing: Response: type 0x00 label 0 PID 0x110e
        AVRCP: GetFolderItems: len 0x0036
          Status: 0x04 (Success)
          UIDCounter: 0x0000 (0)
          NumOfItems: 0x0001 (1)
          Item: 0x01 (Media Player)
          Length: 0x002e (46)
          PlayerID: 0x0001 (1)
          PlayerType: 0x0001 (Audio)
          PlayerSubType: 0x00000000 (None)
          PlayStatus: 0x01 (PLAYING)
          Features: 0x00000000003700050000000000000000
            Advanced Control Player
            Browsing
            NowPlaying

audio/avrcp: Fix ctype code of PlayItem and AddToNowPlaying

Both PlayItem and AddToNowPlaying should use control type not status.

monitor: Decode additional Intel trace information

monitor: Add support for decoding Intel LMP / LL traces

monitor: Update manufacturer information for local version information

monitor: Add support for decoding LMP clock offset response

core/device: Fix disconnect for connect attempt

Right now calling Disconnect from DBus on device that's not advertising
or out of sight will not stop the connect attempt in kernel. This patch
fixes that by making sure that att_io is properly cleaned up on
disconnect.

monitor: Add support for decoding LMP switch request

monitor: Add support for decoding LMP slot offset transaction

monitor: Decode LMP preferred rate indication

monitor: Decode the LMP name request procedure

monitor: Add support for Broadcom diagnostic channel decoding

monitor: Allow padded LL messages from diagnostic channel

monitor: Print LMP master vs slave transaction identification

monitor: Allow padded LMP messages from diagnostic channel

monitor: Fix typo with LMP channel classification parameter

monitor: Forward correct manufacturer value to vendor events

monitor: Fix array boundary issue with version information

monitor: Add support for vendor diagnostic packets

monitor: Add support for showing index information updates

core/advertising: Fix string match in le adv search

Instead of searching for matched entry in the list, the function
match_advertisement was searching for the first non-matched entry, as a
result, it wasn't possible to unregister existing advertisement from LE
Adv manager.

emulator: Refactor le set scan enable command handler

le_set_scan_enable_complete should be called as post hook action.

emulator: Enable Slave-initiated Features Exchange feature

emulator: Add support for LE Remote Connection Parameter Request Negative Reply

< HCI Command: LE Connection Update (0x08|0x0013) plen 14
                                                 [hci1] 915.472416
        Handle: 42
        Min connection interval: 0.00 msec (0x0000)
        Max connection interval: 18.75 msec (0x000f)
        Connection latency: 0x0000
        Supervision timeout: 32000 msec (0x0c80)
        Min connection length: 0.625 msec (0x0001)
        Max connection length: 0.625 msec (0x0001)
> HCI Event: Command Status (0x0f) plen 4         [hci1] 915.472432
      LE Connection Update (0x08|0x0013) ncmd 1
        Status: Success (0x00)
> HCI Event: LE Meta Event (0x3e) plen 11         [hci0] 915.472443
      LE Remote Connection Parameter Request (0x06)
        Handle: 42
        Min connection interval: 0.00 msec (0x0000)
        Max connection interval: 18.75 msec (0x000f)
        Connection latency: 0x0000
        Supervision timeout: 32000 msec (0x0c80)
< HCI Command: LE Remote Connection Parameter Request Negative Reply
(0x08|0x0021) plen 3       [hci0] 915.474095
        Handle: 42
        Reason: Invalid LMP Parameters / Invalid LL Parameters (0x1e)
> HCI Event: Command Complete (0x0e) plen 6        [hci0] 915.474119
      LE Remote Connection Parameter Request Negative Reply
(0x08|0x0021) ncmd 1
        Status: Success (0x00)
        Handle: 42
> HCI Event: LE Meta Event (0x3e) plen 10          [hci1] 915.474124
      LE Connection Update Complete (0x03)
        Status: Invalid LMP Parameters / Invalid LL Parameters (0x1e)
        Handle: 42
        Connection interval: 4800.00 msec (0x0f00)
        Connection latency: 0.00 msec (0x0000)
        Supervision timeout: 327680 msec (0x8000)

LE Connection Update rejected due to invalid parameter:
Min connection interval: 0.00 msec (0x0000)

emulator: Add support for LE Remote Connection Parameter Request Reply

< HCI Command: LE Connection Update (0x08|0x0013) plen 14
                          [hci0] 92750.079447
        Handle: 42
        Min connection interval: 50.00 msec (0x0028)
        Max connection interval: 70.00 msec (0x0038)
        Connection latency: 0x0000
        Supervision timeout: 420 msec (0x002a)
        Min connection length: 0.000 msec (0x0000)
        Max connection length: 0.000 msec (0x0000)
> HCI Event: Command Status (0x0f) plen 4
                                  [hci0] 92750.079459
      LE Connection Update (0x08|0x0013) ncmd 1
        Status: Success (0x00)
> HCI Event: LE Meta Event (0x3e) plen 11
                                          [hci1] 92750.079462
      LE Remote Connection Parameter Request (0x06)
        Handle: 42
        Min connection interval: 50.00 msec (0x0028)
        Max connection interval: 70.00 msec (0x0038)
        Connection latency: 0x0000
        Supervision timeout: 420 msec (0x002a)
< HCI Command: LE Remote Connection Parameter Request Reply
(0x08|0x0020) plen 14   [hci1] 92750.079476
        Handle: 42
        Min connection interval: 50.00 msec (0x0028)
        Max connection interval: 70.00 msec (0x0038)
        Connection latency: 0x0000
        Supervision timeout: 420 msec (0x002a)
        Min connection length: 0.000 msec (0x0000)
        Max connection length: 0.000 msec (0x0000)
> HCI Event: Command Complete (0x0e) plen 6
  [hci1] 92750.079507
      LE Remote Connection Parameter Request Reply (0x08|0x0020)
ncmd 1
        Status: Success (0x00)
        Handle: 42
> HCI Event: LE Meta Event (0x3e) plen 10
                                          [hci0] 92750.079510
      LE Connection Update Complete (0x03)
        Status: Success (0x00)
        Handle: 42
        Connection interval: 50.00 msec (0x0028)
        Connection latency: 0.00 msec (0x0000)
        Supervision timeout: 420 msec (0x002a)
> HCI Event: LE Meta Event (0x3e) plen 10
                                          [hci1] 92750.079509
      LE Connection Update Complete (0x03)
        Status: Success (0x00)
        Handle: 42
        Connection interval: 50.00 msec (0x0028)
        Connection latency: 0.00 msec (0x0000)
        Supervision timeout: 420 msec (0x002a)
@ New Conn Param: 00:AA:01:00:00:23 (1) hint 0 min 0x0028 max 0x0038
latency 0x0000 timeout 0x002a

emulator: Remove duplicated assignment

Connection handle has been already assigned few lines earlier.

emulator: Fix LE Connection Complete Event data

This sets LE Connection Complete Event paramaters based on parameters
from LE Create Connection command.
Event parameters like Conn_Interval and Supervision_Timeout
shouldn't be set to 0.

monitor: Add support for open and close index events