From bc90c4f822b471a1b2a61c0d5f10c0220b2c1e70 Mon Sep 17 00:00:00 2001 From: niftynei Date: Mon, 15 Mar 2021 19:47:14 -0500 Subject: [PATCH] df: don't ignore their sigs, even if we've already seen them In the case of failures/disconnects between dualopend receiving the sigs and the main man disconnecting (as can happen using the testing disconnects), failing to forward their sigs over to lightningd can result in the channel never progressing from DUALOPEND_OPEN_INIT to DUALOPEND_AWAITING_LOCKIN. Always re-relay. --- openingd/dualopend.c | 6 ------ 1 file changed, 6 deletions(-) diff --git a/openingd/dualopend.c b/openingd/dualopend.c index 0b990cb4b..b22b80060 100644 --- a/openingd/dualopend.c +++ b/openingd/dualopend.c @@ -900,12 +900,6 @@ static void handle_tx_sigs(struct state *state, const u8 *msg) "tx_signatures sent after funding_locked %s", tal_hex(msg, msg)); - if (tx_state->remote_funding_sigs_rcvd) { - status_info("Got duplicate WIRE_TX_SIGNATURES, " - "already have their sigs. Ignoring"); - return; - } - if (!tx_state->psbt) open_err_warn(state, "tx_signatures for %s received,"