d0s3t3ch/wownero
1
0
Fork 0
mirror of https://git.wownero.com/wownero/wownero.git synced 2024-08-15 01:03:23 +00:00
Code Issues Projects Releases Wiki Activity

protocol: prefer cycling low score peers

Browse source 
Cycling is meant to opportunistically connecting to a new peer
in case we're connected to a malicious subset of the network.
We now prefer dropping a low score peer, since those will be
either ones which already failed newly at least one test, or
newly connected ones at the same height. In particular, they
can't be a newly connected one that's found another chain, or
they'd be in synchronizing state, not normal state.
This commit is contained in:
moneromooo 2020-10-28 19:23:39 +00:00 • committed by wowario
parent 83977196db
commit d02b3ffa14
No known key found for this signature in database
GPG key ID: 24DCBE762DE9C111
1 changed files with 9 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

11
src/cryptonote_protocol/cryptonote_protocol_handler.inl
View file

@ -1770,6 +1770,7 @@ skip:
MTRACE("Checking for outgoing syncing peers...");
unsigned n_syncing = 0, n_synced = 0;
boost::uuids::uuid last_synced_peer_id(boost::uuids::nil_uuid());
int64_t lowest_score = std::numeric_limits<int64_t>::max();
m_p2p->for_each_connection([&](cryptonote_connection_context& context, nodetool::peerid_type peer_id, uint32_t support_flags)->bool
{
if (!peer_id || context.m_is_income) // only consider connected outgoing peers
@ -1779,8 +1780,14 @@ skip:
if (context.m_state == cryptonote_connection_context::state_normal)
{
++n_synced;
if (!context.m_anchor)
last_synced_peer_id = context.m_connection_id;
if (!context.m_anchor || context.m_score < 0)
{
if (context.m_score <= lowest_score)
{
last_synced_peer_id = context.m_connection_id;
lowest_score = context.m_score;
}
}
}
return true;
});