Page MenuHomePhabricator
Differential D18017

[avalanche] bump the misbehavior for oversized AVAPOLL messages
Needs RevisionPublic
Actions

Authored by PiRK on Tue, Apr 29, 10:45.

Details

Reviewers
Fabien
Group Reviewers
Restricted Project
Summary

20 was an arbitrary value. If a node does not format its messages correctly it will get disconnected soon anyway, so don't give it 4 more chances and just disconnect it immediately.

Depends on D18016

Test Plan

ninja all check-all

Diff Detail

Event Timeline

PiRK created this revision.Tue, Apr 29, 10:45
Herald added a reviewer: Restricted Project. · View Herald TranscriptTue, Apr 29, 10:45
PiRK requested review of this revision.Tue, Apr 29, 10:45
Harbormaster completed remote builds in B33150: Diff 53746.Tue, Apr 29, 10:53
PiRK added a child revision: D18018: net_processing: don't bother disconnecting for multiple version messages.Tue, Apr 29, 10:55
Fabien requested changes to this revision.Tue, Apr 29, 13:53
Fabien added a subscriber: Fabien.

This doesn't break any test ? It's certainly time to add one then

This revision now requires changes to proceed.Tue, Apr 29, 13:53

Revision Contents
Changeset List

PathSize
src/
2 lines

Diff 53746

View Options

src/net_processing.cpp

Loading...