Move checkpoint based heuristic checks to AcceptBlockHeader #4310

pull sipa wants to merge 1 commits into bitcoin:master from sipa:movefillheaders changing 1 files +22 −22
  1. sipa commented at 11:30 PM on June 8, 2014: member

    It was incorrectly moved to CheckBlockHeader, which is not correct, as it is not a context-independent validity check of the block header.

    Fixes #4246.

  2. Move checkpoint based heuristic checks to AcceptBlockHeader 7a9e0b6460
  3. BitcoinPullTester commented at 11:53 PM on June 8, 2014: none

    Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/7a9e0b6460037b72376af293dc87acafe4fca831 for binaries and test log. This test script verifies pulls every time they are updated. It, however, dies sometimes and fails to test properly. If you are waiting on a test, please check timestamps to verify that the test.log is moving at http://jenkins.bluematt.me/pull-tester/current/ Contact BlueMatt on freenode if something looks broken.

  4. laanwj commented at 12:15 PM on June 9, 2014: member

    I can confirm that this solves #4246 ACK

  5. laanwj merged this on Jun 19, 2014
  6. laanwj closed this on Jun 19, 2014

  7. laanwj referenced this in commit b8b98d5642 on Jun 19, 2014
  8. MarcoFalke locked this on Sep 8, 2021

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-04-13 18:15 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me