fix minor data race in bitswap

whyrusleeping · whyrusleeping · commit 934aaca30548 · 2015-05-22T09:08:40.000-07:00
diff --git a/exchange/bitswap/bitswap.go b/exchange/bitswap/bitswap.go
@@ -287,8 +287,8 @@ func (bs *Bitswap) ReceiveMessage(ctx context.Context, p peer.ID, incoming bsmsg
 		if has, err := bs.blockstore.Has(block.Key()); err == nil && has {
 			bs.dupBlocksRecvd++
 		}
-		bs.counterLk.Unlock()
 		log.Infof("got block %s from %s (%d,%d)", block, p, bs.blocksRecvd, bs.dupBlocksRecvd)
+		bs.counterLk.Unlock()
 
 		hasBlockCtx, cancel := context.WithTimeout(ctx, hasBlockTimeout)
 		if err := bs.HasBlock(hasBlockCtx, block); err != nil {

Original file line number	Diff line number	Diff line change
`@@ -287,8 +287,8 @@ func (bs *Bitswap) ReceiveMessage(ctx context.Context, p peer.ID, incoming bsmsg`
`287`	`287`	`if has, err := bs.blockstore.Has(block.Key()); err == nil && has {`
`288`	`288`	`bs.dupBlocksRecvd++`
`289`	`289`	`}`
`290`		`- bs.counterLk.Unlock()`
`291`	`290`	`log.Infof("got block %s from %s (%d,%d)", block, p, bs.blocksRecvd, bs.dupBlocksRecvd)`
	`291`	`+ bs.counterLk.Unlock()`
`292`	`292`
`293`	`293`	`hasBlockCtx, cancel := context.WithTimeout(ctx, hasBlockTimeout)`
`294`	`294`	`if err := bs.HasBlock(hasBlockCtx, block); err != nil {`