grassrootseconomics/openethereum
5
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

fix(network): don't disconnect reserved peers (#9608)

Browse Source 
The priority of && and || was borked.
This commit is contained in:
Andronik Ordian 2018-09-25 11:59:21 +02:00 committed by Marek Kotewicz
parent 5a1dc3eb8a
commit c4af7464e5
1 changed files with 8 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
util/network-devp2p/src/host.rs
View File

@ -751,11 +751,14 @@ impl Host {
let max_ingress = max(max_peers - min_peers, min_peers / 2); let max_ingress = max(max_peers - min_peers, min_peers / 2);
if reserved_only || if reserved_only ||
(s.info.originated && egress_count > min_peers) || (s.info.originated && egress_count > min_peers) ||
(!s.info.originated && ingress_count > max_ingress) && !self.reserved_nodes.read().contains(&id) { (!s.info.originated && ingress_count > max_ingress) {
// only proceed if the connecting peer is reserved. if !self.reserved_nodes.read().contains(&id) {
s.disconnect(io, DisconnectReason::TooManyPeers); // only proceed if the connecting peer is reserved.
kill = true; trace!(target: "network", "Disconnecting non-reserved peer {:?}", id);
break; s.disconnect(io, DisconnectReason::TooManyPeers);
kill = true;
break;
}
} }
if !self.filter.as_ref().map_or(true, |f| f.connection_allowed(&self_id, &id, ConnectionDirection::Inbound)) { if !self.filter.as_ref().map_or(true, |f| f.connection_allowed(&self_id, &id, ConnectionDirection::Inbound)) {