profiles/AVRCP: Add AV Remote Controller service class id to CT

Both AV Remote and AV Remote Controller service classes are mandatory
by AVRCP CT role from 1.3 onwards. Also the assigned numbers page mention
that AV Remote must appear before AV Remote Controller:

  "The AVRCP specification v1.3 and later require that 0x110E also be
  included in the ServiceClassIDList before 0x110F for backwards
  compatibility."

lib/sdp: Replace VIDEO_CONF_SVCLASS_ID with AV_REMOTE_CONTROLLER_SVCLASS_ID

VCP apparently was never adopted so 0x110f was latter reused by AVRCP 1.3
controller as per current assigned numbers page:

https://www.bluetooth.org/en-us/specification/assigned-numbers/service-discovery

obexd/MAP: Fix parsing message handles as decimal numbers

The spec clearly states the handles are hexadecimal:

MAP 1.2 - Page 29

  ""handle" is the message handle in hexadecimal representation with up
  to 16 digits; leading zero digits may be used so the MCE shall accept
  both handles with and without leading zeros (e.g.,"00000012345678AB"
  or "12345678AB")."

obexd/MAP: Fix sending \0 after filler byte

It is not necessary to append \0 after the filler byte (0x30), the spec
just talk about the a byte not bytes.

core/profile: Fix MNS record version

To be able to use OBEX over L2CAP the version number should be at least
0x102, in addition to that ERTM mode should be used to comply to
GOEP 2.0.

monitor: Fix parsing of ATT Read By Type Response

monitor: Allow zero-length hex fields with print_hex_field()

monitor: Add missing EIR field length checks

monitor: Add decoding of L2CAP connectionless channel

monitor: Add structure for L2CAP connectionless channel

monitor: Add support for decoding 3D information data

monitor: Add decoding for major class audio/video values

lib: Add missing structure for accept physical link command

lib: Fix a few whitespace damages

lib: Update company identifiers

tools: Print PAL version for AMP controllers

lib: Add special functions for PAL version mapping

monitor: Differntiate between LMP and PAL version information

monitor: Fix broken assoc_fragment structure value

monitor: Fix wrong event code for channel selected event

obexd/session: Fix crash while disconnecting

Requests need to be cancelled when obc_session_shutdown is called
otherwise they can trigger the callback with invalid/freed data as in
the following backtrace:
Invalid read of size 8
   at 0x426684: setpath_cb (session.c:998)
   by 0x412AEB: handle_response (gobex.c:949)
   by 0x413010: incoming_data (gobex.c:1192)
   by 0x3D46047E05: g_main_context_dispatch (in /usr/lib64/libglib-2.0.so.0.3600.3)
   by 0x3D46048157: ??? (in /usr/lib64/libglib-2.0.so.0.3600.3)
   by 0x3D46048559: g_main_loop_run (in /usr/lib64/libglib-2.0.so.0.3600.3)
   by 0x40D59C: main (main.c:319)
 Address 0x571f598 is 40 bytes inside a block of size 56 free'd
   at 0x4A074C4: free (in /usr/lib64/valgrind/vgpreload_memcheck-amd64-linux.so)
   by 0x3D4604D9AE: g_free (in /usr/lib64/libglib-2.0.so.0.3600.3)
   by 0x426EA9: obc_session_shutdown (session.c:555)
   by 0x4254B4: remove_session (manager.c:62)
   by 0x43DC53: process_message.isra.5 (object.c:259)
   by 0x3D4981CE85: ??? (in /usr/lib64/libdbus-1.so.3.7.4)
   by 0x3D4980FA30: dbus_connection_dispatch (in /usr/lib64/libdbus-1.so.3.7.4)
   by 0x43A9D7: message_dispatch (mainloop.c:76)
   by 0x3D46048962: ??? (in /usr/lib64/libglib-2.0.so.0.3600.3)
   by 0x3D46047E05: g_main_context_dispatch (in /usr/lib64/libglib-2.0.so.0.3600.3)
   by 0x3D46048157: ??? (in /usr/lib64/libglib-2.0.so.0.3600.3)
   by 0x3D46048559: g_main_loop_run (in /usr/lib64/libglib-2.0.so.0.3600.3)

monitor: Fix typo in command constant

monitor: Add support for decoding location data commands

Bluetooth: Add definitions for location data commands

monitor: Add support for decoding flow control mode commands

Bluetooth: Add definitions for flow control mode commands

tools: Fix hciconfig device information printout for AMP controllers

monitor: Fix flow specification modify complete event name

monitor: Fix physical link disconnection complete event name

tools/l2cap-tester: Fix closing server sockets in time

We should use the teardown function instead of the destroy function for
closing server sockets since otherwise tests may conflict with each
other.

emulator: Add support for read local amp assoc command

emulator: Add some extra long option strings

emulator: Make unix server sockets optional

emulator: Exit with failure when Virtual HCI setup fails

emulator: Add command line option for AMP controllers

emulator: Add special Virtual HCI setup command

monitor: Use special hash and randomizer print functions

monitor: Mark hash and randomizer as P-192 versions

monitor: Update EIR and AD information elements

monitor: Fix EIR Data Type / AD Type assigned numbers

The values for Public Target Address and Random Target Address were
swapped. This information can be verified in the Bluetooth SIG Assigned
numbers webpage:

https://www.bluetooth.org/en-us/specification/assigned-numbers/generic-access-profile

monitor: Add translation of Bluetooth version information

android: Android version of log.c

Add logging for Android, currently print logs to stderr and stdout.

build: Add BlueZ Android daemon

android-build: Add BlueZ Android daemon

Define local mapping to glib path, otherwise this has to be inside central
place in the build repository.

android: Add skeleton of BlueZ Android daemon

build: Add skeleton for BlueZ Android

Retrieve Bluetooth version from configure.ac.

build-sys: Add missing $(AM_CFLAGS) to obexd_src_obexd_CFLAGS

This fix not passing extra build flags when --enable-maintainer-mode
is used.

obexd/MAP: Fix protected property value

"Sent" flag value was returned instead of "Protected" one.
This also fix following build error:

  CC     obexd/client/obexd-map.o
obexd/client/map.c:711:17: error: ‘get_protected’ defined but not
    used [-Werror=unused-function]
cc1: all warnings being treated as errors

obexd/client: Remove unused static functions

This fix following build error:

  CC     obexd/client/obexd-dbus.o
obexd/client/dbus.c:70:13: error: ‘append_array_variant’ defined but
    not used [-Werror=unused-function]
obexd/client/dbus.c:97:13: error: ‘append_dict_variant’ defined but
    not used [-Werror=unused-function]

client/transfer: Return "error" for unknown status in status2str

This can happen only if there is a bug in obexd code.
This fix following build error:

  CC     obexd/client/obexd-transfer.o
obexd/client/transfer.c: In function ‘status2str’:
obexd/client/transfer.c:277:1: error: control reaches end of non-void
    function [-Werror=return-type]

obexd/MAP: Fix missing include

This fix following build error:

  CC     obexd/client/obexd-map.o
obexd/client/map.c: In function ‘msg_element’:
obexd/client/map.c:1113:2: error: implicit declaration of function ‘strtoull’ [-Werror=implicit-function-declaration]

obexd/service: Remove unused local variable

This fix following build error:

  CC     obexd/src/obexd-service.o
obexd/src/service.c: In function ‘obex_service_driver_register’:
obexd/src/service.c:100:10: error: unused variable ‘l’ [-Werror=unused-variable]

obexd: Return "error" for unknown status in status2str

This can happend only if there is a bug in obexd code.
This fix following buld error:

  CC     obexd/src/obexd-manager.o
obexd/src/manager.c: In function ‘status2str’:
obexd/src/manager.c:292:1: error: control reaches end of non-void
    function [-Werror=return-type]

obexd: Remove unused local variables

This fix following build errors:

obexd/src/manager.c: In function ‘get_root’:
obexd/src/manager.c:253:23: error: unused variable ‘os’
    [-Werror=unused-variable]

obexd/src/manager.c: In function ‘manager_emit_transfer_started’:
obexd/src/manager.c:534:22: error: unused variable ‘id’
    [-Werror=unused-variable]

obexd/src/manager.c: In function ‘manager_request_authorization’:
obexd/src/manager.c:684:14: error: unused variable ‘type’
    [-Werror=unused-variable]
obexd/src/manager.c:683:14: error: unused variable ‘filename’
    [-Werror=unused-variable]

obexd: Remove unused static functions

This fix following build error:

obexd/src/manager.c: At top level:
obexd/src/manager.c:190:13: error:
     ‘dbus_message_iter_append_dict_entry’ defined but not used
     [-Werror=unused-function]

obexd/MAP: Remove unused mas_drivers variable

This fix following build error:

  CC     obexd/client/obexd-mns.o
obexd/client/mns.c:344:38: error: ‘mas_drivers’ defined but not used
    [-Werror=unused-variable]
cc1: all warnings being treated as errors

obexd/MAP: Fix missing include

This fix following build error:

  CC     obexd/client/obexd-mns.o
obexd/client/mns.c: In function ‘parse_event_report_handle’:
obexd/client/mns.c:187:2: error: implicit declaration of function
    ‘strtoull’ [-Werror=implicit-function-declaration]

obexd/MAP: Fix missing include

This fix following build error:

  CC     obexd/client/obexd-mns.o
obexd/client/mns.c: In function ‘mns_connect’:
obexd/client/mns.c:105:2: error: implicit declaration of function
    ‘manager_register_session’ [-Werror=implicit-function-declaration]
obexd/client/mns.c: In function ‘mns_disconnect’:
obexd/client/mns.c:128:2: error: implicit declaration of function
    ‘manager_unregister_session’ [-Werror=implicit-function-declaration]

obexd/bluetooth: Make dict_append_entry function static

This fix following build error:

  CC     obexd/plugins/obexd-bluetooth.o
obexd/plugins/bluetooth.c:242:6: error: no previous declaration for
     ‘dict_append_entry’ [-Werror=missing-declarations]

obexd/bluetooth: Remove unused local variables

This ix following build errors:

  CC     obexd/plugins/obexd-bluetooth.o
obexd/plugins/bluetooth.c: In function ‘register_profile_reply’:
obexd/plugins/bluetooth.c:202:10: error: unused variable ‘err’
    [-Werror=unused-variable]

obexd/plugins/bluetooth.c: In function ‘name_acquired’:
obexd/plugins/bluetooth.c:367:15: error: unused variable ‘uuid’
    [-Werror=unused-variable]

obexd/plugins/bluetooth.c: In function ‘name_released’:
obexd/plugins/bluetooth.c:389:15: error: unused variable ‘uuid’
    [-Werror=unused-variable]

obexd/plugins/bluetooth.c: In function ‘bluetooth_start’:
obexd/plugins/bluetooth.c:400:10: error: unused variable ‘ios’
    [-Werror=unused-variable]

obexd/MAP: Fix invalid snprintf format string in map_msg_get()

obexd/client/map.c: In function ‘map_msg_get’:
obexd/client/map.c:446:2: warning: format ‘%u’ expects argument of type
     ‘unsigned int’, but argument 4 has type ‘uint64_t’ [-Wformat]
obexd/client/map.c:446:2: warning: format ‘%u’ expects argument of type
     ‘unsigned int’, but argument 4 has type ‘uint64_t’ [-Wformat]

doc: Correct a few minor style issues

doc: Add missing title text to static address command

obexd/MAP: Store message handle in binary format

This is more efficient in terms of memory and hash lookups, it is also
not prone to string format bugs in remote stacks such as leading zeros
being treated as a different handle as can be experience with
Nokia N950 which sends events using a handle with leading zeros but
message listing don't have them.

obexd/session: Fix crash while processing command queue

session_process_queue can call a callback which can cause the session to
be freed:
Invalid write of size 4
   at 0x4265C9: session_process (session.c:716)
   by 0x3D46047E05: g_main_context_dispatch (in /usr/lib64/libglib-2.0.so.0.3600.3)
   by 0x3D46048157: ??? (in /usr/lib64/libglib-2.0.so.0.3600.3)
   by 0x3D46048559: g_main_loop_run (in /usr/lib64/libglib-2.0.so.0.3600.3)
   by 0x40D55C: main (main.c:319)
 Address 0x4d658a8 is 104 bytes inside a block of size 120 free'd
   at 0x4A074C4: free (in /usr/lib64/valgrind/vgpreload_memcheck-amd64-linux.so)
   by 0x3D4604D9AE: g_free (in /usr/lib64/libglib-2.0.so.0.3600.3)
   by 0x4265B1: session_process_queue (session.c:794)
   by 0x4265C8: session_process (session.c:714)
   by 0x3D46047E05: g_main_context_dispatch (in /usr/lib64/libglib-2.0.so.0.3600.3)
   by 0x3D46048157: ??? (in /usr/lib64/libglib-2.0.so.0.3600.3)
   by 0x3D46048559: g_main_loop_run (in /usr/lib64/libglib-2.0.so.0.3600.3)
   by 0x40D55C: main (main.c:319)

obexd: Prefix folders in event reports with leading slash

The internal representation of folder paths starts with a leading slash.
Therefore we have to prepend it to the folders received in event reports.

obexd: Handle message shift and message deleted events

Update Folder propery if the message has been shifted or deleted.

obexd: Update Status property in map documentation

For outgoing messages, the status field values are:
"delivery-success", "sending-success", "delivery-failure", "sending-failure"

obexd: Handle message status events

For outgoing messages, the message status is changed when an event
indicates that the sending/delivery has failed or succeeded.

obexd: Handle new message event

Register new org.bluez.obex.Message1 D-Bus object when a NewMessage
event report has arrived.

tools/l2cap-tester: Make setup_powered_server() enable LE advertising when necessary

tools/l2cap-tester: Rename generic test_bredr_server() to test_server()

tools/l2cap-tester: Fix passing correct psm to create_l2cap_sock()

tools/l2cap-tester: Separate server and client setup routines

tools/l2cap-tester: Add LE support

bnep: improve error-msg if bnep.ko is not loaded

bnep.ko, besides others, can be disabled in custom kernels if
network-support is not required. To avoid strange error messages, handle
EPROTONOSUPPORT as a special case and emit a warning that kernel support
is missing.

plugin: handle ENOSYS as not-supported

Allow plugins to return -ENOSYS during registration and handle it as
"not-supported" error. It makes the error messages slightly more useful in
case kernel-support is missing for a particular subsystem.

monitor: Fail on invalid command line parameters

monitor: Use different tables for BR/EDR and LE L2CAP signal channels

doc/mgmt-api: Add definition for Set Static Address

tools/btmgmt: Fix small copy&paste issue

tools/btmgmt: Add support for setting static address

lib: Add define for mgmt_set_static_address

tools/mgmt-tester: Fix expected response code for start_discovery case

obexd: Fix emitting Type property changed signals for messages

In order to determine if the message Type property has changed,
the stored type needs to be compared with the parsed type and not with
the raw value received from the MSE.

This fixes the issue that the property changed signal for the Type
property is emitted for every message on every ListMessage call.

sdp: Check for correct pkt size received instead of error

Check that the full header is received in recv().

lib: Fix invalid "whitespace" on company identifier string

Somehow, the invalid multibyte sequence "\xC2\xA0" (which is displayed
as a whitespace) was inserted on a company identifier string. Fix this
to avoid a bogus diff when running tools/update_compids.sh.

tools: Fix update_compids.sh to avoid non-ASCII output

Some distros have html2text patches that may generate non-ASCII output
even when -ascii is used. This patch adds another case (seen in Fedora)
where HTML entity   (non-breaking space) is converted into a
multibyte whitespace.

Also add a sanity check to make sure non-ASCII text is not introduced in
lib/bluetooth.c.

doc: Small beautification

tools/mgmt-tester: Add some basic set_bredr set cases

tools/btmgmt: Add support for set_bredr

lib: Add define for mgmt_set_bredr

doc/mgmt-api: Add Set BR/EDR command definition

tools/btmgmt: Fix minor whitespace issue

tools/mgmt-tester: Update expected settings to support recent HS changes

tools/mgmt-tester: Add test cases for enabling high speed

tools/mgmt-tester: Cleanup indentation of test cases

lib: Fix use of uninitialized variable in sdp_set_profile_descs

Error path on default case was not breaking loop. To keep error
handling similar all error path were converted to use goto.

This fix following:
target  C: libbluetooth <= external/bluetooth/bluez/android/../lib/sdp.c
lib/sdp.c: In function 'sdp_set_profile_descs':
lib/sdp.c:487:10: warning: 'values[0]' may be used uninitialized in
    this function [-Wmaybe-uninitialized]
lib/sdp.c:2562:19: note: 'values[0]' was declared here
lib/sdp.c:545:11: warning: 'dtds[0]' may be used uninitialized in this
    function [-Wmaybe-uninitialized]
lib/sdp.c:2562:9: note: 'dtds[0]' was declared here

test/test-hfp: Fix passing missing backlog parameter to sco.listen()

obexd/client: Fix not setting Transfer.Size for GET operations

GET operations may return the length header in the first response which
should be set as transfer Size property.