View Issue Details

IDProjectCategoryView StatusLast Update
0003587GNUnetfile-sharing servicepublic2018-06-06 22:25
Reporteramatus Assigned ToDue Date
PrioritynormalSeveritycrashReproducibilityalways
Status closedResolutionno change required 
Product VersionGit master 
Target Version0.11.0pre66Fixed in Version0.11.0pre66 
Summary0003587: gnunet-service-filesharing exausts memory
DescriptionA bug was introduced or triggered by revision 34633 that causes gnunet-service-filesharing to use up all available memory.
TagsNo tags attached.

Activities

Christian Grothoff

2014-12-20 12:07

manager   ~0008719

Wasn't introduced in that revision, that revision merely fixes the fact that the previous one failed to connect to other peers, a bug that was introduced in SVN 34563 earlier:

34563:
 {
- if (0 == memcmp (&my_id, peer, sizeof (struct GNUNET_PeerIdentity)))
+ if (0 != GNUNET_CRYPTO_cmp_peer_identity (&my_id,
+ peer))
     return;

34633:

- if (0 != GNUNET_CRYPTO_cmp_peer_identity (&my_id,
- peer))
+ if (0 ==
+ GNUNET_CRYPTO_cmp_peer_identity (&my_id,
+ peer))

So the leak was probably present before 34563 as well, just maked in between those two revisions by FS not properly talking to other peers at all.

amatus

2014-12-20 16:48

developer   ~0008720

I'll try the bisect again with the 34633 fix applied to each revision.

amatus

2014-12-20 23:34

developer   ~0008721

Watching the memory behavior more closely this doesn't look like a leak. The network has simply grown big enough that I need to restrict the number of peers I allow.

Issue History

Date ModifiedUsernameFieldChange
2014-12-20 05:16amatusNew Issue
2014-12-20 12:07Christian GrothoffNote Added: 0008719
2014-12-20 16:48amatusNote Added: 0008720
2014-12-20 23:34amatusNote Added: 0008721
2014-12-20 23:34amatusStatusnew => resolved
2014-12-20 23:34amatusResolutionopen => no change required
2014-12-21 00:22Christian GrothoffFixed in Version => 0.11.0pre66
2014-12-21 00:22Christian GrothoffTarget Version => 0.11.0pre66
2018-06-06 22:25Christian GrothoffStatusresolved => closed