Small fuzzing fixes (#7418)

* Addresses an issue in the Fluent API where certain EVM bytecode could lead to executions that mismatch canonical execution. * Uses LOG.debug instead of System.out.println to prevent console spam during fuzzing. Signed-off-by: Guido Vranken <[email protected]>
hyperledger · Aug 1, 2024 · 14d2809 · 14d2809
1 parent 9cb8b06
commit 14d2809
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/evm/src/main/java/org/hyperledger/besu/evm/fluent/EVMExecutor.java b/evm/src/main/java/org/hyperledger/besu/evm/fluent/EVMExecutor.java
@@ -755,6 +755,7 @@ public Bytes execute() {
           })
           .process(messageFrame, tracer);
     }
+    initialMessageFrame.getSelfDestructs().forEach(worldUpdater::deleteAccount);
     if (commitWorldState) {
       worldUpdater.commit();
     }

diff --git a/evm/src/main/java/org/hyperledger/besu/evm/precompile/KZGPointEvalPrecompiledContract.java b/evm/src/main/java/org/hyperledger/besu/evm/precompile/KZGPointEvalPrecompiledContract.java
@@ -140,7 +140,7 @@ public PrecompileContractResult computePrecompile(
             null, Optional.of(ExceptionalHaltReason.PRECOMPILE_ERROR));
       }
     } catch (RuntimeException kzgFailed) {
-      System.out.println(kzgFailed.getMessage());
+      LOG.debug("Native KZG failed", kzgFailed);
 
       return PrecompileContractResult.halt(
           null, Optional.of(ExceptionalHaltReason.PRECOMPILE_ERROR));