Skip to content

Commit 75c63e3

Browse files
committed
GH-1245 Rename fatal_error to invalid_message
1 parent f778d71 commit 75c63e3

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

plugins/net_plugin/include/eosio/net_plugin/auto_bp_peering.hpp

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -270,12 +270,12 @@ class bp_connection_manager {
270270
}
271271

272272
const controller& cc = self()->chain_plug->chain();
273-
bool fatal_error = false;
273+
bool invalid_message = false;
274274
auto is_peer_key_valid = [&](const gossip_bp_peers_message::bp_peer& peer) -> bool {
275275
try {
276276
if (peer.sig.is_webauthn()) {
277277
fc_dlog(self()->get_logger(), "Peer ${p} signature is webauthn, not allowed.", ("p", peer.producer_name));
278-
fatal_error = true;
278+
invalid_message = true;
279279
return false;
280280
}
281281
std::optional<peer_info_t> peer_info = cc.get_peer_info(peer.producer_name);
@@ -294,15 +294,15 @@ class bp_connection_manager {
294294
}
295295
} catch (fc::exception& e) {
296296
fc_dlog(self()->get_logger(), "Exception recovering peer key ${p}, error: ${e}", ("p", peer.producer_name)("e", e.to_detail_string()));
297-
fatal_error = true;
297+
invalid_message = true;
298298
return false; // invalid key
299299
}
300300
return true;
301301
};
302302

303303
fc::lock_guard g(gossip_bps.mtx);
304304
auto& sig_idx = gossip_bps.index.get<by_sig>();
305-
for (auto i = msg.peers.begin(); i != msg.peers.end() && !fatal_error;) {
305+
for (auto i = msg.peers.begin(); i != msg.peers.end() && !invalid_message;) {
306306
const auto& peer = *i;
307307
bool have_sig = sig_idx.contains(peer.sig); // we already have it, already verified
308308
if (!have_sig && !is_peer_key_valid(peer)) {
@@ -313,7 +313,7 @@ class bp_connection_manager {
313313
}
314314
}
315315

316-
if (fatal_error)
316+
if (invalid_message)
317317
return false;
318318
return !msg.peers.empty();
319319
}

0 commit comments

Comments
 (0)