sweepbatcher: fix a bug in dest address selection

starius · starius · commit 72caafa14e8f · 2025-06-26T14:01:16.000-03:00
For presigned possible remaining groups, the destination address of the current
batch was used instead of the destination address of an expected future batch.

TODO: reproduce in unit test "purged". For this, each swap should have a
separate destination address.
diff --git a/sweepbatcher/presigned.go b/sweepbatcher/presigned.go
@@ -247,7 +247,7 @@ func (b *batch) presign(ctx context.Context, newSweeps []*sweep) error {
 
 		// Cache the destination address.
 		destAddr, err := getPresignedSweepsDestAddr(
-			ctx, b.cfg.presignedHelper, b.primarySweepID,
+			ctx, b.cfg.presignedHelper, primarySweepID,
 			b.cfg.chainParams,
 		)
 		if err != nil {

Original file line number	Diff line number	Diff line change
`@@ -247,7 +247,7 @@ func (b batch) presign(ctx context.Context, newSweeps []sweep) error {`
`247`	`247`
`248`	`248`	`// Cache the destination address.`
`249`	`249`	`destAddr, err := getPresignedSweepsDestAddr(`
`250`		`- ctx, b.cfg.presignedHelper, b.primarySweepID,`
	`250`	`+ ctx, b.cfg.presignedHelper, primarySweepID,`
`251`	`251`	`b.cfg.chainParams,`
`252`	`252`	`)`
`253`	`253`	`if err != nil {`