]> git.baikalelectronics.ru Git - kernel.git/commit
btrfs: add missing check for nocow and compression inode flags
authorDavid Sterba <dsterba@suse.com>
Fri, 10 Jul 2020 07:49:56 +0000 (09:49 +0200)
committerDavid Sterba <dsterba@suse.com>
Mon, 27 Jul 2020 10:55:44 +0000 (12:55 +0200)
commit2c704c98ff512e2cb8a511d5173968cf7ecb24b7
treedea02c9a4f8a93ffd131198b7235fb67bb41b838
parent63a6a634763ac7efdb6551d8fef6f8dd46372408
btrfs: add missing check for nocow and compression inode flags

User Forza reported on IRC that some invalid combinations of file
attributes are accepted by chattr.

The NODATACOW and compression file flags/attributes are mutually
exclusive, but they could be set by 'chattr +c +C' on an empty file. The
nodatacow will be in effect because it's checked first in
btrfs_run_delalloc_range.

Extend the flag validation to catch the following cases:

  - input flags are conflicting
  - old and new flags are conflicting
  - initialize the local variable with inode flags after inode ls locked

Inode attributes take precedence over mount options and are an
independent setting.

Nocompress would be a no-op with nodatacow, but we don't want to mix
any compression-related options with nodatacow.

CC: stable@vger.kernel.org # 4.4+
Signed-off-by: David Sterba <dsterba@suse.com>
fs/btrfs/ioctl.c