From dfe484d6a28307de3de52a42ffcde9cc4b2a4129 Mon Sep 17 00:00:00 2001
From: zhiqiangxu <652732310@qq.com>
Date: Fri, 5 Jan 2024 23:49:19 +0800
Subject: [PATCH] should use Lock when mutating the flag

---
 core/blockstm/mvhashmap.go | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/core/blockstm/mvhashmap.go b/core/blockstm/mvhashmap.go
index 59085138cf..51fcce63f1 100644
--- a/core/blockstm/mvhashmap.go
+++ b/core/blockstm/mvhashmap.go
@@ -166,13 +166,13 @@ func (mv *MVHashMap) MarkEstimate(k Key, txIdx int) {
 		panic(fmt.Errorf("path must already exist"))
 	})
 
-	cells.rw.RLock()
+	cells.rw.Lock()
 	if ci, ok := cells.tm.Get(txIdx); !ok {
 		panic(fmt.Sprintf("should not happen - cell should be present for path. TxIdx: %v, path, %x, cells keys: %v", txIdx, k, cells.tm.Keys()))
 	} else {
 		ci.(*WriteCell).flag = FlagEstimate
 	}
-	cells.rw.RUnlock()
+	cells.rw.Unlock()
 }
 
 func (mv *MVHashMap) Delete(k Key, txIdx int) {