nihilist/monero
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

core: fix recalculation of emission/fees

Browse Source 
It was counting things many, many times
This commit is contained in:
moneromooo-monero 2016-12-21 11:18:33 +00:00
parent 049b7e9a93
commit 12abe86ac0
No known key found for this signature in database
GPG Key ID: 686F07454D6CEFC3
1 changed files with 4 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
src/cryptonote_core/cryptonote_core.cpp
View File

@ -619,17 +619,16 @@ namespace cryptonote
std::pair<uint64_t, uint64_t> core::get_coinbase_tx_sum(const uint64_t start_offset, const size_t count) std::pair<uint64_t, uint64_t> core::get_coinbase_tx_sum(const uint64_t start_offset, const size_t count)
{ {
std::list<block> blocks; std::list<block> blocks;
std::list<transaction> txs;
std::list<crypto::hash> missed_txs;
uint64_t coinbase_amount = 0;
uint64_t emission_amount = 0; uint64_t emission_amount = 0;
uint64_t total_fee_amount = 0; uint64_t total_fee_amount = 0;
uint64_t tx_fee_amount = 0;
this->get_blocks(start_offset, count, blocks); this->get_blocks(start_offset, count, blocks);
BOOST_FOREACH(auto& b, blocks) BOOST_FOREACH(auto& b, blocks)
{ {
coinbase_amount = get_outs_money_amount(b.miner_tx); std::list<transaction> txs;
std::list<crypto::hash> missed_txs;
uint64_t coinbase_amount = get_outs_money_amount(b.miner_tx);
this->get_transactions(b.tx_hashes, txs, missed_txs); this->get_transactions(b.tx_hashes, txs, missed_txs);
uint64_t tx_fee_amount = 0;
BOOST_FOREACH(const auto& tx, txs) BOOST_FOREACH(const auto& tx, txs)
{ {
tx_fee_amount += get_tx_fee(tx); tx_fee_amount += get_tx_fee(tx);
@ -637,8 +636,6 @@ namespace cryptonote
emission_amount += coinbase_amount - tx_fee_amount; emission_amount += coinbase_amount - tx_fee_amount;
total_fee_amount += tx_fee_amount; total_fee_amount += tx_fee_amount;
coinbase_amount = 0;
tx_fee_amount = 0;
} }
return std::pair<uint64_t, uint64_t>(emission_amount, total_fee_amount); return std::pair<uint64_t, uint64_t>(emission_amount, total_fee_amount);