pin: fix concurrent map access race

Not sure why this didn't show up sooner. fixes #6418
ipfs · Jun 6, 2019 · 486f013 · 486f013
1 parent 505eeed
commit 486f013
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/pin/pin.go b/pin/pin.go
@@ -498,11 +498,17 @@ func LoadPinner(d ds.Datastore, dserv, internal ipld.DAGService) (Pinner, error)
 
 // DirectKeys returns a slice containing the directly pinned keys
 func (p *pinner) DirectKeys() []cid.Cid {
+	p.lock.RLock()
+	defer p.lock.RUnlock()
+
 	return p.directPin.Keys()
 }
 
 // RecursiveKeys returns a slice containing the recursively pinned keys
 func (p *pinner) RecursiveKeys() []cid.Cid {
+	p.lock.RLock()
+	defer p.lock.RUnlock()
+
 	return p.recursePin.Keys()
 }