View Issue Details

IDProjectCategoryView StatusLast Update
0003252GNUnetutil librarypublic2014-01-06 23:19
ReporterSree Harsha Totakura Assigned ToChristian Grothoff  
PrioritynormalSeverityminorReproducibilityalways
Status closedResolutionno change required 
Product VersionGit master 
Summary0003252: Valgrind reports invalid read in gcrypt calls
Description==23819== Invalid read of size 4
==23819== at 0x5A15D08: transform (rmd160.c:179)
==23819== by 0x5A1768D: _gcry_rmd160_mixblock (rmd160.c:416)
==23819== by 0x5A1D16A: mix_pool (random-csprng.c:627)
==23819== by 0x5A1D3B7: add_randomness (random-csprng.c:1102)
==23819== by 0x5A20015: _gcry_rndlinux_gather_random (rndlinux.c:240)
==23819== by 0x5A1D04F: read_random_source (random-csprng.c:1288)
==23819== by 0x5A1E0EB: _gcry_rngcsprng_randomize (random-csprng.c:1114)
==23819== by 0x5A1CE74: _gcry_create_nonce (random.c:354)
==23819== by 0x4E6656C: GNUNET_CRYPTO_random_u32 (crypto_random.c:178)
==23819== by 0x4E66BDC: GNUNET_CRYPTO_random_init (crypto_random.c:300)
==23819== by 0x400E9C5: call_init (dl-init.c:84)
==23819== by 0x400EAA9: _dl_init (dl-init.c:133)
==23819== Address 0x73d3cc8 is 0 bytes after a block of size 664 alloc'd
==23819== at 0x4C2ABED: malloc (vg_replace_malloc.c:263)
==23819== by 0x59899D4: do_malloc (global.c:777)
==23819== by 0x598A7F8: _gcry_malloc (global.c:799)
==23819== by 0x598AB3E: _gcry_xmalloc (global.c:953)
==23819== by 0x598AC6E: _gcry_xcalloc (global.c:1015)
==23819== by 0x5A1DB30: initialize (random-csprng.c:324)
==23819== by 0x5989DF0: _gcry_vcontrol (global.c:489)
==23819== by 0x598686B: gcry_control (visibility.c:79)
==23819== by 0x4E66BC1: GNUNET_CRYPTO_random_init (crypto_random.c:297)
==23819== by 0x400E9C5: call_init (dl-init.c:84)
==23819== by 0x400EAA9: _dl_init (dl-init.c:133)
==23819== by 0x4001539: ??? (in /lib/x86_64-linux-gnu/ld-2.17.so)

==23819== Invalid read of size 4
==23819== at 0x5A15D18: transform (rmd160.c:178)
==23819== by 0x5A1768D: _gcry_rmd160_mixblock (rmd160.c:416)
==23819== by 0x5A1D16A: mix_pool (random-csprng.c:627)
==23819== by 0x5A1D3B7: add_randomness (random-csprng.c:1102)
==23819== by 0x5A20015: _gcry_rndlinux_gather_random (rndlinux.c:240)
==23819== by 0x5A1D04F: read_random_source (random-csprng.c:1288)
==23819== by 0x5A1E0EB: _gcry_rngcsprng_randomize (random-csprng.c:1114)
==23819== by 0x5A1CE74: _gcry_create_nonce (random.c:354)
==23819== by 0x4E6656C: GNUNET_CRYPTO_random_u32 (crypto_random.c:178)
==23819== by 0x4E66BDC: GNUNET_CRYPTO_random_init (crypto_random.c:300)
==23819== by 0x400E9C5: call_init (dl-init.c:84)
==23819== by 0x400EAA9: _dl_init (dl-init.c:133)
==23819== Address 0x73d3cd0 is 8 bytes after a block of size 664 alloc'd
==23819== at 0x4C2ABED: malloc (vg_replace_malloc.c:263)
==23819== by 0x59899D4: do_malloc (global.c:777)
==23819== by 0x598A7F8: _gcry_malloc (global.c:799)
==23819== by 0x598AB3E: _gcry_xmalloc (global.c:953)
==23819== by 0x598AC6E: _gcry_xcalloc (global.c:1015)
==23819== by 0x5A1DB30: initialize (random-csprng.c:324)
==23819== by 0x5989DF0: _gcry_vcontrol (global.c:489)
==23819== by 0x598686B: gcry_control (visibility.c:79)
==23819== by 0x4E66BC1: GNUNET_CRYPTO_random_init (crypto_random.c:297)
==23819== by 0x400E9C5: call_init (dl-init.c:84)
==23819== by 0x400EAA9: _dl_init (dl-init.c:133)
==23819== by 0x4001539: ??? (in /lib/x86_64-linux-gnu/ld-2.17.so)
Steps To Reproducerun any testbed testcases with valgrind. E.g:
libtool --mode=execute valgrind ./test_testbed_api_testbed_run_topologyscalefree
TagsNo tags attached.

Activities

Christian Grothoff

2014-01-06 23:18

manager   ~0007966

You'll need to update your suppression file. Valgrind can have false-positives, especially when it comes to highly optimized assembly, like what you find in libgcrypt crypto functions.

Issue History

Date Modified Username Field Change
2014-01-06 12:33 Sree Harsha Totakura New Issue
2014-01-06 23:18 Christian Grothoff Note Added: 0007966
2014-01-06 23:19 Christian Grothoff Assigned To => Christian Grothoff
2014-01-06 23:19 Christian Grothoff Status new => closed
2014-01-06 23:19 Christian Grothoff Resolution open => no change required