Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

transfer2 precompile #2636

Merged
merged 2 commits into from
Oct 31, 2023
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
86 changes: 83 additions & 3 deletions runtime/common/src/precompile/multicurrency.rs
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,10 @@ use super::{
target_gas_limit,
};
use crate::WeightToGas;
use frame_support::traits::{Currency, Get};
use frame_support::{
pallet_prelude::IsType,
traits::{Currency, Get},
};
use module_currencies::WeightInfo;
use module_evm::{
precompiles::Precompile,
Expand All @@ -32,7 +35,7 @@ use module_support::Erc20InfoMapping as Erc20InfoMappingT;
use num_enum::{IntoPrimitive, TryFromPrimitive};
use orml_traits::MultiCurrency as MultiCurrencyT;
use primitives::{currency::DexShare, Balance, CurrencyId};
use sp_runtime::{traits::Convert, RuntimeDebug};
use sp_runtime::{traits::Convert, AccountId32, RuntimeDebug};
use sp_std::{marker::PhantomData, prelude::*};

/// The `MultiCurrency` impl precompile.
Expand All @@ -56,12 +59,14 @@ pub enum Action {
QueryTotalIssuance = "totalSupply()",
QueryBalance = "balanceOf(address)",
Transfer = "transfer(address,address,uint256)",
TransferToAccountId = "transferToAccountId(address,bytes32,uint256)",
}

impl<Runtime> Precompile for MultiCurrencyPrecompile<Runtime>
where
Runtime:
module_currencies::Config + module_evm::Config + module_prices::Config + module_transaction_payment::Config,
Runtime::AccountId: IsType<AccountId32>,
module_currencies::Pallet<Runtime>: MultiCurrencyT<Runtime::AccountId, CurrencyId = CurrencyId, Balance = Balance>,
{
fn execute(input: &[u8], target_gas: Option<u64>, context: &Context, _is_static: bool) -> PrecompileResult {
Expand Down Expand Up @@ -188,6 +193,35 @@ where
cost: target_gas_limit(target_gas).unwrap_or_default(),
})?;

Ok(PrecompileOutput {
exit_status: ExitSucceed::Returned,
cost: gas_cost,
output: vec![],
logs: Default::default(),
})
}
Action::TransferToAccountId => {
let from = input.account_id_at(1)?;
let to_data = input.bytes32_at(2)?;
let mut buf = [0u8; 32];
buf.copy_from_slice(&to_data[..]);
let to: Runtime::AccountId = AccountId32::from(buf).into();

let amount = input.balance_at(3)?;
log::debug!(target: "evm", "multicurrency: transferToAccountId from: {:?}, to: {:?}, amount: {:?}", from, to, amount);

<module_currencies::Pallet<Runtime> as MultiCurrencyT<Runtime::AccountId>>::transfer(
currency_id,
&from,
&to,
amount,
)
.map_err(|e| PrecompileFailure::Revert {
exit_status: ExitRevert::Reverted,
output: Output::encode_error_msg("Multicurrency TransferToAccountId failed", e),
cost: target_gas_limit(target_gas).unwrap_or_default(),
})?;

Ok(PrecompileOutput {
exit_status: ExitSucceed::Returned,
cost: gas_cost,
Expand Down Expand Up @@ -235,7 +269,7 @@ where
<Runtime as frame_system::Config>::DbWeight::get().reads(2),
))
}
Action::Transfer => {
Action::Transfer | Action::TransferToAccountId => {
let cost = InputPricer::<Runtime>::read_accounts(2);

// transfer weight
Expand Down Expand Up @@ -555,4 +589,50 @@ mod tests {
);
})
}

#[test]
fn transfer_to_account_id_works() {
new_test_ext().execute_with(|| {
let mut context = Context {
address: Default::default(),
caller: Default::default(),
apparent_value: Default::default(),
};

// transferToAccountId(address,bytes32,uint256) -> 0x89590498
// from
// to
// amount
let input = hex! {"
89590498
000000000000000000000000 1000000000000000000000000000000000000001
65766d3a10000000000000000000000000000000000000020000000000000000
00000000000000000000000000000000 00000000000000000000000000000001
"};

let from_balance = Balances::free_balance(alice());
let to_balance = Balances::free_balance(bob());

// Token
context.caller = aca_evm_address();

let resp = MultiCurrencyPrecompile::execute(&input, None, &context, false).unwrap();
assert_eq!(resp.exit_status, ExitSucceed::Returned);
assert_eq!(resp.output, [0u8; 0].to_vec());

assert_eq!(Balances::free_balance(alice()), from_balance - 1);
assert_eq!(Balances::free_balance(bob()), to_balance + 1);

// DexShare
context.caller = lp_aca_ausd_evm_address();
assert_noop!(
MultiCurrencyPrecompile::execute(&input, Some(100_000), &context, false),
PrecompileFailure::Revert {
exit_status: ExitRevert::Reverted,
output: "Multicurrency TransferToAccountId failed: BalanceTooLow".into(),
cost: target_gas_limit(Some(100_000)).unwrap(),
}
);
})
}
}
Loading