fix(meta): create dentry concurrency cause orphan inodes

Signed-off-by: sunjiapeng <782615313@qq.com>
cubefs · Mar 11, 2024 · 86696fc · 86696fc
1 parent 491c1cb
commit 86696fc
Showing 1 changed file with 3 additions and 7 deletions.
diff --git a/sdk/meta/api.go b/sdk/meta/api.go
@@ -324,10 +324,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 {
@@ -1657,9 +1655,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