summaryrefslogtreecommitdiff
path: root/kernel/cgroup
diff options
context:
space:
mode:
authorDan Carpenter <dan.carpenter@oracle.com>2017-08-09 13:25:21 +0300
committerTejun Heo <tj@kernel.org>2017-08-11 20:33:28 +0300
commit696b98f244f77a79840bf420861c996d61c82637 (patch)
tree8085a1bfa4e633ecc489c9ea9d9ffa690b12620f /kernel/cgroup
parent3e48930cc74f0c212ee1838f89ad0ca7fcf2fea1 (diff)
downloadlinux-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.c4
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);