p2p: Fix not freeing server_data in free_server_data

Free server_data when clearing servers list. This fix following issue
reported by valgrind:

84 bytes in 3 blocks are definitely lost in loss record 167 of 210
   at 0x402BE68: malloc (in /usr/lib/valgrind/vgpreload_memcheck-x86-linux.so)
   by 0x4095C5A: ??? (in /lib/i386-linux-gnu/libglib-2.0.so.0.3306.0)
   by 0x40961DC: g_try_malloc0 (in /lib/i386-linux-gnu/libglib-2.0.so.0.3306.0)
   by 0x8056853: p2p_listen (p2p.c:247)
   by 0x407FA21: g_hash_table_foreach (in /lib/i386-linux-gnu/libglib-2.0.so.0.3306.0)
   by 0x805BA2F: __near_adapter_listen (adapter.c:1048)
   by 0x805C64D: near_device_driver_register (device.c:461)
   by 0x80567E2: p2p_init (p2p.c:374)
   by 0x805E317: __near_plugin_init (plugin.c:180)
   by 0x804E0E1: main (main.c:214)

bluetooth: Fix EIR parsing

Fix parsing of EIR data to avoid reading data from wrong offset.
This also fix invalid memory access reported by valgrind which could
potentially result in daemon crash.

neard[30296]: src/bluetooth.c:bt_parse_eir()
neard[30296]: Unknown EIR: x03 (len: 4)
neard[30296]: Unknown EIR: x00 (len: 253)
==30296== Invalid read of size 1
==30296==    at 0x8064DF8: __near_bluetooth_parse_oob_record (bluetooth.c:531)
==30296==    by 0x8060DF3: parse_mime_type.isra.3 (ndef.c:1299)
==30296==    by 0x8062043: near_ndef_parse (ndef.c:2124)
==30296==    by 0x8063BB7: near_tlv_parse (tlv.c:95)
==30296==    by 0x8053252: data_recv (nfctype2.c:148)
==30296==    by 0x8059D03: execute_recv_cb (adapter.c:846)
==30296==    by 0x408D1BF: ??? (in /lib/i386-linux-gnu/libglib-2.0.so.0.3306.0)
==30296==    by 0x408FEF2: g_main_context_dispatch (in /lib/i386-linux-gnu/libglib-2.0.so.0.3306.0)
==30296==    by 0x409028F: ??? (in /lib/i386-linux-gnu/libglib-2.0.so.0.3306.0)
==30296==    by 0x40906EA: g_main_loop_run (in /lib/i386-linux-gnu/libglib-2.0.so.0.3306.0)
==30296==    by 0x41E44D2: (below main) (libc-start.c:226)
==30296==  Address 0x4556405 is 13 bytes inside a block of size 22 free'd
==30296==    at 0x402B06C: free (in /usr/lib/valgrind/vgpreload_memcheck-x86-linux.so)
==30296==    by 0x4095FCA: ??? (in /lib/i386-linux-gnu/libglib-2.0.so.0.3306.0)
==30296==    by 0x409613F: g_free (in /lib/i386-linux-gnu/libglib-2.0.so.0.3306.0)
==30296==    by 0x805CCAF: near_tag_get_tag (tag.c:94)
==30296==    by 0x8052F36: meta_recv (nfctype2.c:221)
==30296==    by 0x8059D03: execute_recv_cb (adapter.c:846)
==30296==    by 0x408D1BF: ??? (in /lib/i386-linux-gnu/libglib-2.0.so.0.3306.0)
==30296==    by 0x408FEF2: g_main_context_dispatch (in /lib/i386-linux-gnu/libglib-2.0.so.0.3306.0)
==30296==    by 0x409028F: ??? (in /lib/i386-linux-gnu/libglib-2.0.so.0.3306.0)
==30296==    by 0x40906EA: g_main_loop_run (in /lib/i386-linux-gnu/libglib-2.0.so.0.3306.0)
==30296==    by 0x41E44D2: (below main) (libc-start.c:226)

snep: Remove unreachable code in snep_read

snep: Fix possible memory leak in snep_read

Free previously allocated snep_data if nfc_data allocation failed
before returning error.

nfctype3: Always free cookie in case of error

Free cookie allocated in nfctype3_recv_UID in case of error
irrespectively of callback being present.

nfctype3: Fix possible memory leak in nfctype3_read

If call to near_adapter_send() failed cookie should be freed before
returning.

nfctype3: Fix possible NULL pointer dereference in nfctype3_read

Check if memory allocation succeed before dereferencing pointer.

nfctype3: Fix possible NULL pointer dereference in nfctype3_recv_UID

Check if memory allocation succeed before dereferencing pointer and
free any allocated memory before returning. This also fix memory leak
if call to near_adapter_send failed.

nfctype2: Fix possible NULL pointer dereference in nfctype2_format

Check if memory allocation succeed before dereferencing pointers and
free any allocated memory before returning. This also fix memory leaks
if call to near_adapter_send failed.

nfctype2: Fix possible NULL pointer dereference in data_write

Check if memory allocation succeed before dereferencing pointer.

nfctype2: Fix possible memory leak in nfctype2_read_meta

If call to near_adapter_send() failed cookie should be freed before
returning.

nfctype2: Fix possible NULL pointer dereference in nfctype2_read_meta

Check if memory allocation succeed before dereferencing pointer.

nfctype1: Fix possible NULL pointer dereference in nfctype1_read

Check if memory allocation succeed before dereferencing pointer.

mifare: Fix possible NULL pointer dereference in mifare_read

Check if memory allocation succeed before dereferencing pointer.

Fix some common spelling errors and typos

Fix some whitespace issues

TODO: Change Mifare writer mode support owner

nfctype4: Call write callback when write is succesful

tag: Verify tag content only when the write part succeeded

doc: Document the Dual polling loop mode

Dual means the polling loop will alternate between initiator
and target modes.

device: Don't try to push an NDEF when the DEP link is down

adapter: Add a DEP state accessor

agent: Move handover agent implementation to agent.c

manager: Implement NDEF agent registration API

agent: Initial implementation

The NDEF agent API will allow apps to register for record types and get the
corresponding NDEFs back.

ndef: Copy each record raw data

Applications may need raw data for external type records.

ndef: Keep track of the record header length

doc: Update handover agent API

Update API with PushOOB() method that is used to pass OOB data to agent
so it can prepare alternative carrier link.

doc: Fix handover agent API description

OOB data given by agent is used for both Hr and Hs.
Hash must be present if Randomizer is present in dictionary.

nfctype1: Add support for blank tag handling

When using a new tag for the first time, the CC field might not be
initialized (un-formatted tag) this was causing the tag to be dropped.
this patch add handling for such tags:

    * When a tag with uninitialized CC block is read, it is marked as blank
      and is not dropped.
    * Add a new nfctype1_format() function used for initializing the CC block
      in blank t1 flags.
    * When writing data to a tag, in case the tag is marked as blank, this api
      can be used in order to format the tag before the data is being written
      to it.

tag: Improve tag formatting and writing operation

In format_cb() routine, checking if there is a pending message.
If there is no such thing, it simply returns.
If there is a pending message:
   1) If the formatting failed, sends a D-Bus message back to notify
      the writer that the operation failed.
   2) If the formatting succeeded, calling __near_tag_write() again.
      Making sure tag is no longer blank and calling writing operation.

doc: Add Devices to the adapter properties

doc: Fix RequestOOB method description

The OOB data given by the agent is used to build an Hr, not an Hs.

tag: Update tag content after write record

After write record to tag, we should update the tag content by reading
from the adapter.

doc: Sync coding-style.txt with oFono

Revert "build: Use -Wshadow compiler option"

build: Use -Wshadow compiler option

handover: Avoid shadowing connection variable

ndef: Use uri_prefixes to avoid a shadowed variable

AUTHORS: Mention Jeff's contributions

Edit a tag from neard-ui.py

In neard-ui.py, double click a record will display a window. User
can edit the record and write back to tag

netlink: Remove undefined but declared target activation routines

Target activation is done by the kernel when connecting to it.

gdbus: do not check signature twice

Message signature is already checked in generic_message(), so there's no
need to check again in the callback.

gdbus: Fix incorrectly discarded signals

Signals with no arguments were incorrectly filtered out due to the NULL
inequality check.

gdbus: Fix removal of filter after last filter_data

If there's a signal watch that's also watching for name
(data->name_watch) currently we are trying to remove the message_filter
twice since we may have the following call chain:

filter_data_remove_callback()
  filter_data_free()
    g_dbus_remove_watch()
      filter_data_remove_callback()
filter_data_free()
        dbus_connection_remove_filter()
  dbus_connection_remove_filter()

Because of this we can't currently watch for signals passing the bus
name. After this patch we don't have this issue anymore.

We fix it by removing the filter before calling filter_data_free() if we
are the last filter_data and thus avoid calling
dbus_connection_remove_filter() twice.

Release 0.5

handover: Free Hs record after sending it

device: Free the pushed NDEF instead of offloading that to p2p plugins

p2p: Free all p2p plugins data before calling their exit routines

The exit routine will delete the hash tables that free_server_data() might
try to destroy as well, triggering a GLib warning.

device: Async push_ndef returns NULL when reply is pending

The push_cb routine will send the actual reply back.

handover: Factorize push client data freeing routine

free_hr_push_client also calls the push callback.

handover: Remove extra handover_close call

handover: Free handover records

tag: Call near_ndef_records_free when freeing tag records

ndef: Add record list freeing routine

handover: Free allocated handover record if parsing fails

ndef: Implement Hr record building

Although only Bluetooth is supported right now, the Hr record building
will add the right configuration data depending on the carrier.
This record will be sent in order to initiate a handover.

ndef: Start Bluetooth pairing from the Hs parser

When successfully parsing an Hs frame a bluetooth pairing will start.

Fix MB and ME bits checking in handover frame

Handover frames can have nested ndef messages, so we have to validate
MB and ME at different levels (from the first record but also from each
ac and cr nested records).

handover: Handle Hr and Hs parsing

When receiving an Hr neard should reply to the initiator with an Hs frame.
When receiving an Hs neard is the initiator and should start the pairing
(or eventually the association).

handover: Move handover_close function up

handover: Add handover_push

This code path will be called when sending a handover request NDEF.

adapter: Return a better error when powering an already powered adapter

README: Update configuration options description

tag: tag_set_ro returns void

For consistency sake, setters return void.

tag: Remove undefined tag_set_uid routine

nfctype2: Add support for blank tag handling

When using a new tag for the first time, the CC field might not be
initialized (un-formatted tag) this was causing the tag to be dropped.
this patch add handling for such tags:

* When a tag with uninitialized CC block is read, it is marked as blank
  and is not dropped
* Add a new nfctype2_format() function used for initializing the CC block
  in blank t2 flags.
* When writing data to a tag, in case the tag is marked as blank, this api
  can be used in order to format the tag before the data is being written
  to it.

tag: Add blank tag format support

* Add a new near_tag_driver op (format) used for formatting blank tags.
* In case an un-formatted tag is being written __near_tag_write()
  first format the tag and if that succeeds it proceeds to writing
  the data.

tag: Add blank tag handling

Add an API for marking blank tags.

tag: Remove read-only property getter

The check has been moved from plugins to the core code. No need to
export this getter anymore.

nfctype4: Remove duplicate tag read-only check

Tag read-only check while writing is added in tag.c.
So remove duplicate check from plugin.

nfctype3: Remove duplicate tag read-only check

Tag read-only check while writing is added in tag.c.
So remove duplicate check from plugin.

nfctype2: Remove duplicate tag read-only check

Tag read-only check while writing is added in tag.c.
So remove duplicate check from plugin.

nfctype1: Remove duplicate tag read-only check

Tag read-only check while writing is added in tag.c.
So remove duplicate call from plugin.

test: Add Handover type support to push-device

The pushed handover NDEF will be a handover request one, and the argument
to the push-device script is the expected carrier (bluetooth or wifi).

device: Specify service name when pushing NDEFs

Depending on the NDEF type, the service name will be different. In
particular, Handover NDEFs should be sent to the handover service, not
the SNEP one.

tag: Do not attempt to write to a read only tag

adapter: Early p2p binding

The LLCP wks array will be properly built before the MAC link is up by
binding to p2p services as soon as a p2p driver is registered.
The bound will also happen when the MAC link is up for supporting kernels
where the bound LLCP sockets will be closed when the MAC is brought down.
For more recent kernels, the bind operation will happen only once.

p2p: Do not return an error when a service name is already bound

When trying to re-use the same service name, bind will fail but it means
a server is already listening on this service.

p2p: Keep track of allocated servers

p2p: Bind and listen only on a specific target

Bind and listening are related to a particular adapter, not to a target
or device index.

bluetooth: Check for OOB pointers to be NULL before freeing them

handover: Support for handover selector mode

This code parses received handover requester records, create an Hs one
and send it back to the requester.

ndef: Prepare handover record

This patch allows to create handover (Hr or Hs) frames with all the
expecteded records (collision, ac, ...) and the configuration ones
(like e.g. the Bluetooth OOB related data).

ndef: Add NDEF record couting routine

For a defined record type, this routine will count how many records are
found in an NDEF.

ndef: Add handover parsing routines

Because of the handover specific frame structure (out of NDEF record
messages), the routines are slightly different than the existing ones.

ndef: Add handover records freeing routines

ndef: Add handover record definitions

near_ndef_record will point to one of them.

ndef: Add a function to set/reset MB and ME bits in NDEF headers

ndef: Extend the NDEF alocation routine

ndef_message_alloc_complete allows to pass more arguments when allocating
a new NDEF.
ndef_message_alloc is now a wrapper to ndef_message_alloc_complete,
mainly because, in most cases, payload_id is null.

ndef: Add Bluetooth pairing argument to parse_mime_type()

This argument being set to TRUE will initiate a Bluetooth pairing.

ndef: Add near_ndef_record_length function

This function returns the length of a NDEF record.

ndef: Save type length in record header structure

bluetooth: Define Bluetooth MIME types

gdbus: Add macro for methods marked as NOREPLY

Bring gdbus up to sync with other projects. The macro is
not yet used in neard.

AUTHORS: Mention Eyal's contributions

nfctype2: Fix wrong read command length

According to "Type 2 Tag Operation Specification" the read command is 2
bytes long: (Code=0x30 and Block number).

Instead of sending 2 bytes the code was sending 6 bytes which caused some
NFC controllers to send a wrong read command which resulted with a read
timeout.

features: Update

TODO: Update

netlink: Simplfy the netlink initialisation code

The cache and generic netlink family pointer are not needed.