gdbus: Avoid using g_ptr_array_new_full convenience function

The g_ptr_array_new_full function only got introduced with GLib 2.30
and to avoid requiring a newer GLib use g_ptr_array_set_free_func instead.

build: Compile support for new D-Bus client helpers

gdbus: Increase the method call timeout to 5 minutes

gdbus: Add helper for sending async error replies

gdbus: Fix race condition with creating duplicate client proxies

gdbus: Fix issue with unref of client while parsing interfaces

gdbus: Fix issue with unref of client in connect handler

gdbus: Add support for D-Bus client method calls

gdbus: Add callback support for handling property changes

gdbus: Handle property changed signals and update property

gdbus: Set property changed filter match rule for each proxy

gdbus: Use a GPtrArray for the match rules

gdbus: Add support for setting D-Bus client basic properties

gdbus: Deep copy of variants also requires a signature

gdbus: Add support for complex D-Bus client properties

gdbus: Add support for D-Bus client properties

gdbus: Add support for D-Bus client proxies

gdbus: Add helper functions for simple D-Bus clients

p2p: Make p2p_push more robust

Some devices do not provide neither SNEP nor NPP, or only NPP.
For those we need to check the fallback service name and the push op
existence.

bluetooth: Remove not needed forward declaration

bt_do_pairing is not used before it is defined so there is no need to
forward declare it.

test: Add handover_agent for testing

This is a very simple implementation of handover agent useful for
testing (especially if not willing to run BlueZ 5.x). Currently
received data in PushOOB/RequestOOB is printed and reply to RequestOOB
is generated from static data.

In future this might be extended to accept some user input and more
flexible reply generation.

ndef: Remove not needed NULL assignments

There is no need to NULL parameters pointers to free_*_payload functions
as those are local copies.

Fix coding style errors through the code

This fix some style errors reported by checkpatch and found while
working with code.

TODO: Update

gdbus: Add g_dbus_add_properties_watch function

Convenient function to create watches for D-Bus properties.

gdbus: Don't automatically attach ObjectManager

Let each project attach the object manager interface instead of
registering it automatically.

gdbus: Don't register DBus.Properties with no properties

Delay registering DBus.Properties interface until the moment there are
properties on that path. This is needed for objects that currently don't
expose any property to not export the interface.

adapter: Update the Devices array before emitting the D-Bus signal

agent: Provide DBUS_TIMEOUT_USE_DEFAULT if not available

gdbus: Remove not needed NULL pointer checks

g_strdup returns NULL if argument passed is NULL and there is no need
to double check that.

gdbus: Replace leading spaces with tabs

Trivial formatting fix.

gdbus: Fix compilation error due to missing #defines

Since these are simple #define strings, we are defining it here
instead of upgrading to D-Bus 1.5 or later.

Log:
  CC     gdbus/object.o
gdbus/object.c: In function ‘properties_set’:
gdbus/object.c:876:7: error: ‘DBUS_ERROR_UNKNOWN_PROPERTY’ undeclared
 (first use in this function)
gdbus/object.c:876:7: note: each undeclared identifier is reported
 only once for each function it appears in
gdbus/object.c:881:6: error: ‘DBUS_ERROR_PROPERTY_READ_ONLY’
 undeclared (first use in this function)
make[1]: *** [gdbus/object.o] Error 1
make: *** [all] Error 2

gdbus: Remove connection from pending_property functions

The reply to a DBus.Properties.Set() method call should go through the
same D-Bus connection. Thus remove the DBusConnection parameter from the
following functions:

    - g_dbus_pending_property_success()
    - g_dbus_pending_property_error_valist()
    - g_dbus_pending_property_error()

gdbus: Fix invalid memory access during interface removal

If an interface is removed from the root path during the same mainloop
iteration that it was added we need to check for data->added before
doing the check for data->parent == NULL in the remove_interface()
function. Otherwise the added interface doesn't get removed from the
data->added list and will result in accessing freed memory:

==337== Invalid read of size 8
==337==    at 0x4F65AFA: dbus_message_iter_append_basic (in /usr/lib64/libdbus-1.so.3.7.1)
==337==    by 0x1247B5: append_interface (object.c:556)
==337==    by 0x4C8DC5C: g_slist_foreach (gslist.c:840)
==337==    by 0x1261F7: process_changes (object.c:594)
==337==    by 0x126372: generic_unregister (object.c:997)
==337==    by 0x4F69669: ??? (in /usr/lib64/libdbus-1.so.3.7.1)
==337==    by 0x4F5CE51: dbus_connection_unregister_object_path (in /usr/lib64/libdbus-1.so.3.7.1)
==337==    by 0x125E81: object_path_unref (object.c:1236)
==337==    by 0x126136: g_dbus_unregister_interface (object.c:1361)
==337==    by 0x14CDF0: service_exit (service.c:581)
==337==    by 0x177556: plugin_cleanup (plugin.c:242)
==337==    by 0x12221F: main (main.c:559)
==337==  Address 0x5bc1550 is 0 bytes inside a block of size 56 free'd
==337==    at 0x4A079AE: free (vg_replace_malloc.c:427)
==337==    by 0x4C7850E: g_free (gmem.c:252)
==337==    by 0x125DB0: remove_interface (object.c:671)
==337==    by 0x125E3B: object_path_unref (object.c:1230)
==337==    by 0x126136: g_dbus_unregister_interface (object.c:1361)
==337==    by 0x14CDF0: service_exit (service.c:581)
==337==    by 0x177556: plugin_cleanup (plugin.c:242)
==337==    by 0x12221F: main (main.c:559)

gdbus: Add g_dbus_get_properties function

This function can be used to construct custom D-Bus messages containing
the properties for a specific interface on a given path.

gdbus: Add support for invalidated properties

If there's a pending property but its exists() callback returns false
the property should be considered invalidated and included in the
relevant list of the PropertiesChanged signal.

gdbus: Fix processing pending properties in remove_interface()

gdbus: Fix up Properties.Set() code path

Minor fixes to make setter actually work:

- Add propdata in pending_property_set
- Break loop when we are removing propdata from list and we
  found it
- in_args and out_args were swapped
- interface and method name arguments were swapped

gdbus: Fix invalid memory access while unregistering

If an interface is added and removed on the same mailoop iteration,
ObjectManager would try to send InterfacesAdded signal while running the
idler because the interface was added to data->added list.

This is easily reproduced by forcing an error path in a plugin
registration, like on sap_server_register(), resulting in the following
error:

==11795== Invalid read of size 4
==11795==    at 0x496F592: dbus_message_iter_append_basic (dbus-message.c:2598)
==11795==    by 0x117B39: append_interface (object.c:554)
==11795==    by 0x48955E7: g_slist_foreach (gslist.c:840)
==11795==    by 0x11923B: process_changes (object.c:592)
==11795==    by 0x11956D: generic_unregister (object.c:980)
==11795==    by 0x4973BAC: _dbus_object_tree_unregister_and_unlock (dbus-object-tree.c:516)
==11795==    by 0x4965240: dbus_connection_unregister_object_path (dbus-connection.c:5776)
==11795==    by 0x1178A5: object_path_unref (object.c:1219)
==11795==    by 0x118517: g_dbus_unregister_interface (object.c:1344)
==11795==    by 0x19AF5B: sap_exit (sap.c:385)
==11795==    by 0x13E9E2: sap_server_register (server.c:1428)
==11795==    by 0x13C092: sap_server_probe (manager.c:44)

With this patch we don't send the InterfacesAdded signal, removing it
from data->added while unregistering.

gdbus: Implement PropertiesChanged signal

gdbus: Simplify code for appending properties

This reuse append_properties for GetAll and GetManagedObjects

gdbus: Integrates ObjectManager with Properties interface

This appends the properties and its values when using ObjectManager.

gdbus: Only export ObjectManager interface on root path

ObjectManager should be exported only in the root path and list all
the children paths.

gdbus: Group interface changes to reduce the amount of signals emitted

InterfacesAdded and InterfacesRemoved can group all the interfaces
changes together in one message.

gdbus: Add support for org.freedesktop.DBus.ObjectManager interface

This implements initial support for ObjectManager, it automatically adds
objects to its parents so no action is needed by daemons to get their
objects managed by this interface.

ObjectManager is part of D-Bus spec since revision 0.17:
http://dbus.freedesktop.org/doc/dbus-specification.html#standard-interfaces-objectmanager

gdbus: Add properties into Introspectable interface

gdbus: Implement DBus.Properties.Set method

Contrary to Get() and GetAll(), Set() is asynchronous so we pass an id
to the setter so later it can declare the Set() as successful or
otherwise.

gdbus: Implement DBus.Properties.GetAll method

gdbus: Implement DBus.Properties.Get method

gdbus: Add skeleton of DBus.Properties interface

This interface is responsible for handling properties of all objects in
a given path. Right now it only registers itself, doing nothing useful.
A conversion to this new layout will be done by subsequent patches.

org.freedesktop.org.DBus.Properties spec can be found at
http://dbus.freedesktop.org/doc/dbus-specification.html#standard-interfaces-properties

gdbus: Use macros to add annotations

Besides being more readable this way it avoids going over 80 chars.

gdbus: Move typedefs up

Move the typedefs up so they can be used by functions and callbacks.

gdbus: Fix not freeing list node by using g_slist_delete_link

g_slist_remove_link does not free the node which can cause leaks so
replace that with g_slist_delete_link which does free memory properly.

gdbus: Refactor filter_data_find()

Now this function is only used for searching the listeners of a
connection and the other parameters are not needed anymore.

gdbus: Fix wrong signal handler match

When we add a signal handler with g_dbus_add_signal_watch(), this
function tries to multiplex the matches added in libdbus by checking
if there's a previous filter_data with the same fields. However, if the
field is NULL it accepts as being the same. The result is that the
following watches will use the same filter data:

watch1 = g_dbus_add_signal_watch(conn, BUS_NAME, NULL, iface, member,
cb1, data1, NULL);
watch2 = g_dbus_add_signal_watch(conn, BUS_NAME, "/path2", iface, member,
cb2, data2, NULL);
watch3 = g_dbus_add_signal_watch(conn, BUS_NAME, "/path3", iface, member,
cb3, data3, NULL);

The result is that when a signal arrives with path == "/path2", all 3
callbacks above will be called, with the same signal delivered to all of
them.

Another problem is that, if we invert the calls like below, only signals
to cb1 will never be trigerred, nonetheless it used path == NULL.

watch2 = g_dbus_add_signal_watch(conn, BUS_NAME, "/path2", iface, member,
cb2, data2, NULL);
watch1 = g_dbus_add_signal_watch(conn, BUS_NAME, NULL, iface, member,
cb1, data1, NULL);
watch3 = g_dbus_add_signal_watch(conn, BUS_NAME, "/path3", iface, member,
cb3, data3, NULL);

This is fixed by not multiplexing the matchs with filter data if any of
the fields are different, including being NULL. When a signal arrives,
if a field is NULL we accept it as a match, but not when adding the
signal handler.

gdbus: Fix crash when getting disconnected from the bus

When getting disconnected from the bus sometimes (maybe always?)
dbus_watch_handle() can cause the "info" context to be free'd meaning
that we should not try to access it after the call. The only member we
need access to is the connection pointer and as the code already has a
ref() call for it it's only natural to solve the issue by adding a local
variable not dependent on "info".

The backtrace of the crash fixed looks as follows:

 Invalid read of size 8
   at 0x121085: watch_func (mainloop.c:105)
   by 0x4C72694: g_main_context_dispatch (gmain.c:2539)
   by 0x4C729C7: g_main_context_iterate.isra.23 (gmain.c:3146)
   by 0x4C72DC1: g_main_loop_run (gmain.c:3340)
   by 0x120541: main (main.c:551)
 Address 0x5bbcd90 is 16 bytes inside a block of size 24 free'd
   at 0x4A079AE: free (vg_replace_malloc.c:427)
   by 0x4C7837E: g_free (gmem.c:252)
   by 0x4F708BF: dbus_watch_set_data (dbus-watch.c:614)
   by 0x4F70938: _dbus_watch_unref (dbus-watch.c:132)
   by 0x4F6E9A7: _dbus_transport_handle_watch (dbus-transport.c:884)
   by 0x4F59AFB: _dbus_connection_handle_watch (dbus-connection.c:1497)
   by 0x4F70AF9: dbus_watch_handle (dbus-watch.c:683)
   by 0x121084: watch_func (mainloop.c:103)
   by 0x4C72694: g_main_context_dispatch (gmain.c:2539)
   by 0x4C729C7: g_main_context_iterate.isra.23 (gmain.c:3146)
   by 0x4C72DC1: g_main_loop_run (gmain.c:3340)
   by 0x120541: main (main.c:551)

adapter: Update the RF mode D-Bus property

When detecting a tag, or a device, the RF mode changes.
When the link is down, RF mode goes to Idle.

adapter: Expose the adapter radio mode through D-Bus

doc: Some small issues cleanup

Fix some whitespace, line-over-80 and typos issues.

doc: Clarify EIR blob format in handover agent API

Clarify what EIR blob is and where to find how it is specified.

nfctype3: Sending the POLL command should be the default behaviour

commit bd285ab6 changed the default case to be the one for only a
couple of Felica tags.

agent: Don't look for NULL record type on the NDEF agent hash

ndef: Cosmetic fix for action check in parse_mime_type

Match action check with neard code style.

bluetooth: Always enable legacy BT when no HO agent is registered

If the timer expired, that basically means no agent got registered,
regardless of BlueZ being version 5 or 4. In such situation just go
with legacy implementation as there is nothing better that could be
done anyway.

ndef: Select HO agent based on agent presence check

Choose between HO Agent or legacy bluetooth implementation based on
HO agent presence and not on detected BlueZ version.

bluetooth: Cosmetic fix for handover registration check

Support only static handover with Nokia proprietary mime type

Parse nokia.com:bt mime type only if received in Handover Select
message.

ndef: Workaround buggy Nokia BH-505 handover implementation

According to BT Core Specification 4.0 length field in OOB EIR contains
the length of the entire OOB data block including the length field
itself.

Some Nokia BH-505 devices reports OOB EIR length without length field
size. Correct size before sending EIR to agent.

ndef: Workaround buggy Android 4.1 BT handover implementation

Some implementations (e.g. Android 4.1) stores the data_size in big
endian but NDEF forum spec (BT Secure Simple Pairing) requires a little
endian. At the same time, the NDEF forum NDEF spec define a payload
length as single byte (and the payload size IS the oob data size).
So we check the first byte to determine if we have a big or little
endian size.

Use handover agent for Bluetooth data

If detected BlueZ is not version 4 handover agent is used for bluetooth
data handling.

bluetooth: Use bluetooth implementation only for BlueZ 4

BlueZ 5 will break DBus API and will be able to act as Handover Agent.
If handover agent is not registered while initializing/starting legacy
bluetooth implementation delay setting up watchers to give handover
agent time to register. This reduces D-Bus calls in case when BlueZ
registers itself as handover agent.

If handover agent is registered legacy bluetooth is stopped. When agent
unregisters or vanishes legacy bluetooth is reinitialized.

bluetooth: Use struct bt_data in near_ndef_prepare_handover_record

This also fix creating EIR with length field in host order instead of
little endian.

bluetooth: Move name at the end of EIR

This will make sure that hash and randomizer are send even when name is
long. Also limit name to not exceed maximum sensible EIR length of 255
bytes. If name would not fit use EIR_NAME_SHORT data type for it.

bluetooth: Make __near_bluetooth_parse_oob_record use struct bt_data

This will make it similar to handover agent api and thus make switching
between those two interfaces easier.

Add __near_agent_handover_registered function

It will be used to check if handover agent is registered.

agent: Add __near_agent_handover_request_data function

agent: Add __near_agent_handover_push_data function

Release 0.8

netlink: Add netlink event socket

Some netlink command can send multicast event before they're done.
To receive them the netlink_event function should be listening on
an event dedicated socket.

ndef: Remove the cr record from Hs message

NFC forum specs says that cr record should not be found on Hs messages.

As near_ndef_prepare_handover_record() is used to build Hs and Hr,
we have to remove the cr record only for Hs. To handle this case (and also
other cases where e.g. there's no ac record), the message begin/end flags
routines must be more granular (separate MB and ME setting routines).

ndef: Fix handover version check

In Connection Handover, Version check should return an empty Hs (with
the supported version) when major part is different from the local
major version.

handover: Fix to allow two handover requests

Test case 7 verifies that the remote handover server does not close the data
link connection after the first handover request message.

handover: Handle Handover frames without ac records

bluetooth: Extend mime_type with bluetooth properties

To comply with handover specs, if the incoming Hr OOB doesn't include
SP hash/randomizer keys, the reply (Hs) should not include such
information. So we store it to the cfg mime_type.

handover: Change carriers type to bitfield

This would allow multiple carriers at the same time (e.g. Bluetooth
and Wifi)

handover: Change behavior for empty (no ac) frames

Some frame may not have ac records, so we have to jump to the parsing code
right after checking the frame header.

ndef: Skip record header when parsing handover records

ndef: Fix offset check when parsing NDEF records

For each record header entry we need to check that the offest and the
payload length is not longer than the total record length.

agent: Push raw NDEF along with the matching record

ndef: Add records raw data to a D-Bus iterator

This will be needed by the NDEF agent raw data export routines.

ndef: Export record's raw data

AUTHORS: Mention Krzysztof's contributions

ndef: Change near_ndef_parse function name

Add suffix "_msg" to function near_ndef_parse for
name consistency and to stress out its role.

ndef: Use appropriate naming for record payload structures

Word "record" was used not only for records but also
for "payloads" wchich was unclear. Now record
contains general header and specyfic payload.

ndef: Add randomizing function for Collision-RR

p2p: Close a created socket if it's already bound

handover: No need to explicitely close the server socket

gio will do that when the channel refcount drops to 0.

handover: Unref client push channel after adding the watch

g_io_add_watch() bumps the channel refcount to 2, so it can be unreferenced
after that.
Also, since it is set to close on unref there is no need to explicitely
close it when removing the source.

bluetooth: Do not use near_info for displaying successful pairing string

ndef: Use DBG instead of near_info for Hs preparation succes path