fix(meta): create dentry concurrency cause orphan inodes

Signed-off-by: sunjiapeng <[email protected]>
sjp00556 · Jun 19, 2024 · 14b7156 · 14b7156
1 parent cef58ab
commit 14b7156
Showing 1 changed file with 3 additions and 7 deletions.
diff --git a/sdk/meta/api.go b/sdk/meta/api.go
@@ -311,10 +311,8 @@ create_dentry:
 		}
 		return nil, statusToErrno(status)
 	} else if status != statusOK {
-		if status != statusExist {
-			mw.iunlink(mp, info.Inode, mw.Client.GetLatestVer(), 0, fullPath)
-			mw.ievict(mp, info.Inode, fullPath)
-		}
+		mw.iunlink(mp, info.Inode, mw.Client.GetLatestVer(), 0, fullPath)
+		mw.ievict(mp, info.Inode, fullPath)
 		return nil, statusToErrno(status)
 	}
 	if mw.EnableSummary {
@@ -1645,9 +1643,7 @@ func (mw *MetaWrapper) link(parentID uint64, name string, ino uint64, fullPath s
 	if err != nil {
 		return nil, statusToErrno(status)
 	} else if status != statusOK {
-		if status != statusExist {
-			mw.iunlink(mp, ino, mw.Client.GetLatestVer(), 0, fullPath)
-		}
+		mw.iunlink(mp, ino, mw.Client.GetLatestVer(), 0, fullPath)
 		return nil, statusToErrno(status)
 	}
 	return info, nil