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

diff --git a/NEWS b/NEWS
index d04b42e1c126a554522cca8600d699ccfbc30ede..ef62e0f2412e16eb3fac4ff7711b39c79763ab3b 100644 (file)
--- 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 315422b8a6e03c39379371d4ac42487ef21c4f43..39386df26b3d7ca24aed3d881d15d1bcd2250b6c 100644 (file)
--- 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;
     }