deposit: fix sending to finalizedDepositChan

starius · hieblmi · commit e704119f146a · 2025-03-21T10:30:41.000+01:00
diff --git a/staticaddr/deposit/actions.go b/staticaddr/deposit/actions.go
@@ -147,12 +147,9 @@ func (f *FSM) SweptExpiredDepositAction(ctx context.Context,
 	case <-ctx.Done():
 		return fsm.OnError
 
-	default:
-		f.finalizedDepositChan <- f.deposit.OutPoint
-		ctx.Done()
+	case f.finalizedDepositChan <- f.deposit.OutPoint:
+		return fsm.NoOp
 	}
-
-	return fsm.NoOp
 }
 
 // FinalizeDepositAction is the final action after a withdrawal. It signals to
@@ -164,9 +161,7 @@ func (f *FSM) FinalizeDepositAction(ctx context.Context,
 	case <-ctx.Done():
 		return fsm.OnError
 
-	default:
-		f.finalizedDepositChan <- f.deposit.OutPoint
+	case f.finalizedDepositChan <- f.deposit.OutPoint:
+		return fsm.NoOp
 	}
-
-	return fsm.NoOp
 }

Original file line number	Diff line number	Diff line change
`@@ -147,12 +147,9 @@ func (f *FSM) SweptExpiredDepositAction(ctx context.Context,`
`147`	`147`	`case <-ctx.Done():`
`148`	`148`	`return fsm.OnError`
`149`	`149`
`150`		`- default:`
`151`		`- f.finalizedDepositChan <- f.deposit.OutPoint`
`152`		`- ctx.Done()`
	`150`	`+ case f.finalizedDepositChan <- f.deposit.OutPoint:`
	`151`	`+ return fsm.NoOp`
`153`	`152`	`}`
`154`		`-`
`155`		`- return fsm.NoOp`
`156`	`153`	`}`
`157`	`154`
`158`	`155`	`// FinalizeDepositAction is the final action after a withdrawal. It signals to`
`@@ -164,9 +161,7 @@ func (f *FSM) FinalizeDepositAction(ctx context.Context,`
`164`	`161`	`case <-ctx.Done():`
`165`	`162`	`return fsm.OnError`
`166`	`163`
`167`		`- default:`
`168`		`- f.finalizedDepositChan <- f.deposit.OutPoint`
	`164`	`+ case f.finalizedDepositChan <- f.deposit.OutPoint:`
	`165`	`+ return fsm.NoOp`
`169`	`166`	`}`
`170`		`-`
`171`		`- return fsm.NoOp`
`172`	`167`	`}`