Problem: memiavl rootmulti store access map concurrently (#1302)

* Problem: memiavl rootmulti store access map concurrently

Solution:
- code refactoring

* Update CHANGELOG.md

Signed-off-by: yihuang <huang@crypto.com>

---------

Signed-off-by: yihuang <huang@crypto.com>

CHANGELOG.md

@@ -4,6 +4,7 @@
 
 - [#1292](https://github.com/crypto-org-chain/cronos/pull/1292) memiavl cancel background snapshot rewriting when graceful shutdown.
 - [#1294](https://github.com/crypto-org-chain/cronos/pull/1294) Update ethermint to fix and improve of debug_traceCall and eth_feeHistory.
+- [#1302](https://github.com/crypto-org-chain/cronos/pull/1302) Fix concurrent map access in rootmulti store.
 
 *January 5, 2024*
 

store/memiavlstore/store.go

@@ -39,6 +39,10 @@ func New(tree *memiavl.Tree, logger log.Logger) *Store {
  return &Store{tree: tree, logger: logger}
 }
 
+func (st *Store) SetTree(tree *memiavl.Tree) {
+ st.tree = tree
+}
+
 func (st *Store) Commit() types.CommitID {
  panic("memiavl store is not supposed to be committed alone")
 }

store/rootmulti/store.go

@@ -120,14 +120,11 @@ func (rs *Store) Commit() types.CommitID {
  panic(err)
  }
 
- // the underlying memiavl tree might be reloaded, reload the store as well.
+ // the underlying memiavl tree might be reloaded, update the tree.
  for key := range rs.stores {
  store := rs.stores[key]
  if store.GetStoreType() == types.StoreTypeIAVL {
- rs.stores[key], err = rs.loadCommitStoreFromParams(rs.db, key, rs.storesParams[key])
- if err != nil {
- panic(fmt.Errorf("inconsistent store map, store %s not found", key.Name()))
- }
+ store.(*memiavlstore.Store).SetTree(rs.db.TreeByName(key.Name()))
  }
  }