Call Tracer.CaptureState before memory expansion (#8860)

This aligns the tracing behavior with Geth, Besu and Nimbus.
erigontech · Nov 30, 2023 · 015f6eb · 015f6eb
1 parent 8d1758c
commit 015f6eb
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/core/vm/interpreter.go b/core/vm/interpreter.go
@@ -293,11 +293,15 @@ func (in *EVMInterpreter) Run(contract *Contract, input []byte, readOnly bool) (
 			if err != nil || !contract.UseGas(dynamicCost) {
 				return nil, ErrOutOfGas
 			}
+			// Do tracing before memory expansion
+			if in.cfg.Debug {
+				in.cfg.Tracer.CaptureState(_pc, op, gasCopy, cost, callContext, in.returnData, in.depth, err) //nolint:errcheck
+				logged = true
+			}
 			if memorySize > 0 {
 				mem.Resize(memorySize)
 			}
-		}
-		if in.cfg.Debug {
+		} else if in.cfg.Debug {
 			in.cfg.Tracer.CaptureState(_pc, op, gasCopy, cost, callContext, in.returnData, in.depth, err) //nolint:errcheck
 			logged = true
 		}