merged master

rarimo · Dec 12, 2023 · 0769e68 · 0769e68
1 parent 6740f8d
commit 0769e68
Showing 1 changed file with 5 additions and 4 deletions.
diff --git a/contracts/IdentityManager.sol b/contracts/IdentityManager.sol
@@ -66,19 +66,20 @@ contract IdentityManager is IIdentityManager, Signers, MultiOwnable {
     ) external onlyOwner {
         RootData storage _prevRoot = _roots[prevRoot_];
 
+        require(prevRoot_ != postRoot_, "IdentityManager: same prev and post roots");
         require(_prevRoot.replacedAt == 0, "IdentityManager: can't update already stored root");
 
+        if (_prevRoot.replacedBy != 0) {
+            _roots[postRoot_].replacedBy = _prevRoot.replacedBy;
+        }
+
         if (replacedAt_ >= _latestTimestamp) {
             _roots[_latestRoot].replacedBy = postRoot_;
 
             _latestRoot = postRoot_;
             _latestTimestamp = replacedAt_;
         }
 
-        if (_prevRoot.replacedBy != 0) {
-            _roots[postRoot_].replacedBy = _prevRoot.replacedBy;
-        }
-
         _prevRoot.replacedAt = replacedAt_;
         _prevRoot.replacedBy = postRoot_;