aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Make nTimeBestReceived atomicMatt Corallo2017-02-101-1/+1
|
* Move [clean|str]SubVer writes/copyStats into a lockMatt Corallo2017-02-103-4/+13
|
* Make nServices atomicMatt Corallo2017-02-101-1/+1
|
* Make nStartingHeight atomicMatt Corallo2017-02-101-1/+1
|
* Access fRelayTxes with cs_filter lock in copyStatsMatt Corallo2017-02-101-1/+4
|
* Avoid copying CNodeStats to make helgrind OK with buggy std::stringMatt Corallo2017-02-101-3/+2
|
* Make nTimeConnected const in CNodeMatt Corallo2017-02-102-2/+2
|
* net: fix a few races. Credit @TheBlueMattCory Fields2017-02-103-15/+22
| | | | | | | | | | | These are (afaik) all long-standing races or concurrent accesses. Going forward, we can clean these up so that they're not all individual atomic accesses. - Reintroduce cs_vRecv to guard receive-specific vars - Lock vRecv/vSend for CNodeStats - Make some vars atomic. - Only set the connection time in CNode's constructor so that it doesn't change
* Merge #9698: net: fix socket close raceWladimir J. van der Laan2017-02-102-26/+49
|\ | | | | | | | | 9a0b784 net: add a lock around hSocket (Cory Fields) 45e2e08 net: rearrange so that socket accesses can be grouped together (Cory Fields)
| * net: add a lock around hSocketCory Fields2017-02-062-15/+29
| |
| * net: rearrange so that socket accesses can be grouped togetherCory Fields2017-02-061-14/+23
| |
* | Merge #9730: Remove bitseed.xf2.org form the dns seed listJonas Schnelli2017-02-101-1/+0
|\ \ | | | | | | | | | ca9955e Remove bitseed.xf2.org form the dns seed list (Jonas Schnelli)
| * | Remove bitseed.xf2.org form the dns seed listJonas Schnelli2017-02-091-1/+0
|/ /
* | Merge #9718: Qt/Intro: Various fixesWladimir J. van der Laan2017-02-091-3/+8
|\ \ | | | | | | | | | | | | | | | a9baa6d Bugfix: Qt/Intro: Pruned nodes never require *more* space (Luke Dashjr) 93ffba7 Bugfix: Qt/Intro: Chain state needs to be stored even with the full blockchain (Luke Dashjr) c8cee26 Qt/Intro: Update block chain size (Luke Dashjr)
| * | Bugfix: Qt/Intro: Pruned nodes never require *more* spaceLuke Dashjr2017-02-081-1/+4
| | |
| * | Bugfix: Qt/Intro: Chain state needs to be stored even with the full blockchainLuke Dashjr2017-02-081-2/+4
| | |
| * | Qt/Intro: Update block chain sizeLuke Dashjr2017-02-081-1/+1
| | |
* | | Merge #9674: Always enforce strict lock ordering (try or not)Wladimir J. van der Laan2017-02-082-48/+20
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | 618ee92 Further-enforce lockordering by enforcing directly after TRY_LOCKs (Matt Corallo) 2a962d4 Fixup style a bit by moving { to the same line as if statements (Matt Corallo) 8465631 Always enforce lock strict lock ordering (try or not) (Matt Corallo) fd13eca Lock cs_vSend and cs_inventory in a consistent order even in TRY (Matt Corallo)
| * | | Further-enforce lockordering by enforcing directly after TRY_LOCKsMatt Corallo2017-02-071-15/+13
| | | |
| * | | Fixup style a bit by moving { to the same line as if statementsMatt Corallo2017-02-041-6/+3
| | | |
| * | | Always enforce lock strict lock ordering (try or not)Matt Corallo2017-02-021-25/+1
| | | |
| * | | Lock cs_vSend and cs_inventory in a consistent order even in TRYMatt Corallo2017-02-021-5/+6
| | | |
* | | | Merge #9705: build: Add options to override BDB cflags/libsWladimir J. van der Laan2017-02-083-60/+70
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | 5cc2ebb Update OpenBSD and FreeBSD build steps (Wladimir J. van der Laan) 8713de8 build: Add options to override BDB cflags/libs (Wladimir J. van der Laan)
| * | | | Update OpenBSD and FreeBSD build stepsWladimir J. van der Laan2017-02-072-4/+6
| | | | | | | | | | | | | | | | | | | | | | | | | Re-try with most recent versions, and use BDB_CFLAGS/BDB_LIBS to directly point at BerkeleyDB instead of CPPFLAGS hacks.
| * | | | build: Add options to override BDB cflags/libsWladimir J. van der Laan2017-02-071-56/+64
| | |_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add environment settings to specify the CFLAGS and LIBS to be used for BerkeleyDB directly. These will completely by-pass autodetection in the same way as other similar flags. ``` BDB_CFLAGS C compiler flags for BerkeleyDB, bypasses autodetection BDB_LIBS Linker flags for BerkeleyDB, bypasses autodetection ``` Implements #3921.
* | | | Merge #9712: bench: Fix initialization order in registrationMarcoFalke2017-02-082-9/+10
|\ \ \ \ | | | | | | | | | | | | | | | 29c5328 bench: Fix initialization order in registration (Wladimir J. van der Laan)
| * | | | bench: Fix initialization order in registrationWladimir J. van der Laan2017-02-072-9/+10
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The initialization order of global data structures in different implementation units is undefined. Making use of this is essentially gambling on what the linker does, the so-called [Static initialization order fiasco](https://isocpp.org/wiki/faq/ctors#static-init-order). In this case it apparently worked on Linux but failed on OpenBSD and FreeBSD. To create it on first use, make the registration structure local to a function. Fixes #8910.
* | | | Merge #9532: Remove unused variablesWladimir J. van der Laan2017-02-078-11/+2
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | 90fd29b Remove unused int64_t nSinceLastSeen (practicalswift) ac4a095 Remove unused Python variables (practicalswift)
| * | | | Remove unused int64_t nSinceLastSeenpracticalswift2017-01-131-3/+0
| | | | |
| * | | | Remove unused Python variablespracticalswift2017-01-137-8/+2
| | | | |
* | | | | Merge #9604: [Trivial] add comment about setting peer as HB peer.Wladimir J. van der Laan2017-02-071-1/+9
|\ \ \ \ \ | |_|/ / / |/| | | | | | | | | dd5b011 [Trivial] add comment about setting peer as HB peer. (John Newbery)
| * | | | [Trivial] add comment about setting peer as HB peer.John Newbery2017-01-201-1/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This adds a comment to the new logic for setting HB peers based on block validation (and aligns the code below to reflect the comment). It's not obvious why we're checking mapBlocksInFlight. Add a comment to explain.
* | | | | Merge #9227: Make nWalletDBUpdated atomic to avoid a potential race.Wladimir J. van der Laan2017-02-065-33/+43
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | d63ff62 Make nWalletDBUpdated atomic to avoid a potential race. (Patrick Strateman)
| * | | | | Make nWalletDBUpdated atomic to avoid a potential race.Patrick Strateman2016-12-215-33/+43
| | | | | |
* | | | | | Merge #9378: [trivial] Add documentation for CWalletTx::fFromMe member.Wladimir J. van der Laan2017-02-061-0/+5
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | 39c77b0 Add documentation for CWalletTx::fFromMe member. (Russell Yanofsky)
| * | | | | | Add documentation for CWalletTx::fFromMe member.Russell Yanofsky2016-12-151-0/+5
| | | | | | |
* | | | | | | Merge #9650: Better handle invalid parameters to signrawtransactionWladimir J. van der Laan2017-02-063-4/+36
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6dbfe08 [qa] test signrawtransaction merge with missing inputs (Matt Corallo) ec4f7e4 [qa] Add second input to signrawtransaction test case (Matt Corallo) 691710a [qa] Test that decoderawtransaction throws with extra data appended (Matt Corallo) 922bea9 Better handle invalid parameters to signrawtransaction (Matt Corallo) 7ea0ad5 Fail in DecodeHexTx if there is extra data at the end (Matt Corallo)
| * | | | | | | [qa] test signrawtransaction merge with missing inputsMatt Corallo2017-02-021-0/+16
| | | | | | | |
| * | | | | | | [qa] Add second input to signrawtransaction test caseMatt Corallo2017-02-021-3/+5
| | | | | | | |
| * | | | | | | [qa] Test that decoderawtransaction throws with extra data appendedMatt Corallo2017-02-021-0/+10
| | | | | | | |
| * | | | | | | Better handle invalid parameters to signrawtransactionMatt Corallo2017-02-021-1/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This silently skips trying to merge signatures from inputs which do not exist from transactions provided to signrawtransaction, instead of hitting an assert.
| * | | | | | | Fail in DecodeHexTx if there is extra data at the endMatt Corallo2017-02-021-0/+2
| | |_|_|_|/ / | |/| | | | |
* | | | | | | Merge #9659: Net: Turn some methods and params/variables constWladimir J. van der Laan2017-02-065-19/+19
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 0729102 Net: pass interruptMsgProc as const where possible (Jorge Timón) fc7f2ff Net: Make CNetMsgMaker more const (Jorge Timón) d45955f Net: CConnman: Make some methods const (Jorge Timón)
| * | | | | | | Net: pass interruptMsgProc as const where possibleJorge Timón2017-01-312-6/+6
| | | | | | | |
| * | | | | | | Net: Make CNetMsgMaker more constJorge Timón2017-01-312-7/+7
| | | | | | | |
| * | | | | | | Net: CConnman: Make some methods constJorge Timón2017-01-312-6/+6
| | | | | | | |
* | | | | | | | Merge #9651: Fix typosWladimir J. van der Laan2017-02-067-9/+9
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | 5f62e3e Fix typos (practicalswift)
| * | | | | | | | Fix typospracticalswift2017-01-297-9/+9
| | | | | | | | |
* | | | | | | | | Merge #9673: Set correct metadata on bumpfee wallet transactionsWladimir J. van der Laan2017-02-062-0/+14
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 4ec057d [wallet] Set correct metadata on bumpfee wallet transactions (Russell Yanofsky)
| * | | | | | | | | [wallet] Set correct metadata on bumpfee wallet transactionsRussell Yanofsky2017-02-022-0/+14
| | |_|/ / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Preserve comment, order form, and account strings from the original wallet transaction. Also set fTimeReceivedIsTxTime and fFromMe fields for consistency with CWallet::CreateTransaction. The latter two fields don't influence current wallet behavior, but do record that the transaction originated in the wallet instead of coming from the network or sendrawtransaction.