aboutsummaryrefslogtreecommitdiff
path: root/src
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | | | | | | | | | | | RPC, cosmetic: push down ReadHTTPStatus() calls into ReadHTTP() callersJeff Garzik2012-11-041-11/+19
| | |_|/ / / / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ReadHTTPStatus() is currently overloaded: In client mode, it properly parses and receives an HTTP status line. In server mode, it incorrectly parses the HTTP request line as an HTTP status line. This server mode bug has never mattered, because the RPC server never cared about the URI (path) provided in the HTTP request. That will change in the future, so go ahead and begin fixing the problem. This patch is cosmetic, and should result in NO behavior changes. Further renames: ReadHTTPHeader -> ReadHTTPHeaders ReadHTTP -> ReadHTTPMessage
* | | | | | | | | | | | | | Merge pull request #1978 from sipa/nodetachPieter Wuille2012-11-099-52/+6
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ | |_|/ / / / / / / / / / / / |/| | | | | | | | | | | | | Remove -detachdb and stop's detach argument.
| * | | | | | | | | | | | | Remove -detachdb and stop's detach argument.Pieter Wuille2012-11-049-52/+6
| |/ / / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | As the only BDB database left is the wallet, and it is always detached. Also remove IsChainFile() predicate and related chainfile-specific logic.
* | | | | | | | | | | | | Merge pull request #1981 from sipa/cachesPieter Wuille2012-11-087-14/+27
|\ \ \ \ \ \ \ \ \ \ \ \ \ | |_|/ / / / / / / / / / / |/| | | | | | | | | | | | Cache size optimizations
| * | | | | | | | | | | | Cache size optimizationsPieter Wuille2012-11-047-14/+27
| |/ / / / / / / / / / /
* | | | | | | | | | | | Merge pull request #1830 from Diapolo/trans_rem_spacesWladimir J. van der Laan2012-11-046-30/+18
|\ \ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / |/| | | | | | | | | | | fix some double-spaces in strings
| * | | | | | | | | | | fix some double-spaces in stringsPhilip Kaufmann2012-10-256-30/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - remove some unneeded stuff in sendcoinsentry.ui - harmonize some "Error:"-messages
* | | | | | | | | | | | Merge pull request #1971 from sipa/bugfix_norelayspentPieter Wuille2012-11-031-9/+5
|\ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | Bugfix: do not keep relaying spent wallet transactions
| * | | | | | | | | | | | Bugfix: do not keep relaying spent wallet transactionsPieter Wuille2012-11-011-9/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The original test (checking whether the transaction occurs in the txindex) is not usable anymore, as it will miss anything already fully spent. However, as merkle transactions (and by extension, wallet transactions) track which block they were last seen being included in, we can use that to determine the need for rebroadcasting.
* | | | | | | | | | | | | Comments for constantsPieter Wuille2012-11-031-1/+14
| | | | | | | | | | | | |
* | | | | | | | | | | | | Add virtual destructor to CCoinsViewPieter Wuille2012-11-011-0/+3
| |_|/ / / / / / / / / / |/| | | | | | | | | | |
* | | | | | | | | | | | Bitcoin-Qt: use statustips in addition to tooltipsPhilip Kaufmann2012-10-312-16/+45
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - add setStatusTip() in addition to setTooltip() where it makes sense - add only setStatusTip() if GUI element is only used in main- or tray menu - add an event filter on our BitcoinGUI object to prevent garbelled text on the status bar, which happens when we use it for e.g. displaying block-sync state and then a QEvent::StatusTip wants to write own text to it - remove a double translation of "Bitcoin client"
* | | | | | | | | | | | Merge pull request #1932 from Diapolo/thread_printfWladimir J. van der Laan2012-10-302-1/+6
|\ \ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / |/| | | | | | | | | | | fix some thread related log messages
| * | | | | | | | | | | fix some thread related log messagesPhilip Kaufmann2012-10-252-1/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - "ThreadIRCSeed started" was not displayed, even if the thread ran (although only for a short time as the "do we want this thread?"-checks happen IN ThreadIRCSeed2()) - the patch ensures we always get that message - add a "ThreadIRCSeed trying to connect..." message - add missing "ThreadDumpAddress started" message
* | | | | | | | | | | | Merge pull request #1963 from sipa/bugfix_nonewblockPieter Wuille2012-10-301-1/+2
|\ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | Bugfix: don't crash by trying to write unchanged best block
| * | | | | | | | | | | | Bugfix: don't crash by trying to write unchanged best blockPieter Wuille2012-10-281-1/+2
| | |/ / / / / / / / / / | |/| | | | | | | | | |
* | | | | | | | | | | | Merge branch 'rawtx_p2sh'Gavin Andresen2012-10-295-75/+232
|\ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | New createmultisig rpc commandGavin Andresen2012-10-293-15/+52
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is to support the signrawtransaction API call; given the public keys involved in a multisig transaction, this gives back the redeemScript needed to sign it.
| * | | | | | | | | | | | Add redeemScript to listunspent output and signrawtransaction inputGavin Andresen2012-10-292-23/+66
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | signrawtransaction was unable to sign pay-to-script-hash inputs when given the list of private keys to use. With this commit you can provide the p2sh redemption script in the list of inputs.
| * | | | | | | | | | | | Tests for raw transactions argument checkingGavin Andresen2012-10-292-29/+113
| | | | | | | | | | | | |
| * | | | | | | | | | | | No need for test fixture now that multisig is enabled on main network.Gavin Andresen2012-10-291-8/+1
| | | | | | | | | | | | |
* | | | | | | | | | | | | Add missing calls to ranlib in Windows makefilesMatt Corallo2012-10-292-1/+2
| | | | | | | | | | | | |
* | | | | | | | | | | | | Fix linux-mingw makefile's leveldb build by setting CXX.Matt Corallo2012-10-291-1/+1
| | | | | | | | | | | | |
* | | | | | | | | | | | | Use && instead of ; in leveldb calls to makefile.Matt Corallo2012-10-294-4/+4
|/ / / / / / / / / / / /
* / / / / / / / / / / / Remove P2SH transition code: P2SH violations may cause DoS triggerPieter Wuille2012-10-291-7/+1
|/ / / / / / / / / / /
* | | | | | | | | | | First flush block tree, then coin setPieter Wuille2012-10-251-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | As the coinset data refers to the best block, stored in the block tree. Flushing the coin set first can cause inconsistencies if the process gets killed in between.
* | | | | | | | | | | Added checks for null pointers in Shutdowntucenaber2012-10-251-2/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Estetics
* | | | | | | | | | | Merge pull request #1904 from laanwj/2012_10_remove_getorderPieter Wuille2012-10-254-125/+3
|\ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | remove "checkorder" P2P command
| * | | | | | | | | | | remove "checkorder" and "reply" P2P commandsWladimir J. van der Laan2012-10-034-125/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | These command are a leftover from send-to-IP transactions, which have been removed a long time ago. Also removes CNode::mapRequests and CNode::PushRequests, as these were only used for the mentioned commands.
* | | | | | | | | | | | Merge pull request #1899 from Diapolo/proxy_optionsmodelPieter Wuille2012-10-251-5/+12
|\ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | make optionsmodel query real proxy state for ::data()
| * | | | | | | | | | | | make optionsmodel query real proxy state for ::data()Philip Kaufmann2012-10-071-5/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - don't rely on the QSettings for cases ProxyUse and ProxySocksVersion and query the real values via the GetProxy() call - add a missing "succesful =" for case ProxyUse in ::setData()
* | | | | | | | | | | | | Merge pull request #1953 from gmaxwell/createnewblock-racePieter Wuille2012-10-251-2/+3
|\ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|/ / / / / / / / / / |/| | | | | | | | | | | | Fixes a race condition in CreateNewBlock and a future null deref on testnet.
| * | | | | | | | | | | | Fixes a race condition in CreateNewBlock and a future null deref on testnet.Gregory Maxwell2012-10-241-2/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | CreateNewBlock was reading pindexBest at the start before taking the lock so it was possible to have the the block content not match the prevheader and this can also trigger a newly added assert in ConnectBlock. I noticed this during a code review after twobitcoins reported that ab91bf39 (BIP30 for all blocks) could cause a null dereference on a modified node that mined during the IBD, or on testnet when it reached heights 91842 and 91880 due to CreateNewBlock calling ConnectBlock with pindex->phashBlock NULL.
* | | | | | | | | | | | | Merge pull request #1926 from laanwj/2012_10_maindummyboundsWladimir J. van der Laan2012-10-251-4/+1
|\ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix out-of-bounds read in main (issue #1924)
| * | | | | | | | | | | | | Fix out-of-bounds read noticed by Ricardo CorreiaWladimir J. van der Laan2012-10-241-4/+1
| |/ / / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Sizeof() returned the size of a pointer instead of the size of the buffer. Fixes issue #1924.
* | | | | | | | | | | | | Merge pull request #1956 from laanwj/2012_10_prerelease_warningWladimir J. van der Laan2012-10-258-288/+332
|\ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | Show warning when using prerelease version
| * | | | | | | | | | | | | Show warning when using prerelease versionWladimir J. van der Laan2012-10-258-288/+332
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Implements #1948 - Add macro `CLIENT_VERSION_IS_RELEASE` to clientversion.h - When running a prerelease (the above macro is `false`): - In UI, show an orange warning bar at the top. This will be used for other warnings (and alerts) as well, instead of the status bar. - For `bitcoind`, show the warning in the "errors" field in `getinfo` response.
* | | | | | | | | | | | | | Merge pull request #1955 from gmaxwell/gbt_paramfixPieter Wuille2012-10-251-1/+1
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Don't force getblocktemplate to have a parameter.
| * | | | | | | | | | | | | | Don't force getblocktemplate to have a parameter.Gregory Maxwell2012-10-241-1/+1
| |/ / / / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This looks like it was just a munged merge when ultraprune was committed.
* / / / / / / / / / / / / / ensure AskPassphraseDialog::eventFilter forwards eventsPhilip Kaufmann2012-10-252-8/+8
|/ / / / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - instead of "return false;" use "return QDialog::eventFilter(object, event);" to harmonize this event filter with our default behaviour - remove orphan spaces found while editting the files
* | | | | | | | | | | | | Merge pull request #1947 from centromere/freebsd_cpu_fixPieter Wuille2012-10-241-1/+1
|\ \ \ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / / |/| | | | | | | | | | | | Fixed 100% CPU utilization problem on FreeBSD 9
| * | | | | | | | | | | | Fixed 100% CPU utilization problem on FreeBSD 9Alex2012-10-221-1/+1
| | |_|/ / / / / / / / / | |/| | | | | | | | | |
* | | | | | | | | | | | Bugfix: actually use CCoinsViewMemPoolPieter Wuille2012-10-231-2/+14
| | | | | | | | | | | |
* | | | | | | | | | | | Added some commentsPieter Wuille2012-10-232-2/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Some clarifications after a code review by Mike Hearn.
* | | | | | | | | | | | Bugfix: off-by-one in priority calculationPieter Wuille2012-10-231-1/+1
| | | | | | | | | | | |
* | | | | | | | | | | | Bugfix: add missing fee checkPieter Wuille2012-10-231-0/+3
| | | | | | | | | | | |
* | | | | | | | | | | | Bugfix: off-by-one error in coinbase maturity checkPieter Wuille2012-10-231-3/+5
|/ / / / / / / / / / /
* | | | | | | | | | | Merge pull request #1939 from sipa/bugfix_zerobalanceGregory Maxwell2012-10-211-2/+2
|\ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Bugfix: do not mark all future coins spent
| * | | | | | | | | | | Bugfix: do not mark all future coins spentPieter Wuille2012-10-211-2/+2
| | | | | | | | | | | |
* | | | | | | | | | | | change blockchain -> block chain (spelling)Philip Kaufmann2012-10-216-14/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Wiki says "block chain" is correct ;) - remove some unneeded spaces I found in the source, while fixing the spelling