fix: fix evm config init

axonweb3 · Oct 18, 2023 · 5ba15fe · 5ba15fe
1 parent ec26fab
commit 5ba15fe
Show file tree

Hide file tree

Showing 4 changed files with 17 additions and 13 deletions.
diff --git a/core/api/src/jsonrpc/error.rs b/core/api/src/jsonrpc/error.rs
@@ -54,8 +54,8 @@ pub enum RpcError {
     #[display(fmt = "Invalid filter id {}", _0)]
     CannotFindFilterId(u64),
 
-    #[display(fmt = "CKB-VM error {}", "decode_revert_msg(&_0.ret)")]
-    VM(TxResp),
+    #[display(fmt = "EVM error {}", "decode_revert_msg(&_0.ret)")]
+    EVM(TxResp),
     #[display(fmt = "Internal error")]
     Internal(String),
 }
@@ -92,7 +92,7 @@ impl RpcError {
             RpcError::InvalidFromBlockAndToBlockUnion => -40021,
             RpcError::CannotFindFilterId(_) => -40022,
 
-            RpcError::VM(_) => -49998,
+            RpcError::EVM(_) => -49998,
             RpcError::Internal(_) => -49999,
         }
     }
@@ -134,7 +134,7 @@ impl From<RpcError> for ErrorObjectOwned {
             }
             RpcError::CannotFindFilterId(_) => ErrorObject::owned(err_code, err, none_data),
 
-            RpcError::VM(resp) => {
+            RpcError::EVM(resp) => {
                 ErrorObject::owned(err_code, err.clone(), Some(vm_err(resp.clone())))
             }
             RpcError::Internal(msg) => ErrorObject::owned(err_code, err.clone(), Some(msg)),

diff --git a/core/api/src/jsonrpc/impl/web3.rs b/core/api/src/jsonrpc/impl/web3.rs
@@ -522,7 +522,7 @@ impl<Adapter: APIAdapter + 'static> Web3RpcServer for Web3RpcImpl<Adapter> {
             return Ok(call_hex_result);
         }
 
-        Err(RpcError::VM(resp).into())
+        Err(RpcError::EVM(resp).into())
     }
 
     #[metrics_rpc("eth_estimateGas")]
@@ -557,7 +557,7 @@ impl<Adapter: APIAdapter + 'static> Web3RpcServer for Web3RpcImpl<Adapter> {
             return Ok(resp.gas_used.into());
         }
 
-        Err(RpcError::VM(resp).into())
+        Err(RpcError::EVM(resp).into())
     }
 
     #[metrics_rpc("eth_getCode")]

diff --git a/core/executor/src/lib.rs b/core/executor/src/lib.rs
@@ -73,6 +73,7 @@ impl Executor for AxonExecutor {
         value: U256,
         data: Vec<u8>,
     ) -> TxResp {
+        self.init_local_system_contract_roots(backend);
         let config = self.config();
         let metadata = StackSubstateMetadata::new(gas_limit, &config);
         let state = MemoryStackState::new(metadata, backend);
@@ -138,9 +139,8 @@ impl Executor for AxonExecutor {
         let mut hashes = Vec::with_capacity(txs_len);
         let (mut gas, mut fee) = (0u64, U256::zero());
         let precompiles = build_precompile_set();
-        let config = self.config();
-
         self.init_local_system_contract_roots(adapter);
+        let config = self.config();
 
         // Execute system contracts before block hook.
         before_block_hook(adapter);
@@ -309,7 +309,7 @@ impl AxonExecutor {
     /// The `exec()` function is run in `tokio::task::block_in_place()` and all
     /// the read or write operations are in the scope of exec function. The
     /// thread context is not switched during exec function.
-    fn init_local_system_contract_roots<Adapter: ExecutorAdapter>(&self, adapter: &mut Adapter) {
+    fn init_local_system_contract_roots<Adapter: Backend>(&self, adapter: &Adapter) {
         CURRENT_HEADER_CELL_ROOT.with(|root| {
             *root.borrow_mut() =
                 adapter.storage(CKB_LIGHT_CLIENT_CONTRACT_ADDRESS, *HEADER_CELL_ROOT_KEY);
@@ -324,13 +324,14 @@ impl AxonExecutor {
         let mut config = Config::london();
         let create_contract_limit = {
             let latest_hardfork_info = &**HARDFORK_INFO.load();
-            let enable_contract_limit_flag = H256::from_low_u64_be(HardforkName::Andromeda as u64);
+            let enable_contract_limit_flag =
+                H256::from_low_u64_be((HardforkName::Andromeda as u64).to_be());
             if latest_hardfork_info & &enable_contract_limit_flag == enable_contract_limit_flag {
                 let handle = MetadataHandle::new(CURRENT_METADATA_ROOT.with(|r| *r.borrow()));
                 let config = handle.get_consensus_config().unwrap();
                 Some(config.max_contract_limit as usize)
             } else {
-                None
+                Some(0x6000)
             }
         };
         config.create_contract_limit = create_contract_limit;

diff --git a/core/executor/src/tests/system_script/metadata.rs b/core/executor/src/tests/system_script/metadata.rs
@@ -252,8 +252,11 @@ fn prepare_validator() -> ValidatorExtend {
 //         .gas(21000)
 //         .nonce(nonce);
 
-//     let wallet = LocalWallet::from_str(PRIVATE_KEY).expect("failed to create
-// wallet");     let tx = Legacy(transaction_request);
+//     let wallet = LocalWallet::from_str(PRIVATE_KEY).expect(
+//         "failed to create
+// wallet",
+//     );
+//     let tx = Legacy(transaction_request);
 //     let signature: Signature = wallet.sign_transaction(&tx).await.unwrap();
 
 //     provider