c/mirror_zfs
1
0
Fork 0
mirror of https://git.proxmox.com/git/mirror_zfs.git synced 2024-12-26 19:19:32 +03:00
Code Issues Packages Projects Releases Wiki Activity

Ensure we call fput when cloning fails due to different devices.

Browse Source 
Right now, zpl_ioctl_ficlone and zpl_ioctl_ficlonerange do not call
put on the src fd if the source and destination are on two different
devices.  This leaves the source file held open in this case.

Reviewed-by: Kay Pedersen <mail@mkwg.de>
Reviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>
Reviewed-by: Alexander Motin <mav@FreeBSD.org>
Signed-off-by: Daniel Berlin <dberlin@dberlin.org>
Closes #15386
This commit is contained in:
Daniel Berlin 2023-10-10 14:04:32 -04:00 committed by Brian Behlendorf
parent 75a7740574
commit 810fc49a3e
1 changed files with 6 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
module/os/linux/zfs/zpl_file_range.c
View File

@ -202,8 +202,10 @@ zpl_ioctl_ficlone(struct file *dst_file, void *arg)
if (src_file == NULL) if (src_file == NULL)
return (-EBADF); return (-EBADF);
if (dst_file->f_op != src_file->f_op) if (dst_file->f_op != src_file->f_op) {
fput(src_file);
return (-EXDEV); return (-EXDEV);
}
size_t len = i_size_read(file_inode(src_file)); size_t len = i_size_read(file_inode(src_file));
@ -237,8 +239,10 @@ zpl_ioctl_ficlonerange(struct file *dst_file, void __user *arg)
if (src_file == NULL) if (src_file == NULL)
return (-EBADF); return (-EBADF);
if (dst_file->f_op != src_file->f_op) if (dst_file->f_op != src_file->f_op) {
fput(src_file);
return (-EXDEV); return (-EXDEV);
}
size_t len = fcr.fcr_src_length; size_t len = fcr.fcr_src_length;
if (len == 0) if (len == 0)