ethereum / pydevp2p / 254

88%

push

travis-ci

Fix critical unintentional variable name reuse

In `devp2p.discovery.recv_neighbours()` the variable `node` (ostensibly
containing the sender node) was reused as a temporary variable inside a 
for loop. 

Depending on the order of the `neighbours_lst` this lead in many cases
to neighbour information being attributed to the wrong sender node.

Thanks to @konradkonrad for the pair debugging :)

Refs: HydraChain/hydrachain#73

5 of 5 new or added lines in 1 file covered. (100.0%)

1982 of 2280 relevant lines covered (86.93%)

1.68 hits per line