Fix out-of-bounds check #2935

pull sipa wants to merge 1 commits into bitcoin:master from sipa:obounds changing 1 files +2 −1

sipa commented at 9:26 PM on August 24, 2013: member

Fixes #2934.
Fix out-of-bounds check 24e5d7d5ae
BitcoinPullTester commented at 9:59 PM on August 24, 2013: none

Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/24e5d7d5ae735b41beaaa7ef5feed845607c8280 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.
gavinandresen commented at 2:47 AM on August 25, 2013: contributor

ACK.

Candidate for a 0.8.4rc2, if we do one (not a critical DoS issue only because creating an invalid block is so expensive).
gmaxwell commented at 10:43 AM on August 25, 2013: contributor

ACK. (Without an extranonce, I was having a hard time producing a block just for testing)
sipa referenced this in commit 8a9e538227 on Aug 25, 2013
sipa merged this on Aug 25, 2013
sipa closed this on Aug 25, 2013
Bushstar referenced this in commit 1d7710d06a on Apr 8, 2020
DrahtBot locked this on Sep 8, 2021

Contributors