13133 Commits

Author SHA1 Message Date
Sjors Provoost
f503ff1ecd
build: Bump version to 0.14.3 2018-09-20 14:22:40 +01:00
Suhas Daftuar
52965fbaef
Fix crash bug with duplicate inputs within a transaction
Introduced by #9049

Github-Pull: #14247

Tree-SHA512: 54ccf896e4c816ba8532644affc984a091ed801d8387bb01a836953c9ec4a345359d98fb58dd5f929617afd42bce0cc40293fecf943a1584207c82dd78da0ea5
2018-09-18 13:05:49 +02:00
MarcoFalke
b9ca77863b
Merge #11296: [0.14] travis: filter out pyenv (Cory Fields)
a4a0adbc7 travis: filter out pyenv (Cory Fields)

Pull request description:

  This fixes the travis issues on the 0.14 branch

Tree-SHA512: 302495d9d5eb554e32e6aa3a788416395d8efeaee153dc2ad9425e2762ac83e36a0a5f128d16098147e95d610dd2fbc7aae70e39a11f54ed421abd38693378d5
2017-09-10 18:29:30 +02:00
Cory Fields
a4a0adbc70 travis: filter out pyenv
Github-Pull: #11271
Rebased-From: aa2e0f09ec94dd0908f792ebc2249859ad174586
2017-09-10 04:52:47 +02:00
Wladimir J. van der Laan
91be5e3c1e
Merge #10516: Backports for 0.14.3
ff274d3 [doc] Add hint about getmempoolentry to getrawmempool help. (Karl-Johan Alm)
76f9cf9 contrib: Update location of seeds.txt (Wladimir J. van der Laan)
12adedf Trivial: remove extra character from comment (CryptAxe)
d2ec969 Fixed typo in documentation for merkleblock.h (Mikerah)
3612219 contrib/init/bitcoind.openrcconf: Don't disable wallet by default (Luke Dashjr)
692dbb0 [doc] Minor corrections to osx dependencies (fanquake)
87a21d5 Fix: make CCoinsViewDbCursor::Seek work for missing keys (Pieter Wuille)
28b8b8b [wallet] Securely erase potentially sensitive keys/values (Thomas Snider)
ff13f59 [wallet] Make sure pindex is non-null before possibly referencing in LogPrintf call. (Karl-Johan Alm)
e23cef0 Fix some empty vector references (Pieter Wuille)
6ad45b8 Re-enable upnp support in contrib/debian (Matt Corallo)
e9a0d89 Build with QT5 on Debian-based systems using contrib/debian (Matt Corallo)
2ea0358 Bump minimum boost version in contrib/debian (Matt Corallo)
c94e262 Update contrib/debian to latest Ubuntu PPA upload. (Matt Corallo)
96c7f2c Add CheckQueue Tests (Jeremy Rubin)
e207342 Fix CCheckQueue IsIdle (potential) race condition and remove dangerous constructors. (Jeremy Rubin)
ef810c4 [trivial] Fix a typo (introduced two days ago) in the default fee warning (practicalswift)
7abe7bb Qt/Send: Give fallback fee a reasonable indent (Luke Dashjr)
3e4d7bf Qt/Send: Figure a decent warning colour from theme (Luke Dashjr)
c5adf8f [Qt] Show more significant warning if we fall back to the default fee (Jonas Schnelli)
ee1a60d [tests] update disconnect_ban.py test case to work with listbanned (John Newbery)
d289b56 [net] listbanned RPC and QT should show correct banned subnets (John Newbery)
0422693 [tests] disconnect_ban: remove dependency on urllib (John Newbery)
98bd0c3 [tests] disconnect_ban: use wait_until instead of sleep (John Newbery)
bfd1cf6 [tests] disconnectban test - only use two nodes (John Newbery)
5bc75bb [tests] fix nodehandling.py flake8 warnings (John Newbery)
c25d0a8 Update release notes to include RPC error code changes. (John Newbery)
f5efe82 Return correct error codes in fundrawtransaction(). (John Newbery)
4943d7a Return correct error codes in setban(). (John Newbery)
18c109d Return correct error codes in removeprunedfunds(). (John Newbery)
fe51c89 Return correct error codes in blockchain.cpp. (John Newbery)
3ad00b4 Return correct error codes in bumpfee(). (John Newbery)
71463a7 [qa] Test prioritise_transaction / getblocktemplate interaction (Suhas Daftuar)
d28d583 Bugfix: PrioritiseTransaction updates the mempool tx counter (Suhas Daftuar)

Tree-SHA512: fa3628527c8e176e438de992b9c5815cc2f3c296dbe5d81b592d17a907554e9c6af7eb595e96a2c345de399ba5326c07b4791a91b7b07f89dce0787c85891206
2017-07-06 18:57:00 +02:00
Wladimir J. van der Laan
fc61c8322b
Merge #10588: doc: Note preexisting bug in display of fee calculation in coin control
60b967e Note preexisting bug in display of fee calculation in coin control (Alex Morcos)

Tree-SHA512: 0179654d313ea10b776781617cea744757344d8b4455153540f116978a72aabdc0a3092388b9cfbe8d0348b699943a9e41082ed964e161e4f3017dcd056e77fc
v0.14.2
2017-06-15 12:44:01 +02:00
Alex Morcos
60b967ec41 Note preexisting bug in display of fee calculation in coin control 2017-06-14 14:34:29 -04:00
Wladimir J. van der Laan
7a643511b4
doc: Fill in details about miniupnp CVE-2017-8798 2017-06-07 13:06:11 +02:00
Wladimir J. van der Laan
5e408d99a2
doc: Update manpages for 0.14.2 v0.14.2rc2 2017-06-06 16:26:56 +02:00
Wladimir J. van der Laan
cc23438af0
qt: 0.14.2 pre-rc2 translations update 2017-06-06 15:06:16 +02:00
Karl-Johan Alm
ff274d3b00 [doc] Add hint about getmempoolentry to getrawmempool help.
Github-Pull: #10310
Rebased-From: 3a0a5bc2341ea6a453306f6fe1c065b937294cfb
2017-06-05 22:58:53 +00:00
Wladimir J. van der Laan
76f9cf9ac9 contrib: Update location of seeds.txt
Update the steps for updating the hardcoded seed nodes to point to the
new filename on @sipa's server, and add command to decompress it.

Ref: #10163

Github-Pull: #10495
Rebased-From: ac9cd953d91e2448d24851b2f4973c7be8f4c3cd
2017-06-05 22:58:53 +00:00
CryptAxe
12adedff0b Trivial: remove extra character from comment
Github-Pull: #10309
Rebased-From: 3503716f1e37a0cc3ab13c8f4e774061ff4ae2f8
2017-06-05 22:58:53 +00:00
Mikerah
d2ec96909a Fixed typo in documentation for merkleblock.h
Github-Pull: #10258
Rebased-From: dd07068d6b11e738bc954c998800b5660d160959
2017-06-05 22:58:53 +00:00
Luke Dashjr
3612219f9a contrib/init/bitcoind.openrcconf: Don't disable wallet by default
It's harmless if it goes unused, and confused when a wallet is desired

Github-Pull: #10451
Rebased-From: afc693dea695b75721db55d0decee3b11dcb3625
2017-06-05 22:58:53 +00:00
fanquake
692dbb0288 [doc] Minor corrections to osx dependencies
Github-Pull: #10260
Rebased-From: 661caf83b3dcae6376e59b6cae07f0a2a4d37fe9
2017-06-05 22:58:53 +00:00
Pieter Wuille
87a21d5922 Fix: make CCoinsViewDbCursor::Seek work for missing keys
Thanks to Suhas Daftuar for figuring this out.

Github-Pull: #10445
Rebased-From: 822755a424d0abfa408dc34313f4aca4b816f54f
2017-06-05 22:58:25 +00:00
Thomas Snider
28b8b8b603 [wallet] Securely erase potentially sensitive keys/values
Github-Pull: #10308
Rebased-From: 6c914ac176624468c66febdb1ad0e24ff2118a5f
2017-06-05 22:58:24 +00:00
Karl-Johan Alm
ff13f592c5 [wallet] Make sure pindex is non-null before possibly referencing in LogPrintf call.
Github-Pull: #10265
Rebased-From: c36ea693ee16611e0337aec682d1ce80b90e1bd3
2017-06-05 22:58:24 +00:00
Pieter Wuille
e23cef0c94 Fix some empty vector references
streams.h has some methods that can be tricked into dereferencing
null pointers or end() iterators. Fix this.

Github-Pull: #10250
Rebased-From: f478d98fe49d3c0c0f2c79b3f8d9dbfc1aafd407
2017-06-05 22:58:23 +00:00
Luke Dashjr
a40d69e0b8 Merge #10328 via branch 'debianppa-0.14' into 0.14.2_fixes 2017-06-05 22:58:07 +00:00
Luke Dashjr
1b864c9820 Merge #9497 via remote-tracking branch 'origin-pull/9497/head' into 0.14.2_fixes 2017-06-05 22:58:00 +00:00
Luke Dashjr
77de9bbc57 Merge #10008 via branch 'fee_warning-0.14' into 0.14.2_fixes 2017-06-05 22:57:49 +00:00
Luke Dashjr
aca4b6dbcb Merge #9481 via remote-tracking branch 'jonas/2017/01/fee_warning' into 0.14.2_fixes 2017-06-05 22:57:35 +00:00
Luke Dashjr
e05799a381 Merge #10234, #10143 (fixes only), and #9853 via branch 'disconnect_ban_fixes-0.14' into 0.14.2_fixes 2017-06-05 22:57:27 +00:00
John Newbery
ee1a60d156 [tests] update disconnect_ban.py test case to work with listbanned
Github-Pull: #10234
Rebased-From: d6732d832aa901e733e63799260d409821a2c37a
2017-06-05 22:56:22 +00:00
John Newbery
d289b564e3 [net] listbanned RPC and QT should show correct banned subnets
Github-Pull: #10234
Rebased-From: 77c54b270dd3b469d662c8f69e06f7b00fc4136d
2017-06-05 22:56:05 +00:00
John Newbery
04226938a3 [tests] disconnect_ban: remove dependency on urllib
Github-Pull: #10143
Rebased-From: 5cc3ee24d29397737f2746d78b19a2509c0dedbb
2017-06-05 22:56:04 +00:00
John Newbery
98bd0c338b [tests] disconnect_ban: use wait_until instead of sleep
Github-Pull: #10143
Rebased-From: 12de2f252c8f48e05c579cb679866a68af8c660e
2017-06-05 22:55:10 +00:00
Luke Dashjr
99e5dbd0aa Merge #9853 via branch 'fixerrorcodes-0.14' into 0.14.2_fixes 2017-06-05 19:33:18 +00:00
Luke Dashjr
74653a41fe Merge #10196 via branch 'prioritisetx_gbtcache-0.14' into 0.14.2_fixes 2017-06-05 19:33:01 +00:00
John Newbery
bfd1cf6713 [tests] disconnectban test - only use two nodes
Github-Pull: #10143
Rebased-From: 395561becfa612fedec74fd841cb4f28afdc23d7
2017-06-05 19:26:13 +00:00
John Newbery
5bc75bb8ee [tests] fix nodehandling.py flake8 warnings
Github-Pull: #10143
Rebased-From: d6564a26f4afc28d7d1a24b94946916387c9bf24
2017-06-05 19:26:13 +00:00
John Newbery
c25d0a8739 Update release notes to include RPC error code changes.
Github-Pull: #9853
Rebased-From: adaa281da12bcc697779f97973cea6b103eec4ab
2017-06-05 19:02:01 +00:00
John Newbery
f5efe82a83 Return correct error codes in fundrawtransaction().
The fundrawtransaction() RPC was returning misleading or incorrect error
codes (for example RPC_INTERNAL_ERROR when funding the transaction
failed). This commit fixes those error codes:

- RPC_INTERNAL_ERROR should not be returned for application-level
errors, only for genuine internal errors such as corrupted data.

That error code has been replaced with RPC_WALLET_ERROR.

This commit also updates the test cases to explicitly test the error code.

Github-Pull: #9853
Rebased-From: dab804c18a427901684ebe936b2069a97e04a268
2017-06-05 19:01:16 +00:00
John Newbery
4943d7a9fe Return correct error codes in setban().
The setban() RPC was returning misleading or incorrect error
codes (for example RPC_CLIENT_NODE_ALREADY_ADDED when an invalid IP
address was entered). This commit fixes those error codes:

- RPC_CLIENT_INVALID_IP_OR_SUBNET should be returned if the client
  enters an invalid IP address or subnet.

This commit also updates the test cases to explicitly test the error code.

This commit also adds a testcase for trying to setban on an invalid subnet.

Github-Pull: #9853
Rebased-From: a012087667edb35a36f25ae06b42b1644d80e649
2017-06-05 19:01:16 +00:00
John Newbery
18c109ddb1 Return correct error codes in removeprunedfunds().
The removeprunedfunds() RPC was returning misleading or incorrect error
codes (for example RPC_INTERNAL_ERROR when the transaction was
not found in the wallet). This commit fixes those error codes:

- RPC_INTERNAL_ERROR should not be returned for application-level
errors, only for genuine internal errors such as corrupted data.

This error code has been replaced with RPC_WALLET_ERROR.

This commit also updates the test cases to explicitly test the error code.

Github-Pull: #9853
Rebased-From: 960bc7f778d8dd618e65f1e37ec734e2d4734051
2017-06-05 19:01:16 +00:00
John Newbery
fe51c8924e Return correct error codes in blockchain.cpp.
RPCs in blockchain.cpp were returning misleading or incorrect error
codes (for example getblock() returning RPC_INTERNAL_ERROR when the
block had been pruned). This commit fixes those error codes:

- RPC_INTERNAL_ERROR should not be returned for application-level
  errors, only for genuine internal errors such as corrupted data.
- RPC_METHOD_NOT_FOUND should not be returned in response to a
  JSON request for an existing method.

Those error codes have been replaced with RPC_MISC_ERROR or
RPC_INVALID_PARAMETER as appropriate.

Github-Pull: #9853
Rebased-From: c1190963b388590dc0a346bf625c7e84f69cee8d
2017-06-05 19:01:16 +00:00
John Newbery
3ad00b4b32 Return correct error codes in bumpfee().
The bumpfee() RPC was returning misleading or incorrect error codes
(for example RPC_INVALID_ADDRESS_OR_KEY when the transaction was not
BIP125 replacable). This commit fixes those error codes:

- RPC_INVALID_ADDRESS_OR_KEY if an invalid address was provided:
    - Invalid change address given
- RPC_INVALID_PARAMETER if a single (non-address/key) parameter is incorrect
    - confTarget and totalFee options should not both be set.
    - Invalid confTarget
    - Insufficient totalFee (cannot be less than required fee)
- RPC_WALLET_ERROR for any other error
    - Transaction has descendants in the wallet
    - Transaction has descendants in the mempool
    - Transaction has been mined, or is conflicted with a mined transaction
    - Transaction is not BIP 125 replaceable
    - Transaction has already been bumped
    - Transaction contains inputs that don't belong to the wallet
    - Transaction has multiple change outputs
    - Transaction does not have a change output
    - Fee is higher than maxTxFee
    - New fee rate is less than the minimum fee rate
    - Change output is too small.

This commit also updates the test cases to explicitly test the error code.

Github-Pull: #9853
Rebased-From: 6d07c62322f60eb2702c6654e994fc353bcfcf8c
2017-06-05 19:01:16 +00:00
Wladimir J. van der Laan
21e1ed4898
doc: Preliminary release notes 0.14.2 2017-06-05 08:44:09 +02:00
Matt Corallo
6ad45b8f8b Re-enable upnp support in contrib/debian
Github-Pull: #10328
Rebased-From: 91700aa1df57c1a9209284e743224119396604d5
2017-06-03 21:59:37 +00:00
Matt Corallo
e9a0d89bf7 Build with QT5 on Debian-based systems using contrib/debian
Github-Pull: #10328
Rebased-From: c5071e1f03085b4025f2a33fb2c3079d823d660c
2017-06-03 21:59:36 +00:00
Matt Corallo
2ea035832f Bump minimum boost version in contrib/debian
Github-Pull: #10328
Rebased-From: a8e928699a6ab612459e836ba24af143eae37b0f
2017-06-03 21:59:36 +00:00
Matt Corallo
c94e262d4b Update contrib/debian to latest Ubuntu PPA upload.
This:
 * Partially reverts 9f68ed6 (which fixed spelling in a changelog,
   though generally changelogs should be append-only).
 * Disables UPnP support (PPA has not had it for a while, and I
   still don't trust miniupnpc, plus it seems uneccessary - its
   been a while since we needed to care about Bitcoin-Qt home users
   getting their inbound ports auto-mapped).
 * Enables ZMQ.
 * Forces GUI to Qt4 to fix various issues people have been seeing
   on Ubuntu and elsewhere with Qt5.
 * Reverts 70899d70b (Bitcoin does not enable "instant payments",
   not is transaction management "carried out collectively by the
   network", for whatever "transaction management" means, finally
   Bitcoin Core is not the only way to use the Bitcoin currency,
   as seemingly implied in the description).

Github-Pull: #10328
Rebased-From: 997021986e02f8e59024ffca6437857d9d17f396
2017-06-03 21:59:36 +00:00
practicalswift
ef810c4cd4 [trivial] Fix a typo (introduced two days ago) in the default fee warning
Github-Pull: #10008
Rebased-From: a3ca43bb32520fdc049bc2112a29a554188126cc
2017-06-03 21:19:05 +00:00
Wladimir J. van der Laan
4a41de4585
build: bump version to 0.14.2 2017-06-02 07:39:56 +02:00
Cory Fields
9e3ad50078
net: only enforce the services required to connect
also once half of all outgoing nodes have our preferred flags, require only
minimal flags from the rest.

Github-Pull: #10441
Rebased-From: b6fbfc228236947eaea5c14dda299f5a01810e92
v0.14.2rc1
2017-06-01 13:26:12 +02:00
Wladimir J. van der Laan
ff3f9a807d
Merge #10484: 0.14 Backports
1847642 [Wallet] unset change position when there is no change on exact match (Gregory Sanders)
ae479bc build: fix bitcoin-config.h regeneration after touching build files (Cory Fields)
3d395d6 build: remove wonky auto top-level convenience targets (Cory Fields)
4bc99c3 Add missing <atomic> header in clientmodel.h (Jonas Schnelli)
222f377 Set both time/height header caches at the same time (Jonas Schnelli)
7da1337 Declare headers height/time cache mutable, re-set the methods const (Jonas Schnelli)
9ac40e8 Update the remaining blocks left in modaloverlay at init. (Jonas Schnelli)
1e936d7 Reduce cs_main locks during modal overlay by adding an atomic cache (Jonas Schnelli)
0aee4a1 Check interruptNet during dnsseed lookups (Matt Corallo)
2017-05-31 18:05:06 +02:00
Gregory Sanders
18476423fb
[Wallet] unset change position when there is no change on exact match 2017-05-31 10:57:16 +02:00
Cory Fields
ae479bc45b
build: fix bitcoin-config.h regeneration after touching build files
This was a long-standing and annoying problem.

If autogen.sh was not manually run after touching configure.ac,
bitcoin-config.h would not be properly regenerated. This causes very subtle
problems when configure appears to enable a new value, but it does not end up
reflected in the build.
2017-05-31 10:54:19 +02:00