Fix inability to pin two things at once

License: MIT Signed-off-by: Michael Avila <me@michaelavila.com>
ipfs · Sep 23, 2018 · 55338e6 · 55338e6
1 parent 73ac4a8
commit 55338e6
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/pin/pin.go b/pin/pin.go
@@ -228,16 +228,20 @@ func (p *pinner) Pin(ctx context.Context, node ipld.Node, recurse bool) error {
 		if p.directPin.Has(c) {
 			p.directPin.Remove(c)
 		}
-
+		p.lock.Unlock()
 		// fetch entire graph
 		err := mdag.FetchGraph(ctx, c, p.dserv)
+		p.lock.Lock()
 		if err != nil {
 			return err
 		}
 
 		p.recursePin.Add(c)
 	} else {
-		if _, err := p.dserv.Get(ctx, c); err != nil {
+		p.lock.Unlock()
+		_, err := p.dserv.Get(ctx, c)
+		p.lock.Lock()
+		if err != nil {
 			return err
 		}