Skip to content

Do not load corrupted duplicate block indexes.#487

Merged
presstab merged 1 commit intoVeil-Project:masterfrom
presstab:setblockindexempty
Apr 11, 2019
Merged

Do not load corrupted duplicate block indexes.#487
presstab merged 1 commit intoVeil-Project:masterfrom
presstab:setblockindexempty

Conversation

@presstab
Copy link
Contributor

@presstab presstab commented Apr 9, 2019

fixes #405
It is also possible that #422 is related, but not positive.

For some reason multiple indexes are being saved to disk for the same block. The second index will overwrite the first index on first load. The second index has nTx set to 0, as well as all relevant block file locations. The second index causes all subsequent indexes to have nChainTx set to 0, causing them not to be able to be considered block candidates. Also notable is that the second index has the wrong database key which is supposed to be the block hash but is not.

This change simply ignores the second block index on first load. It is still yet to be determined what causes the duplicate index.

@presstab presstab added Tag: Needs Release Notes Any PR or Issue that needs notable mention in release notes Tag: Waiting For QA A pull review is waiting for QA to test the pull request labels Apr 9, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Tag: Needs Release Notes Any PR or Issue that needs notable mention in release notes Tag: Waiting For QA A pull review is waiting for QA to test the pull request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Corruption because of setBlockIndexCandidates.empty() assertion.

2 participants