Revert "staticaddr: use tx hash not *wire.MsgTx as a key"

This reverts commit 6154219.

Author: hieblmi

staticaddr/withdraw/manager.go

@@ -227,18 +227,15 @@ func (m *Manager) recoverWithdrawals(ctx context.Context) error {
 	}
 
 	// Group the deposits by their finalized withdrawal transaction.
-	depositsByWithdrawalTx := make(map[chainhash.Hash][]*deposit.Deposit)
-	hash2tx := make(map[chainhash.Hash]*wire.MsgTx)
+	depositsByWithdrawalTx := make(map[*wire.MsgTx][]*deposit.Deposit)
 	for _, d := range activeDeposits {
 		withdrawalTx := d.FinalizedWithdrawalTx
 		if withdrawalTx == nil {
 			continue
 		}
-		txid := withdrawalTx.TxHash()
-		hash2tx[txid] = withdrawalTx
 
-		depositsByWithdrawalTx[txid] = append(
-			depositsByWithdrawalTx[txid], d,
+		depositsByWithdrawalTx[withdrawalTx] = append(
+			depositsByWithdrawalTx[withdrawalTx], d,
 		)
 	}
 
@@ -247,7 +244,7 @@ func (m *Manager) recoverWithdrawals(ctx context.Context) error {
 	eg := &errgroup.Group{}
 
 	// We can now reinstate each cluster of deposits for a withdrawal.
-	for txid, deposits := range depositsByWithdrawalTx {
+	for tx, deposits := range depositsByWithdrawalTx {
 		eg.Go(func() error {
 			err := m.cfg.DepositManager.TransitionDeposits(
 				ctx, deposits, deposit.OnWithdrawInitiated,
@@ -257,11 +254,6 @@ func (m *Manager) recoverWithdrawals(ctx context.Context) error {
 				return err
 			}
 
-			tx, ok := hash2tx[txid]
-			if !ok {
-				return fmt.Errorf("can't find tx %v", txid)
-			}
-
 			_, err = m.publishFinalizedWithdrawalTx(ctx, tx)
 			if err != nil {
 				return err