aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* test: Create test fixture for walletWladimir J. van der Laan2016-04-188-31/+55
| | | | | Removes all the `#ifdef ENABLE_WALLET` from `test_bitcoin` by making the wallet tests use their own fixture.
* test: move accounting_tests and rpc_wallet_tests to wallet/testWladimir J. van der Laan2016-04-183-2/+2
| | | | Move the two other wallet tests to where they belong.
* Merge #7848: Divergence between 32- and 64-bit when hashing >4GB affects ↵Wladimir J. van der Laan2016-04-186-4/+12
|\ | | | | | | | | | | | | | | `gettxoutsetinfo` 28b400f doc: update release-notes for `gettxoutsetinfo` change (Wladimir J. van der Laan) 76212bb rpc: make sure `gettxoutsetinfo` hash has txids (Wladimir J. van der Laan) 9ad1a51 crypto: bytes counts are 64 bit (Wladimir J. van der Laan)
| * doc: update release-notes for `gettxoutsetinfo` changeWladimir J. van der Laan2016-04-151-0/+7
| |
| * rpc: make sure `gettxoutsetinfo` hash has txidsWladimir J. van der Laan2016-04-151-0/+1
| | | | | | | | | | | | | | | | | | | | The key (transaction id for the following outputs) should be serialized to the HashWriter. This is a problem as it means different transactions in the same position with the same outputs will potentially result in the same hash. Fixes primary concern of #7758.
| * crypto: bytes counts are 64 bitWladimir J. van der Laan2016-04-154-4/+4
| | | | | | | | | | | | Byte counts for SHA256, SHA512, SHA1 and RIPEMD160 must be 64 bits. `size_t` has a different size per platform, causing divergent results when hashing more than 4GB of data.
* | Merge #7833: tests: Check Content-Type header returned from RPC serverWladimir J. van der Laan2016-04-181-0/+5
|\ \ | | | | | | | | | 5078ca4 tests: Check Content-Type header returned from RPC server (Wladimir J. van der Laan)
| * | tests: Check Content-Type header returned from RPC serverWladimir J. van der Laan2016-04-071-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Check the Content-Type header that is returned from the RPC server. Only if it is `application/json` the data is supposed to be parsed as JSON. This gives better reporting if the HTTP server happens to return an error that is not JSON-formatted, which is the case if it happens at a lower level before JSON-RPC kicks in. Before: `Unexpected exception caught during testing: No JSON object could be decoded` After: `JSONRPC error: non-JSON HTTP response with '403 Forbidden' from server`
* | | Merge #7888: prevector: fix 2 bugs in currently unreached code pathsWladimir J. van der Laan2016-04-182-16/+18
|\ \ \ | |_|/ |/| | | | | | | | | | | a7af72a prevector::swap: fix (unreached) data corruption (Kaz Wesley) 4ed41a2 test prevector::swap (Kaz Wesley) 1e2c29f prevector: destroy elements only via erase() (Kaz Wesley)
| * | prevector::swap: fix (unreached) data corruptionKaz Wesley2016-04-161-6/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | swap was using an incorrect condition to determine when to apply an optimization (not swapping the full direct[] when swapping two indirect prevectors). Rather than correct the optimization I'm removing it for simplicity. Removing this optimization minutely improves performance in the typical (currently only) usage of member swap(), which is swapping with a freshly value-initialized object.
| * | test prevector::swapKaz Wesley2016-04-161-2/+13
| | | | | | | | | | | | | | | - add a swap operation to prevector tests (fails due to broken prevector::swap) - fix 2 prevector test operation conditions that were impossible
| * | prevector: destroy elements only via erase()Kaz Wesley2016-04-161-8/+4
|/ / | | | | | | | | | | | | | | | | Fixes a bug in which pop_back did not call the deleted item's destructor. Using the most general erase() implementation to implement all the others prevents similar bugs because the coupling between deallocation and destructor invocation only needs to be maintained in one place. Also reduces duplication of complex memmove logic.
* | Merge #7603: Build System: Use PACKAGE_TARNAME in NSIS scriptWladimir J. van der Laan2016-04-153-19/+24
|\ \ | | | | | | | | | | | | | | | 0528e30 Remove wxwidgets references from NSIS script. (JeremyRand) 26880c3 build: Use PACKAGE_TARNAME and new bin names in NSIS script. (JeremyRand) 0dbf6e4 build: define base filenames for use elsewhere in the buildsystem (Cory Fields)
| * | Remove wxwidgets references from NSIS script.JeremyRand2016-04-111-4/+0
| | | | | | | | | | | | The NSIS script tried to delete wxwidgets-based executables/locales. These files are ancient, and presumably no users have them anymore, so we can simplify the NSIS script by removing those lines.
| * | build: Use PACKAGE_TARNAME and new bin names in NSIS script.JeremyRand2016-04-111-13/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Replaces the hardcoded string "bitcoin" with the autoconf variable PACKAGE_TARNAME; fixes #7265. Places where I chose not to replace: 1. bitcoin.ico wasn't replaced because it doesn't seem to be relevant to the build system and its filename never affects the end user. 2. InstallDir wasn't replaced because the current text has an uppercase B, and I'm not sure of a good way to capitalize the result of PACKAGE_TARNAME. 3. A comment in the Main Installer section wasn't replaced because comments don't ever face the end user. 4. The registry value "URL:Bitcoin" wasn't replaced for the same reason as InstallDir. 5. Startup shortcut wasn't replaced for the same reason as InstallDir. All other appearances of "bitcoin" were replaced with PACKAGE_TARNAME, except for the bin names, which were instead replaced with the new bin name autoconf variables.
| * | build: define base filenames for use elsewhere in the buildsystemCory Fields2016-04-112-3/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | Unfortunately, the target namees defined at the Makefile.am level can't be used for *.in substitution. So these new defines will have to stay synced up with those targets. Using the new variables for the deploy targets in the main Makefile.am will ensure that they stay in sync, otherwise build tests will fail.
* | | Merge #7756: Add cursor to iterate over utxo set, use this in `gettxoutsetinfo`Wladimir J. van der Laan2016-04-156-57/+148
|\ \ \ | | | | | | | | | | | | 509cb00 txdb: Add Cursor() method to CCoinsView to iterate over UTXO set (Wladimir J. van der Laan)
| * | | txdb: Add Cursor() method to CCoinsView to iterate over UTXO setWladimir J. van der Laan2016-04-156-57/+148
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add a method Cursor() to CCoinsView that returns a cursor which can be used to iterate over the whole UTXO set. - rpc: Change gettxoutsetinfo to use new Cursor method - txdb: Remove GetStats method - Now that GetStats is implemented in terms of Cursor, remove it.
* | | | Merge #7518: Add multiple options to fundrawtransactionWladimir J. van der Laan2016-04-156-28/+183
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | f2d0944 Add lockUnspents option to fundrawtransaction (João Barbosa) af4fe7f Add change options to fundrawtransaction (João Barbosa) 41e835d Add strict flag to RPCTypeCheckObj (João Barbosa)
| * | | | Add lockUnspents option to fundrawtransactionJoão Barbosa2016-04-153-6/+19
| | | | |
| * | | | Add change options to fundrawtransactionJoão Barbosa2016-04-154-24/+153
| | | | |
| * | | | Add strict flag to RPCTypeCheckObjJoão Barbosa2016-04-152-2/+15
|/ / / / | | | | | | | | | | | | Strict flag forces type check on all object keys.
* | | | Merge #7870: [contrib] Add MarcoFalke key and move all keys to ↵Wladimir J. van der Laan2016-04-1524-4/+73
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | contrib/gitian-keys faf4c83 [gitian] Move keys to contrib/gitian-keys (MarcoFalke) fa93936 [gitian] Add marcofalke-key.pgp (MarcoFalke)
| * | | | [gitian] Move keys to contrib/gitian-keysMarcoFalke2016-04-1524-4/+4
| | | | |
| * | | | [gitian] Add marcofalke-key.pgpMarcoFalke2016-04-151-0/+69
|/ / / /
* | | | Merge #7856: Only send one GetAddr response per connection.Wladimir J. van der Laan2016-04-153-0/+10
|\ \ \ \ | | | | | | | | | | | | | | | 66b0724 Only send one GetAddr response per connection. (Gregory Maxwell)
| * | | | Only send one GetAddr response per connection.Gregory Maxwell2016-04-113-0/+10
| | |/ / | |/| | | | | | | | | | | | | | | | | | This conserves resources from abusive peers that just send getaddr in a loop. Also makes correlating addr messages against INVs less effective.
* | | | Merge #7877: Change mapRelay to store CTransactionsWladimir J. van der Laan2016-04-153-21/+8
|\ \ \ \ | | | | | | | | | | | | | | | 38c3102 Change mapRelay to store CTransactions (Pieter Wuille)
| * | | | Change mapRelay to store CTransactionsPieter Wuille2016-04-143-21/+8
| | | | |
* | | | | Merge #7809: depends: some base fixes/changesWladimir J. van der Laan2016-04-155-13/+39
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 11d9f6b depends: qt/cctools: fix checksum checksum tests (Cory Fields) bb717f4 depends: fix "unexpected operator" error during "make download" (Cory Fields) fe740f1 depends: fix fallback downloads (Cory Fields) dc4ec6d depends: create a hostid and buildid and add option for salts (Cory Fields)
| * | | | | depends: qt/cctools: fix checksum checksum testsCory Fields2016-04-042-2/+6
| | | | | | | | | | | | | | | | | | | | | | | | Checksums were being verified after download, but not again before extraction
| * | | | | depends: fix "unexpected operator" error during "make download"Cory Fields2016-04-041-3/+3
| | | | | |
| * | | | | depends: fix fallback downloadsCory Fields2016-04-041-6/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In some cases, failed downloads wouldn't trigger a fallback download attempt. Namely, checksum mismatches.
| * | | | | depends: create a hostid and buildid and add option for saltsCory Fields2016-04-043-2/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | These add very simple sanity checks to ensure that the build/host toolchains have not changed since the last run. If they have, all ids will change and packages will be rebuilt. For more complicated usage (like parsing dpkg), HOST_ID_SALT/BUILD_ID_SALT may be used to introduce arbitrary data to the ids.
* | | | | | Merge #7798: [travis] Print the commit which was evaluatedWladimir J. van der Laan2016-04-151-0/+3
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | fac9ca2 [travis] echo $TRAVIS_COMMIT_RANGE (MarcoFalke) faef3b7 [travis] Print the commit which was evaluated (MarcoFalke)
| * | | | | | [travis] echo $TRAVIS_COMMIT_RANGEMarcoFalke2016-04-051-1/+1
| | | | | | |
| * | | | | | [travis] Print the commit which was evaluatedMarcoFalke2016-04-031-0/+3
| | | | | | |
* | | | | | | Merge #6215: add bip32 pub key serializationWladimir J. van der Laan2016-04-156-12/+71
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | 90604f1 add bip32 pubkey serialization (Jonas Schnelli)
| * | | | | | | add bip32 pubkey serializationJonas Schnelli2016-04-146-12/+71
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | CExtPubKey should be serializable like CPubKey
* | | | | | | | Merge #7878: [test] bctest.py: Revert faa41eeWladimir J. van der Laan2016-04-151-2/+1
|\ \ \ \ \ \ \ \ | |/ / / / / / / |/| | | | | | | | | | | | | | | fa7abe0 [test] bctest.py: Revert faa41ee (MarcoFalke)
| * | | | | | | [test] bctest.py: Revert faa41eeMarcoFalke2016-04-141-2/+1
|/ / / / / / /
* | | | | | | Merge #7874: Improve AlreadyHaveWladimir J. van der Laan2016-04-141-1/+3
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | c6cb6f7 Avoid unnecessary database access for unknown transactions (Alex Morcos)
| * | | | | | | Avoid unnecessary database access for unknown transactionsAlex Morcos2016-04-131-1/+3
| | | | | | | |
* | | | | | | | Merge #7853: [qa] py2: Unfiddle strings into bytes explicitlyWladimir J. van der Laan2016-04-1428-213/+205
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | faa41ee [qa] py2: Unfiddle strings into bytes explicitly (MarcoFalke)
| * | | | | | | | [qa] py2: Unfiddle strings into bytes explicitlyMarcoFalke2016-04-1028-213/+205
| | | | | | | | |
* | | | | | | | | Merge #7812: Tiny refactor of `IsRBFOptIn`, avoid exceptionWladimir J. van der Laan2016-04-143-16/+19
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 4f7c959 Refactor IsRBFOptIn, avoid exception (Jonas Schnelli)
| * | | | | | | | | Refactor IsRBFOptIn, avoid exceptionJonas Schnelli2016-04-063-16/+19
| | |_|_|/ / / / / | |/| | | | | | |
* | | | | | | | | Merge #7842: RPC: do not print minping time in getpeerinfo when no ping ↵Wladimir J. van der Laan2016-04-141-5/+7
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | received yet 62a6486 RPC: do not print ping info in getpeerinfo when no ping received yet, fix help (Pavel Janík)
| * | | | | | | | | RPC: do not print ping info in getpeerinfo when no ping received yet, fix helpPavel Janík2016-04-091-5/+7
| | |_|_|/ / / / / | |/| | | | | | |
* | | | | | | | | Merge #7846: Clean up lockorder data of destroyed mutexesWladimir J. van der Laan2016-04-142-23/+65
|\ \ \ \ \ \ \ \ \ | |_|_|_|_|/ / / / |/| | | | | | | | | | | | | | | | | 5eeb913 Clean up lockorder data of destroyed mutexes (Pieter Wuille)