Skip to content

Commit 5aafd96

Browse files
committed
smb: client: get rid of d_drop() in cifs_do_rename()
JIRA: https://issues.redhat.com/browse/RHEL-123665 commit 72ed55b Author: Paulo Alcantara <pc@manguebit.org> Date: Wed Oct 22 21:11:01 2025 -0300 smb: client: get rid of d_drop() in cifs_do_rename() There is no need to force a lookup by unhashing the moved dentry after successfully renaming the file on server. The file metadata will be re-fetched from server, if necessary, in the next call to ->d_revalidate() anyways. Signed-off-by: Paulo Alcantara (Red Hat) <pc@manguebit.org> Reviewed-by: David Howells <dhowells@redhat.com> Cc: stable@vger.kernel.org Cc: linux-cifs@vger.kernel.org Signed-off-by: Steve French <stfrench@microsoft.com> Signed-off-by: Paulo Alcantara <paalcant@redhat.com>
1 parent df37740 commit 5aafd96

File tree

1 file changed

+1
-4
lines changed

1 file changed

+1
-4
lines changed

fs/smb/client/inode.c

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2462,11 +2462,8 @@ cifs_do_rename(const unsigned int xid, struct dentry *from_dentry,
24622462
}
24632463
#endif /* CONFIG_CIFS_ALLOW_INSECURE_LEGACY */
24642464
do_rename_exit:
2465-
if (rc == 0) {
2465+
if (rc == 0)
24662466
d_move(from_dentry, to_dentry);
2467-
/* Force a new lookup */
2468-
d_drop(from_dentry);
2469-
}
24702467
cifs_put_tlink(tlink);
24712468
return rc;
24722469
}

0 commit comments

Comments
 (0)