View Issue Details

IDProjectCategoryView StatusLast Update
0002367GNUnettransport servicepublic2024-05-03 11:59
ReporterMatthias Wachs Assigned ToMatthias Wachs Due Date
PrioritynormalSeveritycrashReproducibilityhave not tried
Status closedResolutionfixed 
Product VersionGit master 
Target Version0.9.3Fixed in Version0.9.3 
Summary0002367: Use after free setup_neighbour (gnunet-service-transport_neighbours.c:1589)
DescriptionRevision 21560 on gnunet9@gnunet.org

==17970== Invalid read of size 8
==17970== at 0x40A4C0: setup_neighbour (gnunet-service-transport_neighbours.c:1589)
==17970== by 0x40BBA4: GST_neighbours_handle_connect (gnunet-service-transport_neighbours.c:2056)
==17970== by 0x403F67: plugin_env_receive_callback (gnunet-service-transport.c:260)
==17970== by 0xA258FF6: handle_tcp_data (plugin_transport_tcp.c:1940)
==17970== by 0x526E7B1: GNUNET_SERVER_inject (server.c:891)
==17970== by 0x526F87F: client_message_tokenizer_callback (server.c:1098)
==17970== by 0x5270061: GNUNET_SERVER_mst_receive (server_mst.c:224)
==17970== by 0x526F2C7: process_mst (server.c:961)
==17970== by 0x526F793: process_incoming (server.c:1041)
==17970== by 0x524BEE1: receive_ready (connection.c:1055)
==17970== by 0x526D0FF: GNUNET_SCHEDULER_run (scheduler.c:602)
==17970== by 0x5276088: GNUNET_SERVICE_run (service.c:1773)
==17970== Address 0xc66f3b8 is 312 bytes inside a block of size 424 free'd
==17970== at 0x4028AAE: free (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==17970== by 0xA25659C: session_disconnect_it (plugin_transport_tcp.c:1362)
==17970== by 0x5253AAC: GNUNET_CONTAINER_multihashmap_get_multiple (container_multihashmap.c:485)
==17970== by 0xA257A06: tcp_plugin_disconnect (plugin_transport_tcp.c:1389)
==17970== by 0x408CE4: free_neighbour (gnunet-service-transport_neighbours.c:889)
==17970== by 0x40BB9C: GST_neighbours_handle_connect (gnunet-service-transport_neighbours.c:2055)
==17970== by 0x403F67: plugin_env_receive_callback (gnunet-service-transport.c:260)
==17970== by 0xA258FF6: handle_tcp_data (plugin_transport_tcp.c:1940)
==17970== by 0x526E7B1: GNUNET_SERVER_inject (server.c:891)
==17970== by 0x526F87F: client_message_tokenizer_callback (server.c:1098)
==17970== by 0x5270061: GNUNET_SERVER_mst_receive (server_mst.c:224)
==17970== by 0x526F2C7: process_mst (server.c:961)
TagsNo tags attached.

Relationships

has duplicate 0002369 closedMatthias Wachs Use after free in GST_blacklist_test_allowed (gnunet-service-transport_blacklist.c:758) 
has duplicate 0002368 closedMatthias Wachs Use after free in GST_neighbours_handle_connect (gnunet-service-transport_neighbours.c:2058) 

Activities

Matthias Wachs

2012-05-23 11:26

reporter   ~0005935

Issue in TCP plugin:

calls plugin_env_receive_callback with session that was freed before.

setup_neighbour is the first trying to access the memory, so crash here.

Matthias Wachs

2012-05-24 11:45

reporter   ~0005940

bug that solved the issue: 0002374

Issue History

Date ModifiedUsernameFieldChange
2012-05-23 07:53Matthias WachsNew Issue
2012-05-23 07:53Matthias WachsStatusnew => assigned
2012-05-23 07:53Matthias WachsAssigned To => Matthias Wachs
2012-05-23 07:55Matthias WachsSeverityminor => crash
2012-05-23 11:26Matthias WachsNote Added: 0005935
2012-05-23 11:26Matthias WachsCategorytransport service => TCP transport
2012-05-23 11:37Matthias WachsRelationship addedrelated to 0002369
2012-05-23 11:38Matthias WachsRelationship deletedrelated to 0002369
2012-05-23 11:38Matthias WachsRelationship addedhas duplicate 0002369
2012-05-23 11:38Matthias WachsRelationship addedhas duplicate 0002368
2012-05-24 11:45Matthias WachsNote Added: 0005940
2012-05-24 11:45Matthias WachsStatusassigned => resolved
2012-05-24 11:45Matthias WachsResolutionopen => fixed
2012-05-27 16:33Christian GrothoffProduct Version => Git master
2012-05-27 16:33Christian GrothoffFixed in Version => 0.9.3
2012-05-27 16:33Christian GrothoffTarget Version => 0.9.3
2012-06-02 17:15Christian GrothoffStatusresolved => closed
2024-05-03 11:59Christian GrothoffCategoryTCP transport => transport service