gdbus: Check for NULL DBusPendingCall in g_dbus_send_message_with_reply

"Warning: if the connection is disconnected or you try to send Unix file
descriptors on a connection that does not support them, the
DBusPendingCall will be set to NULL, so be careful with this."

Check this in g_dbus_send_message_with_reply so that callers don't need
to double check for NULL if g_dbus_send_message_with_reply returned
TRUE.

This also fix crash if passing FD over D-Bus is blocked e.g. by SELinux
policy.

bluetoothd[1894]: profiles/audio/avdtp.c:session_cb()
bluetoothd[1894]: profiles/audio/avdtp.c:avdtp_parse_cmd() Received
    SET_CONFIGURATION_CMD
bluetoothd[1894]: profiles/audio/a2dp.c:endpoint_setconf_ind() Source
    0x6c5000: Set_Configuration_Ind
bluetoothd[1894]: profiles/audio/avdtp.c:avdtp_ref() 0x6df360: ref=1
bluetoothd[1894]: profiles/audio/a2dp.c:setup_ref() 0x6d32b0: ref=1
process 1894: arguments to dbus_pending_call_set_notify() were incorrect,
     assertion "pending != NULL" failed in file dbus-pending-call.c line
     636.
This is normally a bug in some application using the D-Bus library.

tools/sdptool: Clarify 'add' and 'remove' commands in manual

Adapters are no longer notified about external changes in SDP database.
This results in 'add' and 'remove' commands being usefull only for SDP
testing or qualification.

doc: Fix some style issues with the command parameters

doc: Add section about Set Scan Parameters command

tools: Add simple test utility for L2CAP connectionless channels

tools: Print L2CAP address information of connections

lib: Add definitions for Set Scan Parameters command

tools/mgmt-tester: Remove unused variable

tools/mgmt-tester: Update set_hs tests to require SSP to be enabled first

monitor: Add ATT Prepare Write Response decoding

monitor: Add ATT Prepare Write Request decoding

monitor: Add ATT Execute Write Request decoding

monitor: Add ATT Read Multiple Request decoding

emulator: Fix AMP emulation build error

This fix build with GCC 4.6.3.

  CC     emulator/amp.o
emulator/amp.c: In function ‘send_packet’:
emulator/amp.c:172:7: error: ignoring return value of ‘write’,
    declared with attribute warn_unused_result [-Werror=unused-result]
cc1: all warnings being treated as errors
make[1]: *** [emulator/amp.o] Error

monitor: Add ATT over BR/EDR decoding support

lib: Update company identifiers

monitor: Decode Apple identifier and iBeacon data structures

Don't register Device ID record on sdp server start

This makes SDP code no longer depends on main_opts. DID record is now
registered from main() after sdp server was started. This is OK since
mainloop is not yet running and record will be present when first
request comes.

avdtp: Fix minor coding style issue

avdtp: Remove not needed forward declaration

auth_cb function is not used before definition so no need to forward
declare it.

avdtp: Move connection_lost function to avoid forward declaration

avdtp: Rename avdtp_callbacks to state_callbacks

This better describes what list contains.

avdtp: Remove unused avdtp_stream_setup_active function

It is not used anywhere and can be removed.

avdtp: Remove unused avdtp_is_connected function

It is not used anywhere and can be removed.

avdtp: Fix typos in errors from avdtp_strerror

monitor: Fix integer overruns with EIR parsing

For tracking how much has been parsed of the EIR we must use a greater
than uint8 integer to avoid having a large (e.g. 255) field length cause
our counter to wrap around and start from zero.

monitor: Fix zero-length field handling in print_hex_field

doc: Mention that High Speed support depends on Secure Simple Pairing

emulator: Add support for AMP controller emulation

tools: Add test utility for AMP physical connection setup

tools/hcidump: Fix parsing for service class 0x110f

According to assigned number 0x110f is AV Remote Controller not VCP.

tools/sdptool: Fix parsing for service class 0x110f

According to assigned number 0x110f is AV Remote Controller not VCP.

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"