Merge pull request #6549 from oasisprotocol/peternose/bugfix/evidence-round

peternose · web-flow · commit cd5651b2e607 · 2026-06-16T12:52:09.000+02:00
go/consensus/cometbft/apps/roothash: Fix evidence removal round check
diff --git a/.changelog/6549.bugfix.md b/.changelog/6549.bugfix.md
@@ -0,0 +1 @@
+go/consensus/cometbft/apps/roothash: Fix evidence removal round check
diff --git a/go/consensus/cometbft/apps/roothash/state/state.go b/go/consensus/cometbft/apps/roothash/state/state.go
@@ -547,7 +547,7 @@ func (s *MutableState) RemoveExpiredEvidence(ctx context.Context, runtimeID comm
 		if rtID != hID {
 			break
 		}
-		if round > minRound {
+		if round >= minRound {
 			break
 		}
 		toDelete = append(toDelete, it.Key())

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1 @@`
	`1`	`+go/consensus/cometbft/apps/roothash: Fix evidence removal round check`
Original file line number	Diff line number	Diff line change
`@@ -547,7 +547,7 @@ func (s *MutableState) RemoveExpiredEvidence(ctx context.Context, runtimeID comm`
`547`	`547`	`if rtID != hID {`
`548`	`548`	`break`
`549`	`549`	`}`
`550`		`- if round > minRound {`
	`550`	`+ if round >= minRound {`
`551`	`551`	`break`
`552`	`552`	`}`
`553`	`553`	`toDelete = append(toDelete, it.Key())`