Samuel Ortiz [Fri, 5 Apr 2013 13:49:33 +0000 (15:49 +0200)]
nfc_copy: Update with new socket options
Marcin Malagowski [Tue, 26 Mar 2013 18:25:04 +0000 (19:25 +0100)]
unit: Support for SNEP REJECT request
Marcin Malagowski [Tue, 26 Mar 2013 18:25:03 +0000 (19:25 +0100)]
unit: Fixed cleaning SNEP fragments
If sending the fragment fails, server should remove all
the remaining fragments.
Ravi kumar Veeramally [Wed, 3 Apr 2013 12:37:27 +0000 (15:37 +0300)]
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.
Ravi kumar Veeramally [Wed, 3 Apr 2013 12:37:26 +0000 (15:37 +0300)]
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.
Thierry Escande [Mon, 25 Mar 2013 16:45:54 +0000 (17:45 +0100)]
nfctool: Print length of buffer before its hexadecimal dump
Thierry Escande [Mon, 25 Mar 2013 16:45:53 +0000 (17:45 +0100)]
nfctool: Fix some integer sizes and signedness
Also, use glib types to be coherent.
Thierry Escande [Mon, 25 Mar 2013 16:45:52 +0000 (17:45 +0100)]
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.
Szymon Janc [Mon, 25 Mar 2013 10:48:01 +0000 (11:48 +0100)]
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.
Szymon Janc [Mon, 25 Mar 2013 10:48:00 +0000 (11:48 +0100)]
ndef: Remove bogus text from near_ndef_parse_msg comment
This was accidentally pasted in commit
8a141753.
Szymon Janc [Mon, 25 Mar 2013 10:47:59 +0000 (11:47 +0100)]
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.
Szymon Janc [Mon, 25 Mar 2013 10:47:58 +0000 (11:47 +0100)]
ndef: Fix some typos in comments and debug messages
Szymon Janc [Mon, 25 Mar 2013 10:47:56 +0000 (11:47 +0100)]
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
Szymon Janc [Mon, 25 Mar 2013 10:47:55 +0000 (11:47 +0100)]
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
Samuel Ortiz [Mon, 25 Mar 2013 16:48:03 +0000 (17:48 +0100)]
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.
Guillem Jover [Mon, 25 Mar 2013 11:44:07 +0000 (12:44 +0100)]
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.
Samuel Ortiz [Mon, 25 Mar 2013 11:14:59 +0000 (12:14 +0100)]
unit: Sanitize NDEF test names
Samuel Ortiz [Mon, 25 Mar 2013 11:11:13 +0000 (12:11 +0100)]
unit: Sanitize SNEP test names
Samuel Ortiz [Mon, 25 Mar 2013 11:06:44 +0000 (12:06 +0100)]
AUTHORS: Mention Marcin's contributions
Samuel Ortiz [Mon, 25 Mar 2013 11:05:21 +0000 (12:05 +0100)]
unit: Remove SNEP routines forward declarations
Marcin Malagowski [Sat, 23 Mar 2013 11:40:27 +0000 (12:40 +0100)]
unit: Add SNEP PUT/GET NDEF test
Marcin Malagowski [Sat, 23 Mar 2013 11:40:26 +0000 (12:40 +0100)]
unit: Add SNEP GET request "fragmented" test
Marcin Malagowski [Sat, 23 Mar 2013 11:40:25 +0000 (12:40 +0100)]
unit: Add SNEP GET request "not implemented" test
Marcin Malagowski [Sat, 23 Mar 2013 11:40:24 +0000 (12:40 +0100)]
unit: Add SNEP GET request "ok" test
Marcin Malagowski [Sat, 23 Mar 2013 11:40:23 +0000 (12:40 +0100)]
unit: Add SNEP PUT request "fragmented" test
Marcin Malagowski [Sat, 23 Mar 2013 11:40:22 +0000 (12:40 +0100)]
unit: Add SNEP PUT request "feature not implemented" test
Marcin Malagowski [Sat, 23 Mar 2013 11:40:21 +0000 (12:40 +0100)]
unit: Add SNEP PUT request "unsupported version" test
Marcin Malagowski [Sat, 23 Mar 2013 11:40:20 +0000 (12:40 +0100)]
unit: Add SNEP PUT request "ok" test
Marcin Malagowski [Sat, 23 Mar 2013 11:40:19 +0000 (12:40 +0100)]
unit: Add SNEP test utilities
Guillem Jover [Fri, 22 Mar 2013 18:53:12 +0000 (19:53 +0100)]
build: Remove unused linking of neard against libresolv
Waldemar Rymarkiewicz [Tue, 12 Mar 2013 10:46:18 +0000 (11:46 +0100)]
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]
Samuel Ortiz [Fri, 22 Mar 2013 14:21:24 +0000 (15:21 +0100)]
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.
Samuel Ortiz [Fri, 22 Mar 2013 11:40:43 +0000 (12:40 +0100)]
AUTHORS: Mention Guillem's contributions
Guillem Jover [Fri, 22 Mar 2013 11:29:25 +0000 (12:29 +0100)]
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.
Samuel Ortiz [Fri, 22 Mar 2013 11:32:58 +0000 (12:32 +0100)]
configure: Add scripts/neard to the configure output
Guillem Jover [Wed, 20 Mar 2013 16:39:49 +0000 (17:39 +0100)]
ndef: Fix typo in unknown carrier string
Samuel Ortiz [Wed, 20 Mar 2013 00:40:21 +0000 (01:40 +0100)]
AUTHORS: Mention Marcin's contributions
Marcin Małagowski [Sun, 17 Mar 2013 12:59:51 +0000 (13:59 +0100)]
snep: SNEP core moved from plugins to core
This is needed for implementing SNEP unit tests.
Samuel Ortiz [Tue, 19 Mar 2013 16:13:24 +0000 (17:13 +0100)]
build: Install nfctool to @bin@
Thierry Escande [Mon, 11 Mar 2013 16:55:12 +0000 (17:55 +0100)]
doc: Add nfctool manpage
Thierry Escande [Mon, 11 Mar 2013 16:55:11 +0000 (17:55 +0100)]
nfctool: Add --version option
The nfctool version number is based on neard's one.
Samuel Ortiz [Fri, 15 Mar 2013 19:53:20 +0000 (20:53 +0100)]
nfctype2: Get the actual tag data length when trying to write an NDEF
Samuel Ortiz [Fri, 15 Mar 2013 19:53:06 +0000 (20:53 +0100)]
tag: Export tag data length
Samuel Ortiz [Mon, 11 Mar 2013 14:21:37 +0000 (15:21 +0100)]
ndef: Fix local carrier data fetching error message
Samuel Ortiz [Fri, 8 Mar 2013 09:58:32 +0000 (10:58 +0100)]
Release 0.10
Samuel Ortiz [Thu, 7 Mar 2013 14:11:04 +0000 (15:11 +0100)]
build: Add agent-api.txt to the document files list
Samuel Ortiz [Thu, 7 Mar 2013 14:09:07 +0000 (15:09 +0100)]
doc: Add neard man pages
One man page for the daemon and another one for the configuration file.
Samuel Ortiz [Thu, 7 Mar 2013 10:53:05 +0000 (11:53 +0100)]
build: Install neard back to libexec/nfc/
Olivier Guiter [Mon, 4 Mar 2013 14:26:10 +0000 (15:26 +0100)]
build: Install neard to /usr/sbin
Thierry Escande [Mon, 4 Mar 2013 13:49:18 +0000 (14:49 +0100)]
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.
Thierry Escande [Mon, 4 Mar 2013 13:49:17 +0000 (14:49 +0100)]
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.
Samuel Ortiz [Mon, 4 Mar 2013 10:33:13 +0000 (11:33 +0100)]
main: Big fat warning for netlink initialization failure
Szymon Janc [Thu, 28 Feb 2013 15:20:26 +0000 (16:20 +0100)]
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)
Szymon Janc [Thu, 28 Feb 2013 15:20:30 +0000 (16:20 +0100)]
main: Exit neard if netlink initialization failed
There is no point in going further without netlink operational.
Szymon Janc [Thu, 28 Feb 2013 15:20:29 +0000 (16:20 +0100)]
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.
Szymon Janc [Thu, 28 Feb 2013 15:20:28 +0000 (16:20 +0100)]
nfctyp3: Remove set but not used variable
system_code in receive_system_code function was written but never read.
Szymon Janc [Thu, 28 Feb 2013 15:20:27 +0000 (16:20 +0100)]
build: Add some missing files to .gitignore
Add unit test binaries to ignore list.
Ravi kumar Veeramally [Thu, 28 Feb 2013 14:01:49 +0000 (16:01 +0200)]
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.
Ravi kumar Veeramally [Thu, 28 Feb 2013 14:01:48 +0000 (16:01 +0200)]
org.neard.conf: Add the HandoverAgent interface sending permissions
Ravi kumar Veeramally [Thu, 28 Feb 2013 14:01:47 +0000 (16:01 +0200)]
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.
Ravi kumar Veeramally [Thu, 28 Feb 2013 14:01:46 +0000 (16:01 +0200)]
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).
Ravi kumar Veeramally [Thu, 28 Feb 2013 14:01:45 +0000 (16:01 +0200)]
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).
Ravi kumar Veeramally [Thu, 28 Feb 2013 14:01:44 +0000 (16:01 +0200)]
ndef: Implement Hr message preparation routine
To support multiple carriers in handover request message, implemented
seperate function for handover request message.
Ravi kumar Veeramally [Thu, 28 Feb 2013 14:01:43 +0000 (16:01 +0200)]
ndef: Implement Hs message preparation routine
To support multiple carriers in handover select message, implemented
seperate function for handover select message.
Ravi kumar Veeramally [Thu, 28 Feb 2013 14:01:42 +0000 (16:01 +0200)]
ndef: Add helper funtions to create handover messages
Providing helper functions to create Hanodver select and
request messages.
Ravi kumar Veeramally [Thu, 28 Feb 2013 14:01:41 +0000 (16:01 +0200)]
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.
Ravi kumar Veeramally [Thu, 28 Feb 2013 14:01:40 +0000 (16:01 +0200)]
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.
Samuel Ortiz [Thu, 28 Feb 2013 11:05:07 +0000 (12:05 +0100)]
nfctool: Use nfc_copy.h instead of the system NFC header
Thierry Escande [Fri, 22 Feb 2013 18:28:34 +0000 (19:28 +0100)]
nfctool: Move adapter related functions in one place
This moves adapter management functions in adapter.c
Tomasz Bursztyka [Fri, 8 Feb 2013 13:58:10 +0000 (15:58 +0200)]
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.
Samuel Ortiz [Fri, 22 Feb 2013 11:19:54 +0000 (12:19 +0100)]
nfc_copy: Add SNL netlink API
Samuel Ortiz [Fri, 22 Feb 2013 10:51:02 +0000 (11:51 +0100)]
nfc_copy: Add LLCP socket option names
Samuel Ortiz [Thu, 21 Feb 2013 16:37:29 +0000 (17:37 +0100)]
p2p: Remove client data from the server client list when channel closes
Samuel Ortiz [Thu, 21 Feb 2013 16:28:49 +0000 (17:28 +0100)]
nfctool: Close sniffer socket when getting IO errors
Samuel Ortiz [Thu, 21 Feb 2013 11:27:23 +0000 (12:27 +0100)]
snep-validation: Remove DBG statement from init and exit routines
Samuel Ortiz [Fri, 15 Feb 2013 16:59:07 +0000 (17:59 +0100)]
device: Do not double free the device records list
Typo from the previous commit...
Samuel Ortiz [Fri, 15 Feb 2013 15:25:58 +0000 (16:25 +0100)]
device: Do not open code the records freeing routine
Use near_ndef_records_free instead.
Samuel Ortiz [Fri, 15 Feb 2013 15:24:48 +0000 (16:24 +0100)]
monitor-near: Monitor Tags and Devices properties
Samuel Ortiz [Fri, 15 Feb 2013 15:24:25 +0000 (16:24 +0100)]
tag: Send PropertyChanged signal when new records are added
Samuel Ortiz [Fri, 15 Feb 2013 15:23:50 +0000 (16:23 +0100)]
device: Send PropertyChanged signal when new records are added
Samuel Ortiz [Fri, 15 Feb 2013 15:15:02 +0000 (16:15 +0100)]
p2p: Do not try to bind if LLCP is not supported
Denis Kenzior [Thu, 7 Feb 2013 16:22:57 +0000 (10:22 -0600)]
gdbus: Add g_dbus_proxy_set_removed_watch
Vinicius Costa Gomes [Thu, 7 Feb 2013 17:40:31 +0000 (14:40 -0300)]
gdbus: Fix missing PropertiesChanged signal
If D-Bus ObjectManager is not supported, InterfacesAdded signal
checking needs to be ignored otherwise PropertiesChanged signal
will never be sent.
Olivier Guiter [Thu, 14 Feb 2013 15:19:50 +0000 (16:19 +0100)]
nfctype3: Do not call write callback if the initial write command succeeds
When write-tag on a type 3 tag, the command returns
with "Invalid arguments", though it actually writes successfully.
Olivier Guiter [Mon, 11 Feb 2013 15:52:33 +0000 (16:52 +0100)]
snep: Fix Android's handover over SNEP support
Android does handover over SNEP and recent changes broke the offset used
to check the handover frame validity.
Olivier Guiter [Mon, 11 Feb 2013 15:52:32 +0000 (16:52 +0100)]
handover: Reply with no ACs when getting an Hr with an empty AC list
This fixes handover validation test #2.
If the Hr comes with no alternative carriers, the Hs AC list should be
empty as well.
Olivier Guiter [Mon, 11 Feb 2013 15:52:31 +0000 (16:52 +0100)]
ndef: Send empty Hs when an Hr carries a single unknow type carrier
This fixes handover validation test #6.
Check that the remote handover server returns a select message
without alternative carriers if a single carrier of unknown type was
sent with the handover request.
Olivier Guiter [Mon, 11 Feb 2013 15:52:30 +0000 (16:52 +0100)]
ndef: Send empty Hs frame when the Hr version check fails
This fixes handover validation test #3.
Verify that the remote handover server handles historic and future
handover request version numbers. The test failed on step 4
(Check version 15.0).
Olivier Guiter [Mon, 11 Feb 2013 14:33:52 +0000 (15:33 +0100)]
ndef: Do not build a reply to handover select messages
Handover select frames must not be replied to.
This fixes the following bt-handover error:
"ERROR:dbus.connection:Exception in handler for D-Bus signal"
Marcel Holtmann [Mon, 11 Feb 2013 19:19:10 +0000 (20:19 +0100)]
gdbus: Don't call property changed callback during client init
When the client uses ObjectManager to init properties, do not call
property changed callbacks. They should only be called once the proxy
added has been successfully signaled since the proxy itself provides
a full copy of available properties.
Ravi kumar Veeramally [Fri, 8 Feb 2013 12:01:50 +0000 (14:01 +0200)]
test: Update handover agent register/unregister tests
To match the new API.
Ravi kumar Veeramally [Fri, 8 Feb 2013 12:01:49 +0000 (14:01 +0200)]
agent: Add an extra parameter to the handover registration agent api
Current handover registration apis does not support for carrier
specific registration. Added an extra parameter (carrier type) to
RegisterHandoverAgent and UnregisterHandoverAgent. This is an initial
patch for wifi handover agent registration. WiFi NDEF message handling
will be supported soon.
Ravi kumar Veeramally [Fri, 8 Feb 2013 12:01:48 +0000 (14:01 +0200)]
near: Rename struct name bt_data to carrier_data
To be more generic, renaming structure name from bt_data to
carrier_data.
Samuel Ortiz [Wed, 6 Feb 2013 08:00:04 +0000 (09:00 +0100)]
adapter: Disconnect from the target when data read fails to start
Samuel Ortiz [Sun, 3 Feb 2013 20:09:20 +0000 (21:09 +0100)]
unit: Rename test-ndef to test-ndef-parse
Samuel Ortiz [Sun, 3 Feb 2013 20:07:11 +0000 (21:07 +0100)]
unit: Add initial NDEF building test
Johan Hedberg [Fri, 1 Feb 2013 15:03:22 +0000 (09:03 -0600)]
gdbus: Fix unpublished interface signal emission
If we haven't published an interface yet (i.e. it's in the data->added
list), we should just ignore any property changed indications as the
values for the properties will anyway be part of the InterfacesAdded
signal.
Samuel Ortiz [Thu, 31 Jan 2013 09:06:16 +0000 (10:06 +0100)]
include: Add a copy of the NFC kernel header
With the netlink kernel API changes, including a copy of the NFC header and
not relying on the system one will allow clean builds. The drawback could
be missing runtime features but that's better than not being able to run
the daemon at all.
Szymon Janc [Fri, 4 Jan 2013 15:10:38 +0000 (16:10 +0100)]
device: Fix free order in push_ndef
ndef was freed before ndef->data.
Invalid read of size 8
at 0x41CF58: push_ndef (device.c:287)
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)
by 0x408D8E: main (main.c:221)
Address 0x6847fb0 is 16 bytes inside a block of size 24 free'd
at 0x4C2A82E: free (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
by 0x41CF57: push_ndef (device.c:286)
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)
by 0x408D8E: main (main.c:221)
Szymon Janc [Fri, 4 Jan 2013 15:10:37 +0000 (16:10 +0100)]
p2p: Fix memory leak in p2p_listener_event
g_io_add_watch keeps own reference to iochannel so local reference
should be dropped before leaving.
882 (840 direct, 42 indirect) bytes in 7 blocks are definitely lost in loss record 177 of 192
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 0x4EB66F4: g_io_channel_unix_new (in /lib/x86_64-linux-gnu/libglib-2.0.so.0.3200.3)
by 0x416C6F: p2p_listener_event (p2p.c:187)
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)
by 0x408DDE: main (main.c:221)