Give peer time-adjustment data an own lock #3114

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

sipa commented at 2:03 PM on October 19, 2013: member

Instead of relying on cs_main (defined in a different module) to prevent concurrent access to it.

Depends on #2840.
laanwj commented at 7:16 AM on October 23, 2013: member

ACK

Give peer time-adjustment data an own lock

Instead of relying on cs_main (defined in a different module) to
prevent concurrent access to it.

a616206865

sipa commented at 12:46 PM on October 26, 2013: member

Rebased to make pulltester run again.
BitcoinPullTester commented at 12:56 PM on October 26, 2013: none

Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/a616206865d62106944300d7b912e3af8d357610 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.
sipa referenced this in commit 1dffbf0060 on Nov 1, 2013
sipa merged this on Nov 1, 2013
sipa closed this on Nov 1, 2013
promag commented at 10:56 AM on March 9, 2016: member

@sipa sorry to bump this, but could't we drop the mutex?
Bushstar referenced this in commit c5da938518 on Apr 8, 2020
DrahtBot locked this on Sep 8, 2021

Contributors