Merge pull request lightningnetwork#8609 from ziggie1984/master

lnd: fix sweepall argument call.
mohamedawnallah · Apr 2, 2024 · b331e73 · b331e73
2 parents 16c8339 + b019657
commit b331e73
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/docs/release-notes/release-notes-0.18.0.md b/docs/release-notes/release-notes-0.18.0.md
@@ -109,6 +109,9 @@
 
 * [Removed](https://github.com/lightningnetwork/lnd/pull/8577) some unreachable code
 
+[Fixed](https://github.com/lightningnetwork/lnd/pull/8609) fixed a function
+call where arguments were swapped.
+
 # New Features
 ## Functional Enhancements
 

diff --git a/rpcserver.go b/rpcserver.go
@@ -1310,7 +1310,7 @@ func (r *rpcServer) SendCoins(ctx context.Context,
 		// pay to the change address created above if we needed to
 		// reserve any value, the rest will go to targetAddr.
 		sweepTxPkg, err := sweep.CraftSweepAllTx(
-			maxFeeRate, feePerKw, uint32(bestHeight), nil,
+			feePerKw, maxFeeRate, uint32(bestHeight), nil,
 			targetAddr, wallet, wallet, wallet.WalletController,
 			r.server.cc.Signer, minConfs,
 		)
@@ -1363,7 +1363,7 @@ func (r *rpcServer) SendCoins(ctx context.Context,
 			}
 
 			sweepTxPkg, err = sweep.CraftSweepAllTx(
-				maxFeeRate, feePerKw, uint32(bestHeight),
+				feePerKw, maxFeeRate, uint32(bestHeight),
 				outputs, targetAddr, wallet, wallet,
 				wallet.WalletController,
 				r.server.cc.Signer, minConfs,