Commit c0d3a7bd authored Jun 23, 2017 by Chao Yu Committed by Greg Kroah-Hartman Jul 15, 2017

ext4: check return value of kstrtoull correctly in reserved_clusters_store



commit 1ea1516fbbab2b30bf98c534ecaacba579a35208 upstream.

kstrtoull returns 0 on success, however, in reserved_clusters_store we
will return -EINVAL if kstrtoull returns 0, it makes us fail to update
reserved_clusters value through sysfs.

Fixes: 76d33bca
Signed-off-by: Chao Yu <yuchao0@huawei.com>
Signed-off-by: Miao Xie <miaoxie@huawei.com>
Signed-off-by: Theodore Ts'o <tytso@mit.edu>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

parent 71698654

fs/ext4/sysfs.c

+1 −1

Original line number	Diff line number	Diff line
		@@ -100,7 +100,7 @@ static ssize_t reserved_clusters_store(struct ext4_attr *a,
		int ret;

		ret = kstrtoull(skip_spaces(buf), 0, &val);
		if (!ret \|\| val >= clusters)
		if (ret \|\| val >= clusters)
		return -EINVAL;

		atomic64_set(&sbi->s_resv_clusters, val);