aboutsummaryrefslogtreecommitdiff
path: root/src
Commit message (Collapse)AuthorAgeFilesLines
* Fix transaction size comments. Size now refers to virtual size as defined in ↵jonnynewbs2016-12-083-8/+10
| | | | BIP141.
* Introduce convenience type CTransactionRefPieter Wuille2016-11-1916-57/+63
|
* Make CBlock::vtx a vector of shared_ptr<CTransaction>Pieter Wuille2016-11-1924-149/+161
|
* Add deserializing constructors to CTransaction and CMutableTransactionPieter Wuille2016-11-192-0/+13
|
* Add serialization for unique_ptr and shared_ptrPieter Wuille2016-11-191-0/+59
|
* Merge #9186: test: Fix use-after-free in scheduler testsWladimir J. van der Laan2016-11-191-3/+4
|\ | | | | | | 12519bf test: Fix use-after-free in scheduler tests (Wladimir J. van der Laan)
| * test: Fix use-after-free in scheduler testsWladimir J. van der Laan2016-11-181-3/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Make a copy of the boost time-point to wait for, otherwise the head of the queue may be deleted by another thread while this one is waiting, while the boost function still has a reference to it. Although this problem is in non-test code, this is not an actual problem outside of the tests because we use the thread scheduler with only one service thread, so there will never be threads fighting at the head of the queue. The old boost fallback escapes this problem because it passes a scalar value to wait_until instead of a const object reference. Found by running the tests in LLVM-4.0-master asan.
* | Merge #9075: Decouple peer-processing-logic from block-connection-logic (#3)Pieter Wuille2016-11-175-53/+44
|\ \ | |/ |/| | | | | | | ae22357 Replace CValidationState param in ProcessNewBlock with BlockChecked (Matt Corallo) 7c98ce5 Remove pfrom parameter from ProcessNewBlock (Matt Corallo) e2e069d Revert "RPC: Give more details when "generate" fails" (Matt Corallo)
| * Replace CValidationState param in ProcessNewBlock with BlockCheckedMatt Corallo2016-11-095-48/+28
| |
| * Remove pfrom parameter from ProcessNewBlockMatt Corallo2016-11-095-18/+29
| | | | | | | | This further decouples ProcessNewBlock from networking/peer logic.
| * Revert "RPC: Give more details when "generate" fails"Matt Corallo2016-11-081-1/+1
| | | | | | | | | | This only returned information in the case of CheckBlock failure, but breaks future changes.
* | Merge #8838: Calculate size and weight of block correctly in CreateNewBlock()Wladimir J. van der Laan2016-11-171-1/+3
|\ \ | | | | | | | | | 5f274a1 log block size and weight correctly. (jnewbery)
| * | log block size and weight correctly.jnewbery2016-10-031-1/+3
| | |
* | | Merge #9155: [trivial] update comments for tx weightMarcoFalke2016-11-162-2/+2
|\ \ \ | | | | | | | | | | | | 07ede5d update comments for tx weight (Brian Deery)
| * | | update comments for tx weightBrian Deery2016-11-142-2/+2
| | | |
* | | | Merge #9133: Unset fImporting for loading mempoolWladimir J. van der Laan2016-11-161-1/+3
|\ \ \ \ | | | | | | | | | | | | | | | 79f755d Unset fImporting for loading mempool (Alex Morcos)
| * | | | Unset fImporting for loading mempoolAlex Morcos2016-11-111-1/+3
| | | | |
* | | | | Merge #9131: fNetworkActive is not protected by a lock, use an atomicWladimir J. van der Laan2016-11-161-1/+1
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | 079142b fNetworkActive is not protected by a lock, use an atomic (Jonas Schnelli)
| * | | | | fNetworkActive is not protected by a lock, use an atomicJonas Schnelli2016-11-111-1/+1
| | | | | |
* | | | | | Merge #9164: [trivial] credit values are CAmountPieter Wuille2016-11-151-1/+1
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | 20c3215 credit values are CAmount (Gregory Sanders)
| * | | | | | credit values are CAmountGregory Sanders2016-11-151-1/+1
| | | | | | |
* | | | | | | Merge #9140: Bugfix: Correctly replace generated headers and fail cleanlyWladimir J. van der Laan2016-11-152-15/+11
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | b74ff5c Bugfix: Correctly replace generated headers and fail cleanly (Luke Dashjr)
| * | | | | | | Bugfix: Correctly replace generated headers and fail cleanlyLuke Dashjr2016-11-122-15/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Also removes generation of headers for *.raw files in test_bitcoin (none exist anymore)
* | | | | | | | Merge #9132: Make strWalletFile constWladimir J. van der Laan2016-11-151-4/+2
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | f734505 Make strWalletFile const (Jonas Schnelli)
| * | | | | | | | Make strWalletFile constJonas Schnelli2016-11-111-4/+2
| | |_|/ / / / / | |/| | | | | |
* | | | | | | | Merge #9160: [trivial] Fix hungarian variable nameWladimir J. van der Laan2016-11-151-4/+4
|\ \ \ \ \ \ \ \ | |_|_|/ / / / / |/| | | | | | | | | | | | | | | ec34648 [trivial] Fix hungarian variable name (Russell Yanofsky)
| * | | | | | | [trivial] Fix hungarian variable nameRussell Yanofsky2016-11-141-4/+4
| | |_|_|/ / / | |/| | | | | | | | | | | | | | | | | | | Follow up to comment https://github.com/bitcoin/bitcoin/pull/9058#discussion_r87676593
* / | | | | | [qt] Make network disabled icon 50% opaqueMarcoFalke2016-11-137-34/+15
|/ / / / / /
* | | | | | Merge #9124: Use better name for local variable to prevent -Wshadow compiler ↵Wladimir J. van der Laan2016-11-111-2/+2
|\ \ \ \ \ \ | |/ / / / / |/| | | | | | | | | | | | | | | | | | | | | | | warning bf49f10 Use better name for local variable to prevent -Wshadow compiler warning (Pavel Janík)
| * | | | | Use better name for local variable to prevent -Wshadow compiler warningPavel Janík2016-11-101-2/+2
| | | | | |
* | | | | | Merge #8996: Network activity toggleJonas Schnelli2016-11-1116-13/+282
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 19f46f1 Qt: New network_disabled icon (Luke Dashjr) 54cf997 RPC/Net: Use boolean consistently for networkactive, and remove from getinfo (Luke Dashjr) b2b33d9 Overhaul network activity toggle (Jonas Schnelli) 32efa79 Qt: Add GUI feedback and control of network activity state. (Jon Lund Steffensen) e38993b RPC: Add "togglenetwork" method to toggle network activity temporarily (Jon Lund Steffensen) 7c9a98a Allow network activity to be temporarily suspended. (Jon Lund Steffensen)
| * | | | | | Qt: New network_disabled iconLuke Dashjr2016-10-242-0/+68
| | | | | | |
| * | | | | | RPC/Net: Use boolean consistently for networkactive, and remove from getinfoLuke Dashjr2016-10-243-13/+11
| | | | | | |
| * | | | | | Overhaul network activity toggleJonas Schnelli2016-10-248-14/+74
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Rename RPC command "togglenetwork" to "setnetworkactive (true|false)" - Add simple test case - GUI toggle added to connections icon in statusbar
| * | | | | | Qt: Add GUI feedback and control of network activity state.Jon Lund Steffensen2016-10-248-7/+97
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add getNetworkActive()/setNetworkActive() method to client model. Send network active status through NotifyNetworkActiveChanged. Indicate in tool tip of gui status bar network indicator whether network activity is disabled. Indicate in debug window whether network activity is disabled and add button to allow user to toggle network activity state.
| * | | | | | RPC: Add "togglenetwork" method to toggle network activity temporarilyJon Lund Steffensen2016-10-243-1/+23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | RPC command "togglenetwork" toggles network and returns new state after command. RPC command "getinfo" returns "networkactive" field in output.
| * | | | | | Allow network activity to be temporarily suspended.Jon Lund Steffensen2016-10-242-0/+31
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Added the function SetNetworkActive() which when called with argument set to false disconnects all nodes and sets the flag fNetworkActive to false. As long as this flag is false no new connections are attempted and no incoming connections are accepted. Network activity is reenabled by calling the function with argument true.
* | | | | | | Merge #9058: Fixes for p2p-compactblocks.py test timeouts on travis (#8842)Wladimir J. van der Laan2016-11-111-0/+12
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | dac53b5 Modify getblocktxn handler not to drop requests for old blocks (Russell Yanofsky) 55bfddc [qa] Fix stale data bug in test_compactblocks_not_at_tip (Russell Yanofsky) 47e9659 [qa] Fix bug in compactblocks v2 merge (Russell Yanofsky)
| * | | | | | | Modify getblocktxn handler not to drop requests for old blocksRussell Yanofsky2016-11-071-0/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The current getblocktxn implementation drops and ignores requests for old blocks, which causes occasional sync_block timeouts during the p2p-compactblocks.py test as reported in https://github.com/bitcoin/bitcoin/issues/8842. The p2p-compactblocks.py test setup creates many new blocks in a short period of time, which can lead to getblocktxn requests for blocks below the hardcoded depth limit of 10 blocks. This commit changes the getblocktxn handler not to ignore these requests, so the peer nodes in the test setup will reliably be able to sync. The protocol change is documented in BIP-152 update "Allow block responses to getblocktxn requests" at https://github.com/bitcoin/bips/pull/469. The protocol change is not expected to affect nodes running outside the test environment, because there shouldn't normally be lots of new blocks being rapidly added that need to be synced.
* | | | | | | | Merge #9112: Avoid ugly exception in log on unknown inv typeWladimir J. van der Laan2016-11-111-1/+5
|\ \ \ \ \ \ \ \ | |_|_|/ / / / / |/| | | | | | | | | | | | | | | e9f25dd Avoid ugly exception in log on unknown inv type (Wladimir J. van der Laan)
| * | | | | | | Avoid ugly exception in log on unknown inv typeWladimir J. van der Laan2016-11-091-1/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It is unexpected behavior for `ToString` to raise an exception. It is expected to do a best-effort attempt at formatting but never fail. Catch the exception and simply print unknown inv types as hexadecimal. Fixes #9110.
* | | | | | | | Merge #9049: Remove duplicatable duplicate-input check from CheckTransactionWladimir J. van der Laan2016-11-105-11/+82
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | e2b3fb3 Optimize vInOutPoints insertion a bit (Matt Corallo) eecffe5 Remove redundant duplicate-input check from CheckTransaction (Matt Corallo) b2e178a Add deserialize + CheckBlock benchmarks, and a full block hex (Matt Corallo)
| * | | | | | | | Optimize vInOutPoints insertion a bitMatt Corallo2016-11-091-2/+1
| | | | | | | | |
| * | | | | | | | Remove redundant duplicate-input check from CheckTransactionMatt Corallo2016-11-092-10/+12
| | |_|_|/ / / / | |/| | | | | |
| * | | | | | | Add deserialize + CheckBlock benchmarks, and a full block hexMatt Corallo2016-11-093-1/+71
| | |_|_|_|_|/ | |/| | | | |
* | | | | | | Merge #9122: fix getnettotals RPC description about timemillis.MarcoFalke2016-11-101-1/+1
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | a79f864 fix getnettotals RPC description about timemillis. (Masahiko Hyuga)
| * | | | | | | fix getnettotals RPC description about timemillis.Masahiko Hyuga2016-11-101-1/+1
| | | | | | | |
* | | | | | | | Merge #9121: Initialize variable to prevent compiler warningWladimir J. van der Laan2016-11-101-1/+1
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | bdcba6d Initialize variable to prevent compiler warning (Pavel Janík)
| * | | | | | | | Initialize variable to prevent compiler warningPavel Janík2016-11-101-1/+1
| |/ / / / / / /
* | | | | | | | Merge #9120: bug: Missed one "return false" in recent refactoring in #9067Wladimir J. van der Laan2016-11-101-1/+1
|\ \ \ \ \ \ \ \ | |/ / / / / / / |/| | | | | | | | | | | | | | | 45d372f Missed one "return false" in recent refactoring in #9067 (UdjinM6)