Skip to content

Conversation

@fanquake
Copy link
Member

@fanquake fanquake commented Mar 9, 2019

Backports #15486 to the 0.18 branch.

sdaftuar added 4 commits March 9, 2019 15:01
Fixes a bug where feelers could be stuck trying to resolve a collision in the
tried table that is to an address in the same netgroup as an existing outbound peer.

Thanks to Muoi Tran for the original bug report and detailed debug logs to track
this down.

Github-Pull: bitcoin#15486
Rebased-From: 4991e3c
After 40 minutes, time out a test-before-evict entry and just evict without
testing. Otherwise, if we were unable to test an entry for some reason, we
might break using feelers altogether.

Github-Pull: bitcoin#15486
Rebased-From: f71fdda
@fanquake
Copy link
Member Author

fanquake commented Mar 9, 2019

Done by @laanwj in f810f14...b80dedb.

@fanquake fanquake closed this Mar 9, 2019
@fanquake fanquake deleted the backport-15486-addrman-collisions branch March 9, 2019 07:15
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Dec 16, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants