bitswap: clean up ledgers when disconnecting

License: MIT
Signed-off-by: Jeromy <[email protected]>

Author: whyrusleeping

exchange/bitswap/bitswap.go

@@ -393,6 +393,7 @@ func (bs *Bitswap) updateReceiveCounters(b blocks.Block) error {
 // Connected/Disconnected warns bitswap about peer connections
 func (bs *Bitswap) PeerConnected(p peer.ID) {
 	bs.wm.Connected(p)
+	bs.engine.PeerConnected(p)
 }
 
 // Connected/Disconnected warns bitswap about peer connections

exchange/bitswap/decision/engine.go

@@ -295,8 +295,24 @@ func (e *Engine) MessageSent(p peer.ID, m bsmsg.BitSwapMessage) error {
 	return nil
 }
 
+func (e *Engine) PeerConnected(p peer.ID) {
+	l := e.findOrCreate(p)
+	l.lk.Lock()
+	l.ref++
+	l.lk.Unlock()
+}
+
 func (e *Engine) PeerDisconnected(p peer.ID) {
-	// TODO: release ledger
+	e.lock.Lock()
+	defer e.lock.Unlock()
+	l, ok := e.ledgerMap[p]
+	if !ok {
+		return
+	}
+	l.ref--
+	if l.ref <= 0 {
+		delete(e.ledgerMap, p)
+	}
 }
 
 func (e *Engine) numBytesSentTo(p peer.ID) uint64 {

exchange/bitswap/decision/ledger.go

@@ -43,6 +43,10 @@ type ledger struct {
 	// to a given peer
 	sentToPeer map[string]time.Time
 
+	// ref is the reference count for this ledger, its used to ensure we
+	// don't drop the reference to this ledger in multi-connection scenarios
+	ref int
+
 	lk sync.Mutex
 }