dual-funding: don't not update the state! log the issue and move on with

with your life
This commit is contained in:
niftynei 2021-03-15 19:49:01 -05:00 committed by Rusty Russell
parent bc90c4f822
commit fc64ebdb53

View File

@ -1505,17 +1505,17 @@ static void handle_channel_locked(struct subd *dualopend,
assert(channel->scid); assert(channel->scid);
assert(channel->remote_funding_locked); assert(channel->remote_funding_locked);
if (channel->state != DUALOPEND_AWAITING_LOCKIN) { /* This can happen if we missed their sigs, for some reason */
if (channel->state != DUALOPEND_AWAITING_LOCKIN)
log_debug(channel->log, "Lockin complete, but state %s", log_debug(channel->log, "Lockin complete, but state %s",
channel_state_name(channel)); channel_state_name(channel));
} else {
channel_set_state(channel, channel_set_state(channel,
DUALOPEND_AWAITING_LOCKIN, channel->state,
CHANNELD_NORMAL, CHANNELD_NORMAL,
REASON_UNKNOWN, REASON_UNKNOWN,
"Lockin complete"); "Lockin complete");
channel_record_open(channel); channel_record_open(channel);
}
/* FIXME: LND sigs/update_fee msgs? */ /* FIXME: LND sigs/update_fee msgs? */
peer_start_channeld(channel, pps, NULL, false); peer_start_channeld(channel, pps, NULL, false);