aboutsummaryrefslogtreecommitdiff
path: root/src/main.cpp
Commit message (Collapse)AuthorAgeFilesLines
...
| * | Update transaction statisticsPieter Wuille2013-04-041-2/+3
| |/ | | | | | | | | | | | | | | As these were not updated when 'backporting' the 225430 checkpoint into head. Additionally, also report verification progress in debug.log, and tweak the sigcheck-verification-speed-factor a bit.
* | Merge pull request #2420 from sipa/globcleanGavin Andresen2013-04-021-0/+26
|\ \ | |/ |/| Global cleanups
| * Clean up global datastructures at shutdown.Pieter Wuille2013-03-291-0/+26
| | | | | | | | This should make detecting leaks much easier.
* | Process getdata invs separately until send buffer overflowsPieter Wuille2013-03-301-94/+116
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There exists a per-message-processed send buffer overflow protection, where processing is halted when the send buffer is larger than the allowed maximum. This protection does not apply to individual items, however, and getdata has the potential for causing large amounts of data to be sent. In case several hundreds of blocks are requested in one getdata, the send buffer can easily grow 50 megabytes above the send buffer limit. This commit breaks up the processing of getdata requests, remembering them inside a CNode when too many are requested at once.
* | Use per-message send buffer, rather than per connectionPieter Wuille2013-03-291-5/+8
| |
* | Some fixes to CNetMessage processingPieter Wuille2013-03-291-14/+14
| | | | | | | | | | | | * Change CNode::vRecvMsg to be a deque instead of a vector (less copying) * Make sure to acquire cs_vRecvMsg in CNode::CloseSocketDisconnect (as it may be called without that lock).
* | P2P: parse network datastream into header/data components in socket threadJeff Garzik2013-03-291-38/+27
|/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | Replaces CNode::vRecv buffer with a vector of CNetMessage's. This simplifies ProcessMessages() and eliminates several redundant data copies. Overview: * socket thread now parses incoming message datastream into header/data components, as encapsulated by CNetMessage * socket thread adds each CNetMessage to a vector inside CNode * message thread (ProcessMessages) iterates through CNode's CNetMessage vector Message parsing is made more strict: * Socket is disconnected, if message larger than MAX_SIZE or if CMessageHeader deserialization fails (latter is impossible?). Previously, code would simply eat garbage data all day long. * Socket is disconnected, if we fail to find pchMessageStart. We do not search through garbage, to find pchMessageStart. Each message must begin precisely after the last message ends. ProcessMessages() always processes a complete message, and is more efficient: * buffer is always precisely sized, using CDataStream::resize(), rather than progressively sized in 64k chunks. More efficient for large messages like "block". * whole-buffer memory copy eliminated (vRecv -> vMsg) * other buffer-shifting memory copies eliminated (vRecv.insert, vRecv.erase)
* Before 15 May, limit created block size to 500KGavin Andresen2013-03-171-2/+6
|
* CheckBlock rule until 15-May for 10,000 BDB lock compatibilityGavin Andresen2013-03-171-0/+19
|
* Merge pull request #2186 from Diapolo/misc_stuffWladimir J. van der Laan2013-02-231-13/+11
|\ | | | | small changes in init, main, checkpoints.h and bitcoin-qt.pro
| * small changes in init, main, checkpoints.h and bitcoin-qt.proPhilip Kaufmann2013-02-201-13/+11
| | | | | | | | | | | | | | | | | | - remove an unneeded MODAL flag, as MSG_ERROR sets MODAL - re-order an if-clause in main to have bool checks before a function call - fix some log messages that used wrong function names - make a log message use a correct ellipsis - remove some unneded spaces, brackets and line-breaks - fix style for adding files in the Qt project
* | Merge pull request #2221 from sipa/perfoGavin Andresen2013-02-221-18/+19
|\ \ | | | | | | Various performance tweaks to CCoinsView
| * | Various performance tweaks to CCoinsViewPieter Wuille2013-01-261-18/+19
| | | | | | | | | | | | | | | | | | | | | | | | * Pass txid's to CCoinsView functions by reference instead of by value * Add a method to swap CCoins, and use it in some places to avoid a allocating copy + destruct. * Optimize CCoinsViewCache::FetchCoins to do only a single search through the backing map.
* | | Merge pull request #2229 from sipa/preallocGavin Andresen2013-02-221-2/+6
|\ \ \ | |_|/ |/| | Native versions for AllocateFileRange()
| * | Drop padding in blk* when finalizing filePieter Wuille2013-01-301-2/+6
| | |
* | | Improve block database load error reportingPieter Wuille2013-02-171-0/+12
| | |
* | | Merge pull request #2279 from sipa/cvrbip30Gavin Andresen2013-02-061-1/+1
|\ \ \ | | | | | | | | Bugfix CValidationResult for BIP30 + add DoS
| * | | Bugfix CValidationResult for BIP30 + add DoSPieter Wuille2013-02-061-1/+1
| | | |
* | | | Do not call ResendWalletTransactions when reindexing, importing or on IBDRubén Darío Ponticelli2013-02-051-1/+6
|/ / / | | | | | | | | | | | | Calling ResendWalletTransactions when reindexing, importing or on IBD spams other nodes with our old transactions, because they become unconfirmed.
* | | Merge branch 'reindexgen' of git://github.com/sipa/bitcoinGavin Andresen2013-02-051-19/+27
|\ \ \
| * | | Make sure the genesis block is present after reindexPieter Wuille2013-02-011-19/+27
| | | |
* | | | Merge pull request #2273 from gavinandresen/txsizeGavin Andresen2013-02-051-0/+8
|\ \ \ \ | | | | | | | | | | Make transactions larger than 100K non-standard
| * | | | Make transactions larger than 100K non-standardGavin Andresen2013-02-051-0/+8
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Extremely large transactions with lots of inputs can cost the network almost as much to process as they cost the sender in fees. We would never create transactions larger than 100K big; this change makes transactions larger than 100K non-standard, so they are not relayed/mined by default. This is most important for miners that might create blocks larger than 250K big, who could be vulnerable to a make-your-blocks-so-expensive-to-verify-they-get-orphaned attack.
* / / / Bugfix: Enable ConnectBestBlock to properly report back validation problems, ↵Luke Dashjr2013-01-311-4/+7
|/ / / | | | | | | | | | | | | | | | and ensure orphan processing (when their parents are found) cannot be used to counter-DDoS the node providing the parent Also fix a minor typo
* | | Merge pull request #2224 from sipa/valstateGavin Andresen2013-01-291-200/+193
|\ \ \ | | | | | | | | Improve error handling during validation
| * | | Deal with LevelDB errorsPieter Wuille2013-01-301-28/+42
| | | |
| * | | Improve dealing with abort conditionsPieter Wuille2013-01-301-35/+34
| | | |
| * | | Add disk space checks before flushing CCoins cachePieter Wuille2013-01-301-0/+7
| | | |
| * | | Treat coinbase value violation as DoSPieter Wuille2013-01-301-1/+1
| | | |
| * | | CValidationState frameworkPieter Wuille2013-01-301-175/+148
| |/ /
* / / Fix two clang3.3 warningsGavin Andresen2013-01-291-9/+0
|/ /
* | Merge pull request #2223 from gavinandresen/nonfinalnonstandardGavin Andresen2013-01-281-0/+3
|\ \ | | | | | | Treat non-final transactions as non-standard
| * | Treat non-final transactions as non-standardGavin Andresen2013-01-261-0/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | At least one service that accepted zero-confirmation transactions was vulnerable because an attacker could send a transaction with a lock time far in the future, and then have plenty of time in which to get a double-spend mined (perhaps from a miner who wasn't on the network when the first transaction was broadcast). That is a variation on the "Finney attack". We still don't recommend anybody accept 0-confirmation transactions as final payment for anything. This change keeps non-final transactions from appearing in the wallet, and, assuming most of the network accepts this change, will prevent them from being relayed until they are final.
* | | Merge pull request #2182 from gavinandresen/addressoracleGavin Andresen2013-01-261-27/+26
|\ \ \ | |/ / |/| | Remove IsFromMe() check in CTxMemPool::accept()
| * | Let limitfreerelay=0 reject ALL free transactionsGavin Andresen2013-01-231-1/+1
| | |
| * | Remove IsFromMe() check in CTxMemPool::accept()Gavin Andresen2013-01-151-27/+26
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes issue #2178 : attacker could penny-flood with invalid-signature transactions to deduce which addresses belonged to your node. I'm committing this early for code review; I still need to write up a test plan. Executive summary of fix: check all transactions received from the network for penny-flood rate-limiting before adding to the memory pool. But do NOT ratelimit transactions added to the memory pool: - because of blockchain reorgs - stored in the wallet and added at startup - sent from the GUI or one of the send* RPC commands (CWallet::CommitTransaction) The limit-free-transactions code really should be a method on CNode, with counters per-peer. But that is a bigger change for another day.
* | | Check only 288 blocks at startup by defaultPieter Wuille2013-01-261-1/+1
| |/ |/|
* | Merge pull request #2168 from sipa/txindexGavin Andresen2013-01-251-4/+40
|\ \ | | | | | | Add optional transaction index to databases
| * | Add optional transaction index to databasesPieter Wuille2013-01-181-4/+40
| | | | | | | | | | | | | | | | | | | | | | | | | | | By specifying -txindex when initializing the database, a txid-to-diskpos index is maintained in the blktree database. This database is used to help answering getrawtransaction() RPC queries, when enabled. Changing the -txindex value requires a -reindex; the client will abort at startup if the database and the specified -txindex mismatch.
* | | Merge pull request #2187 from CodeShark/SyncWithWalletsFixGavin Andresen2013-01-231-2/+1
|\ \ \ | | | | | | | | Bugfix - Moved SyncWithWallets out of ProcessMessage and into CTxMemPool::accept()
| * | | Moved SyncWithWallets out of ProcessMessage and into CTxMemPool::accept() so ↵Eric Lombrozo2013-01-181-2/+1
| |/ / | | | | | | | | | that when adding multiple wallets they will be aware of each other's transactions.
* | | Merge pull request #2192 from mikehearn/notfoundmsgGavin Andresen2013-01-231-1/+17
|\ \ \ | | | | | | | | Add a notfound message to getdata.
| * | | Add a notfound message to getdata that is sent if any transactions that ↵Mike Hearn2013-01-191-1/+17
| |/ / | | | | | | | | | aren't in the relayable set are requested.
* | | Merge pull request #2188 from TheBlueMatt/bloomGavin Andresen2013-01-231-2/+2
|\ \ \ | | | | | | | | Send transactions after a CMerkleBlock when asked for it in an inv.
| * | | Replace 520 constant with MAX_SCRIPT_ELEMENT_SIZEMatt Corallo2013-01-181-1/+1
| | | |
| * | | Send transactions after a CMerkleBlock when asked for it in an inv.Matt Corallo2013-01-181-1/+1
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This actually simplifies some SPV code, as they can keep track of a filtered block and its txn before accepting both in one step. The previous argument was that SPV nodes should handle the txn the same as any other free txn and then mark them as connected to a block when they get the filtered block itself. However, it now appears that SPV nodes will need to put in more effort to verify loose txn than they would to verify txn in blocks, thus making it more approriate to send the txn after the filtered block.
* / / Bugfix + simplify special case for genesisPieter Wuille2013-01-191-15/+8
|/ /
* | Merge pull request #2060 from sipa/parallelGavin Andresen2013-01-171-8/+56
|\ \ | | | | | | Parallel script verification
| * | Remove contention on signature cache during block validationPieter Wuille2013-01-081-1/+4
| | | | | | | | | | | | | | | | | | | | | | | | Since block validation happens in parallel, multiple threads may be accessing the signature cache simultaneously. To prevent contention: * Turn the signature cache lock into a shared mutex * Make reading from the cache only acquire a shared lock * Let block validations not store their results in the cache
| * | Parallelize script verificationPieter Wuille2013-01-081-4/+36
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * During block verification (when parallelism is requested), script check actions are stored instead of being executed immediately. * After every processed transactions, its signature actions are pushed to a CScriptCheckQueue, which maintains a queue and some synchronization mechanism. * Two or more threads (if enabled) start processing elements from this queue, * When the block connection code is finished processing transactions, it joins the worker pool until the queue is empty. As cs_main is held the entire time, and all verification must be finished before the block continues processing, this does not reach the best possible performance. It is a less drastic change than some more advanced mechanisms (like doing verification out-of-band entirely, and rolling back blocks when a failure is detected). The -par=N flag controls the number of threads (1-16). 0 means auto, and is the default.
generated by cgit v1.2.3 (git 2.25.1) at 2026-03-17 22:58:53 +0000