eth/tracers: always pop precompiles stack in callTracer (#11004)

made a mistake in previous PR #10986 should always pop the precompiles stack for correctness
erigontech · Jul 4, 2024 · 1a050db · 1a050db
1 parent 9710e98
commit 1a050db
Showing 1 changed file with 7 additions and 2 deletions.
diff --git a/eth/tracers/native/call.go b/eth/tracers/native/call.go
@@ -251,8 +251,13 @@ func (t *callTracer) CaptureExit(output []byte, gasUsed uint64, err error) {
 		return
 	}
 	precompilesLastIdx := len(t.precompiles) - 1
-	if !t.config.IncludePrecompiles && precompilesLastIdx > -1 && t.precompiles[precompilesLastIdx] {
-		t.precompiles = t.precompiles[:precompilesLastIdx]
+	if precompilesLastIdx < 0 {
+		return
+	}
+	// pop precompile
+	precompile := t.precompiles[precompilesLastIdx]
+	t.precompiles = t.precompiles[:precompilesLastIdx]
+	if precompile && !t.config.IncludePrecompiles {
 		return
 	}
 	// pop call