summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDenys Vlasenko2019-11-09 17:32:43 +0100
committerDenys Vlasenko2019-11-09 17:32:43 +0100
commitb230fdfa9da2f31e6cc90f90579c6d020c770f86 (patch)
tree33614c264bc528461c429f36980a8bca626107e6
parenta82fb1b9d8a046e01dbad33bb57d064815d60e5c (diff)
downloadbusybox-b230fdfa9da2f31e6cc90f90579c6d020c770f86.zip
busybox-b230fdfa9da2f31e6cc90f90579c6d020c770f86.tar.gz
taskset: tighten the check for stride values
function old new delta taskset_main 986 987 +1 Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
-rw-r--r--util-linux/taskset.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/util-linux/taskset.c b/util-linux/taskset.c
index df1bc0a..b542f8c 100644
--- a/util-linux/taskset.c
+++ b/util-linux/taskset.c
@@ -149,8 +149,8 @@ static void parse_cpulist(ul *mask, unsigned max, char *s)
if ((*s != ',' && *s != '\0')
|| bit > end
|| end == UINT_MAX /* bb_strtou returns this on malformed / ERANGE numbers */
- || stride == 0
- || stride == UINT_MAX
+ || (stride - 1) > (UINT_MAX / 4)
+ /* disallow 0, malformed input, and too large stride prone to overflows */
) {
bb_error_msg_and_die("bad affinity '%s'", aff);
}