Charon/blockchain
1
0
Fork 0
forked from lthn/blockchain
Code Pull requests Activity

validate_alt_block_input fixed rare bug for cases g3-g5 when an output is spent in the next block

Browse source
This commit is contained in:
sowle 2021-01-07 19:40:55 +03:00
parent 920d104e1b
commit bf9195db58
No known key found for this signature in database
GPG key ID: C07A24B2D89D49FC
1 changed files with 3 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
src/currency_core/blockchain_storage.cpp
View file

@ -6047,9 +6047,8 @@ bool blockchain_storage::validate_alt_block_input(const transaction& input_tx, s
if (abg_it != alt_chain.back()->second.gindex_lookup_table.end())
{
amount_touched_altchain = true;
//Notice: since transactions is not allowed to refer to each other in one block, then we can consider that index in
//tx input would be always less then top for previous block, so just take it
global_outs_for_amount = abg_it->second;
// local gindex lookup table contains last used gindex, so we can't get total number of outs
// just skip setting global_outs_for_amount
}
else
{
@ -6074,7 +6073,7 @@ bool blockchain_storage::validate_alt_block_input(const transaction& input_tx, s
if (off.type() == typeid(uint64_t))
{
uint64_t offset_gindex = boost::get<uint64_t>(off);
CHECK_AND_ASSERT_MES(offset_gindex < global_outs_for_amount, false,
CHECK_AND_ASSERT_MES(amount_touched_altchain || (offset_gindex < global_outs_for_amount), false,
"invalid global output index " << offset_gindex << " for amount=" << input.amount <<
", max is " << global_outs_for_amount <<
", referred to by offset #" << pk_n <<