grevm(fix): check if the TxEnv.data complies with ABI specifications (p…

…aradigmxyz#37)
AshinGau · Oct 13, 2024 · 3e2ed77 · 3e2ed77
1 parent 612c5be
commit 3e2ed77
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/crates/grevm/src/hint.rs b/crates/grevm/src/hint.rs
@@ -149,7 +149,11 @@ impl ParallelExecutionHints {
         if code.is_none() && data.is_empty() {
             return;
         }
-        println!("code {:?} data {:?}", code, data);
+        // println!("code {:?} data {:?}", code, data);
+        if data.len() < 4 || (data.len() - 4) % 32 != 0 {
+            // Invalid tx, or tx that triggers fallback CALL
+            return;
+        }
         let (func_id, parameters) = Self::decode_contract_parameters(data);
         // TODO(gravity_richard.zhz): refactor the judgement later with contract template
         // ERC20_transfer