diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2017-08-09 13:25:21 +0300 |
|---|---|---|
| committer | Tejun Heo <tj@kernel.org> | 2017-08-11 20:33:28 +0300 |
| commit | 696b98f244f77a79840bf420861c996d61c82637 (patch) | |
| tree | 8085a1bfa4e633ecc489c9ea9d9ffa690b12620f /kernel/cgroup | |
| parent | 3e48930cc74f0c212ee1838f89ad0ca7fcf2fea1 (diff) | |
| download | linux-696b98f244f77a79840bf420861c996d61c82637.tar.xz | |
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); |