diff options
author | David Sterba <dsterba@suse.com> | 2017-06-21 17:43:24 +0200 |
---|---|---|
committer | David Sterba <dsterba@suse.com> | 2017-06-29 20:06:11 +0200 |
commit | 1164a9fb9c7be09c2fb7489365a82d4260c81d36 (patch) | |
tree | 334fed60267c2f73ee6c9affc13d5042f7ac96af /fs | |
parent | fbc326159a01104db6163bc8e5f0a4c0ab11864d (diff) | |
download | op-kernel-dev-1164a9fb9c7be09c2fb7489365a82d4260c81d36.zip op-kernel-dev-1164a9fb9c7be09c2fb7489365a82d4260c81d36.tar.gz |
btrfs: fix validation of XATTR_ITEM dir items
The XATTR_ITEM is a type of a directory item so we use the common
validator helper. Unlike other dir items, it can have data. The way the
name len validation is currently implemented does not reflect that. We'd
have to adjust by the data_len when comparing the read and item limits.
However, this will not work for multi-item xattr dir items.
Example from tree dump of generic/337:
item 7 key (257 XATTR_ITEM 751495445) itemoff 15667 itemsize 147
location key (0 UNKNOWN.0 0) type XATTR
transid 8 data_len 3 name_len 11
name: user.foobar
data 123
location key (0 UNKNOWN.0 0) type XATTR
transid 8 data_len 6 name_len 13
name: user.WvG1c1Td
data qwerty
location key (0 UNKNOWN.0 0) type XATTR
transid 8 data_len 5 name_len 19
name: user.J3__T_Km3dVsW_
data hello
At the point of btrfs_is_name_len_valid call we don't have access to the
data_len value of the 2nd and 3rd sub-item. So simple btrfs_dir_data_len(leaf,
di) would always return 3, although we'd need to get 6 and 5 respectively to
get the claculations right. (read_end + name_len + data_len vs item_end)
We'd have to also pass data_len externally, which is not point of the
name validation. The last check is supposed to test if there's at least
one dir item space after the one we're processing. I don't think this is
particularly useful, validation of the next item would catch that too.
So the check is removed and we don't weaken the validation. Now tests
btrfs/048, btrfs/053, generic/273 and generic/337 pass.
Signed-off-by: David Sterba <dsterba@suse.com>
Diffstat (limited to 'fs')
-rw-r--r-- | fs/btrfs/dir-item.c | 8 |
1 files changed, 0 insertions, 8 deletions
diff --git a/fs/btrfs/dir-item.c b/fs/btrfs/dir-item.c index 2b00dd7..41cb919 100644 --- a/fs/btrfs/dir-item.c +++ b/fs/btrfs/dir-item.c @@ -550,14 +550,6 @@ bool btrfs_is_name_len_valid(struct extent_buffer *leaf, int slot, goto out; } - /* - * This may be the last item in the slot - * Or same type item(s) is left between read_end and item_end - */ - if (item_end != read_end && item_end - read_end < size) { - ret = false; - goto out; - } out: if (!ret) btrfs_crit(fs_info, "invalid dir item name len: %u", |