adapter: Use dbus_bool_t for D-Bus functions

nfctype1: Use the right offset after the CC

The Capability container is 4 bytes long and the NDEF data can be found
right after that. The offset for the static memory on a dynamic tag
should thus be the length of the CC and not more.

src: Remove redundant include config.h

tag.h: Remove header declared but not defined

adapter.h: Remove header declared but not defined

handover: Fix memory leak on reallocation error

A temporay variable is used for checking the realloc() return value.
Otherwise ndef->ndef is leaked when realloc fails as it is then set to
NULL.
With this fix, ndef->ndef is updated iff realloc() succeeds.

Reported by Szymon Janc <szymon.janc@tieto.com>

handover: Fix incorrect NULL check

Need to check the return value of g_try_realloc()
instead of (ndef == NULL).

nfc_copy: Update nfc_copy.h with SE and firmware upload APIs

TODO: Update and cleanup

unit: Add test for Handover Select

This adds test for HS. Currently test is not enabled as it is not
possible to run HO code as unit test now as this code does both
parsing and acting (eg. trying to access HO agent) in same run.

This test is added for reference and should be enabled when handover
code is splitted into parsing and acting functions.

doc: Remove monitor-tag and monitor-device

monitor-near also monitor tags and devices, so let's avoid redundancy.

Fixes #NFC-51

TODO: Add NDEF split

build: Add unit/test-snep-read to .gitignore

ndef: Make near_ndef_prepare_ac_message ready for multibyte cdr

This allows to acctually create multibye cdr if needed. Currently neard
always use one byte cdr but let us be in match with spec and a bit more
future proof.

ndef: Make near_ndef_prepare_cfg_message accept cdr as pointer

This allows to acctually create multibye cdr if needed. Currently neard
always use one byte cdr but let us be in match with spec and a bit more
future proof.

ndef: Remove unused struct near_ndef_carrier_data

This is not used anywhere in the code.

ndef: Make cdr in near_ndef_ac_payload multibyte

cdr is a multibyte field. This removes assumption in code that cdr is
one byte only.

doc: Document the Push method dictionary content

device: Remove the SetProperty method

Records are read only.

manager: Remove the SetProperty method

The neard Manager properties are read only and they only contain the
Adapters.

adapter: Restart polling when failing to connect to a new target

If a newly detected target failed to be connected then a new polling
loop is started if constant poll is enabled.

Signed-off-by: Samuel Ortiz <sameo@linux.intel.com>

AUTHORS: Mention Louis' contributions

ndef: Add raw MIME payload support

Additional to wifi_wsc MIME type, raw parameter is provided for
sending raw MIME payload as NDEF.
See test/push-device for usage.

main: Use signalfd instead of raw signals

Copied from ConnMan's commit 6ad51830 where Daniel Wagner was
explaining:

   "It's unsafe to call syslog in the terminated signal handler
    because syslog takes a lock. So when the signal handler
    kicks in and we were already in syslog, we have a nice
    deadlock."

ndef: Display AAR package name as a debug string

device: Cancel pending SNEP message when removing a device

adapter: Add a DEP link establishment timer

Once a device in target mode is found, the polling loop restarts after 1
second if the dep_link_is_up is not received.

nfctool: Add a null pointer check in llcp_decode_cleanup

g_hash_table_destroy raises an assert when called with a null pointer.
connection_hash can be NULL when sniffer_init() fails before calling
llcp_init().

unit: Add Android Application Record NDEF record unit test

ndef: Parse AAR payload

Android Application Record payload simply contains the Android Market
package name.

ndef: Initial support for Android Application Records

AAR are Android external NDEF records that contain an Android package
name.

test: Add a device monitoring script

It displays the device detection event and the received NDEF record
payloads if something is received from the device.

test: Add a tag monitoring script

It displays the record payload available from a detected tag.

nfctool: sniffer: Fix ndef read-ahead check macro

This macro checks if we can safely read the next field in a ndef
record. So the check needs to be done at the current read position
in the current record.

Release 0.12

device: Pass the NDEF push cb status back to the caller

Instead of hard coding it to EINVAL.

error: Add I/O error routine

ndef: Support receiving empty Handover select frames

Empty handover select frames are received when the peer notifies us
about not supporting any of the carrier sent in the handover request
frame.
In that case there is nothing to do but disconnecting from the handover
service.

ndef: Send empty WiFi carrier in Hr message when running as a STA

When running as a 802.11 STA, the Hr should contain an empty WiFi
carrier as we're just asking for WiFi availability from the peer.
Also when running in AP mode, an Hr is not a valid frame to send as
an AP will not be able to initiate the association.

ndef: Implement static handover

static handover is going through a different D-Bus path as it would not
make sense to handle both static and dynamic handover from the same code
path.
Static handover means an application explicitely requires to send a
handover select without having previously received a handover request.
This could typically be used for handling WiFi handover through
configuration token from the registrar perspective. The registrar
(a.k.a. the Access Point) would send a configuration token via Handover
and expects the Enrolle (a.k.a. the WiFi STA) to initiate the WiFi
association.

ndef: Generalize near_ndef_prepare_hr_message()

near_ndef_prepare_hr_message can be used for building Hr frames but also
Hs ones for static handover. So generalize and factorize it and also
rename it to near_ndef_prepare_ho_message().

test: Check for dump-* scripts argument

Check that there at least is one argument and dump a usage example
otherwise.

test: Explicitely unregister the NDEF agent when being interrupted

ndef: Rename near_ndef_prepare_hs_message

near_ndef_prepare_hs_message() does not prepare a generic handover
select message but rather a reply to a handover request.
Those 2 have to be differentiated in order to implement static handover
where a handover select frame is sent without having first seen a
handover request.

ndef: Rename build_ho_record

build_ho_record is used for initiating negotiated handover, so it will
only send handover requests.
As such, rename build_ho_record to build_hr_record.

nfctool: sniffer: Dump unknown WSC data elements

Don't drop them silently.

gdbus: Remove proxies when client disconnects

Remove proxies and generate proxy_removed callbacks even when there are
no corresponding InterfaceRemoved signals.

This patch fixes having zombie gdbus proxy object when a server
disconnects without sending InterfaceRemoved signals. These objects
may interact with new server instances, for example, making
InterfaceAdded signals of new objects with the same name be filtered
out as duplicated, or staying allocated, but unused, if the new server
doesn't reuse the object paths. Note that as a side-effect, the
lifetime of a gdbus proxy becomes stricter: it lives at most for the
duration of a single connection to a single instance of a server
process.

nfctool: sniffer: Decode application/vnd.wfa.wsc MIME records

test: Add MIME type to the Device pushing options

Only wifi_wsc is supported for now, i.e. one can only push WSC
configuration tokens over SNEP.

ndef: Fallback to the WiFi agent when missing an SSID

When the D-Bus message does not contain an SSID (and optionalyy a
passphrase) then we fallback to the WiFi WSC agent. If it's running
in tethering mode then it will reply with an already built WSC TLV
array.
This is needed to push WiFi WSC configuration token over SNEP. Android
is going to support that as a simplified WiFi Handover.

nfctype4: Fix segmentation fault on invalid access key

If length is < 0, further access to the resp buffer will cause neard to
seg fault.

bluetooth: Fix memory leaks on error or when not pairing

Warned-by: cppcheck

npp: Fix memory leak on allocation error

The pointer to a realloc() function does not get touched on error, so
we need to check if the function failed and either free or update the
pointer.

Warned-by: cppcheck

p2p: Coding style violation fixes

p2p: Check for an already existing p2p driver from near_p2p_register

If there already is a driver handling this service name, -EALREADY is
returned.

nfctool: sniffer: Print Handover I frames as NDEF

Handover information frames are actually pure NDEFs, so decode them as
such since it is now possible to detect Handover specific traffic.

nfctool: sniffer: CONNECT and CC frames tracking

In order to map an SAP with the righ service name, CONNECT and CC frames
need to be tracked along with their dsap/ssap.
When finding an SN, it means a CONNECT frame has been monitored. The SN
string is then associated with the SSAP. Upon CC sniffing, the DSAP is
fetched and if a pending SN for this DSAP is found, the service name
lookup is done.
With this mechanism it is now possible to know the e.g. the Handover SAP
and decode its I frames as pure NDEF ones.
This tracking will only work with connection by name (Which is what the
vast majority of platforms do). An enhancement would be to track SNL
packets as well.

nfctool: sniffer: Handle Bluetooth Handover records

If the MIME string is "application/vnd.bluetooth.ep.oob" then the
Bluetooth OOB data is parsed and output appropriately.
AC and CR records are not parsed yet.

nfctool: sniffer: Output the MIME type value

This is needed for handover sniffing.

nfctool: sniffer: A record's TNF is not a boolean

A record TNF (Type Name Format) is on 3 bits and thus should not be the
result of a boolean comparison.

nfctool: sniffer: Support multiple records per NDEF

Two offsets are needed to track the record current pointer and the NDEF
one.

agent: Do not call NDEF agent Release when agent leaves

The Release method should only be called when neard leaves. neard
doesn't need to notify an agent that's explicitly leaving.

test: Fix Text NDEF representation

The Text NDEF representation could be non ASCII strings (UTF-8 or
UTF-16) and the dumping scripts need to know about that.

Fixes #NFC-48

ndef: Do not fail if NDEF record MIME type is not a handover one

NDEFs with MIME type records that are not WSC or Bluetooth ones are not
wrong. Their payload won't be processed but the record is kept and
linked to the discovered tag.
NDEF agents will get the binary payload.

Fixes #NFC-45

adapter: Pass the right adapter index when auto powering adapters

When calling __near_netlink_adapter_enable(), adapter->idx is not yet
set. The right adapter index to pass is the __near_adapter_create()
argument.

validation: Add MIUX support

The MIUX value is now read from the socket options, allowing for a dynamic
SDU buffer allocation.

snep: Fix fragmented GET

When receiving a CONTINUE while nfc_data_length is set to 0, it must
be a CONTINUE for a fragmented GET. The headers are thus read in order
to modify snep_data->request and then process the right request.
This fixes the GET fragmented unit test.

snep: Fix SNEP fragmentation

The intial request is kept through the snep_data and
snep_core_process_request no longer relies on the last read frame to
build an answer but only on the snep_data pointer.
Without this fix, each fragment from a fragmented PUT was handled as
a complete SNEP frame (header included) which caused process_request
to get completely confused about how to handle fragments.
Now that snep_core_read is fixed, the fragmentation SNEP unit test can
be fixed as well.

validation: Fix memory leak from llcp_send_data

When send or sendto fails the SDU still needs to be freed.

doc: Fix NDEF agent GetNDEF prototype

GetNDEF does not return any dictionary, it takes one as a parameter.

nfctype4: Add support for type4 B tags

4a and 4b tags follow the same specs: the type 4 one from the NFC
Forum.

tag: Set type B tag type properly

When receiving a 14443-B protocol mask, we should set the tag type to
NFC-B.

nfctool: sniffer: Fix fragmented SNEP output

Display the right string depending ont the direction.

p2p: Fall back to blocking sockets if the async connection fails

Due to a kernel bug, trying to connect through a non blocking socket
may fail and return POLLHUP. In that case the connection procedure falls
back to using blocking sockets and this needs to be delayed until the
kernel sends a DM to the peer.
This is slightly hackish but this code path should get obsolete once
neard runs on top of 3.10 or later kernels.

p2p: Non blocking client socket

The p2p client connection used to be a blocking one, which can be quite
harmful when for some reason the peer does not reply to a connection
request.
The client socket is non blocking now and the actual push is handled
by a GLib watch callback.

p2p: Set connection timeout

The connect() call should not block forever. The 8 seconds timeout makes
it for approximately 3 times the maximum LTO and that seems to be plenty
of time for a peer to reply to an LLCP connection.

nfctool: sniffer: Set the time string first character to 0

If the show_timestamp option is disabled, the string needs to be properly
terminated.

nfctool: Power up the device when start poll

with --poll, --enable is now implicit

nfctool: Add command line options to power up or down a device

--enable (-1 for short) turns the specified device on
--disable (-0) turns it off

If both are specified, only --enable is applied

gdbus: Fix using NULL DBusPendingCall when disconnected from D-Bus

From D-Bus documentation for dbus_connection_send_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."

Fix these errors when killing D-Bus daemon with the client still
running:

process 5712: arguments to dbus_pending_call_set_notify() were
incorrect, assertion "pending != NULL" failed in file
../../dbus/dbus-pending-call.c line 596.
This is normally a bug in some application using the D-Bus library.
process 5712: arguments to dbus_pending_call_unref() were incorrect,
assertion "pending != NULL" failed in file
../../dbus/dbus-pending-call.c line 572.
This is normally a bug in some application using the D-Bus library.

gdbus: Fix segfault when D-Bus daemon exits

Fix this crash if D-Bus exits while the client is still connected to it:

==5570== Invalid read of size 1
==5570==    at 0x402D28E: strcmp (in
/usr/lib/valgrind/vgpreload_memcheck-x86-linux.so)
==5570==    by 0x4070E22: g_str_equal (ghash.c:1704)
==5570==    by 0x8055F61: message_filter (client.c:1123)
==5570==    by 0x4141500: dbus_connection_dispatch (in
/lib/i386-linux-gnu/libdbus-1.so.3.5.8)
==5570==    by 0x80506F7: message_dispatch (mainloop.c:76)
==5570==    by 0x4081A7E: g_timeout_dispatch (gmain.c:3882)
==5570==    by 0x4080D85: g_main_context_dispatch (gmain.c:2539)
==5570==    by 0x4081124: g_main_context_iterate.isra.21 (gmain.c:3146)
==5570==    by 0x408156A: g_main_loop_run (gmain.c:3340)
==5570==    by 0x41BF4D2: (below main) (libc-start.c:226)
==5570==  Address 0x0 is not stack'd, malloc'd or (recently) free'd
==5570==
==5570==

gdbus: Fix calling GetManagedObjects twice in a row

Calling g_dbus_client_new followed by g_dbus_client_set_proxy_handlers
cause two calls to GetManagedObjects in a row as GetNameOwner reply is
asyncronously it triggers the second call because the handlers have
been set by g_dbus_client_set_proxy_handlers.

nfctool: sniffer: Add NDEF decoding support

This adds support for decoding NDEF records embedded in SNEP packets

nfctool: sniffer: Reduce hexa dump line width

If we display less than 0xFFFF bytes (which will be the case 99.999%
of the time) the offset value is truncated and the 2 first zero bytes
are not displayed.

nfctool: sniffer: Add a snep_printf_error

This will print SNEP decoding errors in bold red

nfctool: sniffer: Fix over 80 chars macro

nftool: sniffer: Remove new line from snep_printf_msg calls

snep_printf_msg already contains one.

nfctool: sniffer: Color LLCP dumps

nfctool: sniffer: Simplify print_pdu routine

And prepare for more colors...

nfctool: Initial color support

SNEP decoding is colored now, for a more readable output.

nfctool: sniffer: Change the sniffer hexdump prototype

Take an indentation length instead of a generic prefix, as this is
what's only used anyway.

nfctool: sniffer: Fix SNEP output string

SNEP stands for Simple NDEF Exchange Protocol, not Messaging Protocol.

nfctool: sniffer: Add SNEP decoding

This adds support for decoding Simple NDF Message Protocol. The SNEP decoder
supports aggregation of fragmented messages.

nfctool: sniffer: ssap and dsap were displayed switched

In the LLCP header of a received frame, dsap is the local port and ssap is the
remote port. ssap and dsap fields of the sniffer_packet structure have been
renamed to local_sap and remote_sap to avoid confusion.

nfctool: sniffer: Group sniffed packet info into the same struct

Raw socket header (direction and adapter index) and LLCP info are now stored
into the same struct (sniffer_packet). This structure is then passed to PDU
specific decoding functions.

nfctool: Fix a memory leak

The handlers hash table was not freed

nfctyp3: Set cookie cb to NULL when read succeeds

Otherwise the callback function may be call twice and
write_msg is unreferenced twice:

process 1868: arguments to dbus_message_unref() were incorrect,
assertion "message != NULL" failed in file dbus-message.c line 1616
0  0xb7fe1424 in __kernel_vsyscall ()
1  0xb7bcfcc1 in raise () from /lib/libc.so.6
2  0xb7bd30ee in abort () from /lib/libc.so.6
3  0xb7da5045 in _dbus_abort () from /usr/lib/libdbus-1.so.3
4  0xb7d99f86 in _dbus_warn_check_failed () from /usr/lib/libdbus-1.so.3
5  0xb7d8856e in dbus_message_unref () from /usr/lib/libdbus-1.so.3
6  0x08062a63 in tag_read_cb (adapter_idx=0, target_idx=6, status=0)
    at src/tag.c:240
7  0x08063277 in near_tag_add_records (tag=0x8081a30, records=0x8080270,
    cb=0x8062a30 <tag_read_cb>, status=0) at src/tag.c:814

bluetooth: Get name value from right message iterator

We need to get name value from var instead of the container iter
else it may also cause neard crash:
process 1044: type variant 118 not a basic type
0  0xb7fe1424 in __kernel_vsyscall ()
1  0xb7bcfcc1 in raise () from /lib/libc.so.6
2  0xb7bd30ee in abort () from /lib/libc.so.6
3  0xb7da5045 in _dbus_abort () from /usr/lib/libdbus-1.so.3
4  0xb7d99f86 in _dbus_warn_check_failed () from /usr/lib/libdbus-1.so.3
5  0xb7d9bc9c in _dbus_marshal_read_basic () from /usr/lib/libdbus-1.so.3
6  0xb7d84e0f in _dbus_type_reader_read_basic () from /usr/lib/libdbus-1.so.3
7  0xb7d89f0b in dbus_message_iter_get_basic () from /usr/lib/libdbus-1.so.3
8  0x0806b638 in bt_adapter_property_changed (conn=0x80814c8,
    message=0x8081870, user_data=0x0) at src/bluetooth.c:445

Release 0.11

unit: Fix broken SNEP read unit test

In "crash-1" loop, the snep_data was added to the slist, but not removed
because of a specific exit condition. This fix swaps the test condition
and the action on slist, in order to add the fragment only when the length
test passed.