fix: validate gas_limit and max_fee_per_gas before transaction execution

[Romsters]

What 💻

• Added validation for gas_limit and max_fee_per_gas before transaction execution

Why ✋

• Atm when gasLimit doesn't fit in U32 the system crashes.

Evidence 📷

2023-10-27T23:09:19.788918Z  INFO era_test_node::node: Executing 0xcfa772ee8b45f2d54ee5a31f3e42be5018862627fe4785d812bf1617bb8de5e7
thread 'tokio-runtime-worker' panicked at /Users/user/.cargo/registry/src/index.crates.io-6f17d22bba15001f/primitive-types-0.12.2/src/lib.rs:38:1:
Integer overflow when casting to u32
stack backtrace:
   0: rust_begin_unwind
             at /rustc/aa1a71e9e90f6eb3aed8cf79fc80bea304c17ecb/library/std/src/panicking.rs:597:5
   1: core::panicking::panic_fmt
             at /rustc/aa1a71e9e90f6eb3aed8cf79fc80bea304c17ecb/library/core/src/panicking.rs:72:14
   2: primitive_types::U256::as_u32
             at /Users/user/.cargo/registry/src/index.crates.io-6f17d22bba15001f/uint-0.9.5/src/uint.rs:633:6
   3: multivm::versions::vm_virtual_blocks::types::internals::transaction_data::TransactionData::overhead_gas
             at /Users/user/.cargo/git/checkouts/zksync-era-657998444dcd9ed9/73a1e8f/core/lib/multivm/src/versions/vm_virtual_blocks/types/internals/transaction_data.rs:203:31
   4: multivm::versions::vm_virtual_blocks::implementation::tx::<impl multivm::versions::vm_virtual_blocks::vm::Vm<S,H>>::push_transaction_with_compression
             at /Users/user/.cargo/git/checkouts/zksync-era-657998444dcd9ed9/73a1e8f/core/lib/multivm/src/versions/vm_virtual_blocks/implementation/tx.rs:64:24
   5: multivm::versions::vm_virtual_blocks::vm::Vm<S,H>::push_transaction
             at /Users/user/.cargo/git/checkouts/zksync-era-657998444dcd9ed9/73a1e8f/core/lib/multivm/src/versions/vm_virtual_blocks/vm.rs:48:9
   6: era_test_node::node::InMemoryNode<S>::run_l2_tx_inner
             at ./src/node.rs:1289:9
   7: era_test_node::node::InMemoryNode<S>::run_l2_tx
             at ./src/node.rs:1471:13
   8: <era_test_node::node::InMemoryNode<S> as zksync_core::api_server::web3::backend_jsonrpc::namespaces::eth::rpc_impl_EthNamespaceT::gen_server::EthNamespaceT>::send_raw_transaction
             at ./src/node.rs:2019:15
]

[Romsters]

Closes #206

[MexicanAce]

When adding Evidence to the PR description, please show evidence that the issue is now resolved with these changes. This provides another level of confidence for the reviewer that things are working end-to-end after these changes.

[Romsters]

When adding Evidence to the PR description, please show evidence that the issue is now resolved with these changes. This provides another level of confidence for the reviewer that things are working end-to-end after these changes.

I guess adding unit tests should resolve this one, in terms of other evidences not sure how do you see it, in my case there is just no crash or errors in console after the fix.

[MexicanAce]

I guess adding unit tests should resolve this one, in terms of other evidences not sure how do you see it, in my case there is just no crash or errors in console after the fix.

Yup, unit tests would work! Just any way to prove that the problem is resolved with these changes 👍

[MexicanAce]

LGTM