View Issue Details

IDProjectCategoryView StatusLast Update
0002981GNUnetutil librarypublic2013-12-24 20:54
ReporterMatthias Wachs Assigned ToChristian Grothoff  
PrioritynormalSeveritymajorReproducibilityalways
Status closedResolutionfixed 
Product VersionGit master 
Target Version0.10.0Fixed in Version0.10.0 
Summary0002981: Valgrind reports memory leaks in ecc crypto
DescriptionWhile running a peer:

==886075== 960 (720 direct, 240 indirect) bytes in 30 blocks are definitely lost in loss record 594 of 709
==886075== at 0x4C2BE2B: malloc (vg_replace_malloc.c:270)
==886075== by 0x6177950: do_malloc.constprop.4 (global.c:874)
==886075== by 0x6178C2C: _gcry_xmalloc (global.c:896)
==886075== by 0x61C3AC1: _gcry_mpi_alloc (mpiutil.c:84)
==886075== by 0x61C46EF: _gcry_mpi_point_init (ec.c:72)
==886075== by 0x61C4742: point_copy (ec.c:48)
==886075== by 0x61C4CF3: _gcry_mpi_ec_set_point (ec.c:642)
==886075== by 0x6175F08: gcry_mpi_ec_set_point (visibility.c:499)
==886075== by 0x567C8A5: decode_public_key (crypto_ecc.c:371)
==886075== by 0x567E65A: GNUNET_CRYPTO_ecc_verify (crypto_ecc.c:822)
==886075== by 0x414B0F: GST_neighbours_handle_disconnect_message (gnunet-service-transport_neighbours.c:3175)
==886075== by 0x403E4A: GST_receive_callback (gnunet-service-transport.c:270)
==886075==
==886075== 1,024 (384 direct, 640 indirect) bytes in 16 blocks are definitely lost in loss record 601 of 709
==886075== at 0x4C2BE2B: malloc (vg_replace_malloc.c:270)
==886075== by 0x6177950: do_malloc.constprop.4 (global.c:874)
==886075== by 0x6178C2C: _gcry_xmalloc (global.c:896)
==886075== by 0x61C3AC1: _gcry_mpi_alloc (mpiutil.c:84)
==886075== by 0x61C115C: _gcry_mpi_scan (mpicoder.c:436)
==886075== by 0x617B263: _gcry_sexp_nth_mpi (sexp.c:779)
==886075== by 0x6183BE5: sexp_to_key (pubkey.c:1896)
==886075== by 0x6187F28: _gcry_pk_sign (pubkey.c:3241)
==886075== by 0x567E282: GNUNET_CRYPTO_ecc_sign (crypto_ecc.c:752)
==886075== by 0x40D78A: send_disconnect (gnunet-service-transport_neighbours.c:1205)
==886075== by 0x40D939: disconnect_neighbour (gnunet-service-transport_neighbours.c:1263)
==886075== by 0x414C6A: GST_neighbours_handle_disconnect_message (gnunet-service-transport_neighbours.c:3190)
==886075==
TagsNo tags attached.

Activities

Christian Grothoff

2013-08-12 16:05

manager   ~0007357

libgcrypt fixed some leaks in Git, I fixed some leaks in SVN 28529, now valgrind is happy with test_crypto_ecc, which should cover the leaks mentioned in this report.

Issue History

Date Modified Username Field Change
2013-08-08 14:40 Matthias Wachs New Issue
2013-08-10 11:11 Christian Grothoff Assigned To => Christian Grothoff
2013-08-10 11:11 Christian Grothoff Status new => assigned
2013-08-12 16:05 Christian Grothoff Note Added: 0007357
2013-08-12 16:06 Christian Grothoff Status assigned => resolved
2013-08-12 16:06 Christian Grothoff Fixed in Version => 0.10.0
2013-08-12 16:06 Christian Grothoff Resolution open => fixed
2013-08-12 16:06 Christian Grothoff Product Version => Git master
2013-08-12 16:06 Christian Grothoff Target Version => 0.10.0
2013-12-24 20:54 Christian Grothoff Status resolved => closed