Tracer should only fetch the currently active stack. (#1085)

0xPolygon · Jan 10, 2023 · ae79516 · ae79516
1 parent 25f36e5
commit ae79516
Showing 1 changed file with 7 additions and 2 deletions.
diff --git a/state/runtime/tracer/structtracer/tracer.go b/state/runtime/tracer/structtracer/tracer.go
@@ -143,7 +143,7 @@ func (t *StructTracer) CaptureState(
 
 	t.captureMemory(memory)
 
-	t.captureStack(stack)
+	t.captureStack(stack, sp)
 
 	t.captureStorage(
 		stack,
@@ -169,14 +169,19 @@ func (t *StructTracer) captureMemory(
 
 func (t *StructTracer) captureStack(
 	stack []*big.Int,
+	sp int,
 ) {
 	if !t.Config.EnableStack {
 		return
 	}
 
-	t.currentStack = make([]*big.Int, len(stack))
+	t.currentStack = make([]*big.Int, sp)
 
 	for i, v := range stack {
+		if i >= sp {
+			break
+		}
+
 		t.currentStack[i] = new(big.Int).Set(v)
 	}
 }