Differential D11363

[avalanche] Don't ban peers sending us unsolicited avalanche messages
ClosedPublic
Actions

Authored by Fabien on Apr 15 2022, 08:54.

Details

Reviewers

PiRK

Group Reviewers

Restricted Project

Commits

rABC3b93fbd4fe78: [avalanche] Don't ban peers sending us unsolicited avalanche messages

Summary

This doesn't make sense to ban a peer for sending a message when some local option is missing, as there is no way for that peer to know about that. Just do the same than for any other message and ignore it.

Test Plan

ninja all check-all

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

Fabien created this revision.Apr 15 2022, 08:54

Herald added a reviewer: Restricted Project. · View Herald TranscriptApr 15 2022, 08:54

Fabien requested review of this revision.Apr 15 2022, 08:54

Harbormaster completed remote builds in B18833: Diff 33230.Apr 15 2022, 09:09

Fabien added a child revision: D11365: [avalanche] Always send an avahello message if we have a proof.Apr 15 2022, 10:58

PiRK accepted this revision.Apr 15 2022, 15:55

This revision is now accepted and ready to land.Apr 15 2022, 15:55

Closed by commit rABC3b93fbd4fe78: [avalanche] Don't ban peers sending us unsolicited avalanche messages (authored by Fabien). · Explain WhyApr 15 2022, 20:57

This revision was automatically updated to reflect the committed changes.

Fabien added a commit: rABC3b93fbd4fe78: [avalanche] Don't ban peers sending us unsolicited avalanche messages.

Revision Contents
Changeset List

Path

Size

src/

net_processing.cpp

2 lines

test/

functional/

p2p_invalid_messages.py

24 lines

Diff 33240

View Options

src/net_processing.cpp