aboutsummaryrefslogtreecommitdiff
path: root/src/consensus
Commit message (Collapse)AuthorAgeFilesLines
* really s/Doge/Dis/g this timeTomo Ueda2021-09-021-1/+1
|
* Fixate BIP65 softfork heightsPatrick Lodder2021-02-131-1/+1
| | | | | | | | | - Re-introduce Params.BIP65Height - Fixate block 1854705 for testnet - Fixate block 3464751 for mainnet - Fixate block 1351 for regtest - Stop using IsSuperMajority() in validation for BIP65 - Simplify RPC SoftForkMajorityDesc
* Clean up RPC tests (#1465)Ross Nicoll2018-09-191-2/+0
| | | | | | | | | | * Enable full block tests * Fix invalidblocktest * Move watch only address funding to immediately before it's used, so node 0 doesn't spend the output before it checks it later. * Fix `fundrawtransaction` tests and sanitize fee calculation at the same time * Correct resolution of chain parameters when validating tx inputs, especially from previous coinbase transactions * Set block versions on full block tests so that the generated blocks are AuxPoW compatible
* Initialise left/right consensus branches to null (#1435)Ross Nicoll2018-09-191-2/+2
| | | | Strictly speaking the left/right consensus branches haven't been initialised so contain random values. Most of the time these happen to be null, but sometimes causes client crashes (at least it does in 1.15, but fixing in earlier versions to be sure).
* Move COINBASE_MATURITY to the consensus parameters (#1426)Ross Nicoll2018-09-192-5/+2
|
* Modify chain consensus parameters to be height aware (#1396)Ross Nicoll2018-09-191-13/+11
| | | | | | | | | * Modify chain consensus parameters to be height aware * Correct implementation of simplified rewards in parameters * Correct max money * Use base block version in IsSuperMajority() instead of full version * Correct mining of blocks in AuxPoW tests * Add in missing pre-AuxPoW consensus checks
* Change BIP65/66 enforcement to match Dogecoin (#1403)Ross Nicoll2018-09-191-1/+6
| | | | | | | | | | | | | | | | | | * Introduce first estimates at BIP lock-in blocks * Introduce Dogecoin BIP parameters * Re-introduce supermajority rules for BIP65 * Add BIP65 supermajority rules * Tighten v3 block constraints * Don't enforce coinbase in v2 blocks * Correct testnet majority params * Change to using base version when checking supermajority
* Merge AuxPoW support from NamecoreRoss Nicoll2018-09-191-0/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Changes are as below: Wrap CBlockHeader::nVersion into a new class (CBlockVersion). This allows to take care of interpreting the field into a base version, auxpow flag and the chain ID. Update getauxblock.py for new 'generate' RPC call. Add 'auxpow' to block JSON. Accept auxpow as PoW verification. Add unit tests for auxpow verification. Add check for memory-layout of CBlockVersion. Weaken auxpow chain ID checks for the testnet. Allow Params() to overrule when to check the auxpow chain ID and for legacy blocks. Use this to disable the checks on testnet. Introduce CPureBlockHeader. Split the block header part that is used by auxpow and the "real" block header (that uses auxpow) to resolve the cyclic dependency between the two. Differentiate between uint256 and arith_uint256. This change was done upstream, modify the auxpow code. Add missing lock in auxpow_tests. Fix REST header check for auxpow headers. Those can be longer, thus take that into account. Also perform the check actually on an auxpow header. Correctly set the coinbase for getauxblock results. Call IncrementExtraNonce in getauxblock so that the coinbase is actually initialised with the stuff it should be. (BIP30 block height and COINBASE_FLAGS.) Implement getauxblock plus regression test. Turn auxpow test into FIXTURE test. This allows using of the Params() calls. Move CMerkleTx code to auxpow.cpp. Otherwise we get linker errors when building without wallet. Fix rebase with BIP66. Update the code to handle BIP66's nVersion=3. Enforce that auxpow parent blocks have no auxpow block version. This is for compatibility with namecoind. See also https://github.com/namecoin/namecoin/pull/199. Move auxpow-related parameters to Consensus::Params.
* Replace consensus values with Dogecoin equivalentsRoss Nicoll2018-09-191-1/+5
| | | | | | | * Replace chain parameters with Dogecoin values * Update maximum coins to match Dogecoin * Disable version 2 block requirement * Update coinbase maturity to match Dogecoin
* [trivial] Fix typos in commentspracticalswift2017-01-271-1/+1
|
* Introduce assumevalid setting to skip presumed valid scripts.Gregory Maxwell2017-01-131-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This disentangles the script validation skipping from checkpoints. A new option is introduced "assumevalid" which specifies a block whos ancestors we assume all have valid scriptsigs and so we do not check them when they are also burried under the best header by two weeks worth of work. Unlike checkpoints this has no influence on consensus unless you set it to a block with an invalid history. Because of this it can be easily be updated without risk of influencing the network consensus. This results in a massive IBD speedup. This approach was independently recommended by Peter Todd and Luke-Jr since POW based signature skipping (see PR#9180) does not have the verifiable properties of a specific hash and may create bad incentives. The downside is that, like checkpoints, the defaults bitrot and older releases will sync slower. On the plus side users can provide their own value here, and if they set it to something crazy all that will happen is more time will be spend validating signatures. Checkblocks and checklevel are also moved to the hidden debug options: Especially now that checkblocks has a low default there is little need to change these settings, and users frequently misunderstand them as influencing security or IBD speed. By hiding them we offset the space added by this new option.
* Increment MIT Licence copyright header year on files modified in 2016isle29832016-12-314-4/+4
| | | | | | Edited via: $ contrib/devtools/copyright_header.py update .
* Make CBlock::vtx a vector of shared_ptr<CTransaction>Pieter Wuille2016-11-191-3/+3
|
* IBD check uses minimumchain work instead of checkpoints.Gregory Maxwell2016-11-021-0/+1
| | | | | | | | | | | | | This introduces a 'minimum chain work' chainparam which is intended to be the known amount of work in the chain for the network at the time of software release. If you don't have this much work, you're not yet caught up. This is used instead of the count of blocks test from checkpoints. This criteria is trivial to keep updated as there is no element of subjectivity, trust, or position dependence to it. It is also a more reliable metric of sync status than a block count.
* Define start and end time for segwit deploymentPieter Wuille2016-10-171-1/+1
|
* Merge #8607: [doc] Fix doxygen off-by-one comments, fix typosWladimir J. van der Laan2016-08-281-3/+3
|\ | | | | | | | | | | | | fafe7b3 contrib: Make fix-copyright-headers.py more portable (MarcoFalke) fa27c0a [doc] Fix typos in comments, doxygen: Fix comment syntax (MarcoFalke) fabfd5d [qa] pull-tester: Don't mute zmq ImportError (MarcoFalke) 67a5502 init: Fix typo in help message for -whitelistforcerelay (Wladimir J. van der Laan)
| * [doc] Fix typos in comments, doxygen: Fix comment syntaxMarcoFalke2016-08-221-3/+3
| |
* | Consensus: Remove ISMNicolasDorier2016-07-231-4/+4
| |
* | Rename "block cost" to "block weight"Suhas Daftuar2016-07-181-2/+2
|/
* BIP141: Other consensus critical limits, and BIP145Pieter Wuille2016-06-221-3/+9
| | | | Includes changes by Suhas Daftuar, Luke-jr, and mruddy.
* BIP141: Commitment structure and deploymentPieter Wuille2016-06-224-0/+21
| | | | Includes a fix by Suhas Daftuar and LongShao007
* Implement BIP 9 GBT changesLuke Dashjr2016-06-061-0/+1
| | | | | | | - BIP9DeploymentInfo struct for static deployment info - VersionBitsDeploymentInfo: Avoid C++11ism by commenting parameter names - getblocktemplate: Make sure to set deployments in the version if it is LOCKED_IN - In this commit, all rules are considered required for clients to support
* Add CHECKSEQUENCEVERIFY softfork through BIP9Pieter Wuille2016-03-181-0/+1
|
* Add testing of ComputeBlockVersionSuhas Daftuar2016-03-151-1/+2
|
* BIP9 ImplementationPieter Wuille2016-03-151-0/+28
| | | | | Inspired by former implementations by Eric Lombrozo and Rusty Russell, and based on code by Jorge Timon.
* Merge #7184: Implement SequenceLocks functions for BIP 68Wladimir J. van der Laan2016-02-121-1/+4
|\ | | | | | | | | | | | | b043c4b fix sdaftuar's nits again (Alex Morcos) a51c79b Bug fix to RPC test (Alex Morcos) da6ad5f Add RPC test exercising BIP68 (mempool only) (Suhas Daftuar) c6c2f0f Implement SequenceLocks functions (Alex Morcos)
| * Implement SequenceLocks functionsAlex Morcos2016-02-101-1/+4
| | | | | | | | | | | | | | SequenceLocks functions are used to evaluate sequence lock times or heights per BIP 68. The majority of this code is copied from maaku in #6312 Further credit: btcdrak, sipa, NicolasDorier
* | Add missing copyright headersMarcoFalke2016-01-051-0/+4
| |
* | Bump copyright headers to 2015MarcoFalke2015-12-133-3/+3
|/
* Switch blocks to a constant-space Merkle root/branch algorithm.Pieter Wuille2015-11-272-0/+35
| | | | | | | | | | | This switches the Merkle tree logic for blocks to one that runs in constant (small) space. The old code is moved to tests, and a new test is added that for various combinations of block sizes, transaction positions to compute a branch for, and mutations: * Verifies that the old code and new code agree for the Merkle root. * Verifies that the old code and new code agree for the Merkle branch. * Verifies that the computed Merkle branch is valid. * Verifies that mutations don't change the Merkle root. * Verifies that mutations are correctly detected.
* Add merkle.{h,cpp}, generic merkle root/branch algorithmPieter Wuille2015-11-272-0/+169
|
* Merge pull request #6931Pieter Wuille2015-11-121-0/+3
|\ | | | | | | | | 33c90cf Make skipping BIP30 check chain agnostic (Alex Morcos) 06d81ad Skip BIP30 check after BIP34 activation (Alex Morcos)
| * Make skipping BIP30 check chain agnosticAlex Morcos2015-11-021-0/+3
| |
* | Restore MedianTimePast for locktime.Gregory Maxwell2015-11-031-0/+6
|/ | | | | | | | Revert "Revert "Add rules--presently disabled--for using GetMedianTimePast as endpoint for lock-time calculations"" This reverts commit 40cd32e835092c3158175511da5193193ec54939. After careful analysis it was determined that the change was, in fact, safe and several people were suffering momentary confusion about locktime semantics.
* Revert "Add rules--presently disabled--for using GetMedianTimePast as ↵Gregory Maxwell2015-11-011-6/+0
| | | | | | | | | | | | | endpoint for lock-time calculations" This reverts commit 9d55050773d57c0e12005e524f2e54d9e622c6e2. As noted by Luke-Jr, under some conditions this will accept transactions which are invalid by the network rules. This happens when the current block time is head of the median time past and a transaction's locktime is in the middle. This could be addressed by changing the rule to MAX(this_block_time, MTP+offset) but this solution and the particular offset used deserve some consideration.
* Add rules--presently disabled--for using GetMedianTimePast as endpoint for ↵Mark Friedenbach2015-10-231-0/+6
| | | | | | | | lock-time calculations The lock-time code currently uses CBlock::nTime as the cutoff point for time based locked transactions. This has the unfortunate outcome of creating a perverse incentive for miners to lie about the time of a block in order to collect more fees by including transactions that by wall clock determination have not yet matured. By using CBlockIndex::GetMedianTimePast from the prior block instead, the self-interested miner no longer gains from generating blocks with fraudulent timestamps. Users can compensate for this change by simply adding an hour (3600 seconds) to their time-based lock times. If enforced, this would be a soft-fork change. This commit only adds the functionality on an unexecuted code path, without changing the behaviour of Bitcoin Core.
* Added fPowNoRetargeting field to Consensus::Params that disables nBits ↵Eric Lombrozo2015-10-191-0/+1
| | | | recalculation.
* Add debug message to CValidationState for optional extra informationWladimir J. van der Laan2015-08-061-4/+9
| | | | | | Add a field `strDebugMessage` which can be passed to DoS or Invalid, and queried using GetDebugMessage() to add extra troubleshooting information to the validation state.
* Add absurdly high fee message to validation state (for RPC propagation)Shaul Kfir2015-06-301-4/+4
|
* Merge pull request #6124Wladimir J. van der Laan2015-06-261-2/+0
|\ | | | | | | | | | | | | ffd75ad Enable CHECKLOCKTIMEVERIFY as a standard script verify flag (Peter Todd) bc60b2b Replace NOP2 with CHECKLOCKTIMEVERIFY (BIP65) (Peter Todd) 48e9c57 Move LOCKTIME_THRESHOLD to src/script/script.h (Peter Todd) 99088d6 Make CScriptNum() take nMaxNumSize as an argument (Peter Todd)
| * Move LOCKTIME_THRESHOLD to src/script/script.hPeter Todd2015-05-091-2/+0
| | | | | | | | Will now be needed by CHECKLOCKTIMEVERIFY code.
* | use const references where appropriatePhilip Kaufmann2015-06-041-1/+1
| |
* | Consensus: MOVEONLY: Move CValidationState from main consensus/validationjtimon2015-05-151-0/+80
|/
* Consensus: Create consensus/consensus.h with some constantsjtimon2015-04-201-0/+18
|
* ensure consistent header comment naming conventionsPhilip Kaufmann2015-04-201-3/+3
| | | | - BITCOIN_FOLDER_SUBFOLDER_FILENAME_H
* consensus: don't use arith_uint256 in consensus.hCory Fields2015-04-101-2/+1
| | | | Requiring arith_uint256 at such a base level is not good for modularity.
* Consensus: Refactor: Introduce Consensus::Params classJorge Timón2015-03-251-0/+32