cifs: Fix xid leak in cifs_copy_file_range()
[ Upstream commit 9a97df404a402fe1174d2d1119f87ff2a0ca2fe9 ]
If the file is used by swap, before return -EOPNOTSUPP, should
free the xid, otherwise, the xid will be leaked.
Fixes: 4e8aea30f7
("smb3: enable swap on SMB3 mounts")
Reviewed-by: Paulo Alcantara (SUSE) <pc@cjr.nz>
Signed-off-by: Zhang Xiaoxu <zhangxiaoxu5@huawei.com>
Signed-off-by: Steve French <stfrench@microsoft.com>
Signed-off-by: Sasha Levin <sashal@kernel.org>
This commit is contained in:

committed by
Greg Kroah-Hartman

parent
05cc22c008
commit
bf49d4fe4a
@@ -1221,8 +1221,11 @@ static ssize_t cifs_copy_file_range(struct file *src_file, loff_t off,
|
|||||||
ssize_t rc;
|
ssize_t rc;
|
||||||
struct cifsFileInfo *cfile = dst_file->private_data;
|
struct cifsFileInfo *cfile = dst_file->private_data;
|
||||||
|
|
||||||
if (cfile->swapfile)
|
if (cfile->swapfile) {
|
||||||
return -EOPNOTSUPP;
|
rc = -EOPNOTSUPP;
|
||||||
|
free_xid(xid);
|
||||||
|
return rc;
|
||||||
|
}
|
||||||
|
|
||||||
rc = cifs_file_copychunk_range(xid, src_file, off, dst_file, destoff,
|
rc = cifs_file_copychunk_range(xid, src_file, off, dst_file, destoff,
|
||||||
len, flags);
|
len, flags);
|
||||||
|
Reference in New Issue
Block a user