Peerplays_Blockchain/peerplays_migrated
1
0
Fork 0
Code Issues 35 Pull requests 4 Projects Releases 20 Packages Wiki Activity Actions

transfer_evaluator.cpp: Remove unused variable and redundant check #566

Browse source 
This check was the pre-419 check of whether the fee asset is authorized,
which was (due to a typo) buggily checking the sent asset rather
then the fee asset.
This commit is contained in:
theoreticalbts 2016-02-11 03:23:15 -05:00
parent 6f7f2605c1
commit e4f7483ec3
1 changed files with 0 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
libraries/chain/transfer_evaluator.cpp
View file

@ -36,7 +36,6 @@ void_result transfer_evaluator::do_evaluate( const transfer_operation& op )
const account_object& from_account = op.from(d);
const account_object& to_account = op.to(d);
const asset_object& asset_type = op.amount.asset_id(d);
const asset_object& fee_asset_type = op.fee.asset_id(d);
try {
@ -55,12 +54,6 @@ void_result transfer_evaluator::do_evaluate( const transfer_operation& op )
("asset",op.amount.asset_id)
);
if( d.head_block_time() <= HARDFORK_419_TIME )
{
FC_ASSERT( is_authorized_asset( d, from_account, asset_type ) );
}
// the above becomes no-op after hardfork because this check will then be performed in evaluator
if( asset_type.is_transfer_restricted() )
{
GRAPHENE_ASSERT(
@ -105,7 +98,6 @@ void_result override_transfer_evaluator::do_evaluate( const override_transfer_op
const account_object& from_account = op.from(d);
const account_object& to_account = op.to(d);
const asset_object& fee_asset_type = op.fee.asset_id(d);
FC_ASSERT( is_authorized_asset( d, to_account, asset_type ) );
FC_ASSERT( is_authorized_asset( d, from_account, asset_type ) );