The partition checking code was using chainActive timestamps to detect partitioning; with headers-first syncing, it should use (and with this pull request, does use) pIndexBestHeader timestamps.
Fixes issue #6251
Use best header chain timestamps to detect partitioning #6256
pull gavinandresen wants to merge 1 commits into bitcoin:master from gavinandresen:PartitionCheck_Issue6251 changing 4 files +16 −14-
gavinandresen commented at 8:38 PM on June 8, 2015: contributor
-
65b9454503
Use best header chain timestamps to detect partitioning
The partition checking code was using chainActive timestamps to detect partitioning; with headers-first syncing, it should use (and with this pull request, does use) pIndexBestHeader timestamps. Fixes issue #6251
- laanwj added the label Bug on Jun 9, 2015
- laanwj added the label P2P on Jun 9, 2015
-
laanwj commented at 2:41 PM on June 12, 2015: member
utACK
- laanwj merged this on Jun 12, 2015
- laanwj closed this on Jun 12, 2015
- laanwj referenced this in commit 8ccc07c077 on Jun 12, 2015
- gavinandresen referenced this in commit fce474c9df on Jun 15, 2015
-
laanwj commented at 8:38 AM on June 15, 2015: member
Cherry-picked to 0.11 as fce474c9dfa1fd3f98f14f7c2866a6001b21ac9a
- laanwj referenced this in commit e6334f4aad on Jun 21, 2015
- MarcoFalke locked this on Sep 8, 2021
Contributors