ndef: Fix error message in near_ndef_prepare_handover_record

This function is used to create both Hs and Hr records.

snep: Check ndef length before applying Android workaround

With crafted NDEF message data outside of buffer could be read from
and write to. Potentially leading to segmentation fault or some other
undefined behaviour.

ndef: Fix checking for error in parse_mime_type

Upper layer don't know if action will be held or not when providing
ndef_message for reply. It is possible that reply was provided but
not used and this is not always an error

ndef: Remove not needed AC payload creation in build_ho_record

It was not used to create ndef message as AC recors is created inside
near_ndef_prepare_handover_record based on response from HO agent.

ndef: Use stack variable for record in build_ho_record

There is no need to dynamically allocate record as it is not used
after build_ho_record returns.

ndef: Fix memory leak in build_ho_record

record data needs to be freed before returning ndef.

144 (80 direct, 64 indirect) bytes in 1 blocks are definitely lost in loss record 142 of 190
   at 0x4C2B6CD: malloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
   by 0x4E7FBFD: g_try_malloc0 (in /lib/x86_64-linux-gnu/libglib-2.0.so.0.3200.3)
   by 0x424477: __ndef_build_from_message (ndef.c:2950)
   by 0x41CEE3: push_ndef (device.c:276)
   by 0x40B850: process_message.isra.4 (object.c:285)
   by 0x5143C40: ??? (in /lib/x86_64-linux-gnu/libdbus-1.so.3.5.8)
   by 0x5135B0F: dbus_connection_dispatch (in /lib/x86_64-linux-gnu/libdbus-1.so.3.5.8)
   by 0x409247: message_dispatch (mainloop.c:76)
   by 0x4E7A91A: ??? (in /lib/x86_64-linux-gnu/libglib-2.0.so.0.3200.3)
   by 0x4E79D52: g_main_context_dispatch (in /lib/x86_64-linux-gnu/libglib-2.0.so.0.3200.3)
   by 0x4E7A09F: ??? (in /lib/x86_64-linux-gnu/libglib-2.0.so.0.3200.3)
   by 0x4E7A499: g_main_loop_run (in /lib/x86_64-linux-gnu/libglib-2.0.so.0.3200.3)

bluetooth: Track adapter properties changes

Watch for PropertyChanged signal to keep track of changed properties.
Only properties used for EIR creation are tracked (name, cod, powered).

test: Add 'State' support for handover-agent

This allows to pass power state hint with handover-agent.

ndef: Prepare ac record with proper BT power state

When adding AC record for BT handover user state provided in bt_data
to properly indicate BT carrier power state.

agent: Set carrier power state from hint in parse_reply

Set carrier power state in struct bt_data if HO agent provided power
state hint in RequestOOB reply.

agent: Set carrier power state hint in prepare_bt_data

If alternative carrier is power state is known add it to dictionary.

Add carrier power state to struct bt_data

Add state to bt_data and set it according to power state received in
Hs/Hr messages.

ndef: Validate CPS when parsing AC record

CPS is 2-bit field so mask out reserved bits from first byte of AC
record.

ndef: Move carrier_power_state definition to header file

It will be also used by handover agent implementation

doc: Remove not really needed errors from Handover Agent API

Only InProgress error is really usefull as in such case neard could
retry operation. Other errors don't give any useful information
comparing to simply returning Failed with proper error message.

doc: Add 'State' field to handover agent API

This will allow both agent and neard to be informed about power state
of carrier. It is up to agent to decide what to do with information
provided e.g. in case of 'activating' state it could delay connection
attempt. This will also allow to properly set Carrier Power State in
ac record.

gdbus: Check for valid path before handling properties

build: Add unit tests during distribution building

unit: Fix includes for build testing

build: Header files are also part of the build material

build: Add unit/test-ndef to .gitignore list

build: Include unit test in make check run

unit: Hide all g_print statements behind g_test_verbose switch

Release 0.9

build: Do not use deprecated AM_CONFIG_HEADER

The long-obsoleted AM_CONFIG_HEADER macro was removed in automake 1.13.
Use AC_CONFIG_HEADERS instead.

build: Use AM_CPPFLAGS instead of INCLUDES

Makefile.am:50: warning: 'INCLUDES' is the old name for 'AM_CPPFLAGS' (or '*_CPPFLAGS')

types: Fix build error due to unaligned memory access

This fix number of build error on architectures that can't handle
unaligned memory access (eg. ARM).

Most of the code was originally written in BlueZ's bluetooth.h.

snep: SNEP validation test server implementation

NFC forum defined a SNEP Validation specification. This set of patches
makes neard to pass all the SNEP test cases, providing a SNEP validation
server as a default one.

snep: Separate snep default server code from snep core

In order to introduce the SNEP Validation server, we separate core
functions from snep default server code. This split leads to factorize
some parts of the process.
This code also improves the SNEP fragmentation code, more specifically
in snep_core_response_with_info.

ndef: Add records comparison helpers

These functions propose to check ndef records for specific fields
like mime type or ID field in record header.

unit: Add Title and URI records SmartPoster NDEF unit test

unit: URI NDEF test should check for the field length

unit: Add single record SmartPoster NDEF unit test

unit: Add Text NDEF record unit test

unit: Add test-ndef unit test

test-ndef checks for the URI record parsing validity.

test: Remove the NDEF argument from ndef-agent

Since ndef-agent won't implement the handover agent API, no need to
specify the NDEF argument.

test: Rename simple-agent to ndef-agent

nfctool: Fix build failure with libnl v1

With libnl v1 neard fails to build with:

tools/nfctool/netlink.c: In function ‘nl_send_msg’:
tools/nfctool/netlink.c:135:2: error: passing argument 1 of
‘nl_send_auto_complete’ from incompatible pointer type [-Werror]
/usr/include/netlink/netlink.h:48:14: note: expected ‘struct nl_handle *’ but
argument is of type ‘struct nl_sock *’
tools/nfctool/netlink.c:153:3: error: passing argument 1 of ‘nl_recvmsgs’ from
incompatible pointer type [-Werror]
/usr/include/netlink/netlink.h:58:14: note: expected ‘struct nl_handle *’ but
argument is of type ‘struct nl_sock *’
tools/nfctool/netlink.c: In function ‘nl_get_multicast_id’:
tools/nfctool/netlink.c:212:2: error: passing argument 1 of
‘genl_ctrl_resolve’ from incompatible pointer type [-Werror]
/usr/include/netlink/genl/ctrl.h:29:14: note: expected ‘struct nl_handle *’
but argument is of type ‘struct nl_sock *’
[...]

Adding the regular compatibility layer fixes it.

adapter: Fix memory leaks on adapter free

Destroy hash tables when removing adapter.

184 (88 direct, 96 indirect) bytes in 1 blocks are definitely lost in loss record 132 of 156
   at 0x4C2B6CD: malloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
   by 0x4E7FA78: g_malloc (in /lib/x86_64-linux-gnu/libglib-2.0.so.0.3200.3)
   by 0x4E92CA2: g_slice_alloc (in /lib/x86_64-linux-gnu/libglib-2.0.so.0.3200.3)
   by 0x4E69A02: g_hash_table_new_full (in /lib/x86_64-linux-gnu/libglib-2.0.so.0.3200.3)
   by 0x418AD7: __near_adapter_create (adapter.c:613)
   by 0x41790F: __near_manager_adapter_add (manager.c:79)
   by 0x41D1CB: get_devices_handler (netlink.c:173)
   by 0x579A4A9: nl_recvmsgs (in /lib/libnl-3.so.200.3.0)
   by 0x41C562: nl_send_msg (netlink.c:136)
   by 0x41D38A: __near_netlink_get_adapters (netlink.c:200)
   by 0x4084F1: main (main.c:210)

184 (88 direct, 96 indirect) bytes in 1 blocks are definitely lost in loss record 133 of 156
   at 0x4C2B6CD: malloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
   by 0x4E7FA78: g_malloc (in /lib/x86_64-linux-gnu/libglib-2.0.so.0.3200.3)
   by 0x4E92CA2: g_slice_alloc (in /lib/x86_64-linux-gnu/libglib-2.0.so.0.3200.3)
   by 0x4E69A02: g_hash_table_new_full (in /lib/x86_64-linux-gnu/libglib-2.0.so.0.3200.3)
   by 0x418AF8: __near_adapter_create (adapter.c:617)
   by 0x41790F: __near_manager_adapter_add (manager.c:79)
   by 0x41D1CB: get_devices_handler (netlink.c:173)
   by 0x579A4A9: nl_recvmsgs (in /lib/libnl-3.so.200.3.0)
   by 0x41C562: nl_send_msg (netlink.c:136)
   by 0x41D38A: __near_netlink_get_adapters (netlink.c:200)
   by 0x4084F1: main (main.c:210)

gdbus: Fix memory leak on properties_set()

The pointer returned by dbus_message_iter_get_signature() must be freed
with dbus_free().

Fixes this memory leak:

==1857== 16 bytes in 1 blocks are definitely lost in loss record 104 of
251
==1857==    at 0x402BF52: realloc (in
/usr/lib/valgrind/vgpreload_memcheck-x86-linux.so)
==1857==    by 0x415E286: dbus_realloc (in
/lib/i386-linux-gnu/libdbus-1.so.3.5.8)
==1857==    by 0x415E70B: ??? (in
/lib/i386-linux-gnu/libdbus-1.so.3.5.8)
==1857==    by 0x415F17B: ??? (in
/lib/i386-linux-gnu/libdbus-1.so.3.5.8)
==1857==    by 0x414CB33: dbus_message_iter_get_signature (in
/lib/i386-linux-gnu/libdbus-1.so.3.5.8)
==1857==    by 0x8053239: properties_set (object.c:899)
==1857==    by 0x5FFFFF: ???
==1857==

neard: nfctool: Add LLCP frame decoding support

nfctool now dumps LLCP frames in a human readable manner

neard: nfctool: Add LLCP traffic sniffing feature

nfctool -d nfcX --sniffer
This dumps LLCP frames to stdout (Hex+ASCII display)

nfctool -d nfcX --sniffer --pcap-file FILENAME
This saves LLCP frames in pcap format to file FILENAME. This pcap file
can be opened in wireshark (v>=1.8.2) with the wireshark-nfc plugin
available at http://code.google.com/p/wireshark-nfc/

neard: nfctool: Add an option to set LTO, RW, and MIUX parameters

nfctool --device nfcX --set-param lto=x,rw=y,miux=z
This sets one or more of the LTO, RW, or MIUX parameters for device nfcX

neard: nfctool: Add start_poll command

nfctool --device nfcX --poll [Initiator|Target|Both]
This starts polling on device nfcX as initiator, target, or both.

neard: Add nfctool native application

nfctool is a native application used as a tool box for various nfc related
operations. First goal is to be able to do such operations on embedded
platforms that don't have a python interpretor (since all neard test tools
are pyhton scripts)

This initial release can list nfc adapters, targets, etc.

AUTHORS: Mention Christophe's contributions

test: Support writing SMS and e-mail address to a tag

gdbus: Check signature of property value before calling setter

gdbus: Don't include just added interfaces in GetManagedObjects

If we received a call to ObjectManager.GetManagedObject we should not
include in the response the interfaces in data->added. This is because
it's not guaranteed that those interfaces will trigger an
InterfacesAdded signal, which is the case if the interface is removed in
the same mainloop iteration.

gdbus: Simplify generated introspection

The generated introspection is not supposed to be read as is by human,
so there's no point in printing the indentation or writing more code to
use auto-close tags.

If it's desired to read the raw xml file, user can always use other
tools to transform the output such as "xmllint --format".

This also fixes a missing </property> when property is deprecated.

gdbus: Hold client reference during get name owner reply

gdbus: Call check_signals when sending signals with g_dbus_send_message

If message passed to g_dbus_send_message is a signal verify if it is a
valid and there really exists an interface with respective signal name.

gdbus: Check if the interface being registered is valid

This prevent registering interfaces that are empty or have all members
marked as experiemental.

gdbus: Introduce G_DBUS_PROPERTY_FLAG_EXPERIMENTAL

This flag can be used to mark properties as experimental, marked
properties are disabled by default and can be enabled by setting
G_DBUS_FLAG_ENABLE_EXPERIMENTAL using g_dbus_set_flags.

gdbus: Introduce G_DBUS_SIGNAL_FLAG_EXPERIMENTAL

This flag can be used to mark signals as experimental, marked
signals are disabled by default and can be enabled by setting
G_DBUS_FLAG_ENABLE_EXPERIMENTAL using g_dbus_set_flags.

gdbus: Introduce G_DBUS_METHOD_FLAG_EXPERIMENTAL

This flag can be used to mark methods as experimental, marked
methods are disable by default and can be enabled by setting
G_DBUS_FLAG_ENABLE_EXPERIMENTAL using g_dbus_set_flags.

gdbus: Include changed properties only once per signal

gdbus: Update properties on D-Bus client re-connections

gdbus: Add function to manually refresh properties

gdbus: Add support for proxy property change notifications

gdbus: Fix handling of client connect/disconnect signals

gdbus: Protect standalone proxy creation with client reference

gdbus: Use object manager only if callback functions are set

gdbus: Use client service name and not hardcoded org.bluez

gdbus: Add support for creating D-Bus proxies without object manager

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.