treat CallGasCostMoreThanGasLimit as error

ArkEcosystem · Dec 20, 2024 · 3500a24 · 3500a24
1 parent 8215582
commit 3500a24
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 8 deletions.
diff --git a/packages/crypto-transaction-evm-call/source/handlers/evm-call.ts b/packages/crypto-transaction-evm-call/source/handlers/evm-call.ts
@@ -60,7 +60,7 @@ export class EvmCallTransactionHandler extends Handlers.TransactionHandler {
 
 			return receipt;
 		} catch (error) {
-			return this.app.terminate("invalid EVM call", error);
+			throw new Error(`invalid EVM call: ${error.message}`);
 		}
 	}
 

diff --git a/packages/evm/bindings/src/lib.rs b/packages/evm/bindings/src/lib.rs
@@ -424,13 +424,8 @@ impl EvmInner {
                 match err {
                     EVMError::Transaction(err) => {
                         match err {
-                            revm::primitives::InvalidTransaction::CallGasCostMoreThanGasLimit => {
-                                return Ok(TxReceipt {
-                                    gas_used: gas_limit,
-                                    ..Default::default()
-                                });
-                            }
-                            revm::primitives::InvalidTransaction::NonceTooHigh { .. }
+                            revm::primitives::InvalidTransaction::CallGasCostMoreThanGasLimit
+                            | revm::primitives::InvalidTransaction::NonceTooHigh { .. }
                             | revm::primitives::InvalidTransaction::NonceTooLow { .. } => {
                                 return Err(EVMError::Transaction(err));
                             }