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.

gobex: Revert g_obex_pending_req_abort to static pending_req_abort

This reverts the changes introduced in
9095deb82572112fc0870095bf2222964610eafe that made pending_req_abort
public which is not necessary considering g_obex_cancel_req can do the
same and is safe to call even if the request is not pending.

gobex: Fix crash on g_obex_pending_req_abort

It is not safe to call g_obex_pending_req_abort directly as pending_req
can be NULL:
Invalid read of size 4
   at 0x41231E: g_obex_pending_req_abort (gobex.c:693)
   by 0x416A8A: g_obex_cancel_transfer (gobex-transfer.c:647)
   by 0x42DEF2: obc_transfer_cancel (transfer.c:180)
   by 0x43D833: process_message.isra.5 (object.c:259)
   by 0x3B0701CE85: ??? (in /usr/lib64/libdbus-1.so.3.7.4)
   by 0x3B0700FA30: dbus_connection_dispatch (in /usr/lib64/libdbus-1.so.3.7.4)
   by 0x43A5B7: message_dispatch (mainloop.c:76)
   by 0x3B03C48962: ??? (in /usr/lib64/libglib-2.0.so.0.3600.3)
   by 0x3B03C47E05: g_main_context_dispatch (in /usr/lib64/libglib-2.0.so.0.3600.3)
   by 0x3B03C48157: ??? (in /usr/lib64/libglib-2.0.so.0.3600.3)
   by 0x3B03C48559: g_main_loop_run (in /usr/lib64/libglib-2.0.so.0.3600.3)
   by 0x40D53C: main (main.c:319)
 Address 0x30 is not stack'd, malloc'd or (recently) free'd

tools: Limit controller type to two bits of device info type

lib: Update company identifiers

obexd/bluetooth: Fix memory leak when adapter is off

sdp_connect fails when Bluetooth adapter is off which leads to the
following leak:
37 bytes in 1 blocks are definitely lost in loss record 68 of 165
   at 0x4A06409: malloc (in /usr/lib64/valgrind/vgpreload_memcheck-amd64-linux.so)
   by 0x3B03C4D89E: g_malloc (in /usr/lib64/libglib-2.0.so.0.3600.3)
   by 0x3B03C64BAE: g_strdup (in /usr/lib64/libglib-2.0.so.0.3600.3)
   by 0x427D5D: bluetooth_connect (bluetooth.c:410)
   by 0x426CC9: obc_session_create (session.c:454)
   by 0x425693: create_session (manager.c:203)
   by 0x43D8A3: process_message.isra.5 (object.c:259)
   by 0x3B0701CE85: ??? (in /usr/lib64/libdbus-1.so.3.7.4)
   by 0x3B0700FA30: dbus_connection_dispatch (in /usr/lib64/libdbus-1.so.3.7.4)
   by 0x43A627: message_dispatch (mainloop.c:76)
   by 0x3B03C48962: ??? (in /usr/lib64/libglib-2.0.so.0.3600.3)
   by 0x3B03C47E05: g_main_context_dispatch (in /usr/lib64/libglib-2.0.so.0.3600.3)

obexd: Clarify the folder property of PushMessage

The folder property of PushMessages does not accept path information,
it allows only to request the messages to be added to a subfolder of the
current folder.

obexd: Clarify the folder property of ListMessages

The folder property of ListMessages does not accept path information,
it allows only to request the messages of a subfolder of the current
folder.

obexd: Fix setting message folder for relative folder in ListMessages

The method ListMessages allows to specify a relative subfolder.
This subfolder needs to be added to the current path when registering
a new message interface.

obexd: Add folder property to map_msg_create

Message interfaces are not necessarily created for the current folder,
therefore the folder needs to be specified in a parameter.

For example, messages can be created for sub folders when using the folder
parameter in ListMessages.

doc: Update description for Set Advertising command

tools/mgmt-tester: Add basic rejection test case for LE advertising

tools/mgmt-tester: Add basic LE advertising success test cases

tools/mgmt-tester: Take into account new "advertising" supported setting

monitor: Add support for LE Advertising mgmt setting

emulator/bthost: Add support for creating LE connections

tools/btmgmt: Add support for setting LE advertising

mgmt: Add defines for new advertising setting

doc: Fix mgmt_set_le error code list

doc: Add mgmt_set_advertising details to mgmt API

obexd/service: Fix possible leaking drivers list

When registering a new driver with obex_service_driver_register there
could exist another driver for the service which will cause the drivers
list to leak.

obexd/server: Fix leaking drivers list

The leak can be detected by using G_SLICE=always-malloc which will
produce the following trace using valgrind:
112 bytes in 7 blocks are definitely lost in loss record 123 of 167
   at 0x4A06409: malloc (in /usr/lib64/valgrind/vgpreload_memcheck-amd64-linux.so)
   by 0x3B03C4D89E: g_malloc (in /usr/lib64/libglib-2.0.so.0.3600.3)
   by 0x3B03C6344D: g_slice_alloc (in /usr/lib64/libglib-2.0.so.0.3600.3)
   by 0x3B03C647A5: g_slist_append (in /usr/lib64/libglib-2.0.so.0.3600.3)
   by 0x424DD3: obex_service_driver_list (service.c:76)
   by 0x42517F: obex_server_init (server.c:64)
   by 0x40D439: main (main.c:304)

unit: Remove not needed functions from test-sdp

SDP code no longer depends on adapter code.

core: Remove not needed sdp_init_services_list function

It is doing nothing now and can be removed.

adapter: Handle removing of SDP records

Make adapter in charge of updating SDP database. This allow to decouple
SDP of code used for notifying adapters about SDP database change.

adapter: Handle adding new SDP records

Make adapter in charge of updating SDP database. This allow to decouple
SDP of code used for notifying adapters about SDP database change.

adapter: Insert DeviceID record if setting DeviceID data to controller

DeviceID is special since its record is registered on start by
sdpd-server and always has handle 0x10000.

lib: Update company identifiers

core: Fix passing timeout when disabling discoverable mode

We should only pass a timeout value when enabling discoverable mode.
When disabling passing something non-zero would yield a "invalid
parameters" error.

emulator/bthost: Fix using correct address type for LE connections

emulator/bthost: Fix ACL handle packing

emulator/bthost: Track address type for connections

emulator/bthost: Use right CID for LE signaling

emulator/bthost: Pass full ACL connection context around

emulator: Add initial LE L2CAP signaling support to bthost

emulator: Add LE Connection Complete handling to bthost

emulator: Refactor bthost ACL connection tracking

emulator: Add skeleton for LE meta event handler in bthost

emulator: Add basic advertise enable support for bthost

tools/l2cap-tester: Fix using right hciemu type

emulator: Add basic support for LE connections

monitor: Add define for "Connection Failed To Be Established"

emulator: Remove redundant empty line

sdp: Check correct number of bytes received in recv

Instead of checking for error check that correct number of bytes received.

doc: Update supported A2DP version to 1.3

tools/l2cap-tester: Add invalid config request server test

tools/l2cap-tester: Add invalid CID server test case

tools/l2cap-tester: Fix sending right req code in tests

tools/l2cap-tester: Add expected response data to nval PDU test

Release 5.9

obexd/bluetooth: Fix memory leak

g_io_channel_unix_new creates a reference which is then passed to
obex_session_start which creates its on reference via g_io_channel_ref
leading to the following leak:

at 0x4A06409: malloc (in /usr/lib64/valgrind/vgpreload_memcheck-amd64-linux.so)
by 0x3B03C4D89E: g_malloc (in /usr/lib64/libglib-2.0.so.0.3600.3)
by 0x3B03C88224: g_io_channel_unix_new (in /usr/lib64/libglib-2.0.so.0.3600.3)
by 0x418967: profile_new_connection (bluetooth.c:148)
by 0x43D763: process_message.isra.5 (object.c:259)

obexd: Remove msg from MAP session structure

The D-Bus message for a pending method call is now stored in the
pending_request struct.