doc: Fix optional properties not marked as such

Commit 826023de56896e83f35ae69a73d6a120a2f5b11e made some of the
device's properties optional but the documentation was not updated
accordingly. More specifically, the properties 'Name' and 'UUIDs' won't
be present if the information is not available.

monitor: Add an option to dump SCO traffic

attrib: Change match_event return type from gboolean

There is no reason to use gboolean as return type of match_event.

attrib: Change is_response return type from gboolean

There is no reason to use gboolean as return type of is_response.

attrib: Change command->sent type from gboolean

There is no reason to command->sent be gboolean.

attrib: Change GAttrib->stale type from gboolean

There is no reason to GAttrib->stale be gboolean.

attrib: Simplify boolean tests

Remove unnecessary TRUE/FALSE in boolean tests.

tools/mgmt-tester: Fix expect_len assignment to use right variable

tools/mgmt-tester: Add valid test case for LE-only discovery

tools/mgmt-tester: Update LE-only test case pre-conditions

Starting with Linux 3.10 single mode controllers will always have the LE
setting bit enabled in the current_settings mask.

tools: Fix avrcp_media_element_item_dump to print elements properly

core: Delay Paired property until SDP completes

To make it easier for clients to know when SDP has completed, and to
make the property consistent with the timing of the Device1.Pair()
method return it makes sense to delay the "Paired" property change until
SDP has completed.

input: Automatically attempt a reconnect when required.

The HID 1.1 spec requires a host to attempt a reconnection when a
HID device goes out of range and comes back. There is test (see
TP/HCE/BV-04-I [Host Initiated Re-connection]) to ensure that the
host initiates the connection when that happens.

This patch adds a reconnection timer for HID devices trying to
reconnect every 30s for a maximum of 3 minutes after the connection
to a HID device with a ReconnectMode of "host" or "any" is closed.

input: Fix memory leak for hidp_connadd_req.

If the struct input_device is destroyed while a hidp_connad_req
is pending, the input_device will be destroyed leaking the associated
memory.

==30790== 492 (168 direct, 324 indirect) bytes in 1 blocks are definitely lost in loss record 199 of 216
==30790==    at 0x4C29DB4: calloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==30790==    by 0x4E7FAE0: g_malloc0 (gmem.c:189)
==30790==    by 0x4378E4: hidp_add_connection (device.c:365)
==30790==    by 0x438055: input_device_connected (device.c:525)
==30790==    by 0x4380DA: interrupt_connect_cb (device.c:548)
==30790==    by 0x443508: connect_cb (btio.c:230)
==30790==    by 0x4E79D52: g_main_context_dispatch (gmain.c:2539)
==30790==    by 0x4E7A09F: g_main_context_iterate.isra.23 (gmain.c:3146)
==30790==    by 0x4E7A499: g_main_loop_run (gmain.c:3340)
==30790==    by 0x448CCC: main (main.c:583)

input: Convert gboolean to bool

Remove some usage of gboolean in favor of bool.

input: Implement the new ReconnectMode Input1 property.

The "Connectability" of a HID device, as defined by the HID spec,
governs the way the host may and should connect to a HID device or
expect a connection from it. In the comon case of mice and keyboards
the HIDNormallyConnectable is FALSE and HIDReconnectInitiate is TRUE,
since those devices only attempt a connection to the host when they
have some data to transfer. A connection attempt initiated from the
host after the device drops the connection (while still paired) will
result in a Page timeout.

This patch exposes a new property called "ReconnectMode" combining the
those two SDP attributes as shown in the Connectability section of the
HID spec (see section 5.4.2). The property can have one of the following
four values: "None", "Device", "Host", "Any", and is derived from the
SDP cached value on device creation even if the device is off.

input: Documentation for new Input1 interface

Adds documentation for a new Input1 interface explaining a new
"ReconnectMode" property that exposes the Connectability mode of
a HID device.

tools: Fix AVRCP parsing of GetFolderItems for folders and items

Length of name is 2 bytes not 1

emulator: Remove extended features from LE-only controllers

emulator: Move Event Mask Page 2 from BR/EDR to BR/EDR/LE

emulator: Add proper supported commands response values

emulator: Add missing command support checks

emulator: Add basic HCI_Set_Event_Mask_Page_2 support

emulator: Add sync train parameter reading support

emulator: Fix extended features invalid params response

monitor: Fix missing byte order conversions

monitor: Add support for decoding read sync train params response

obexd: Fix make distcheck

../obexd/client/map.c:43:17: fatal error: sdp.h: No such file or directory

tools: Fix mpris-player not forwarding method calls properly

This fixes not sending methods calls to players, it now copies the
contents of the original message into the copy instead of relying on
dbus_message_copy and send the reply back.

emulator: Add support for local features page 2

sdptool: Use macro to calculate array size

tools: Add MAS profile attributes

test: Fix opp-client not printing progress

This avoid KeyError expection and fixes print usage.

obexd: Read service attributes when instantiating MAP session

Parse the service attributes mas_instance_id and supported_message_types
from the transport's service attributes as soon as a connection is
established.

lib: Add attributes MASInstanceID and SupportedMessageTypes

obexd: Add obc_session_get_attribute to session

This adds a obc_session_get_attribute function to the client session.
The function allows querying the session specific service attributes.

obexd: Add bluetooth_getattribute to Bluetooth transport

This function makes the cached SDP attributes available for the profile
implementations.

obexd: Add getattribute to obc_transport

This adds a getattribute function pointer to obc_transport, which allows
transports to provide the session specific service attributes.

obexd: Cache SDP record in bluetooth transport

This caches the SDP record of the active session in the Bluetooth
transport.

gatttool: Use error message prefixes consistently

Replace rl_printf() calls with macros that add appropriate prefixes.

gatttool: Show info message when no primary service is found

gatttool: Use bluetoothctl strategy for unix signal handling

Now to exit from gatttool at interactive mode, user must use SIGTERM(15)
or press Ctrl+D. SIGINT(2) or Ctrl+C just flushes typed characters and
breaks line.

gatttool: Use bluetoothctl strategy to handle standard input

Use GSource id returned by g_io_add_watch() instead of raw GIOChannel to
handle stdin and glib main loop.

gatttool: Remove connecting message from prompt

When trying to connect, the "connecting" message is not set as prompt
anymore. Instead, it is printed just like other status messages and the
prompt is kept, allowing the user to send commands while connection is
being established.

gatttool: Show connection state using color

When device is connected, the Bluetooth address is shown in blue color.

gatttool: Use bluetoothctl behavior when pressing Enter on empty prompt

gatttool: Replace more usages of printf() with rl_printf()

rl_printf() works better with readline than plain printf(), specially on
asynchronous callbacks.

gatttool: Fix issues with printf() and readline prompt

Use rl_printf() from bluetoothctl to consistently display messages. This
is specially important for asynchronous callbacks, where using printf()
may mess up the prompt created by readline.

gatttool: Use GError to propage error messages to caller

Handle runtime errors outside gatt_connect(), using GError to propagate
error messages to caller.

monitor: Remove bogus entries from event mask table

The event mask is 64 bits so any bit offsets greater than that do not
make sense.

audio: Remove unused constants

Removes the never used constants AUDIO_SINK_INTERFACE and
AUDIO_SOURCE_INTERFACE which are defined to "org.bluez.AudioSink" and
"org.bluez.AudioSource", respectively.

audio: Remove unused function

Removes the never used function audio_device_is_active.

audio: Remove unused constant

Removes the never used constant AUDIO_INTERFACE which is defined to
"org.bluez.Audio".

Release 5.4

build: Update library version

lib: Update list of company identifiers

obexd: Automatically setup notification connection for MAP client

This automatically establishes the MNS connection when the MAS client
session is started and terminates the MNS connection when the MAS client
session is closed.

The MAP client controls the notification channel using the
SetNotificationRegistration function. The MSE will connect/disconnect
the MNS connection accordingly.

obexd: Enable MNS server

build: Add support for building MNS server

obexd: Add Message Notification Service (MNS) server

This implements the server role of the MAP Message Notification Service
(MNS) which is part of the MAP Client Equipment (MCE) device.

After successful registration, the MNS will receive event reports,
notifying about state changes on the server side.

Possible events are: NewMessages, DeliverySuccess, SendingSuccess,
DeliveryFailure, SendingFailure, MemoryFull, MemoryAvailable,
MessageDeleted, MessageShift

profile: Add MNS server record

This adds the MNS server record to the profile default settings.

shared: Use gcc builtin instead of g_atomic

g_atomic_* end up using G_STATIC_ASSERT, causing gcc 4.8 to yell due to
-Wunused-local-typedefs.

/usr/include/glib-2.0/glib/gmacros.h:162:53: error: typedef ‘_GStaticAssertCompileTimeAssertion_2’ locally defined but not used [-Werror=unused-local-typedefs]
 #define G_STATIC_ASSERT(expr) typedef char G_PASTE (_GStaticAssertCompileTimeAssertion_, __COUNTER__)[(expr) ? 1 : -1]

obexd: Use gcc builtin instead of g_atomic

g_atomic_* end up using G_STATIC_ASSERT, causing gcc 4.8 to yell due to
-Wunused-local-typedefs.

/usr/include/glib-2.0/glib/gmacros.h:162:53: error: typedef ‘_GStaticAssertCompileTimeAssertion_2’ locally defined but not used [-Werror=unused-local-typedefs]
 #define G_STATIC_ASSERT(expr) typedef char G_PASTE (_GStaticAssertCompileTimeAssertion_, __COUNTER__)[(expr) ? 1 : -1]

Most of the uses of atomic operations were wrong. They were fixed as
well. If we are using atomic operations, reading the variable again
later for logging is not an option, we should use the return of the
atomic function used to fetch the variable.

gobex: Use gcc builtin instead of g_atomic

g_atomic_* end up using G_STATIC_ASSERT, causing gcc 4.8 to yell due to
-Wunused-local-typedefs.

/usr/include/glib-2.0/glib/gmacros.h:162:53: error: typedef ‘_GStaticAssertCompileTimeAssertion_2’ locally defined but not used [-Werror=unused-local-typedefs]
 #define G_STATIC_ASSERT(expr) typedef char G_PASTE (_GStaticAssertCompileTimeAssertion_, __COUNTER__)[(expr) ? 1 : -1]

Most of the uses of atomic operations were wrong. They were fixed as
well. If we are using atomic operations, reading the variable again
later for logging is not an option, we should use the return of the
atomic function used to fetch the variable.

attrib: Use gcc builtin instead of g_atomic

g_atomic_* end up using G_STATIC_ASSERT, causing gcc 4.8 to yell due to
-Wunused-local-typedefs.

/usr/include/glib-2.0/glib/gmacros.h:162:53: error: typedef ‘_GStaticAssertCompileTimeAssertion_2’ locally defined but not used [-Werror=unused-local-typedefs]
 #define G_STATIC_ASSERT(expr) typedef char G_PASTE (_GStaticAssertCompileTimeAssertion_, __COUNTER__)[(expr) ? 1 : -1]

Most of the uses of atomic operations were wrong. They were fixed as
well. If we are using atomic operations, reading the variable again
later for logging is not an option, we should use the return of the
atomic function used to fetch the variable.

gdbus: Use gcc builtin instead of g_atomic

g_atomic_* end up using G_STATIC_ASSERT, causing gcc 4.8 to yell due to
-Wunused-local-typedefs.

gdbus/client.c: In function ‘g_dbus_client_ref’:
/usr/include/glib-2.0/glib/gmacros.h:162:53: error: typedef ‘_GStaticAssertCompileTimeAssertion_2’ locally defined but not used [-Werror=unused-local-typedefs]
 #define G_STATIC_ASSERT(expr) typedef char G_PASTE (_GStaticAssertCompileTimeAssertion_, __COUNTER__)[(expr) ? 1 : -1]

gitignore: Ignore file generated by Automake 1.13

Automake >= 1.13 enables parallel-tests option which uses a test-driver
script (copied by automake). Ignore this file.

gatttool: Remove check for bluetooth address parameter on gatt_connect()

This check is more appropriate to be done by callers of gatt_connect().
This is already done on interactive mode in attrib/interactive.c and is
now also done in non-interactive mode.

gatttool: Remote address is mandatory for non-interactive gatttool

For gatttool non-interactive mode, command line parameter with
bluetooth remote address is mandatory.

gatttool: Fix memory leak after parsing input line

The callback is responsible for freeing input line buffer that comes
from rl_callback_handler_install().

tools/hciattach: Fix TI firmware path

Change firmware path to /lib/firmware/ti-connectivity/ to be
conform with linux-firmware folder structure. All wl1xxx firmware
resides in ti-connectivity. See linux-firmware repository
https://git.kernel.org/cgit/linux/kernel/git/firmware/linux-firmware.git/
for reference.

audio: Track connections to AVRCP roles separately

Currently the code assumes when one role connects the other must be
connected as well which is bogus and can create problems.

lib: Fix invalid memory access in sdp_service_search_attr_req

Browsing services using sdptool can lead to writing to invalid heap
locations.

valgrind's output of exemplary call: sdptool browse local

==2203== HEAP SUMMARY:
==2203==     in use at exit: 0 bytes in 0 blocks
==2203==   total heap usage: 251 allocs, 251 frees, 140,156 bytes allocated
==2203==
==2203== All heap blocks were freed -- no leaks are possible
==2203==
==2203== ERROR SUMMARY: 6 errors from 2 contexts (suppressed: 0 from 0)
==2203==
==2203== 1 errors in context 1 of 2:
==2203== Invalid write of size 2
==2203==    at 0x805B882: bt_put_be16 (in /home/xpu/gits/bluez.bin/bin/sdptool)
==2203==    by 0x8062BD0: sdp_service_search_attr_req (in /home/xpu/gits/bluez.bin/bin/sdptool)
==2203==    by 0x8052457: do_search (in /home/xpu/gits/bluez.bin/bin/sdptool)
==2203==    by 0x80525AE: do_search (in /home/xpu/gits/bluez.bin/bin/sdptool)
==2203==    by 0x805277F: cmd_browse (in /home/xpu/gits/bluez.bin/bin/sdptool)
==2203==    by 0x8053199: main (in /home/xpu/gits/bluez.bin/bin/sdptool)
==2203==  Address 0x4391359 is 7 bytes before a block of size 2,048 alloc'd
==2203==    at 0x402B6A8: malloc (in /usr/lib/valgrind/vgpreload_memcheck-x86-linux.so)
==2203==    by 0x8062B4B: sdp_service_search_attr_req (in /home/xpu/gits/bluez.bin/bin/sdptool)
==2203==    by 0x8052457: do_search (in /home/xpu/gits/bluez.bin/bin/sdptool)
==2203==    by 0x80525AE: do_search (in /home/xpu/gits/bluez.bin/bin/sdptool)
==2203==    by 0x805277F: cmd_browse (in /home/xpu/gits/bluez.bin/bin/sdptool)
==2203==    by 0x8053199: main (in /home/xpu/gits/bluez.bin/bin/sdptool)
==2203==
==2203==
==2203== 5 errors in context 2 of 2:
==2203== Invalid write of size 1
==2203==    at 0x402D363: memcpy (in /usr/lib/valgrind/vgpreload_memcheck-x86-linux.so)
==2203==    by 0x80613E7: gen_dataseq_pdu (in /home/xpu/gits/bluez.bin/bin/sdptool)
==2203==    by 0x8061472: gen_attridseq_pdu (in /home/xpu/gits/bluez.bin/bin/sdptool)
==2203==    by 0x8062C00: sdp_service_search_attr_req (in /home/xpu/gits/bluez.bin/bin/sdptool)
==2203==    by 0x8052457: do_search (in /home/xpu/gits/bluez.bin/bin/sdptool)
==2203==    by 0x80525AE: do_search (in /home/xpu/gits/bluez.bin/bin/sdptool)
==2203==    by 0x805277F: cmd_browse (in /home/xpu/gits/bluez.bin/bin/sdptool)
==2203==    by 0x8053199: main (in /home/xpu/gits/bluez.bin/bin/sdptool)
==2203==  Address 0x439135b is 5 bytes before a block of size 2,048 alloc'd
==2203==    at 0x402B6A8: malloc (in /usr/lib/valgrind/vgpreload_memcheck-x86-linux.so)
==2203==    by 0x8062B4B: sdp_service_search_attr_req (in /home/xpu/gits/bluez.bin/bin/sdptool)
==2203==    by 0x8052457: do_search (in /home/xpu/gits/bluez.bin/bin/sdptool)
==2203==    by 0x80525AE: do_search (in /home/xpu/gits/bluez.bin/bin/sdptool)
==2203==    by 0x805277F: cmd_browse (in /home/xpu/gits/bluez.bin/bin/sdptool)
==2203==    by 0x8053199: main (in /home/xpu/gits/bluez.bin/bin/sdptool)
==2203==
==2203== ERROR SUMMARY: 6 errors from 2 contexts (suppressed: 0 from 0)

attrib: Fix status on memory allocation failure

If memory allocation fails, ATT_ECODE_INSUFF_RESOURCES should be
reported to the callback.

attrib: Fix coding style in read_char_helper()

attrib: Fix memory leak on low memory condition

If g_try_new0() fails due to low memory condition, "list" should be
freed before returning from primary_all_cb().

AVCTP: Add mapping for mandatory pass-through operations

This add mapping for mandatory pass-through operations for categories
supported and a few other operations considered useful.

tools/mgmt-tester: Refine stop discovery test case

Add HCI test and discovering event test for stop discovery.

tools/mgmt-tester: Refine start discovery test case

Add HCI command test and discovering event test for start discovery.

attrib: Fix use after free of attrib

If attrib is freed in cmd->func(), then it will be used if either
request or response queue has some data to send.

This patch moves calling wake_up_sender() which increases the ref
count of attrib so that it wont get freed in cmd->func().

attrib: Remove norequests and noresponses variables

Passing NULL is fine to g_queue_is_empty(), so removing these
variables makes the code more readable

core: Fix a double free on adapter_stop

The discovery_list list has the list of current discovery clients and is
removed on adapter_stop (for example due a "power off" command). The
g_slist_free_full will call discovery_free on every element of the list
and remove the nodes of the list, but discovery_destroy (called by
discovery_free) will not only free the element, but also remove it from
the list. This causes the list node to be freed twice, once by
g_slist_free_full and once by g_slist_remove.

This fix calls successively discovery_destroy and lets it remove the
list's elements one by one.

core: Memory leak on device_free for eir_uuids field.

The eir_uuids list is usually freed by device_svc_resolved, but that
doesn't happen if the device is removed before a SDP browse ends.
This fix deletes the eir_uuids list on device_free.

sdptool: Fix mem leak in do_search

gatttool: fix parameter type/name from le to psm

The parameter le was changed to psm and its type was modified from
boolean to int, but header was unmodified. That change occurred in commit
0cc24599637ed6666469c68d55f4e42ff722e447.

client: Agent's RequestPasskey implementation

Implements the uint32 RequestPasskey(object device) method.

client: Agent's DisplayPasskey implementation

Implements the DisplayPasskey(object device, uint32 passkey, uint16 entered)
method.

client: Agent's DisplayPincode implementation

Implements the DisplayPinCode(object device, string pincode) method.

client: "agent" command capability argument and autocompletion

This patch enables argument autocompletion for the agent command with
the list of capabilities an agent can have, adding also "on" (for the
default "") and "off". The command passes the argument (parsing and
verifying it) to the dbus method call.

client: Right prompt management on agent input

Registering an agent shares the user input interface with the normal console
command interface. The way it is implemented (using rl_message, rl_save_prompt
and rl_restore_prompt) conflicts with the rl_printf calls that may appear
while waiting for user input, loosing the [bluetooth]# prompt.
This patch fixes this and makes clear if the expected input is a command or an
agent reply changing the color and text of the prompt.

client: Add color modifiers to NEW, CHG and DEL events.

Events like [NEW], [CHG] and [DEL] can appear in the command console at
any time, even when the user is typing a command. The last line is kept
making the event line appear just before it, something that can be
unnoticed. This patch add meaningful colors for those three event to make
it easier to see them.

sdptool: Fix recursive search

When doing recursive SDP searches, we must not overwrite our own values
after we retrieved them from SDP. Otherwise, the UUID test will always we
false.

lib: Update list of company identifiers

core: Fix wrong argument on agent *_cb functions.

The last argument for the functions pincode_cb, passkey_cb and
confirm_cb is a user data pointer with the "void *". Those functions
expect a "struct authentication_req *", but insted a "struct
btd_device*" without any complaint from the compiler due the void* cast.
This fix passes the right argument to those functions.

Other uses of those functions as callbacks (agent_request_passkey,
agent_request_pincode, agent_request_authorization and
agent_request_confirmation) pass the right struct authentication_req *.

AVRCP: Fallback to GetElementAttributes if GetItemAttributes fails

If GetItemAttributes fails with error set to player not browsable
fallback to GetElementAttributes.

AVRCP: Reduce number of request to refresh the player list

In case the addressed player changes but it features is already known
the list of players don't need to be refreshed.

player: Check if values really changed before emitting signals

AVRCP: Add support for available player changed event for CT role

This adds support for registering to available player changed event if
supported by TG.

tools: Add basic invalid parameters test for stop discovery

tools: Add basic rejected test for stop discovery