[PATCH v2 2/2] smb: client: flush deferred handles of other names before a hard link open

[Paulo Alcantara <pc@manguebit.org>]

cifs_open() flushes an inode's deferred closes before opening a different
name (hard link) of it, so the open can take a fresh lease rather than be
reissued without one (see "smb: client: retry a hard link open without a
lease").  cifs_get_hardlink_path() only triggered that flush when a cached
handle had the same open flags as the new open, so a differently accessed
name - written then read, say - skipped it and opened without a lease.

Match on the name instead: flush when a cached handle for another name of
the inode is open, so those opens keep lease caching too.

Fixes: 262b73ef442e ("smb3 client: fix open hardlink on deferred close file error")
Cc: stable@vger.kernel.org
Reviewed-by: Namjae Jeon <linkinjeon@kernel.org>
Signed-off-by: Paulo Alcantara <pc@manguebit.org>
---
 fs/smb/client/file.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/fs/smb/client/file.c b/fs/smb/client/file.c
index 1247d73740a0..f51653cb6432 100644
--- a/fs/smb/client/file.c
+++ b/fs/smb/client/file.c
@@ -2737,7 +2737,7 @@ cifs_get_hardlink_path(struct cifs_tcon *tcon, struct inode *inode,
 	spin_lock(&cinode->open_file_lock);
 
 	list_for_each_entry(open_file, &cinode->openFileList, flist) {
-		if (file->f_flags == open_file->f_flags) {
+		if (open_file->dentry != file_dentry(file)) {
 			rc = -EINVAL;
 			break;
 		}
-- 
2.56.0