]> xenbits.xensource.com Git - arm/linux.git/commit
bcache: fix input overflow to sequential_cutoff
authorColy Li <colyli@suse.de>
Sat, 9 Feb 2019 04:53:01 +0000 (12:53 +0800)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 5 Apr 2019 20:31:34 +0000 (22:31 +0200)
commit70e8b1e0062454377cf23acda7af83f8b7d40a3c
treea3ff2f8a2bd0c31460aa7746a0caeec9c3db6047
parent77f895edd49d6d0e1703afb3944650077db6eb5b
bcache: fix input overflow to sequential_cutoff

[ Upstream commit 8c27a3953e92eb0b22dbb03d599f543a05f9574e ]

People may set sequential_cutoff of a cached device via sysfs file,
but current code does not check input value overflow. E.g. if value
4294967295 (UINT_MAX) is written to file sequential_cutoff, its value
is 4GB, but if 4294967296 (UINT_MAX + 1) is written into, its value
will be 0. This is an unexpected behavior.

This patch replaces d_strtoi_h() by sysfs_strtoul_clamp() to convert
input string to unsigned integer value, and limit its range in
[0, UINT_MAX]. Then the input overflow can be fixed.

Signed-off-by: Coly Li <colyli@suse.de>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
Signed-off-by: Sasha Levin <sashal@kernel.org>
drivers/md/bcache/sysfs.c