tapchannel: properly use htlc keys when making htlc tap descs

Author: Roasbeef

tapchannel/aux_sweeper.go

@@ -632,6 +632,7 @@ func commitNoDelaySweepDesc(keyRing *lnwallet.CommitmentKeyRing,
 		input.ScriptPathSuccess,
 	)
 	if err != nil {
+		return lfn.Err[returnType](err)
 	}
 	ctrlBlockBytes, err := ctrlBlock.ToBytes()
 	if err != nil {
@@ -673,6 +674,7 @@ func commitDelaySweepDesc(keyRing *lnwallet.CommitmentKeyRing,
 		input.ScriptPathSuccess,
 	)
 	if err != nil {
+		return lfn.Err[returnType](err)
 	}
 	ctrlBlockBytes, err := ctrlBlock.ToBytes()
 	if err != nil {
@@ -737,8 +739,9 @@ func remoteHtlcTimeoutSweepDesc(keyRing *lnwallet.CommitmentKeyRing,
 	// We're sweeping a timed out HTLC, which means that we'll need to
 	// create the receiver's HTLC script tree (from the remote party's PoV).
 	htlcScriptTree, err := input.ReceiverHTLCScriptTaproot(
-		htlcExpiry, keyRing.ToLocalKey, keyRing.ToRemoteKey,
-		keyRing.RevocationKey, payHash, lntypes.Remote, input.NoneTapLeaf(),
+		htlcExpiry, keyRing.LocalHtlcKey, keyRing.RemoteHtlcKey,
+		keyRing.RevocationKey, payHash, lntypes.Remote,
+		input.NoneTapLeaf(),
 	)
 	if err != nil {
 		return lfn.Err[tapscriptSweepDescs](err)
@@ -771,15 +774,15 @@ func remoteHtlcTimeoutSweepDesc(keyRing *lnwallet.CommitmentKeyRing,
 // the remote party's commitment transaction that we can sweep with the
 // preimage.
 func remoteHtlcSuccessSweepDesc(keyRing *lnwallet.CommitmentKeyRing,
-	payHash []byte, csvDelay uint32,
-	auxSigs lfn.Option[lnwallet.AuxSigDesc]) lfn.Result[tapscriptSweepDescs] {
+	payHash []byte, csvDelay uint32) lfn.Result[tapscriptSweepDescs] {
 
 	// We're planning on sweeping an HTLC that we know the preimage to,
 	// which the remote party sent, so we'll construct the sender version of
 	// the HTLC script tree (from their PoV, they're the sender).
 	htlcScriptTree, err := input.SenderHTLCScriptTaproot(
-		keyRing.ToLocalKey, keyRing.ToRemoteKey, keyRing.RevocationKey,
-		payHash, lntypes.Local, input.NoneTapLeaf(),
+		keyRing.RemoteHtlcKey, keyRing.LocalHtlcKey,
+		keyRing.RevocationKey, payHash, lntypes.Remote,
+		input.NoneTapLeaf(),
 	)
 	if err != nil {
 		return lfn.Err[tapscriptSweepDescs](err)
@@ -1733,7 +1736,6 @@ func (a *AuxSweeper) resolveContract(
 		// sweep desc for the timeout txn.
 		sweepDesc = remoteHtlcSuccessSweepDesc(
 			req.KeyRing, payHash[:], req.CsvDelay,
-			req.AuxSigDesc,
 		)
 
 	// In this case, we broadcast a commitment transaction which held an