From 5e9c522eeade3a2cb1f5679db84d8164b405860d Mon Sep 17 00:00:00 2001 From: Craig Small Date: Sat, 2 Jul 2016 14:49:51 +1000 Subject: [PATCH] kill: report error if cannot kill process Shell kill would report a problem if you tried to kill a process while procps kill was silent. This meant it looked like kill worked when it actually failed. References: commit 07642b8ea6d4d61518808de2e5e7f713a2f36618 https://bugs.debian.org/733172 --- NEWS | 1 + kill.c | 1 + 2 files changed, 2 insertions(+) diff --git a/NEWS b/NEWS index d04b42e1..ef62e0f2 100644 --- a/NEWS +++ b/NEWS @@ -5,6 +5,7 @@ procps-ng-NEXT * ps: Fallback to attr/current for context Debian #786956 * tests: Conditionally add prctl Debian #816237 * pidof: check cmd if space in argv0. GitLab #4 + * kill: report error if cannot kill process #733172 procps-ng-3.3.11 ---------------- diff --git a/kill.c b/kill.c index 315422b8..39386df2 100644 --- a/kill.c +++ b/kill.c @@ -132,6 +132,7 @@ int main(int argc, char **argv) pid = strtol_or_err(argv[i], _("failed to parse argument")); if (!kill((pid_t) pid, signo)) continue; + error(0, errno, "(%d)", pid); exitvalue = EXIT_FAILURE; continue; }