alsactl: fix error handling for sched_setscheduler() call

As man page says: "If successful, the sched_setparam() function shall return zero."

Without update I got this output in the syslog (journalctl):
abr 16 09:25:30 mypc alsactl[1652]: alsactl 1.2.2 daemon started
abr 16 09:25:30 mypc alsactl[1652]: /usr/bin/alsactl: do_nice:165sched_setparam failed: No such file or directory

If sched_setscheduler() returns 0, so it means that the call was successful.

Signed-off-by: Oscar Megía <megia.oscar@gmail.com>
Signed-off-by: Jaroslav Kysela <perex@perex.cz>

diff --git a/alsactl/alsactl.c b/alsactl/alsactl.c
index dfb1db7f4a8312dc58d9189b8b79725a36b07a98..60d39459dedfc545d067f49285c6578dbcb76e57 100644 (file)
--- a/alsactl/alsactl.c
+++ b/alsactl/alsactl.c
@@ -161,7 +161,7 @@ static void do_nice(int use_nice, int sched_idle)
        if (sched_idle) {
                if (sched_getparam(0, &sched_param) >= 0) {
                        sched_param.sched_priority = 0;
-                       if (!sched_setscheduler(0, SCHED_IDLE, &sched_param))
+                       if (sched_setscheduler(0, SCHED_IDLE, &sched_param) < 0)
                                error("sched_setparam failed: %s", strerror(errno));
                } else {
                        error("sched_getparam failed: %s", strerror(errno));