[net processing] ignore unknown INV types in GETDATA messages
ClosedPublic
Actions

Authored by PiRK on Jan 27 2021, 16:58.

Details

Reviewers

majcosta

Group Reviewers

Restricted Project

Commits

rABCa3dac181b44c: [net processing] ignore unknown INV types in GETDATA messages

Summary

Co-Authored-By: John Newbery <john@johnnewbery.com>

This is a backport of Core PR18808 [2/4]
https://github.com/bitcoin/bitcoin/pull/18808/commits/e257cf71c851e25e1a533bf1d4296f6b55c81332
Depends on D9084

Test Plan

ninja all check-all

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 15017
Build 29985: Build Diff	lint-circular-dependencies · build-without-wallet · build-diff · build-debug · build-clang-tidy · build-clang
Build 29984: arc lint + arc unit

Event Timeline

PiRK created this revision.Jan 27 2021, 16:58

Herald added a reviewer: Restricted Project. · View Herald TranscriptJan 27 2021, 16:58

PiRK requested review of this revision.Jan 27 2021, 16:58

teamcity edited the summary of this revision. (Show Details)Jan 27 2021, 16:58

PiRK added a child revision: D9085: [test] test that an invalid GETDATA doesn't prevent processing of future messages.Jan 27 2021, 17:03

PiRK edited the summary of this revision. (Show Details)

Harbormaster completed remote builds in B15017: Diff 27366.Jan 27 2021, 17:29

majcosta accepted this revision.Jan 27 2021, 17:46

This revision is now accepted and ready to land.Jan 27 2021, 17:46

Closed by commit rABCa3dac181b44c: [net processing] ignore unknown INV types in GETDATA messages (authored by Amiti Uttarwar <amiti@uttarwar.org>, committed by PiRK). · Explain WhyJan 28 2021, 08:14

This revision was automatically updated to reflect the committed changes.

PiRK added a commit: rABCa3dac181b44c: [net processing] ignore unknown INV types in GETDATA messages.

Revision Contents
Changeset List

Path

Size

src/

net_processing.cpp

10 lines

Diff 27366

View Options

[net processing] ignore unknown INV types in GETDATA messagesClosedPublicActions