ksmbd: add mnt_want_write to ksmbd vfs functions

ksmbd is doing write access using vfs helpers. There are the cases that
mnt_want_write() is not called in vfs helper. This patch add missing
mnt_want_write() to ksmbd vfs functions.

Signed-off-by: Namjae Jeon <[email protected]>

Author: namjaejeon

smb1pdu.c

@@ -2612,7 +2612,7 @@ int smb_nt_create_andx(struct ksmbd_work *work)
 			da.create_time = fp->create_time;
 
 			err = ksmbd_vfs_set_dos_attrib_xattr(mnt_user_ns(path.mnt),
-							     path.dentry, &da);
+							     &path, &da);
 			if (err)
 				ksmbd_debug(SMB, "failed to store creation time in xattr\n");
 			err = 0;
@@ -7359,7 +7359,7 @@ static int create_dir(struct ksmbd_work *work)
 				XATTR_DOSINFO_ITIME;
 
 			err = ksmbd_vfs_set_dos_attrib_xattr(mnt_user_ns(path.mnt),
-							     path.dentry, &da);
+							     &path, &da);
 			if (err)
 				ksmbd_debug(SMB, "failed to store creation time in EA\n");
 			path_put(&path);
@@ -7531,7 +7531,7 @@ int smb_mkdir(struct ksmbd_work *work)
 				XATTR_DOSINFO_ITIME;
 
 			err = ksmbd_vfs_set_dos_attrib_xattr(mnt_user_ns(path.mnt),
-							     path.dentry, &da);
+							     &path, &da);
 			if (err)
 				ksmbd_debug(SMB, "failed to store creation time in xattr\n");
 			path_put(&path);
@@ -8271,7 +8271,7 @@ int smb_open_andx(struct ksmbd_work *work)
 				XATTR_DOSINFO_ITIME;
 
 			err = ksmbd_vfs_set_dos_attrib_xattr(mnt_user_ns(path.mnt),
-							     path.dentry, &da);
+							     &path, &da);
 			if (err)
 				ksmbd_debug(SMB, "failed to store creation time in xattr\n");
 			err = 0;

smb2pdu.c

@@ -2334,7 +2334,7 @@ static int smb2_set_ea(struct smb2_ea_info *eabuf, unsigned int buf_len,
 #else
 			rc = ksmbd_vfs_setxattr(user_ns,
 #endif
-						path->dentry, attr_name, value,
+						path, attr_name, value,
 						le16_to_cpu(eabuf->EaValueLength), 0);
 			if (rc < 0) {
 				ksmbd_debug(SMB,
@@ -2400,9 +2400,9 @@ static noinline int smb2_set_stream_name_xattr(const struct path *path,
 	}
 
 #if LINUX_VERSION_CODE >= KERNEL_VERSION(6, 3, 0)
-	rc = ksmbd_vfs_setxattr(idmap, path->dentry,
+	rc = ksmbd_vfs_setxattr(idmap, path,
 #else
-	rc = ksmbd_vfs_setxattr(user_ns, path->dentry,
+	rc = ksmbd_vfs_setxattr(user_ns, path,
 #endif
 				xattr_stream_name, NULL, 0, 0);
 	if (rc < 0)
@@ -2490,10 +2490,10 @@ static void smb2_new_xattrs(struct ksmbd_tree_connect *tcon, const struct path *
 
 #if LINUX_VERSION_CODE >= KERNEL_VERSION(6, 3, 0)
 	rc = ksmbd_vfs_set_dos_attrib_xattr(mnt_idmap(path->mnt),
-					    path->dentry, &da);
+					    path, &da);
 #else
 	rc = ksmbd_vfs_set_dos_attrib_xattr(mnt_user_ns(path->mnt),
-					    path->dentry, &da);
+					    path, &da);
 #endif
 	if (rc)
 		ksmbd_debug(SMB, "failed to store file attribute into xattr\n");
@@ -3157,11 +3157,11 @@ int smb2_open(struct ksmbd_work *work)
 
 #if LINUX_VERSION_CODE >= KERNEL_VERSION(6, 3, 0)
 		posix_acl_rc = ksmbd_vfs_inherit_posix_acl(idmap,
-							   path.dentry,
+							   &path,
 							   d_inode(path.dentry->d_parent));
 #else
 		posix_acl_rc = ksmbd_vfs_inherit_posix_acl(user_ns,
-							   path.dentry,
+							   &path,
 							   d_inode(path.dentry->d_parent));
 #endif
 		if (posix_acl_rc)
@@ -3179,10 +3179,10 @@ int smb2_open(struct ksmbd_work *work)
 				if (posix_acl_rc)
 #if LINUX_VERSION_CODE >= KERNEL_VERSION(6, 3, 0)
 					ksmbd_vfs_set_init_posix_acl(idmap,
-								     path.dentry);
+								     &path);
 #else
 					ksmbd_vfs_set_init_posix_acl(user_ns,
-								     path.dentry);
+								     &path);
 #endif
 
 				if (test_share_config_flag(work->tcon->share_conf,
@@ -3235,7 +3235,7 @@ int smb2_open(struct ksmbd_work *work)
 #else
 								    user_ns,
 #endif
-								    path.dentry,
+								    &path,
 								    pntsd,
 								    pntsd_size);
 					kfree(pntsd);
@@ -5852,7 +5852,7 @@ static int smb2_rename(struct ksmbd_work *work,
 			goto out;
 
 		rc = ksmbd_vfs_setxattr(file_mnt_idmap(fp->filp),
-					fp->filp->f_path.dentry,
+					&fp->filp->f_path,
 					xattr_stream_name,
 					NULL, 0, 0);
 		if (rc < 0) {
@@ -5955,7 +5955,7 @@ static int smb2_rename(struct ksmbd_work *work,
 #else
 		rc = ksmbd_vfs_setxattr(user_ns,
 #endif
-					fp->filp->f_path.dentry,
+					&fp->filp->f_path,
 					xattr_stream_name,
 					NULL, 0, 0);
 		if (rc < 0) {
@@ -6177,7 +6177,7 @@ static int set_file_basic_info(struct ksmbd_file *fp,
 #else
 		rc = ksmbd_vfs_set_dos_attrib_xattr(user_ns,
 #endif
-						    filp->f_path.dentry, &da);
+						    &filp->f_path, &da);
 		if (rc)
 			ksmbd_debug(SMB,
 				    "failed to restore file attribute in EA\n");
@@ -8125,7 +8125,7 @@ static inline int fsctl_set_sparse(struct ksmbd_work *work, u64 id,
 #else
 		ret = ksmbd_vfs_set_dos_attrib_xattr(user_ns,
 #endif
-						     fp->filp->f_path.dentry, &da);
+						     &fp->filp->f_path, &da);
 		if (ret)
 			fp->f_ci->m_fattr = old_fattr;
 	}

smbacl.c

@@ -1301,7 +1301,7 @@ int smb_inherit_dacl(struct ksmbd_conn *conn,
 #else
 		ksmbd_vfs_set_sd_xattr(conn, user_ns,
 #endif
-				       path->dentry, pntsd, pntsd_size);
+				       path, pntsd, pntsd_size);
 		kfree(pntsd);
 	}
 
@@ -1551,9 +1551,9 @@ int set_info_sec(struct ksmbd_conn *conn, struct ksmbd_tree_connect *tcon,
 	newattrs.ia_mode = (inode->i_mode & ~0777) | (fattr.cf_mode & 0777);
 
 #if LINUX_VERSION_CODE >= KERNEL_VERSION(6, 3, 0)
-	ksmbd_vfs_remove_acl_xattrs(idmap, path->dentry);
+	ksmbd_vfs_remove_acl_xattrs(idmap, path);
 #else
-	ksmbd_vfs_remove_acl_xattrs(user_ns, path->dentry);
+	ksmbd_vfs_remove_acl_xattrs(user_ns, path);
 #endif
 	/* Update posix acls */
 	if (IS_ENABLED(CONFIG_FS_POSIX_ACL) && fattr.cf_dacls) {
@@ -1626,7 +1626,7 @@ int set_info_sec(struct ksmbd_conn *conn, struct ksmbd_tree_connect *tcon,
 		ksmbd_vfs_remove_sd_xattrs(user_ns, path->dentry);
 		ksmbd_vfs_set_sd_xattr(conn, user_ns,
 #endif
-				       path->dentry, pntsd, ntsd_len);
+				       path, pntsd, ntsd_len);
 	}
 
 out: