View Issue Details

IDProjectCategoryView StatusLast Update
0002932GNUnetcadet servicepublic2013-12-24 20:54
ReporterFlorian Dold Assigned ToBart Polot  
PrioritynormalSeverityminorReproducibilityalways
Status closedResolutionfixed 
Product VersionGit master 
Target Version0.10.0Fixed in Version0.10.0 
Summary0002932: mesh api sends uninitialized memory
DescriptionDestroying a tunnel sometimes sends uninitialized memory over a socket.

Valgrind detects this (see Additional Information), and attaching via VGDB gives the following:

(gdb) print ntohs (*((uint16_t *) buf + 1))
$1 = 274

Looking at gnunet_protocols.h finds the culprit:
#define GNUNET_MESSAGE_TYPE_MESH_LOCAL_TUNNEL_DESTROY 274
Additional InformationValgrind message:
==4541== Syscall param socketcall.sendto(msg) points to uninitialised byte(s)
==4541== at 0x539EEBD: send (send.c:27)
==4541== by 0x4E6C7CF: GNUNET_NETWORK_socket_send (network.c:697)
==4541== by 0x4E4F3E3: transmit_ready (connection.c:1325)
==4541== by 0x4E77318: GNUNET_SCHEDULER_run (scheduler.c:597)
==4541== by 0x4E81EF5: GNUNET_SERVICE_run (service.c:1816)
==4541== by 0x401805: main (gnunet-service-set.c:974)
==4541== Address 0x688ec38 is 8 bytes inside a block of size 33,364 alloc'd
==4541== at 0x4C2CF8E: realloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==4541== by 0x4E4925C: GNUNET_xrealloc_ (common_allocation.c:177)
==4541== by 0x4E4F554: transmit_ready (connection.c:1311)
==4541== by 0x4E77318: GNUNET_SCHEDULER_run (scheduler.c:597)
==4541== by 0x4E81EF5: GNUNET_SERVICE_run (service.c:1816)
==4541== by 0x401805: main (gnunet-service-set.c:974)
==4541==
==4541== (action on error) vgdb me ...
TagsNo tags attached.

Activities

Florian Dold

2013-07-09 23:04

developer   ~0007215

fixed in r27845

Issue History

Date Modified Username Field Change
2013-07-09 23:02 Florian Dold New Issue
2013-07-09 23:02 Florian Dold Status new => assigned
2013-07-09 23:02 Florian Dold Assigned To => Bart Polot
2013-07-09 23:04 Florian Dold Note Added: 0007215
2013-07-09 23:04 Florian Dold Status assigned => resolved
2013-07-09 23:04 Florian Dold Resolution open => fixed
2013-07-10 14:41 Christian Grothoff Fixed in Version => 0.10.0
2013-07-10 14:41 Christian Grothoff Target Version => 0.10.0
2013-12-24 20:54 Christian Grothoff Status resolved => closed
2014-05-09 18:34 Christian Grothoff Category mesh service => cadet service