btrfs: defrag: reject unknown flags of btrfs_ioctl_defrag_range_args
commit 173431b274a9a54fc10b273b46e67f46bcf62d2e upstream. Add extra sanity check for btrfs_ioctl_defrag_range_args::flags. This is not really to enhance fuzzing tests, but as a preparation for future expansion on btrfs_ioctl_defrag_range_args. In the future we're going to add new members, allowing more fine tuning for btrfs defrag. Without the -ENONOTSUPP error, there would be no way to detect if the kernel supports those new defrag features. CC: stable@vger.kernel.org # 4.14+ Reviewed-by: Filipe Manana <fdmanana@suse.com> Signed-off-by: Qu Wenruo <wqu@suse.com> Reviewed-by: David Sterba <dsterba@suse.com> Signed-off-by: David Sterba <dsterba@suse.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
parent
86aff7c5f7
commit
52e02f26d0
|
@ -3500,6 +3500,10 @@ static int btrfs_ioctl_defrag(struct file *file, void __user *argp)
|
||||||
ret = -EFAULT;
|
ret = -EFAULT;
|
||||||
goto out;
|
goto out;
|
||||||
}
|
}
|
||||||
|
if (range.flags & ~BTRFS_DEFRAG_RANGE_FLAGS_SUPP) {
|
||||||
|
ret = -EOPNOTSUPP;
|
||||||
|
goto out;
|
||||||
|
}
|
||||||
/* compression requires us to start the IO */
|
/* compression requires us to start the IO */
|
||||||
if ((range.flags & BTRFS_DEFRAG_RANGE_COMPRESS)) {
|
if ((range.flags & BTRFS_DEFRAG_RANGE_COMPRESS)) {
|
||||||
range.flags |= BTRFS_DEFRAG_RANGE_START_IO;
|
range.flags |= BTRFS_DEFRAG_RANGE_START_IO;
|
||||||
|
|
|
@ -585,6 +585,9 @@ struct btrfs_ioctl_clone_range_args {
|
||||||
*/
|
*/
|
||||||
#define BTRFS_DEFRAG_RANGE_COMPRESS 1
|
#define BTRFS_DEFRAG_RANGE_COMPRESS 1
|
||||||
#define BTRFS_DEFRAG_RANGE_START_IO 2
|
#define BTRFS_DEFRAG_RANGE_START_IO 2
|
||||||
|
#define BTRFS_DEFRAG_RANGE_FLAGS_SUPP (BTRFS_DEFRAG_RANGE_COMPRESS | \
|
||||||
|
BTRFS_DEFRAG_RANGE_START_IO)
|
||||||
|
|
||||||
struct btrfs_ioctl_defrag_range_args {
|
struct btrfs_ioctl_defrag_range_args {
|
||||||
/* start of the defrag operation */
|
/* start of the defrag operation */
|
||||||
__u64 start;
|
__u64 start;
|
||||||
|
|
Loading…
Reference in New Issue