aboutsummaryrefslogtreecommitdiff
path: root/src
Commit message (Collapse)AuthorAgeFilesLines
* Make SCRIPT_VERIFY_STRICTENC compatible with BIP62Pieter Wuille2014-10-088-164/+234
| | | | | | | | * Delete canonical_tests.cpp, and move the tests to script_tests.cpp. * Split off SCRIPT_VERIFY_DERSIG from SCRIPT_VERIFY_STRICTENC (the BIP62 part of it). * Change signature STRICTENC/DERSIG semantics to fail the script entirely rather than the CHECKSIG result (softfork safety, and BIP62 requirement). * Add many autogenerated tests for several odd cases. * Mention specific BIP62 rules in the script verification flags.
* Merge pull request #4834Pieter Wuille2014-10-0817-147/+393
|\ | | | | | | | | | | | | | | 7c70438 Get rid of the dummy CCoinsViewCache constructor arg (Pieter Wuille) ed27e53 Add coins_tests with a large randomized CCoinViewCache test. (Pieter Wuille) 058b08c Do not keep fully spent but unwritten CCoins entries cached. (Pieter Wuille) c9d1a81 Get rid of CCoinsView's SetCoins and SetBestBlock. (Pieter Wuille) f28aec0 Use ModifyCoins instead of mutable GetCoins. (Pieter Wuille)
| * Get rid of the dummy CCoinsViewCache constructor argPieter Wuille2014-09-2414-24/+24
| |
| * Add coins_tests with a large randomized CCoinViewCache test.Pieter Wuille2014-09-232-0/+179
| |
| * Do not keep fully spent but unwritten CCoins entries cached.Pieter Wuille2014-09-234-31/+88
| | | | | | | | | | | | | | | | | | | | | | Instead of storing CCoins entries directly in CCoinsMap, store a CCoinsCacheEntry which additionally keeps track of whether a particular entry is: * dirty: potentially different from its parent view. * fresh: the parent view is known to not have a non-pruned version. This allows us to skip non-dirty cache entries when pushing batches of changes up, and to remove CCoins entries about transactions that are fully spent before the parent cache learns about them.
| * Get rid of CCoinsView's SetCoins and SetBestBlock.Pieter Wuille2014-09-237-44/+8
| | | | | | | | | | | | All direct modifications are now done through ModifyCoins, and BatchWrite is used for pushing batches of queued modifications up, so we don't need the low-level SetCoins and SetBestBlock anymore in the top-level CCoinsView class.
| * Use ModifyCoins instead of mutable GetCoins.Pieter Wuille2014-09-235-62/+108
| | | | | | | | | | | | | | | | | | Replace the mutable non-copying GetCoins method with a ModifyCoins, which returns an encapsulated iterator, so we can keep track of concurrent modifications (as iterators can be invalidated by those) and run cleanup code after a modification is finished. This also removes the overloading of the 'GetCoins' name.
* | Merge pull request #4979Wladimir J. van der Laan2014-10-083-8/+8
|\ \ | | | | | | | | | de10efd add -timeout default as constant and use them (Philip Kaufmann)
| * | add -timeout default as constant and use themPhilip Kaufmann2014-10-063-8/+8
| | | | | | | | | | | | | | | - update help message text - simplify code in init to check for -timeout
* | | Merge pull request #5055Wladimir J. van der Laan2014-10-083-40/+179
|\ \ \ | | | | | | | | | | | | 2c930d9 Prettify JSON tests and minimize diffs (Pieter Wuille)
| * | | Prettify JSON tests and minimize diffsPieter Wuille2014-10-073-40/+179
| | | |
* | | | Merge pull request #4976Wladimir J. van der Laan2014-10-071-0/+2
|\ \ \ \ | | | | | | | | | | | | | | | 27c3e91 qt: add proxy to options overridden if necessary. (Cory Fields)
| * | | | qt: add proxy to options overridden if necessary.Cory Fields2014-09-251-0/+2
| | |_|/ | |/| | | | | | | | | | | | | | If proxy is disabled in the gui but enabled via the command line, it needs to be added to the override list.
* | | | Merge pull request #5047Wladimir J. van der Laan2014-10-072-3/+5
|\ \ \ \ | | | | | | | | | | | | | | | 3e61eb9 minor txindex documentation improvement (Cozz Lovan)
| * | | | minor txindex documentation improvementCozz Lovan2014-10-062-3/+5
| | | | |
* | | | | Merge pull request #4954Wladimir J. van der Laan2014-10-071-2/+13
|\ \ \ \ \ | |_|_|/ / |/| | | | | | | | | 9d7cd4c Don't return an address for invalid pubkeys (Andy Alness)
| * | | | Don't return an address for invalid pubkeysAndy Alness2014-10-061-2/+13
| | | | |
* | | | | fix a lock indentation in main.cppPhilip Kaufmann2014-10-061-5/+5
| | | | | | | | | | | | | | | | | | | | - also ensure alphabetical ordering in file header
* | | | | minor license, header end comment etc. cleanup in /scriptPhilip Kaufmann2014-10-063-6/+6
| |_|_|/ |/| | | | | | | | | | | | | | | | | | | - ensure all licenses are just MIT - add a missing header end comment - ensure alphabetical ordering
* | | | Merge pull request #5036Wladimir J. van der Laan2014-10-064-220/+182
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | eb6b3b2 Update English translation (Wladimir J. van der Laan) b9b2e3f Don't translate state.Abort() messages (Wladimir J. van der Laan)
| * | | | Update English translationWladimir J. van der Laan2014-10-022-201/+161
| | | | |
| * | | | Don't translate state.Abort() messagesWladimir J. van der Laan2014-10-022-19/+21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There is only one message passed to AbortNode() that makes sense to translate to the user specifically: Disk space is low. For the others show a generic message and refer to debug.log for details. Reduces the number of confusing jargon translation messages.
* | | | | Merge pull request #4890Wladimir J. van der Laan2014-10-0615-135/+215
|\ \ \ \ \ | |_|_|/ / |/| | | | | | | | | | | | | | | | | | | e790c37 Replace SCRIPT_VERIFY_NOCACHE by flag directly to checker (Pieter Wuille) 5c1e798 Make signature cache optional (Pieter Wuille) c7829ea Abstract out SignatureChecker (Pieter Wuille)
| * | | | Replace SCRIPT_VERIFY_NOCACHE by flag directly to checkerPieter Wuille2014-10-0213-74/+64
| | | | |
| * | | | Make signature cache optionalPieter Wuille2014-10-029-88/+155
| | | | |
| * | | | Abstract out SignatureCheckerPieter Wuille2014-10-024-14/+37
| |/ / /
* | | | Merge pull request #4982Wladimir J. van der Laan2014-10-031-18/+24
|\ \ \ \ | | | | | | | | | | | | | | | b5ec5fe update getnetworkinfo help with subversion (Philip Kaufmann)
| * | | | update getnetworkinfo help with subversionPhilip Kaufmann2014-10-011-18/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - add missing subversion field to getnetworkinfo help - fix style and format errors in help string - remove unneeded int casts
* | | | | Merge pull request #4887Pieter Wuille2014-10-031-9/+9
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | 4bb30a1 Correct logging AcceptBlock()->AcceptBlockHeader() (R E Broadley)
| * | | | | Correct logging AcceptBlock()->AcceptBlockHeader()R E Broadley2014-10-031-9/+9
| | |/ / / | |/| | |
* | | | | Merge pull request #4994Wladimir J. van der Laan2014-10-021-30/+7
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | 54510f2 DRY: Avoid repetitions in script_test (jtimon)
| * | | | | DRY: Avoid repetitions in script_testjtimon2014-10-021-30/+7
| |/ / / /
* | | | | Merge pull request #4978Wladimir J. van der Laan2014-10-022-16/+41
|\ \ \ \ \ | |/ / / / |/| | | | | | | | | | | | | | | | | | | 938bcce CAutoFile: make file private (Philip Kaufmann) 0c35486 CBufferedFile: add explicit close function (Philip Kaufmann) c9fb27d CBufferedFile: convert into a non-refcounted RAII wrapper (Philip Kaufmann)
| * | | | CAutoFile: make file privatePhilip Kaufmann2014-10-021-3/+4
| | | | |
| * | | | CBufferedFile: add explicit close functionPhilip Kaufmann2014-10-021-6/+18
| | | | | | | | | | | | | | | | | | | | | | | | | - also use identical close function for CAutoFile (avoids setting file to NULL under wrong conditions)
| * | | | CBufferedFile: convert into a non-refcounted RAII wrapperPhilip Kaufmann2014-10-022-10/+22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - it now takes over the passed file descriptor and closes it in the destructor - this fixes a leak in LoadExternalBlockFile(), where an exception could cause the file to not getting closed - disallow copies (like recently added for CAutoFile) - make nType and nVersion private
* | | | | Merge pull request #4985Wladimir J. van der Laan2014-10-022-79/+87
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | bc470c4 Changed mixed indentation to four spaces (Eric Shaw) 31d6390 Fixed setaccount accepting foreign address (Eric Shaw)
| * | | | | Changed mixed indentation to four spacesEric Shaw2014-10-021-74/+73
| | | | | | | | | | | | | | | | | | | | | | | | Put `test/rpc_wallet_tests.cpp` through clang-format.
| * | | | | Fixed setaccount accepting foreign addressEric Shaw2014-10-022-9/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixed issue #4209 where using setaccount with a foreign address causes the address to be added to your receiving addresses.
* | | | | | Merge pull request #5030Wladimir J. van der Laan2014-10-0210-4/+95
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | fbd36d8 Avoid introducing a virtual into CChainParams (Wladimir J. van der Laan) f0fd00c Switch testing framework from MAIN to new UNITTEST network (SergioDemianLerner)
| * | | | | | Avoid introducing a virtual into CChainParamsWladimir J. van der Laan2014-10-022-5/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Treat fSkipProofOfWorkCheck the same as other parameters.
| * | | | | | Switch testing framework from MAIN to new UNITTEST networkSergioDemianLerner2014-10-0210-4/+97
| | | | | | | | | | | | | | | | | | | | | | | | | | | | UNITTEST inherites from MAIN but allows synamically changing its parameters using the ModifiableParams() interface
* | | | | | | amount.h: update license, add header end comment + rem from util.hPhilip Kaufmann2014-10-022-3/+2
|/ / / / / /
* | | | | | Revert merge of pull #4845Wladimir J. van der Laan2014-10-0212-371/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It breaks the new mingw tests! See - https://travis-ci.org/bitcoin/bitcoin/jobs/36845581 - https://travis-ci.org/bitcoin/bitcoin/jobs/36845582 This reverts commit 470590277782cce2fe73275c74523aef59a51eab, 5e2e7fcb99738d9254d4030d53e4f711b2fc5ee0, a25fd6be138ff2bff7e2ad6a1a789db523c0193f.
* | | | | | Merge pull request #4845Wladimir J. van der Laan2014-10-0212-4/+371
|\ \ \ \ \ \ | |_|/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | 4705902 Avoid introducing a virtual into CChainParams (Wladimir J. van der Laan) 5e2e7fc Suggested corrections on comments, variable names. Also new test case testing the PoW skip in UNITTEST. (SergioDemianLerner) a25fd6b Switch testing framework from MAIN to new UNITTEST network (SergioDemianLerner)
| * | | | | Avoid introducing a virtual into CChainParamsWladimir J. van der Laan2014-09-292-5/+3
| | | | | | | | | | | | | | | | | | | | | | | | Treat fSkipProofOfWorkCheck the same as other parameters.
| * | | | | Suggested corrections on comments, variable names.SergioDemianLerner2014-09-293-48/+91
| | | | | | | | | | | | | | | | | | | | | | | | Also new test case testing the PoW skip in UNITTEST.
| * | | | | Switch testing framework from MAIN to new UNITTEST networkSergioDemianLerner2014-09-2912-3/+329
| | | | | | | | | | | | | | | | | | | | | | | | UNITTEST inherites from MAIN but allows synamically changing its parameters using the ModifiableParams() interface
* | | | | | Merge pull request #5017Wladimir J. van der Laan2014-10-021-2/+4
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | f74fc9b Print input index when signature validation fails, to aid debugging. (Mark Friedenbach) 217a5c9 When transaction outputs exceed inputs, show the offending amounts so as to aid debugging. (Mark Friedenbach)
| * | | | | | Print input index when signature validation fails, to aid debugging.Mark Friedenbach2014-10-011-1/+1
| | | | | | |