summaryrefslogtreecommitdiff
path: root/include/linux
diff options
context:
space:
mode:
authorBaokun Li <libaokun1@huawei.com>2024-03-19 19:33:17 +0800
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2024-06-27 13:49:10 +0200
commit82d5a4ba46b3c1a2671f2eb25a866306c76884d9 (patch)
tree293df0bc14e742706c33f9110acd16c47910e958 /include/linux
parentdb3d39869e53bad763bf3cf2c4c5a6c7d9e32d6b (diff)
ext4: avoid overflow when setting values via sysfs
commit 9e8e819f8f272c4e5dcd0bd6c7450e36481ed139 upstream. When setting values of type unsigned int through sysfs, we use kstrtoul() to parse it and then truncate part of it as the final set value, when the set value is greater than UINT_MAX, the set value will not match what we see because of the truncation. As follows: $ echo 4294967296 > /sys/fs/ext4/sda/mb_max_linear_groups $ cat /sys/fs/ext4/sda/mb_max_linear_groups 0 So we use kstrtouint() to parse the attr_pointer_ui type to avoid the inconsistency described above. In addition, a judgment is added to avoid setting s_resv_clusters less than 0. Signed-off-by: Baokun Li <libaokun1@huawei.com> Reviewed-by: Jan Kara <jack@suse.cz> Link: https://lore.kernel.org/r/20240319113325.3110393-2-libaokun1@huawei.com Signed-off-by: Theodore Ts'o <tytso@mit.edu> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'include/linux')
0 files changed, 0 insertions, 0 deletions