aboutsummaryrefslogtreecommitdiff
path: root/src
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | | | | | | | | | | Merge pull request #3000 from gmaxwell/more_osx_fsyncJeff Garzik2013-09-161-0/+2
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | More fixes for blockchain corruption on OSX.
| * | | | | | | | | | | | | | More fixes for blockchain corruption on OSX.Gregory Maxwell2013-09-151-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | As we'd previously learned, OSX's fsync is a data eating lie. Since 0.8.4 we're still getting some reports of disk corruption on OSX but now all of it looks like the block files have gotten out of sync with the database. It turns out that we were still using fsync() on the block files, so this isn't surprising.
* | | | | | | | | | | | | | | autotools: fix a parallel build race condition spotted by the pull-testerCory Fields2013-09-161-6/+5
|/ / / / / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | libleveldb.a and libmemenv.a should be able to build in parallel, but in practice calling the leveldb makefile ends up rewriting build_config.mk. If one target tries to build while the other is halfway through writing the .mk, the make ends up in an undefined state. Fix that by making one depend on the other. This also reorders the variables to be passed by param rather than via the environment, and combines the targets into a single rule to avoid needless duplication.
* | | | | | | | | | | | | | Merge pull request #2996 from theuni/qt-translationGavin Andresen2013-09-142-16/+32
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / / / |/| | | | | | | | | | | | | Update translation process for autotools
| * | | | | | | | | | | | | autotools: add translate target for qt translationsCory Fields2013-09-142-0/+12
| | | | | | | | | | | | | |
| * | | | | | | | | | | | | autotools: rearrange qt sources to make them more flexibleCory Fields2013-09-141-16/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add BITCOIN_MM QR_CPP and BITCOIN_CPP in order to better accomodate complicated targets. This is a no-op change.
* | | | | | | | | | | | | | Make validateaddress provide redeemScriptPeter Todd2013-09-121-0/+1
| | | | | | | | | | | | | |
* | | | | | | | | | | | | | Add RPC call decodescriptPeter Todd2013-09-124-5/+31
| | | | | | | | | | | | | |
* | | | | | | | | | | | | | Move Parse{Hash|Hex} to be usable by all RPC codePeter Todd2013-09-123-33/+37
|/ / / / / / / / / / / / /
* | | | | | | | | | | | | Merge pull request #2979 from luke-jr/autoconfGregory Maxwell2013-09-108-5/+92
|\ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | Autoconf fixes
| * | | | | | | | | | | | | Bugfix: Since test_bitcoin is being built and run inside src/test/, try ↵Luke Dashjr2013-09-103-3/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | using relative directories from that point
| * | | | | | | | | | | | | configure: Check common include subdirectories for bdb headers, and refuse ↵Luke Dashjr2013-09-104-0/+84
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | to use any version other than 4.8 by default
| * | | | | | | | | | | | | autotools: use an absolute path to test dataCory Fields2013-09-101-2/+2
| | | | | | | | | | | | | |
* | | | | | | | | | | | | | Merge pull request #2982 from gmaxwell/20130908_ccoins_corruptGregory Maxwell2013-09-101-1/+6
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / / / |/| | | | | | | | | | | | | Longer term workaround for chainstate corruption from negative versions.
| * | | | | | | | | | | | | Longer term workaround for chainstate corruption from negative versions.Gregory Maxwell2013-09-091-1/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This also makes negative transaction versions non-standard. This avoids an issue triggered in block 256818 where transactions with negative version numbers were incorrectly serialized into the UTXO set. On restart nodes detect the inconsistency and refuse to start so long as a block with these transactions is inside the self-consistency check window, logging "coin database inconsistencies found". The software recommends reindexing, but reindexing does not correct the problem. This should be fixed by changing the chainstate serialization, but working around it seems harmless for now because the version is not used by any network rule currently. A patch free workaround is to start with -checklevel=2 which skips the consistency checks, but the IsStandard change is important for miners in order to protect unpatched nodes.
* | | | | | | | | | | | | | fix some cosmetic glitches in the codebasePhilip Kaufmann2013-09-0910-36/+32
|/ / / / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - rename URL into URI in paymentserver where correct - add some missing Qt-coding-stuff in paymentserver - change QSpinBox to QLineEdit as base for BitcoinAmountField in .ui files (as this is the result when converting the BAF back into base) - remove some c_str() and replace with QString::fromStdString() - remove several new-lines - remove unneeded spaces - indentation fixes
* | | | | | | | | | | | | Merge pull request #2968 from TheBlueMatt/masterGavin Andresen2013-09-081-0/+5
|\ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|/ / / / / / / |/| | | | | | | | | | | | Don't warn about forks while fIsInitialDownload
| * | | | | | | | | | | | Don't warn about forks while fIsInitialDownloadMatt Corallo2013-09-031-0/+5
| | | | | | | | | | | | |
* | | | | | | | | | | | | Merge pull request #2943 from theuni/autotools-finalJeff Garzik2013-09-0634-670/+1933
|\ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | Autotools buildsystem
| * | | | | | | | | | | | | autotools: switch to autotools buildsystemCory Fields2013-09-0534-670/+1933
| | | | | | | | | | | | | |
* | | | | | | | | | | | | | Merge pull request #2976 from laanwj/2013_09_txid_detailsWladimir J. van der Laan2013-09-065-9/+19
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | qt: add vout index to transaction id in transactions details dialog
| * | | | | | | | | | | | | | qt: add vout index to transaction id in transactions details dialogWladimir J. van der Laan2013-09-065-9/+19
| | | | | | | | | | | | | | |
* | | | | | | | | | | | | | | Bitcoin-Qt: Use qDebug() for printing to debug.logPhilip Kaufmann2013-09-067-52/+67
|/ / / / / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - removes all usages of PrintDebugStringF from Qt code - ensure same format for all debug.log messages "functionname : Message"
* / / / / / / / / / / / / / Make settxfee clear that units are btc/kb.Gregory Maxwell2013-09-051-2/+2
|/ / / / / / / / / / / / /
* | | | | | | | | | | | | Merge pull request #2944 from gavinandresen/acceptheaderGavin Andresen2013-09-031-0/+4
|\ \ \ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / / |/| | | | | | | | | | | | Set Accept: application/bitcoin-paymentrequest header
| * | | | | | | | | | | | Set Accept: application/bitcoin-paymentrequest headerGavin Andresen2013-08-281-0/+4
| |/ / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Suggestion from BitPay: set an Accept: HTTP header when fetching payment requests / payment ACKs.
* | | | | | | | | | | | Merge pull request #2948 from jgarzik/changeaddrGregory Maxwell2013-09-023-0/+25
|\ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | RPC: add getrawchangeaddress, for raw transaction change destinations
| * | | | | | | | | | | | RPC: add getrawchangeaddress, for raw transaction change destinationsJeff Garzik2013-08-283-0/+25
| |/ / / / / / / / / / /
* | | | | | | | | | | | Merge pull request #2958 from laanwj/2013_08_txfee2Wladimir J. van der Laan2013-09-026-45/+203
|\ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | [Qt] Display txfee in first sendCoinsDialog message box
| * | | | | | | | | | | | qt: Display txfee in first sendCoinsDialog message boxJonas Schnelli2013-08-316-45/+203
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Jonas Schnelli <[email protected]>
* | | | | | | | | | | | | Fix typo in a comment: it's base58, not base48.Timothy Stranex2013-09-021-2/+2
| | | | | | | | | | | | |
* | | | | | | | | | | | | Merge pull request #2951 from laanwj/2013_08_addressbook_purposeWladimir J. van der Laan2013-09-027-33/+54
|\ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | [Qt] Handle address purpose in incremental updates
| * | | | | | | | | | | | | qt: Handle address purpose in incremental updatesWladimir J. van der Laan2013-08-297-33/+54
| | |_|_|_|_|/ / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Correctly use the purpose of addresses that are added after the start of the client. Addresses with purpose "refund" and "change" should not be visible in the GUI. This is now handled correctly.
* | | | | | | | | | | | | Bitcoin-Qt: allow to differentiate Qt log entries from corePhilip Kaufmann2013-08-311-9/+8
| |/ / / / / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - prepend "Bitcoin-Qt" in front of debug.log entries, which come from Qt - move DebugMessageHandler installation upwards to the event handler installation, which fits much better
* | | | | | | | | | | | qt: Use GUIUtil::HtmlEscape to escape HTMLWladimir J. van der Laan2013-08-301-10/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is why we created this function. Avoids some #ifdef.
* | | | | | | | | | | | qt: Add comment to sorting step in refreshAddressTableWladimir J. van der Laan2013-08-301-0/+2
| | | | | | | | | | | |
* | | | | | | | | | | | Merge pull request #2930 from Diapolo/displayUnitWladimir J. van der Laan2013-08-299-50/+67
|\ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | Bitcoin-Qt: fixes for using display unit from options
| * | | | | | | | | | | | Bitcoin-Qt: fixes for using display unit from optionsPhilip Kaufmann2013-08-299-50/+67
| |/ / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - extend PaymentServer with setOptionsModel() and rework initNetManager() to make use of that - fix all other places in the code to use display unit from options and no hard-coded unit
* | | | | | | | | | | | Merge pull request #2940 from Diapolo/checkpointPieter Wuille2013-08-291-2/+1
|\ \ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / |/| | | | | | | | | | | move Checkpoints:fEnabled from step 2 to step 3 in init
| * | | | | | | | | | | move Checkpoints:fEnabled from step 2 to step 3 in initPhilip Kaufmann2013-08-251-2/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - it belongs to parameter-to-internal flags step, not parameter interactions phase
* | | | | | | | | | | | Merge pull request #2931 from Diapolo/introWladimir J. van der Laan2013-08-283-3/+8
|\ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | Bitcoin-Qt: add testnet check and icon to intro dialog
| * | | | | | | | | | | | Bitcoin-Qt: add testnet check and icon to intro dialogPhilip Kaufmann2013-08-243-3/+8
| | |_|_|_|/ / / / / / / | |/| | | | | | | | | |
* | | | | | | | | | | | Merge pull request #2904 from gmaxwell/newaddr-no-passphraseGregory Maxwell2013-08-286-22/+20
|\ \ \ \ \ \ \ \ \ \ \ \ | |_|_|/ / / / / / / / / |/| | | | | | | | | | | [QT] Don't ask for a passphrase to getnewaddress.
| * | | | | | | | | | | Remove fAllowReuse from GetKeyFromPool.Gregory Maxwell2013-08-236-14/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | With the GUI password fix this was always false.
| * | | | | | | | | | | [QT] Don't ask for a passphrase to getnewaddress.Gregory Maxwell2013-08-231-10/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | With an encrypted wallet the GUI was prompting for a passphrase every time the user requested a new address. This is unnecessary, increases the exposure to keyboard sniffers, and discourages using fresh addresses for every transaction. Instead only prompt for a passphrase when the keypool runs out, also call the new address function with the flag that prevents reuse. Thanks to AlexNagy on IRC for pointing this out and who wouldn't take any lip from a curmudgeonly developer and insisted on what he knew to be true.
* | | | | | | | | | | | Merge pull request #2928 from jgarzik/cnb-txoutJeff Garzik2013-08-254-20/+28
|\ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | CreateNewBlock / getblocktemplate cleaning
| * | | | | | | | | | | | miner: constify CreateNewBlock() arg scriptPubKeyInJeff Garzik2013-08-252-2/+2
| | | | | | | | | | | | |
| * | | | | | | | | | | | RPC: getblocktemplate does not require a key, to create a block templateJeff Garzik2013-08-241-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | getblocktemplate only uses certain portions of the coinbase transaction, notably ignoring the coinbase TX output entirely. Use CreateNewBlock() rather than CreateNewBlockWithKey(), eliminating the needless key passing. Should be zero behavior changes.
| * | | | | | | | | | | | CreateNewBlock() now takes scriptPubKey argument,Jeff Garzik2013-08-244-20/+27
| |/ / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | rather than a key. CreateNewBlockWithKey() helper is added to restore existing functionality, making this an equivalent-transformation change.
* | | | | | | | | | | | Merge pull request #2938 from petertodd/op-reserved-weirdnessGavin Andresen2013-08-255-3/+18
|\ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | Document and test OP_RESERVED weirdness