j3d1/tinc
1
0
Fork 0
Code Issues 1 Pull requests Releases Wiki Activity

Always remove a node from the UDP tree before freeing it.

Browse source 
Valgrind caught tinc reading free'd memory during a purge(). This was caused by
first removing it from the main node tree, which will already call free_node(),
and then removing it from the UDP tree. This might cause spurious segmentation
faults.
This commit is contained in:
Guus Sliepen 2009-09-08 16:35:28 +02:00
parent de029ce460
commit 5a7fc58012
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
src/node.c
View file

@ -137,8 +137,8 @@ void node_del(node_t *n)
edge_del(e);
}
avl_delete(node_tree, n);
avl_delete(node_udp_tree, n);
avl_delete(node_tree, n);
}
node_t *lookup_node(char *name)