Page MenuHomePhabricator
Differential D6433

Don't relay addr messages to block-relay-only peers
ClosedPublic
Actions

Authored by deadalnix on Jun 7 2020, 12:46.

Details

Reviewers
majcosta
Group Reviewers
Restricted Project
Commits
rABCcdf97a64a55c: Don't relay addr messages to block-relay-only peers
Summary

We don't want relay of addr messages to leak information about
these network links.

This is a partial backport of Core PR15759 : https://github.com/bitcoin/bitcoin/pull/15759/commits/430f489027f15c1e4948ea4378954df24e3fee88

Depends on D6429

Test Plan
ninja all check-all

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

deadalnix created this revision.Jun 7 2020, 12:46
Herald added a reviewer: Restricted Project. · View Herald TranscriptJun 7 2020, 12:46
deadalnix requested review of this revision.Jun 7 2020, 12:46
teamcity edited the summary of this revision. (Show Details)Jun 7 2020, 12:47

[Bot Message]
One or more PR numbers were detected in the summary.
Links to those PRs have been inserted into the summary for reference.

Harbormaster completed remote builds in B11288: Diff 21038.Jun 7 2020, 12:58
majcosta accepted this revision.Jun 7 2020, 22:21
This revision is now accepted and ready to land.Jun 7 2020, 22:21
deadalnix updated this revision to Diff 21085.Jun 8 2020, 13:57

rebase

Harbormaster completed remote builds in B11315: Diff 21085.Jun 8 2020, 14:09
Closed by commit rABCcdf97a64a55c: Don't relay addr messages to block-relay-only peers (authored by Suhas Daftuar <sdaftuar@gmail.com>, committed by deadalnix). · Explain WhyJun 8 2020, 20:39
This revision was automatically updated to reflect the committed changes.
deadalnix added a commit: rABCcdf97a64a55c: Don't relay addr messages to block-relay-only peers.

Revision Contents
Changeset List

PathSize
src/
4 lines
6 lines
19 lines

Diff 21113

View Options

src/net.h

Loading...
View Options

src/net.cpp

Loading...
View Options

src/net_processing.cpp

Loading...