11504 Commits

Author SHA1 Message Date
Jonas Schnelli
0cc07f82f0 [QA] add fundrawtransaction test on a locked wallet with empty keypool
Github-Pull: #9295
Rebased-From: 1a6eacbf3b7e3d5941fec1154079bbc4678ce861
2016-12-14 12:58:11 +01:00
Wladimir J. van der Laan
782328660e
Merge #9323: [0.13] Add release notes for wallet/mempool rejections. (PR #9302 and #9290)
57aec3b Add release notes for wallet/mempool rejections. (PR #9302 and #9290) (Gregory Maxwell)
2016-12-14 09:35:08 +01:00
Wladimir J. van der Laan
423659c951
Merge #9264: 0.13.2 backports
f26dab7 Adapt ZMQ/rest serialization to take rpcserialversion arg (instagibbs)
21ccb9f Add option to return non-segwit serialization via rpc (Gregory Sanders)
a710a43 Improvement of documentation of command line parameter 'whitelist' (wodry)
58eab24 [qa] rpc-tests: Apply random offset to portseed (MarcoFalke)
ecd7db5 [qa] test_framework: Exit when tmpdir exists (MarcoFalke)
387ec9d Add script tests for FindAndDelete in pre-segwit and segwit scripts (Johnson Lau)
87fbced Change all instance of 'GMT epoch' to 'Unix epoch' (matthias)
b1e978c instance of 'mem pool' to 'mempool' (S. Matthew English)
ff55a2d Update gitian signing key of jl2012 (Johnson Lau)
28d0f22 Fix calculation of number of bound sockets to use (Matt Corallo)
396c405 Include select.h when WIN32 is not defined (Ivo van der Sangen)
eebc699 bench: Fix subtle counting issue when rescaling iteration count (Wladimir J. van der Laan)
0c09d9f Send tip change notification from invalidateblock (Russell Yanofsky)
6f7841c qt: Avoid OpenSSL certstore-related memory leak (Wladimir J. van der Laan)
e5ad693 qt: Avoid shutdownwindow-related memory leak (Wladimir J. van der Laan)
e4bea4f qt: Avoid splash-screen related memory leak (Wladimir J. van der Laan)
c12f4e9 qt: Prevent thread/memory leak on exiting RPCConsole (Wladimir J. van der Laan)
dc46b10 qt: Plug many memory leaks (Wladimir J. van der Laan)
ff423cc [Qt] Clean up and fix coincontrol tree widget handling (Wladimir J. van der Laan)
6d70a73 [Qt] fix coincontrol sort issue (Jonas Schnelli)
3fffbf7 Doxygen: Set PROJECT_NAME = "Bitcoin Core" (MarcoFalke)
f82c81b fix getnettotals RPC description about timemillis. (Masahiko Hyuga)
6fe3981 net: don't send feefilter messages before the version handshake is complete (Cory Fields)
5f3a12c qt: Use correct conversion function for boost::path datadir (Wladimir J. van der Laan)
08d1c90 Missed one "return false" in recent refactoring in #9067 (UdjinM6)
f27596a Every main()/exit() should return/use one of EXIT_ codes instead of magic numbers (UdjinM6)
f85ee01 Fix exit codes: - `--help`, `--version` etc should exit with `0` i.e. no error ("not enough args" case should still trigger an error) - error reading config file should exit with `1` (UdjinM6)
5bcb05d [rpc] ParseHash: Fail when length is not 64 (MarcoFalke)
973ca1e Fix doxygen comment: the transaction is returned in txOut (Pavel Janík)
6f86b53 [Qt] make warnings label selectable (Jonas Schnelli)
106da69 Sync bitcoin-tx with tx version policy (BtcDrak)
12428b4 add software-properties-common (Steven)
40169dc Set minimum required Boost to 1.47.0 (fanquake)
c134d92 [build-aux] Boost_Base serial 27 (fanquake)
4a974b2 Simple Update to File 'bitcoin-qt.desktop' (matthias)
975ab12 Update INSTALL landing redirection notice for build instructions. (randy-waterhouse)
3a3bcbf Use RelevantServices instead of node_network in AttemptToEvict. (Gregory Maxwell)
ca1fd75 Make orphan parent fetching ask for witnesses. (Gregory Maxwell)
b96a8f7 [qa] Test getblocktemplate default_witness_commitment (Suhas Daftuar)
99477c7 Always add default_witness_commitment with GBT client support (Pieter Wuille)
da5a16b Always drop the least preferred HB peer when adding a new one. (Gregory Maxwell)
094848b log block size and weight correctly. (jnewbery)
d1b4da9 build: fix qt5.7 build under macOS (Cory Fields)
2016-12-14 09:32:29 +01:00
Gregory Maxwell
57aec3b0b5 Add release notes for wallet/mempool rejections. (PR #9302 and #9290) 2016-12-12 20:08:04 +00:00
Wladimir J. van der Laan
e591c1049f
Merge #9293: [0.13 Backport] IBD using chainwork instead of height and not using header timestamp (#9053)
5998a09 IsInitialBlockDownload no longer uses header-only timestamps. (Gregory Maxwell)
5b93eee Remove GetTotalBlocksEstimate and checkpoint tests that test nothing. (Gregory Maxwell)
ad20cdd IBD check uses minimumchain work instead of checkpoints. (Gregory Maxwell)
4c71fc4 Remove duplicate nBlocksEstimate cmp (we already checked IsIBD()) (Matt Corallo)
2016-12-08 13:11:48 +01:00
Gregory Maxwell
5998a09546 IsInitialBlockDownload no longer uses header-only timestamps.
This avoids a corner case (mostly visible on testnet) where bogus
 headers can keep nodes in IsInitialBlockDownload.

Github-Pull: #9053
Rebased-From: e141beb6a9816b7e1e680fb0a8bae16d42a3e557
2016-12-06 17:16:54 +00:00
Gregory Maxwell
5b93eeebb4 Remove GetTotalBlocksEstimate and checkpoint tests that test nothing.
GetTotalBlocksEstimate is no longer used and it was the only thing
 the checkpoint tests were testing.

Since checkpoints are on their way out it makes more sense to remove
 the test file than to cook up a new pointless test.

Github-Pull: #9053
Rebased-From: 2082b5574cec783f4ff99941492d92e05680b293
2016-12-06 17:16:13 +00:00
Gregory Maxwell
ad20cddce2 IBD check uses minimumchain work instead of checkpoints.
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.

Github-Pull: #9053
Rebased-From: fd46136dfaf68a7046cf7b8693824d73ac6b1caf
2016-12-06 17:14:19 +00:00
Matt Corallo
4c71fc42fb Remove duplicate nBlocksEstimate cmp (we already checked IsIBD())
Github-Pull: #8865
Rebased-From: 0278fb5f48ae9e42ec0772f85e201051077f633c
2016-12-06 17:09:50 +00:00
instagibbs
f26dab7e90 Adapt ZMQ/rest serialization to take rpcserialversion arg
Github-Pull: #9194
Rebased-From: ad5c4c93cae53a2a6f74880ca11b4d788677a378
2016-12-05 08:11:50 +01:00
Gregory Sanders
21ccb9f253 Add option to return non-segwit serialization via rpc
Github-Pull: #9194
Rebased-From: 835c75acaac004c3315395dcd7d1f193dfb9e5da
2016-12-05 08:11:03 +01:00
wodry
a710a43040 Improvement of documentation of command line parameter 'whitelist'
Github-Pull: #9251
Rebased-From: 8a70a9da3da0df5c376042b18b87f0a63e88d2e3
2016-12-03 12:03:29 +01:00
Wladimir J. van der Laan
0a4aa87623
Merge #9267: Disable fee estimates for a confirm target of 1 block
3688866 Disable fee estimates for a confirm target of 1 block (Alex Morcos)
2016-12-03 11:58:04 +01:00
Alex Morcos
3688866880 Disable fee estimates for a confirm target of 1 block
Backport of #9239 without GUI changes and fixing conflicts in tests.
2016-12-02 10:35:34 -05:00
MarcoFalke
58eab244fb [qa] rpc-tests: Apply random offset to portseed
This helps to skip over resources, which are blocked by regtest bitcoind
zombie nodes

Github-Pull: #9098
Rebased-From: fab0f07dec8d6e21ab70843fdce101f1703588fd
2016-12-02 07:58:23 +00:00
MarcoFalke
ecd7db5767 [qa] test_framework: Exit when tmpdir exists
Github-Pull: #9098
Rebased-From: fae19aa1da0858678874815b344de83e1ee3a1bd
2016-12-02 07:58:19 +00:00
Johnson Lau
387ec9d963 Add script tests for FindAndDelete in pre-segwit and segwit scripts
Github-Pull: #8927
Rebased-From: acf853df910339412bafb1743f42af1774f5b910
2016-12-02 07:56:12 +00:00
matthias
87fbcede5c Change all instance of 'GMT epoch' to 'Unix epoch'
Github-Pull: #9041
Rebased-From: 7f61b49de82989dc692e7750860eb1ec4044db7a
2016-12-02 07:55:58 +00:00
S. Matthew English
b1e978cfd6 instance of 'mem pool' to 'mempool'
there was only one instance of  'mem pool' and not 'mempool', so I changed it to conform to the others

Github-Pull: #9029
Rebased-From: 1c3ecc70c8cd6c33cf3ff4e2099c8e7d8a8ca9d2
2016-12-02 07:54:57 +00:00
Johnson Lau
ff55a2d22e Update gitian signing key of jl2012
Github-Pull: #8950
Rebased-From: 9b0d8efe811058928a6ed3a55c0b6d58e13b22ba
2016-12-02 07:54:15 +00:00
Matt Corallo
28d0f224fd Fix calculation of number of bound sockets to use
Github-Pull: #9253
Rebased-From: 9e1f46821d5bb69e2cbf25738eefa7c6cb99c838
2016-12-02 07:53:40 +00:00
Ivo van der Sangen
396c405e7b Include select.h when WIN32 is not defined
Github-Pull: #9224
Rebased-From: 498a1d75e75649b02caeca7b1bc9bbc19e51a566
2016-12-02 07:52:28 +00:00
Wladimir J. van der Laan
eebc699d30 bench: Fix subtle counting issue when rescaling iteration count
Make sure that the count is a zero modulo the new mask before
scaling, otherwise the next time until a measure triggers
will take only 1/2 as long as accounted for. This caused
the 'min time' to be potentially off by as much as 100%.

Github-Pull: #9200
Rebased-From: e0a9cb25b0af87723d50cb8d8cffa10f1ebf7dcc
2016-12-02 07:52:12 +00:00
Russell Yanofsky
0c09d9f00e Send tip change notification from invalidateblock
This change is needed to prevent sync_blocks timeouts in the mempool_reorg
test after the sync_blocks update in the upcoming commit
"[qa] Change sync_blocks to pick smarter maxheight".

This change was initially suggested by Suhas Daftuar <sdaftuar@chaincode.com>
in https://github.com/bitcoin/bitcoin/pull/8680#r78209060

Github-Pull: #9196
Rebased-From: 67c6326abd1788e6f411feb4f44b69774e76aae2
2016-12-02 07:50:37 +00:00
Wladimir J. van der Laan
6f7841c4d4 qt: Avoid OpenSSL certstore-related memory leak
- Correctly manage the X509 and X509_STORE objects lifetime.

Github-Pull: #9190
Rebased-From: ed998ea7a0ecf294211b06e9ef82f1548a621a1d
2016-12-02 07:50:11 +00:00
Wladimir J. van der Laan
e5ad693f91 qt: Avoid shutdownwindow-related memory leak
Store a reference to the shutdown window on BitcoinApplication,
so that it will be deleted when exiting the main loop.

Github-Pull: #9190
Rebased-From: 5204598f8d07d7432d91e9b8781806d2f3d16415
2016-12-02 07:50:11 +00:00
Wladimir J. van der Laan
e4bea4fb84 qt: Avoid splash-screen related memory leak
Make splash screen queue its own deletion when it receives the finished
command, instead of relying on WA_DeleteOnClose which doesn't work under
these circumstances.

Github-Pull: #9190
Rebased-From: e4f126a7ba66e7317718c30276dff6db92dc1986
2016-12-02 07:50:11 +00:00
Wladimir J. van der Laan
c12f4e93b9 qt: Prevent thread/memory leak on exiting RPCConsole
Make ownership of the QThread object clear, so that the RPCConsole
can wait for the executor thread to quit before shutdown is called. This
increases overall thread safety, and prevents some objects from leaking
on exit.

Github-Pull: #9190
Rebased-From: 693384eedb1ac7f449e226edd53e2cb52a86e279
2016-12-02 07:50:11 +00:00
Wladimir J. van der Laan
dc46b10a08 qt: Plug many memory leaks
None of these are very serious, and are leaks in objects that are
created at most one time.

In most cases this means properly using the QObject parent hierarchy,
except for BanTablePriv/PeerTablePriv which are not QObject,
so use a std::unique_ptr instead.

Github-Pull: #9190
Rebased-From: 47db07537746940ee7dd0739a8c73e328837813f
2016-12-02 07:44:15 +00:00
Wladimir J. van der Laan
ff423cc6b0 [Qt] Clean up and fix coincontrol tree widget handling
- Do sorting for date, amount and confirmations column as longlong, not
  unsigned longlong.
- Use `UserRole` to store our own data. This makes it treated as
  ancillary data prevents it from being displayed.
- Get rid of `getMappedColumn` `strPad` - these are no longer necessary.
- Get rid of hidden `_INT64` columns.
- Start enumeration from 0 (otherwise values are undefined).

Github-Pull: #9185
Rebased-From: 4231032bfcb9728f0f629b3d67884ba9de63e4ff
2016-12-02 07:41:11 +00:00
Jonas Schnelli
6d70a73968 [Qt] fix coincontrol sort issue
Github-Pull: #9185
Rebased-From: 76af4eb876814a916dd4f26cf71faa20bdc54f2d
2016-12-02 07:41:11 +00:00
MarcoFalke
3fffbf7ef0 Doxygen: Set PROJECT_NAME = "Bitcoin Core"
Github-Pull: #9178
Rebased-From: fa63ee8e3eeef8379360c125eeb690a37f097da2
2016-12-02 07:41:11 +00:00
Masahiko Hyuga
f82c81b177 fix getnettotals RPC description about timemillis.
Github-Pull: #9122
Rebased-From: a79f864945d7abe05e777a942864c181311e5070
2016-12-02 07:41:11 +00:00
Cory Fields
6fe3981ac8 net: don't send feefilter messages before the version handshake is complete
Github-Pull: #9117
Rebased-From: 46625538d674a5aaf1bcfa7c8be8a49e5a23fa9e
2016-12-02 07:41:11 +00:00
Wladimir J. van der Laan
5f3a12c97c qt: Use correct conversion function for boost::path datadir
Fixes #9089.

Github-Pull: #9094
Rebased-From: e760b307f6210bee293796d606bcca87f86be200
2016-12-02 07:41:11 +00:00
UdjinM6
08d1c90113 Missed one "return false" in recent refactoring in #9067
Github-Pull: #9120
Rebased-From: 45d372f88900bda74835a20a44246746c2ac94e6
2016-12-02 07:41:11 +00:00
UdjinM6
f27596a7ec Every main()/exit() should return/use one of EXIT_ codes instead of magic numbers
Github-Pull: #9067
Rebased-From: 4441018d0860fce64ee74fa78da79bbb21114ca9
2016-12-02 07:41:11 +00:00
UdjinM6
f85ee01303 Fix exit codes:
- `--help`, `--version` etc should exit with `0` i.e. no error ("not enough args" case should still trigger an error)
- error reading config file should exit with `1`

Slightly refactor AppInitRPC/AppInitRawTx to return standard exit codes (EXIT_FAILURE/EXIT_SUCCESS) or CONTINUE_EXECUTION (-1)

Github-Pull: #9067
Rebased-From: bd0de1386e1c7f9b875d52290de0d561c8d56bc9
2016-12-02 07:41:11 +00:00
MarcoFalke
5bcb05d4e1 [rpc] ParseHash: Fail when length is not 64
Github-Pull: #9042
Rebased-From: fa326193ad739d1f93da456b3fa73af0bbf9fdd1
2016-12-02 07:41:11 +00:00
Pavel Janík
973ca1e491 Fix doxygen comment: the transaction is returned in txOut
Github-Pull: #8993
Rebased-From: 1d8e12ba48154b2c59213e92efb50958d430a007
2016-12-02 07:41:11 +00:00
Jonas Schnelli
6f86b53643 [Qt] make warnings label selectable
Github-Pull: #8972
Rebased-From: ef0c9ee2960fda58b2fe5b0c3b4c672231b30322
2016-12-02 07:41:11 +00:00
BtcDrak
106da691a5 Sync bitcoin-tx with tx version policy
Github-Pull: #8932
Rebased-From: b0aea8057921f0ed2288cf07048f652c7d6d9cf3
2016-12-02 07:41:11 +00:00
Steven
12428b4d8a add software-properties-common
This allows using `add-apt-repository`.

Github-Pull: #8929
Rebased-From: aa9d3c9a503952589b359624bbb0da11dd7a5e3f
2016-12-02 07:41:11 +00:00
fanquake
40169dcda9 Set minimum required Boost to 1.47.0
Github-Pull: #8920
Rebased-From: 6dd37237222f7102e223ece948150cb5c5087e3c
2016-12-02 07:41:11 +00:00
fanquake
c134d9206d [build-aux] Boost_Base serial 27
Github-Pull: #8920
Rebased-From: 282abd8358e254d976cad05d0eb48586db276bcd
2016-12-02 07:41:11 +00:00
matthias
4a974b2837 Simple Update to File 'bitcoin-qt.desktop'
Github-Pull: #8908
Rebased-From: 164196b7c8aecb72dd20c8681bb24b81b18aecaa
2016-12-02 07:41:11 +00:00
randy-waterhouse
975ab1254e Update INSTALL landing redirection notice for build instructions.
Github-Pull: #8896
Rebased-From: 2920be2a6994cfbffd93e72c6cf4c1ed19ac4339
2016-12-02 07:41:11 +00:00
Gregory Maxwell
3a3bcbf775 Use RelevantServices instead of node_network in AttemptToEvict.
Use of node_network here is really meant to be a proxy of "likely to
 send us blocks in the future".  RelevantServices is the right criteria
 now.

Github-Pull: #9052
Rebased-From: d32036a47d9ccdc38628a7a75bb8b711af462e4f
2016-12-02 07:41:11 +00:00
Gregory Maxwell
ca1fd7521e Make orphan parent fetching ask for witnesses.
In 0.13 orphan transactions began being treated as implicit
 INVs for their parents.  But the resulting getdata were
 not getting the witness flag.

This fixes issue #9182 reported by chjj and roasbeef on IRC.

Github-Pull: #9188
Rebased-From: 5b0150a060208faf436c09b0ca9463407a869c72
2016-12-02 08:38:02 +01:00
Suhas Daftuar
b96a8f7df2 [qa] Test getblocktemplate default_witness_commitment
Github-Pull: #9189
Rebased-From: 95f4a03777ec2ad82a94a3e2890192a93ad83509
2016-12-02 08:38:02 +01:00