aboutsummaryrefslogtreecommitdiff
path: root/src
Commit message (Collapse)AuthorAgeFilesLines
* [Qt] group variables below initial if-clauses in AmountSpinBox::stepEnabledPhilip Kaufmann2015-01-131-2/+3
|
* [Qt] don't allow amount changes when AmountSpinBox is read-onlyPhilip Kaufmann2015-01-101-17/+21
| | | | | | | | - before it was possible to use the steps to change e.g. amouns of authenticated or unauthenticated payment requests (AmountSpinBox is already set to read-only here) - this is now fixed - also move the reimplemented stepEnabled() function to the protected section of our class, where it belongs (see Qt doc)
* fail immediately on an empty signatureCory Fields2015-01-091-0/+3
|
* consensus: guard against openssl's new strict DER checksCory Fields2015-01-091-3/+13
| | | | | | New versions of OpenSSL will reject non-canonical DER signatures. However, it'll happily decode them. Decode then re-encode before verification in order to ensure that it is properly consumed.
* Fix tests after #5413Wladimir J. van der Laan2015-01-082-5/+5
| | | | | Pull #5413 was not rebased after deterministic signing was merged (#5227), so the testcases had to be regenerated using UPDATE_JSON_TESTS.
* Merge pull request #5589Wladimir J. van der Laan2015-01-082-4/+4
|\ | | | | | | 23f3435 Rename MAX_TX_SIGOPS to MAX_STANDARD_TX_SIGOPS to match similar policy constant MAX_STANDARD_TX_SIZE (Luke Dashjr)
| * Rename MAX_TX_SIGOPS to MAX_STANDARD_TX_SIGOPS to match similar policy ↵Luke Dashjr2015-01-012-4/+4
| | | | | | | | constant MAX_STANDARD_TX_SIZE
* | Merge pull request #5604Wladimir J. van der Laan2015-01-082-0/+19
|\ \ | | | | | | | | | d58c5d6 tests: run sanity checks in tests too (Cory Fields)
| * | tests: run sanity checks in tests tooCory Fields2015-01-052-0/+19
| | | | | | | | | | | | If these are going to fail in bitcoind, they should fail in the tests as well.
* | | Merge pull request #5143Wladimir J. van der Laan2015-01-088-14/+81
|\ \ \ | | | | | | | | | | | | | | | | | | | | da918ac Make SCRIPT_VERIFY_CLEANSTACK a standardness requirement (Pieter Wuille) b6e03cc Add SCRIPT_VERIFY_CLEANSTACK (BIP62 rule 6) (Pieter Wuille) ae4151b No semantic change: reuse stack variable in P2SH evaluation (Pieter Wuille)
| * | | Make SCRIPT_VERIFY_CLEANSTACK a standardness requirementPieter Wuille2014-11-251-1/+2
| | | |
| * | | Add SCRIPT_VERIFY_CLEANSTACK (BIP62 rule 6)Pieter Wuille2014-11-257-6/+69
| | | | | | | | | | | | | | | | | | | | Based on an earlier patch by Peter Todd, though the rules here are different (P2SH scripts should not have a CLEANSTACK check before the P2SH evaluation).
| * | | No semantic change: reuse stack variable in P2SH evaluationPieter Wuille2014-11-251-7/+10
| | | |
* | | | Merge pull request #5559Wladimir J. van der Laan2015-01-082-21/+87
|\ \ \ \ | | | | | | | | | | | | | | | e179eb3 Make the command-line-args dialog better (Thomas Zander)
| * | | | Make the command-line-args dialog betterThomas Zander2015-01-052-21/+87
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Instead of using a fixed-width font in a label, which virtually guarentees a horizontal scrollbar, use a proper text-document that can re-layout based on user input.
* | | | | Merge pull request #5542Wladimir J. van der Laan2015-01-082-3/+3
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 2ecd294 Bugfix: configure: Correctly detect "nothing to build" condition (Luke Dashjr) b7a4ecc Bugfix: Only check for boost when building code that requires it (Luke Dashjr) a19eeac Bugfix: configure: Check for openssl/ec.h (Luke Dashjr) fe925e2 Use EXTRA_LIBRARIES instead of noinst_LIBRARIES so we can avoid building unused code (Cory Fields)
| * | | | | Use EXTRA_LIBRARIES instead of noinst_LIBRARIES so we can avoid building ↵Cory Fields2015-01-062-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | unused code
* | | | | | Merge pull request #5521Wladimir J. van der Laan2015-01-071-23/+20
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | 0ea28ba Reject non-final txs even in testnet/regtest (Peter Todd)
| * | | | | | Reject non-final txs even in testnet/regtestPeter Todd2015-01-041-23/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previous behavior with IsFinalTx() being an IsStandard() rule was rather confusing and interferred with testing of protocols that depended on nLockTime.
* | | | | | | Merge pull request #5611Wladimir J. van der Laan2015-01-071-4/+6
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | f0b8afc tests: fix spurious windows test failures after 012598880c (Cory Fields)
| * | | | | | | tests: fix spurious windows test failures after 012598880cCory Fields2015-01-061-4/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | on rare occasions, rand() was returning duped values, causing duplicate transactions. BuildMerkleTree happily used these, but CPartialMerkleTree caught them and returned a null merkle root. Rather than taking changes with rand(), use the loop counter to guarantee unique values. At sipa's request, also remove the remaining uses of rand().
* | | | | | | | Merge pull request #5597Wladimir J. van der Laan2015-01-072-2/+8
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | e413457 Catch LevelDB errors during flush (Pieter Wuille) 02bced1 Bugfix: only track UTXO modification after lookup (Pieter Wuille)
| * | | | | | | | Catch LevelDB errors during flushPieter Wuille2015-01-041-0/+4
| | | | | | | | |
| * | | | | | | | Bugfix: only track UTXO modification after lookupPieter Wuille2015-01-041-2/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Otherwise, if CCoinsViewCache::ModifyCoins throws an exception in between setting hasModifier and constructing the CCoinsModifier, the cache ends up in an inconsistent state, resulting in an assert failure in the next modification. Bug discovered by Wladimir J. van der Laan.
* | | | | | | | | Merge pull request #5535Wladimir J. van der Laan2015-01-072-1/+7
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 1c52aad Require sufficent priority for relay of free transactions (Pieter Wuille)
| * | | | | | | | | Require sufficent priority for relay of free transactionsPieter Wuille2014-12-302-1/+7
| |/ / / / / / / /
* | | | | | | | | Merge pull request #5513Wladimir J. van der Laan2015-01-0620-128/+90
|\ \ \ \ \ \ \ \ \ | |_|/ / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 856e862 namespace: drop most boost namespaces and a few header cleanups (Cory Fields) 9b1ab86 namespace: drop boost::assign altogether here (Cory Fields) a324199 namespace: remove boost namespace pollution (Cory Fields)
| * | | | | | | | namespace: drop most boost namespaces and a few header cleanupsCory Fields2015-01-0211-49/+29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | A few boost::asio were left around because they're very wordy otherwise.
| * | | | | | | | namespace: drop boost::assign altogether hereCory Fields2015-01-021-12/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Standard functions are even simpler
| * | | | | | | | namespace: remove boost namespace pollutionCory Fields2015-01-0213-67/+53
| | | | | | | | |
* | | | | | | | | Merge pull request #5520Wladimir J. van der Laan2015-01-062-3/+1
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | bdb6a71 IsNull doesn't change CBlockLocator, add const hint (Pavel Janík) 1b37333 Remove no longer needed declaration of CBlockLocator (Pavel Janík)
| * | | | | | | | | IsNull doesn't change CBlockLocator, add const hintPavel Janík2014-12-201-1/+1
| | | | | | | | | |
| * | | | | | | | | Remove no longer needed declaration of CBlockLocatorPavel Janík2014-12-201-2/+0
| | | | | | | | | |
* | | | | | | | | | Merge pull request #5598Wladimir J. van der Laan2015-01-064-10/+0
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | eb0d34b Remove unused chainparam networkID (jtimon)
| * | | | | | | | | | Remove unused chainparam networkIDjtimon2014-12-114-10/+0
| | |_|_|_|/ / / / / | |/| | | | | | | |
* | | | | | | | | | Merge pull request #5564Wladimir J. van der Laan2015-01-062-5/+5
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | de236f5 clarify obscure uses of EvalScript() (Pavel Vasin)
| * | | | | | | | | | clarify obscure uses of EvalScript()Pavel Vasin2014-12-302-5/+5
| | |_|_|/ / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | The 3rd argument of EvalScript() is the unsigned int flags, not a bool.
* | | | | | | | | | Merge pull request #5532Wladimir J. van der Laan2015-01-061-590/+502
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | be22b3d Update seed IPs, based on bitcoin.sipa.be crawler data (Pieter Wuille)
| * | | | | | | | | | Update seed IPs, based on bitcoin.sipa.be crawler dataPieter Wuille2015-01-051-590/+502
| | | | | | | | | | |
* | | | | | | | | | | Move arith_uint256 out of consensus and utilPieter Wuille2015-01-061-2/+1
| |_|_|_|_|_|_|_|/ / |/| | | | | | | | |
* | | | | | | | | | arith_uint256: remove initialization from byte vectorWladimir J. van der Laan2015-01-053-55/+21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Remove initialization from vector (as this is only used in the tests). Also implement SetHex and GetHex in terms of uint256, to avoid duplicate code as well as avoid endianness issues (as they work in term of bytes).
* | | | | | | | | | Remove now-unused methods from arith_uint256 and base_uintWladimir J. van der Laan2015-01-053-148/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Methods that access the guts of arith_uint256 are removed, as these are incompatible between endians. Use uint256 instead - Serialization is no longer needed as arith_uint256's are never read or written - GetHash is never used on arith_uint256
* | | | | | | | | | Remove arith_uint160Wladimir J. van der Laan2015-01-053-294/+23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We never do 160-bit arithmetic.
* | | | | | | | | | Add tests for new uint256Wladimir J. van der Laan2015-01-052-0/+269
| | | | | | | | | |
* | | | | | | | | | Add conversion functions arith_uint256<->uint_256Wladimir J. van der Laan2015-01-052-0/+24
| | | | | | | | | |
* | | | | | | | | | uint256->arith_uint256 blob256->uint256Wladimir J. van der Laan2015-01-057-701/+985
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Introduce new opaque implementation of `uint256`, move old "arithmetic" implementation to `arith_uint256.
* | | | | | | | | | Use arith_uint256 where necessaryWladimir J. van der Laan2015-01-0511-33/+44
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Also add conversion from/to uint256 where needed.
* | | | | | | | | | String conversions uint256 -> uint256SWladimir J. van der Laan2015-01-0511-66/+66
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If uint256() constructor takes a string, uint256(0) will become dangerous when uint256 does not take integers anymore (it will go through std::string(const char*) making a NULL string, and the explicit keyword is no help).
* | | | | | | | | | Replace uint256(1) with static constantWladimir J. van der Laan2015-01-052-4/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | SignatureHash and its test function SignatureHashOld return uint256(1) as a special error signaling value. Return a local static constant with the same value instead.
* | | | | | | | | | Replace GetLow64 with GetCheapHashWladimir J. van der Laan2015-01-052-5/+5
| | | | | | | | | |