Problem: memiavl's root hash is not safe to retain in commit info

CHANGELOG.md

@@ -37,6 +37,7 @@
 - [#1123](https://github.com/crypto-org-chain/cronos/pull/1123) Fix memiavl snapshot switching
 - [#1125](https://github.com/crypto-org-chain/cronos/pull/1125) Fix genesis migrate for feeibc, evm, feemarket and gravity.
 - [#1130](https://github.com/crypto-org-chain/cronos/pull/1130) Fix lock issues when state-sync with memiavl.
+- [#]() Fix memiavl's unsafe retain of the root hashes.
 
 ### Features
 

memiavl/mem_node.go

@@ -70,6 +70,10 @@ func (node *MemNode) Mutate(version, cowVersion uint32) *MemNode {
 	return n
 }
 
+func (node *MemNode) SafeHash() []byte {
+	return node.Hash()
+}
+
 // Computes the hash of the node without computing its descendants. Must be
 // called on nodes which have descendant node hashes already computed.
 func (node *MemNode) Hash() []byte {

memiavl/node.go

@@ -20,6 +20,9 @@ type Node interface {
 	Right() Node
 	Hash() []byte
 
+	// SafeHash returns byte slice that's safe to retain
+	SafeHash() []byte
+
 	// PersistedNode clone a new node, MemNode modify in place
 	Mutate(version, cowVersion uint32) *MemNode
 

memiavl/persisted_node.go

@@ -131,6 +131,10 @@ func (node PersistedNode) Right() Node {
 	return PersistedNode{snapshot: node.snapshot, index: node.index - 1}
 }
 
+func (node PersistedNode) SafeHash() []byte {
+	return bytes.Clone(node.Hash())
+}
+
 func (node PersistedNode) Hash() []byte {
 	if node.isLeaf {
 		return node.leafNode().Hash()

memiavl/tree.go

@@ -162,12 +162,13 @@ func (t *Tree) Version() int64 {
 	return int64(t.version)
 }
 
-// RootHash updates the hashes and return the current root hash
+// RootHash updates the hashes and return the current root hash,
+// it clones the persisted node's bytes, so the returned bytes is safe to retain.
 func (t *Tree) RootHash() []byte {
 	if t.root == nil {
 		return emptyHash
 	}
-	return t.root.Hash()
+	return t.root.SafeHash()
 }
 
 func (t *Tree) GetWithIndex(key []byte) (int64, []byte) {