fix: cast overflow in 32-bits OS (#978)

* fix: cast overflow in 32-bits OS

* Fix type conversion issues in bitwise.rs, host.rs, host_env.rs, macros.rs, and system.rs

* Fix error in as_usize_or_fail macro

* Update CI configuration and add Cross.toml

* Fix command arguments in ethereum-tests.yml

* Update Ethereum tests workflow

* Update Cross.toml with pre-build commands for i686 target

Author: dyxushuai

.github/workflows/ci.yml

@@ -51,7 +51,7 @@ jobs:
       - run: |
           cd crates/revm
           cargo check --no-default-features
-  
+
   check-serde:
     name: check serde
     runs-on: ubuntu-latest

.github/workflows/ethereum-tests.yml

@@ -10,7 +10,6 @@ on:
   pull_request:
     branches: [main, "release/**"]
 
-
 jobs:
   tests-stable:
     name: Ethereum Tests (Stable)
@@ -19,6 +18,7 @@ jobs:
     strategy:
       matrix:
         profile: [ethtests, release]
+        target: [i686-unknown-linux-gnu, x86_64-unknown-linux-gnu]
     steps:
       - name: Checkout sources
         uses: actions/checkout@v3
@@ -37,11 +37,13 @@ jobs:
         with:
           cache-on-failure: true
 
+      - name: Install cross
+        run: cargo install cross
+
       - name: Run Ethereum tests
         run: |
-          cargo run --profile ${{ matrix.profile }} -p revme -- statetest \
+          cross run --target ${{matrix.target}} --profile ${{ matrix.profile }} -p revme -- statetest \
             ethtests/GeneralStateTests/ \
             ethtests/LegacyTests/Constantinople/GeneralStateTests/ \
             ethtests/EIPTests/StateTests/stEIP1153-transientStorage/ \
             ethtests/EIPTests/StateTests/stEIP4844-blobtransactions/ \
-            ethtests/EIPTests/StateTests/stEIP5656-MCOPY/

Cross.toml

@@ -0,0 +1,10 @@
+[build]
+pre-build = [
+    "apt-get update && apt-get install --assume-yes --no-install-recommends llvm-dev clang libclang-dev",
+]
+
+[target.i686-unknown-linux-gnu]
+image = "ghcr.io/cross-rs/i686-unknown-linux-gnu:main"
+
+[target.x86_64-unknown-linux-gnu]
+image = "ghcr.io/cross-rs/x86_64-unknown-linux-gnu:main"

crates/interpreter/src/instructions/macros.rs

@@ -190,7 +190,7 @@ macro_rules! as_u64_saturated {
 
 macro_rules! as_usize_saturated {
     ($v:expr) => {
-        as_u64_saturated!($v) as usize
+        ::core::convert::TryInto::<usize>::try_into(as_u64_saturated!($v)).unwrap_or(usize::MAX)
     };
 }
 
@@ -205,6 +205,10 @@ macro_rules! as_usize_or_fail {
             $interp.instruction_result = $reason;
             return;
         }
-        x[0] as usize
+        let Ok(val) = ::core::convert::TryInto::<usize>::try_into(x[0]) else {
+            $interp.instruction_result = $reason;
+            return;
+        };
+        val
     }};
 }