1
62%
master: 62%

Ran 24 Mar 2017 11:50PM UTC

Files 57

Run time 2s

Badge

Embed ▾

Committed 24 Mar 2017 11:40PM UTC coverage: 69.039% (+0.005%) from 69.034%

Job # 684.1

Build Type

push

travis-ci

Committed by

Roasbeef

Commit Message

htlcswitch: add HTLC overflow flow-control via bounded channels

This commit fixes a prior bug which would cause the set of HTLC’s on a
node’s commitment to potentially overflow if an HTLC was accepted or
attempted to be forwarded that but the commitment transaction over the
maximum allowed HTLC’s on a commitment transaction. This would cause
the HTLC to silently be rejected or cause a connection disconnect. In
either case, this would cause the two states to be desynchronized any
pending HTLC’s to be ignored.

We fix this issue by introducing the concept of a bounded channel,
which is a channel in which the number of items send and recevied over
the channel must be balanced in order to allow a new send to succeed
w/o blocking. We achieve this by using a chan struct{} as a semaphore
and decrement it each time a packet it sent, increasing the semaphore
one a packet is received. This creates a channel that we can use to
ensure the switch never sends more than N HTLC’s to a link before any
of the HTLC’s have been settled.

With this bug fix, it’s now once again possible to trigger sustained
bursts of payments through lnd nodes.

Coverage Stats

7140 of 10342 relevant lines covered (69.04%)

4818.96 hits per line

lightningnetwork / lnd / 684 / 1
62%
master: 62%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 684.1

lightningnetwork / lnd / 684 / 1 62% master: 62%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 684.1

lightningnetwork / lnd / 684 / 1
62%
master: 62%

README BADGES
x