From 2e31d1c3a5d729c4a9062e37210a1c01623327d6 Mon Sep 17 00:00:00 2001 From: Sami Kerola Date: Sun, 3 Apr 2016 09:35:29 +0100 Subject: chrt: validate priority before trying to use it Earlier message: $ chrt -i 1 ls chrt: failed to set pid 0's policy: Invalid argument basically told 'something failed', while the new one tries to be more helpful. $ chrt -i 1 ls chrt: unsupported priority value for the policy: 1: see --max for valid range Addresses: https://bugs.debian.org/791707 Signed-off-by: Sami Kerola --- schedutils/chrt.c | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/schedutils/chrt.c b/schedutils/chrt.c index 64a58584f..202ce2526 100644 --- a/schedutils/chrt.c +++ b/schedutils/chrt.c @@ -504,7 +504,11 @@ int main(int argc, char **argv) #endif if (ctl->pid == -1) ctl->pid = 0; - + if (ctl->priority < sched_get_priority_min(ctl->policy) || + sched_get_priority_max(ctl->policy) < ctl->priority) + errx(EXIT_FAILURE, + _("unsupported priority value for the policy: %d: see --max for valid range"), + ctl->priority); set_sched(ctl); if (ctl->verbose) -- cgit v1.2.3-55-g7522