Skip to content

Conversation

@volbil
Copy link
Contributor

@volbil volbil commented Aug 29, 2019

Couple system specific check to prevent utf8_codecvt_facet warnings spamming during compilation on macOS.

Before:
Before
After:
After

TheBlueMatt and others added 19 commits July 1, 2019 10:18
There is no reason to wish to store blocks on disk always just
because a peer is whitelisted. This appears to be a historical
quirk to avoid breaking things when the accept limits were added.

Github-Pull: #11531
Rebased-From: 3d9c70ca0fd5d42f31114b689714af1825f0ed30
ProcessMessages will now return earlier when processing headers
messages, rather than continuing on (and do nothing).

Github-Pull: #11568
Rebased-From: 4637f18522429473e68f6f512a03040e121a446d
Github-Pull: #11568
Rebased-From: 37886d5e2f9992678dea4b1bd893f4f10d61d3ad
Reading the variable mapBlockIndex requires holding the mutex cs_main.

The new "Disconnect outbound peers relaying invalid headers" code
added in commit 37886d5e2f9992678dea4b1bd893f4f10d61d3ad and merged
as part of #11568 two days ago did not lock cs_main prior to accessing
mapBlockIndex.

Github-Pull: #11578
Rebased-From: 2530bf27b72e53cc6ffec27de35f3b487984833d
This tracks the set of all known invalid-themselves blocks (ie
blocks which we attempted to connect but which were found to be
invalid). This is used to cheaply check if new headers build on an
invalid chain.

While we're at it we also resolve an edge-case in invalidateblock
on pruned nodes which results in them needing a reindex if they
fail to reorg.

Github-Pull: #11531
Rebased-From: 015a5258adffb0cf394f387a95ac9c8afc34cfc3
Github-Pull: #11560
Rebased-From: db32a6589720e5b15931cef82e477118dfd92669
@volbil volbil changed the base branch from master to develop August 29, 2019 06:00
@volbil volbil changed the base branch from develop to master August 29, 2019 06:01
@volbil
Copy link
Contributor Author

volbil commented Aug 29, 2019

Sorry, not sure in which branch this PR should be addressed, so for now just leave it to master.

cryptozeny referenced this pull request in cryptozeny/sugarchain-v0.16.3 Sep 2, 2019
@blondfrogs
Copy link
Contributor

@volbil Can you point this PR to the develop branch? That is the branch that will be released next.

@volbil volbil changed the base branch from master to develop September 2, 2019 18:16
@volbil
Copy link
Contributor Author

volbil commented Sep 2, 2019

@blondfrogs done :)

@blondfrogs
Copy link
Contributor

#733

@blondfrogs blondfrogs closed this Jan 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.