nihilist/monero
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Revert "fix accessing an network address in a deleted context"

Browse Source 
This reverts commit 4e74385a1a.
This commit is contained in:
selsta 2021-01-12 12:18:23 +01:00
parent 964ad0e51a
commit 76fe73b8bd
No known key found for this signature in database
GPG Key ID: 2EA0A99A8B07AE5E
2 changed files with 9 additions and 15 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
src/cryptonote_protocol/cryptonote_protocol_handler.inl
View File

@ -2840,15 +2840,12 @@ skip:
epee::string_tools::to_string_hex(context.m_pruning_seed) << epee::string_tools::to_string_hex(context.m_pruning_seed) <<
"), score " << score << ", flush_all_spans " << flush_all_spans); "), score " << score << ", flush_all_spans " << flush_all_spans);
if (score > 0)
m_p2p->add_host_fail(context.m_remote_address, score);
m_block_queue.flush_spans(context.m_connection_id, flush_all_spans); m_block_queue.flush_spans(context.m_connection_id, flush_all_spans);
// copy since dropping the connection will invalidate the context, and thus the address
const auto remote_address = context.m_remote_address;
m_p2p->drop_connection(context); m_p2p->drop_connection(context);
if (score > 0)
m_p2p->add_host_fail(remote_address, score);
} }
//------------------------------------------------------------------------------------------------------------------------ //------------------------------------------------------------------------------------------------------------------------
template<class t_core> template<class t_core>

15
src/p2p/net_node.inl
View File

@ -1217,9 +1217,8 @@ namespace nodetool
if(!handle_remote_peerlist(rsp.local_peerlist_new, context)) if(!handle_remote_peerlist(rsp.local_peerlist_new, context))
{ {
LOG_WARNING_CC(context, "COMMAND_TIMED_SYNC: failed to handle_remote_peerlist(...), closing connection."); LOG_WARNING_CC(context, "COMMAND_TIMED_SYNC: failed to handle_remote_peerlist(...), closing connection.");
const auto remote_address = context.m_remote_address;
m_network_zones.at(context.m_remote_address.get_zone()).m_net_server.get_config_object().close(context.m_connection_id ); m_network_zones.at(context.m_remote_address.get_zone()).m_net_server.get_config_object().close(context.m_connection_id );
add_host_fail(remote_address); add_host_fail(context.m_remote_address);
} }
if(!context.m_is_income) if(!context.m_is_income)
m_network_zones.at(context.m_remote_address.get_zone()).m_peerlist.set_peer_just_seen(context.peer_id, context.m_remote_address, context.m_pruning_seed, context.m_rpc_port, context.m_rpc_credits_per_hash); m_network_zones.at(context.m_remote_address.get_zone()).m_peerlist.set_peer_just_seen(context.peer_id, context.m_remote_address, context.m_pruning_seed, context.m_rpc_port, context.m_rpc_credits_per_hash);
@ -1383,7 +1382,7 @@ namespace nodetool
if(just_take_peerlist) if(just_take_peerlist)
{ {
zone.m_net_server.get_config_object().close(con->m_connection_id); zone.m_net_server.get_config_object().close(con->m_connection_id);
MDEBUG(na.str() << "CONNECTION HANDSHAKED OK AND CLOSED."); LOG_DEBUG_CC(*con, "CONNECTION HANDSHAKED OK AND CLOSED.");
return true; return true;
} }
@ -1445,7 +1444,7 @@ namespace nodetool
zone.m_net_server.get_config_object().close(con->m_connection_id); zone.m_net_server.get_config_object().close(con->m_connection_id);
MDEBUG(na.str() << "CONNECTION HANDSHAKED OK AND CLOSED."); LOG_DEBUG_CC(*con, "CONNECTION HANDSHAKED OK AND CLOSED.");
return true; return true;
} }
@ -2473,14 +2472,12 @@ namespace nodetool
template<class t_payload_net_handler> template<class t_payload_net_handler>
int node_server<t_payload_net_handler>::handle_handshake(int command, typename COMMAND_HANDSHAKE::request& arg, typename COMMAND_HANDSHAKE::response& rsp, p2p_connection_context& context) int node_server<t_payload_net_handler>::handle_handshake(int command, typename COMMAND_HANDSHAKE::request& arg, typename COMMAND_HANDSHAKE::response& rsp, p2p_connection_context& context)
{ {
// copy since dropping the connection will invalidate the context, and thus the address
const auto remote_address = context.m_remote_address;
if(arg.node_data.network_id != m_network_id) if(arg.node_data.network_id != m_network_id)
{ {
LOG_INFO_CC(context, "WRONG NETWORK AGENT CONNECTED! id=" << arg.node_data.network_id); LOG_INFO_CC(context, "WRONG NETWORK AGENT CONNECTED! id=" << arg.node_data.network_id);
drop_connection(context); drop_connection(context);
add_host_fail(remote_address); add_host_fail(context.m_remote_address);
return 1; return 1;
} }
@ -2488,7 +2485,7 @@ namespace nodetool
{ {
LOG_WARNING_CC(context, "COMMAND_HANDSHAKE came not from incoming connection"); LOG_WARNING_CC(context, "COMMAND_HANDSHAKE came not from incoming connection");
drop_connection(context); drop_connection(context);
add_host_fail(remote_address); add_host_fail(context.m_remote_address);
return 1; return 1;
} }