The call method in executor should handle system contracts as exec method does.

[yangby-cryptape]

Description

Current status

• The exec method in executor do special operations for system contracts.

  axon/core/executor/src/lib.rs

  Lines 152 to 153 in 264a676

  	let mut r = system_contract_dispatch(adapter, tx)
  	.unwrap_or_else(|| Self::evm_exec(adapter, &config, &precompiles, tx));

• But call method doesn't.

  axon/core/executor/src/lib.rs

  Line 65 in 264a676

  fn call<B: Backend>(

Issues introduced by the above Design

When call system contracts —— for example, call eth_estimateGas —— it may lead to unexpected results.

Also, unexpected results may be occurred when replay system contracts (ref: #1234 tried).

Proposals

• Do same operations for both exec and call.

• Or, just reject system contracts directly in the call method.

[Flouse]

confirmed solution

• just reject system contracts directly in the call method

[yangby-cryptape]

I found that #1597 will resolve this issue.

p.s. This comment is created via GitHub CLI.