diff options
| author | Sami Kerola | 2016-04-03 10:35:29 +0200 |
|---|---|---|
| committer | Karel Zak | 2016-04-13 12:29:16 +0200 |
| commit | 2e31d1c3a5d729c4a9062e37210a1c01623327d6 (patch) | |
| tree | 1f0d80d4a491bd902bf936890d56276f4c3d47b9 /schedutils | |
| parent | colcrt: avoid the command getting hung [afl] (diff) | |
| download | kernel-qcow2-util-linux-2e31d1c3a5d729c4a9062e37210a1c01623327d6.tar.gz kernel-qcow2-util-linux-2e31d1c3a5d729c4a9062e37210a1c01623327d6.tar.xz kernel-qcow2-util-linux-2e31d1c3a5d729c4a9062e37210a1c01623327d6.zip | |
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 <kerolasa@iki.fi>
Diffstat (limited to 'schedutils')
| -rw-r--r-- | schedutils/chrt.c | 6 |
1 files changed, 5 insertions, 1 deletions
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) |