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.

nfctool: Correct unsigned int comparison

An uint can't be less than 0...

nfctool: Make good use of snap_len option

snap_len now really limits sniffer output.

nfctool: Set sniffer buffer large enough to handle MAX_MIU bytes frames

This fixes the size of the recv buffer for the sniffer. Frames can be
up to 7ff+128+(llcp, seq, and raw headers) bytes.

main.conf: Add DefaultPowered configuration

DefaultPowered is a boolean and will force neard into turning any detected
adapter on when set to TRUE.

gdbus: Use gcc builtin instead of g_atomic

g_atomic_* end up using G_STATIC_ASSERT, causing gcc 4.8 to yell due to
-Wunused-local-typedefs.

gdbus/client.c: In function ‘g_dbus_client_ref’:
/usr/include/glib-2.0/glib/gmacros.h:162:53: error: typedef ‘_GStaticAssertCompileTimeAssertion_2’ locally defined but not used [-Werror=unused-local-typedefs]
 #define G_STATIC_ASSERT(expr) typedef char G_PASTE (_GStaticAssertCompileTimeAssertion_, __COUNTER__)[(expr) ? 1 : -1]

gdbus: Fix not calling property_changed callback

In case a property did not appear by the time proxy_added was called
property_changed has to be called if it appear latter otherwise the
application will be unaware of it.

llcp: Validation test server

This llcp validation server follows NFC Forum requirements and scenarios
and supports the initial list of 9 test cases.
Most of these tests rely on sending frames to an echo server, using
connection less or connection oriented modes.

p2p: Add support for connection less binding

Some validation tests are done in connection less mode. near_p2p_driver
now handles not only stream socket but also datagram socket.

unit: Add WSC NDEF build test case

Added test cases to test WSC MIME type NDEF build.

ndef: Add api to prepare mime type wsc record

Added api to prepare MIME type WSC record for unit tests.

ndef: Remove unnecessary debug statement

Removed unnecessary debug statement which is added with
56a275c8841352a9c8a60bdcb3e63ba69fe783c6 commit.

nfc_copy: Update with new socket options

unit: Support for SNEP REJECT request

unit: Fixed cleaning SNEP fragments

If sending the fragment fails, server should remove all
the remaining fragments.

test: Add test script to write mime type on tag

Added test script to write mime type on tag. Currently it
supports application/vnd.wfa.wsc mime type.

ndef: Add support to build wifi_wsc mime type ndef record

Implemented mime type record building upon dbus request.
Currently it supports application/vnd.wfa.wsc.

nfctool: Print length of buffer before its hexadecimal dump

nfctool: Fix some integer sizes and signedness

Also, use glib types to be coherent.

nfctool: Fix aggregated frame support

This adds the llcp sniffer pseudo header to PDUs extracted from the
aggregated frames before re-entering llcp_print_pdu.

ndef: Remove not used Doxygen annotations

As previous commit shows doxygen annotation are not really used.
Those annotations require also more focus when reading (and mostly
describe obvious parts of the code) comparing to simple comments.
Also not used doxygen annotations tend to drift away from actual code
and simply confuse developer.

Simply remove doxygen annotations as functions and params names are
self-explanatory anyway. Some comments where left to document
non-obvious behaviour.

ndef: Remove bogus text from near_ndef_parse_msg comment

This was accidentally pasted in commit 8a141753.

ndef: Use sizeof instead of strlen to get BT_MIME_STRING_2_0 len

This is a constant string and there is no need to calculate its length
in runtime.

ndef: Fix some typos in comments and debug messages

ndef: Simplify near_ndef_prepare_ac_and_cfg_records function

Make error path a bit cleaner and easier to understand. This also makes
code less confusing to some GCC version (eg. 4.6.3 on Debian ARM) about
variables mime_type and cdr being used without initialization.

make --no-print-directory all-am
  CC     src/ndef.o
src/ndef.c: In function ‘near_ndef_prepare_ac_and_cfg_records’:
src/ndef.c:1758:6: error: ‘mime_type’ may be used uninitialized in this
  function [-Werror=uninitialized]
src/ndef.c:1781:8: note: ‘mime_type’ was declared here
src/ndef.c:1835:36: error: ‘cdr’ may be used uninitialized in this
  function [-Werror=uninitialized]
cc1: all warnings being treated as errors
make[1]: *** [src/ndef.o] Error 1
make: *** [all] Error 2

snep-validation: Fix compilation on ARM due to unaligned memory access

Use near_get_be32 helper function to access nfc_data. Although there is
most likely no risk of unaligned access since nfc_data is allocated
from heap (and properly alligned as access starts from first byte) this
result in compilation error on ARM (Debian GCC 4.6.3).

plugins/snep-validation.c: In function ‘snep_validation_server_req_get’:
plugins/snep-validation.c:155:22: error: cast increases required alignment
  of target type [-Werror=cast-align]
cc1: all warnings being treated as errors
make[1]: *** [plugins/snep-validation.o] Error 1
make: *** [all] Error 2

p2p: Support for single connection server

The LLCP validation tests require to reject any connection once a client
is connected. Which means a new listen should start iff the current
connection is closed. For that a single_connection field is added to the
p2p_driver structure and the p2p_bind code is factorized so that the
client close hook can call it without re-allocating a server data pointer.

main: Wait to daemonize to just before entering the main loop

To be able to get correct exit codes in case of error, we have to delay
the daemonization to just before entering the main loop. Otherwise the
init scripts might assume everything started fine and go ahead when that
was not the case.

The initialization error messages are printed to syslog instead of
stderr, which would need a rework in the future of either the logging
functions, or the call sites to distinguish what belongs where. But at
least for now we get correct exit values.

unit: Sanitize NDEF test names

unit: Sanitize SNEP test names

AUTHORS: Mention Marcin's contributions

unit: Remove SNEP routines forward declarations

unit: Add SNEP PUT/GET NDEF test

unit: Add SNEP GET request "fragmented" test

unit: Add SNEP GET request "not implemented" test

unit: Add SNEP GET request "ok" test

unit: Add SNEP PUT request "fragmented" test

unit: Add SNEP PUT request "feature not implemented" test

unit: Add SNEP PUT request "unsupported version" test

unit: Add SNEP PUT request "ok" test

unit: Add SNEP test utilities

build: Remove unused linking of neard against libresolv

nfctool: Fix pointer casting errors on ARM

On GLib 2.28, Macro GINT_TO_POINTER on ARM will fail
as it won't cast implicitly to int before casting to gpointer.

It results in casting uint8 directly to void *, which will
not do the job on some ARM processors.

Error message:

tools/nfctool/netlink.c: In function ‘nl_nfc_event_cb’:
tools/nfctool/netlink.c:686:39: error: cast to pointer from
integer of different size [-Werror=int-to-pointer-cast]
tools/nfctool/netlink.c: In function ‘nl_add_event_handler’:
tools/nfctool/netlink.c:731:35: error: cast to pointer from
integer of different size [-Werror=int-to-pointer-cast]

configure: Remove init script skeleton

All distros use their own flavor of it, and several are systemd
based. An init script skeleton is probably useless.

AUTHORS: Mention Guillem's contributions

build: Generate correct dependencies for the pkg-config file

This fixes the bogus libnl dependency in the pkg-config file, and
centralizes all dependencies in configure.ac, to replace them at
configure time so that they accurately define the ones detected.
This also makes the build system future-proof against changing
dependencies in one place and not the other.

configure: Add scripts/neard to the configure output

ndef: Fix typo in unknown carrier string

AUTHORS: Mention Marcin's contributions

snep: SNEP core moved from plugins to core

This is needed for implementing SNEP unit tests.

build: Install nfctool to @bin@

doc: Add nfctool manpage

nfctool: Add --version option

The nfctool version number is based on neard's one.

nfctype2: Get the actual tag data length when trying to write an NDEF

tag: Export tag data length

ndef: Fix local carrier data fetching error message

Release 0.10

build: Add agent-api.txt to the document files list

doc: Add neard man pages

One man page for the daemon and another one for the configuration file.

build: Install neard back to libexec/nfc/

build: Install neard to /usr/sbin

nfctool: Add Service Name Lookup support

This adds a --snl option to get the sap number for the passed service name
(i.e. urn:nfc:sn:snep). nfctool will wait until it receives a result for
all the requested services before exiting.

nfctool: Add a handler mechanism for netlink events

Instead of one callback function in main.c for all netlink events, these
changes allow to register per event callback functions. this will be used
more intensively for service name lookup support.

main: Big fat warning for netlink initialization failure

snep: Fix missing call to snep_exit

snep_core should call snep_exit to unregister p2p driver.

16 bytes in 1 blocks are still reachable in loss record 31 of 153
   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 0x4E93ABD: g_slist_prepend (in /lib/x86_64-linux-gnu/libglib-2.0.so.0.3200.3)
   by 0x4178B7: near_p2p_register (p2p.c:382)
   by 0x418DE5: snep_core_init (snep-core.c:820)
   by 0x4173BC: p2p_init (p2p.c:399)
   by 0x41FC61: __near_plugin_init (plugin.c:180)
   by 0x4092AE: main (main.c:214)

main: Exit neard if netlink initialization failed

There is no point in going further without netlink operational.

bluetooth: Fix possible NULL pointer dereference

If DBus message allocation failed NULL pointer msg would be
passed to libdbus functions later on resulting in libdbus NULL pointer
assertion or crash.

nfctyp3: Remove set but not used variable

system_code in receive_system_code function was written but never read.

build: Add some missing files to .gitignore

Add unit test binaries to ignore list.

test: Add wifi carrier support for handover_agent test

Currently received data in PushOOB/RequestOOB is printed
and reply to RequestOOB ss generated from static data.

org.neard.conf: Add the HandoverAgent interface sending permissions

agent: Add WSC field to handover agent API

WiFi out of band data is added to PushOOB request.
Field name is WSC. OOB data format is following the WiFi simple
configuration technical specification.

ndef: Add support for multiple carriers

Support for mutliple carrier handling for tag and p2p device.

In case of tag (tag with simple mime type or with static
handover data): First detects number of carriers in a tag
and try to connect/associate with carrier. If connection or
association failed then go for next carrier otherwise return
success.

In case of p2p device (either neard receiving Hr and sending Hr):
If neard receives Hr message with multiple carriers (e.g. bt and
wifi), first extracts multiple carriers data and reply back with
neard's alternative carrier capacity (e.g. It could be 'bt' or 'wifi'
or both or empty). If neard wants to negotiate handover with other
p2p device it sends Hr with available carriers (e.g. bt ot wifi or
both, if there are no carriers replies error to dbus request).
After receiving Hs from other p2p device, it tries to
connect/associate with carriers (ho agents).

ndef: Add support for multiple carriers in Hr message

Added support for multiple carrriers request for Hr message. For
multiple carriers dbus request has to be comma separated strings
(e.g. bluetooth,wifi).

ndef: Implement Hr message preparation routine

To support multiple carriers in handover request message, implemented
seperate function for handover request message.

ndef: Implement Hs message preparation routine

To support multiple carriers in handover select message, implemented
seperate function for handover select message.

ndef: Add helper funtions to create handover messages

Providing helper functions to create Hanodver select and
request messages.

ndef: Return empty Hs in case of unsupported Hr version

Return empty handover select message for unsupported versions of
handover request message.

Implementing separate functions for empty Hs, Hs with carriers
and Hr messages. Next patches contain these functionality.
Separating them because each message requires different
input parameters and preparation conditions. Handover message
preparation is bit different and bigger than other ndef message
types. Also handover messages contain multiple alterative
carriers, configuration messages, collision and error records.

ndef: Add helper function to free ndef messages

Added helper funtion to free ndef message structure data.
It is quite helpful to reduce code redundancy.

nfctool: Use nfc_copy.h instead of the system NFC header

nfctool: Move adapter related functions in one place

This moves adapter management functions in adapter.c

gdbus: Always unreference the message in g_dbus_send_message()

A quick fix on g_dbus_send_message(), if check_signal() fails it returns
FALSE without unreferencing the message as it should. This patch fixes
it.

nfc_copy: Add SNL netlink API

nfc_copy: Add LLCP socket option names

p2p: Remove client data from the server client list when channel closes

nfctool: Close sniffer socket when getting IO errors

snep-validation: Remove DBG statement from init and exit routines

device: Do not double free the device records list

Typo from the previous commit...

device: Do not open code the records freeing routine

Use near_ndef_records_free instead.

monitor-near: Monitor Tags and Devices properties

tag: Send PropertyChanged signal when new records are added

device: Send PropertyChanged signal when new records are added

p2p: Do not try to bind if LLCP is not supported

gdbus: Add g_dbus_proxy_set_removed_watch