aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorSun Ke2022-08-12 09:14:40 +0800
committerTrond Myklebust2022-08-13 13:02:13 -0400
commit2067231a9e2cbbcae0a4aca6ac36ff2dd6a7b701 (patch)
treef10ae762d95b8ba57fcc995ab4b464c1a51fcf1b
parent3fa5cbdc44de190f2c5605ba7db015ae0d26f668 (diff)
NFS: Fix missing unlock in nfs_unlink()
Add the missing unlock before goto. Fixes: 3c59366c207e ("NFS: don't unhash dentry during unlink/rename") Signed-off-by: Sun Ke <sunke32@huawei.com> Signed-off-by: Trond Myklebust <trond.myklebust@hammerspace.com>
-rw-r--r--fs/nfs/dir.c4
1 files changed, 3 insertions, 1 deletions
diff --git a/fs/nfs/dir.c b/fs/nfs/dir.c
index dbab3caa15ed..1b879584d4fe 100644
--- a/fs/nfs/dir.c
+++ b/fs/nfs/dir.c
@@ -2484,8 +2484,10 @@ int nfs_unlink(struct inode *dir, struct dentry *dentry)
*/
error = -ETXTBSY;
if (WARN_ON(dentry->d_flags & DCACHE_NFSFS_RENAMED) ||
- WARN_ON(dentry->d_fsdata == NFS_FSDATA_BLOCKED))
+ WARN_ON(dentry->d_fsdata == NFS_FSDATA_BLOCKED)) {
+ spin_unlock(&dentry->d_lock);
goto out;
+ }
if (dentry->d_fsdata)
/* old devname */
kfree(dentry->d_fsdata);