From ab11322b39d6c6b75b390f1a110624f12466f1c6 Mon Sep 17 00:00:00 2001 From: Rusty Russell Date: Sun, 6 Nov 2016 14:44:16 +1030 Subject: [PATCH] peer: don't close due to too low fees if we're already closed. This fixes half of #59 (the part which crashes). It doesn't fix the fact that we should never be doing this for testnet. Reported-by: Thomas Daede Signed-off-by: Rusty Russell --- daemon/peer.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/daemon/peer.c b/daemon/peer.c index 492835422..994d6cced 100644 --- a/daemon/peer.c +++ b/daemon/peer.c @@ -3035,7 +3035,8 @@ static enum watch_result anchor_depthchange(struct peer *peer, * timely manner (see "Risks With HTLC Timeouts"). */ /* FIXME: BOLT should say what to do if it can't! We drop conn. */ - if (peer->local.commit->cstate->fee_rate < get_feerate(peer->dstate)) { + if (!state_is_onchain(peer->state) && !state_is_error(peer->state) + && peer->local.commit->cstate->fee_rate < get_feerate(peer->dstate)) { log_broken(peer->log, "fee rate %"PRIu64" lower than %"PRIu64, peer->local.commit->cstate->fee_rate, get_feerate(peer->dstate));