Fix a bug introduced in #4420: it always walked back until it reaches blocks in the this chain.
This is currently not exposed, as we always call CChain::GetLocator with blocks in the main chain, but in #4468 it gets used for more.
Fix a bug in CChain::GetLocator #4470
pull sipa wants to merge 2 commits into bitcoin:master from sipa:fixlocator changing 2 files +62 −8-
sipa commented at 11:41 AM on July 6, 2014: member
-
Fix CChain::GetLocator 3c85d2ec37
-
Add CChain::GetLocator() unit test f4b00beae5
-
BitcoinPullTester commented at 1:03 PM on July 6, 2014: none
Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/p4470_f4b00beae52f22fc6958855a85e1012387c3c33e/ 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.
-
ghost commented at 1:16 PM on July 6, 2014: none
Untested ACK
- laanwj merged this on Jul 7, 2014
- laanwj closed this on Jul 7, 2014
- laanwj referenced this in commit 5e1b7a23b0 on Jul 7, 2014
- MarcoFalke locked this on Sep 8, 2021
