Merge pull request #584 from codenotary/fix_data_race

jeroiraz · web-flow · commit 97f10cbe65a3 · 2020-12-29T11:34:22.000-03:00
fix(embedded/tbtree): sync GetTs to prevent data races
diff --git a/embedded/tbtree/tbtree.go b/embedded/tbtree/tbtree.go
@@ -507,6 +507,9 @@ func (t *TBtree) Get(key []byte) (value []byte, ts uint64, err error) {
 }
 
 func (t *TBtree) GetTs(key []byte, limit int64) (ts []uint64, err error) {
+	t.mutex.Lock()
+	defer t.mutex.Unlock()
+
 	if t.closed {
 		return nil, ErrAlreadyClosed
 	}

Original file line number	Diff line number	Diff line change
`@@ -507,6 +507,9 @@ func (t *TBtree) Get(key []byte) (value []byte, ts uint64, err error) {`
`507`	`507`	`}`
`508`	`508`
`509`	`509`	`func (t *TBtree) GetTs(key []byte, limit int64) (ts []uint64, err error) {`
	`510`	`+ t.mutex.Lock()`
	`511`	`+ defer t.mutex.Unlock()`
	`512`	`+`
`510`	`513`	`if t.closed {`
`511`	`514`	`return nil, ErrAlreadyClosed`
`512`	`515`	`}`