diff options
author | Riccardo Spagni <ric@spagni.net> | 2017-01-20 20:54:03 -0500 |
---|---|---|
committer | Riccardo Spagni <ric@spagni.net> | 2017-01-20 20:54:04 -0500 |
commit | d10425025a0e0c7501d359aa6f738ca2ddddf7d0 (patch) | |
tree | 9468f8035a5a179df154f33947764349f4e60937 /src/p2p/net_node.inl | |
parent | Merge pull request #1585 (diff) | |
parent | Reduce to one connection per IP (diff) | |
download | monero-d10425025a0e0c7501d359aa6f738ca2ddddf7d0.tar.xz |
Merge pull request #1572
0e0e6c5f Reduce to one connection per IP (Miguel Herranz)
3f269e98 Limit incoming connections from the same IP (Miguel Herranz)
Diffstat (limited to 'src/p2p/net_node.inl')
-rw-r--r-- | src/p2p/net_node.inl | 30 |
1 files changed, 30 insertions, 0 deletions
diff --git a/src/p2p/net_node.inl b/src/p2p/net_node.inl index 34a3f9363..60e51c222 100644 --- a/src/p2p/net_node.inl +++ b/src/p2p/net_node.inl @@ -1456,6 +1456,14 @@ namespace nodetool drop_connection(context); return 1; } + + if(has_too_many_connections(context.m_remote_ip)) + { + LOG_PRINT_CCONTEXT_L1("CONNECTION FROM " << epee::string_tools::get_ip_string_from_int32(context.m_remote_ip) << " REFUSED, too many connections from the same address"); + drop_connection(context); + return 1; + } + //associate peer_id with this connection context.peer_id = arg.node_data.peer_id; @@ -1674,4 +1682,26 @@ namespace nodetool return true; } + + template<class t_payload_net_handler> + bool node_server<t_payload_net_handler>::has_too_many_connections(const uint32_t ip) + { + const uint8_t max_connections = 1; + uint8_t count = 0; + + m_net_server.get_config_object().foreach_connection([&](const p2p_connection_context& cntxt) + { + if (cntxt.m_is_income && cntxt.m_remote_ip == ip) { + count++; + + if (count > max_connections) { + return false; + } + } + + return true; + }); + + return count > max_connections; + } } |