From 9eddb9844c9ade83ce13eb9f19c88337c440a027 Mon Sep 17 00:00:00 2001 From: ThomasV Date: Fri, 7 Jun 2019 12:24:46 +0200 Subject: [PATCH] on_commitment_signed: distinguish between exceptions --- electrum/lnpeer.py | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/electrum/lnpeer.py b/electrum/lnpeer.py index 07e8ab42c..a9dd215d8 100644 --- a/electrum/lnpeer.py +++ b/electrum/lnpeer.py @@ -1097,12 +1097,14 @@ class Peer(Logger): self.logger.info("on_commitment_signed") channel_id = payload['channel_id'] chan = self.channels[channel_id] - ctn_to_recv = chan.get_current_ctn(LOCAL) + 1 # make sure there were changes to the ctx, otherwise the remote peer is misbehaving if (chan.hm.pending_htlcs(LOCAL) == chan.hm.current_htlcs(LOCAL) - and chan.pending_feerate(LOCAL) == chan.constraints.feerate) \ - or ctn_to_recv == self.recv_commitment_for_ctn_last[chan]: - raise RemoteMisbehaving('received commitment_signed without any change') + and chan.pending_feerate(LOCAL) == chan.constraints.feerate): + raise RemoteMisbehaving('received commitment_signed without pending changes') + # make sure ctn is new + ctn_to_recv = chan.get_current_ctn(LOCAL) + 1 + if ctn_to_recv == self.recv_commitment_for_ctn_last[chan]: + raise RemoteMisbehaving('received commitment_signed with same ctn') self.recv_commitment_for_ctn_last[chan] = ctn_to_recv data = payload["htlc_signature"]