diff options
author | 2017-08-09 13:25:21 +0300 | |
---|---|---|
committer | 2017-08-11 10:33:28 -0700 | |
commit | 696b98f244f77a79840bf420861c996d61c82637 (patch) | |
tree | 8085a1bfa4e633ecc489c9ea9d9ffa690b12620f /kernel/cgroup | |
parent | cgroup: misc changes (diff) | |
download | linux-696b98f244f77a79840bf420861c996d61c82637.tar.gz linux-696b98f244f77a79840bf420861c996d61c82637.tar.bz2 linux-696b98f244f77a79840bf420861c996d61c82637.zip |
cgroup: remove unneeded checks
"descendants" and "depth" are declared as int, so they can't be larger
than INT_MAX. Static checkers complain and it's slightly confusing for
humans as well so let's just remove these conditions.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Tejun Heo <tj@kernel.org>
Diffstat (limited to 'kernel/cgroup')
-rw-r--r-- | kernel/cgroup/cgroup.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/kernel/cgroup/cgroup.c b/kernel/cgroup/cgroup.c index c038ccf95b5d..1591e9b20122 100644 --- a/kernel/cgroup/cgroup.c +++ b/kernel/cgroup/cgroup.c @@ -3223,7 +3223,7 @@ static ssize_t cgroup_max_descendants_write(struct kernfs_open_file *of, return ret; } - if (descendants < 0 || descendants > INT_MAX) + if (descendants < 0) return -ERANGE; cgrp = cgroup_kn_lock_live(of->kn, false); @@ -3266,7 +3266,7 @@ static ssize_t cgroup_max_depth_write(struct kernfs_open_file *of, return ret; } - if (depth < 0 || depth > INT_MAX) + if (depth < 0) return -ERANGE; cgrp = cgroup_kn_lock_live(of->kn, false); |