|
 |
923a60 |
From 3b573e5c39f452f98084b3d36486369882455b72 Mon Sep 17 00:00:00 2001
|
|
|
923a60 |
From: =?UTF-8?q?Zbigniew=20J=C4=99drzejewski-Szmek?= <zbyszek@in.waw.pl>
|
|
|
923a60 |
Date: Sat, 7 Feb 2015 11:16:04 -0500
|
|
|
923a60 |
Subject: [PATCH] systemctl: check validity of PID we received
|
|
|
923a60 |
|
|
|
923a60 |
(cherry picked from commit d028e01814a405e83c400c60545785d35dba2a17)
|
|
|
923a60 |
---
|
|
|
923a60 |
src/systemctl/systemctl.c | 5 ++++-
|
|
|
923a60 |
1 file changed, 4 insertions(+), 1 deletion(-)
|
|
|
923a60 |
|
|
|
923a60 |
diff --git a/src/systemctl/systemctl.c b/src/systemctl/systemctl.c
|
|
|
923a60 |
index 6b93ec8446..3da4d3d4f1 100644
|
|
|
923a60 |
--- a/src/systemctl/systemctl.c
|
|
|
923a60 |
+++ b/src/systemctl/systemctl.c
|
|
|
923a60 |
@@ -2881,6 +2881,9 @@ static int check_inhibitors(sd_bus *bus, enum action a) {
|
|
|
923a60 |
if (!sv)
|
|
|
923a60 |
return log_oom();
|
|
|
923a60 |
|
|
|
923a60 |
+ if ((pid_t) pid < 0)
|
|
|
923a60 |
+ return log_error_errno(ERANGE, "Bad PID %"PRIu32": %m", pid);
|
|
|
923a60 |
+
|
|
|
923a60 |
if (!strv_contains(sv,
|
|
|
923a60 |
a == ACTION_HALT ||
|
|
|
923a60 |
a == ACTION_POWEROFF ||
|
|
|
923a60 |
@@ -2892,7 +2895,7 @@ static int check_inhibitors(sd_bus *bus, enum action a) {
|
|
|
923a60 |
user = uid_to_name(uid);
|
|
|
923a60 |
|
|
|
923a60 |
log_warning("Operation inhibited by \"%s\" (PID "PID_FMT" \"%s\", user %s), reason is \"%s\".",
|
|
|
923a60 |
- who, pid, strna(comm), strna(user), why);
|
|
|
923a60 |
+ who, (pid_t) pid, strna(comm), strna(user), why);
|
|
|
923a60 |
|
|
|
923a60 |
c++;
|
|
|
923a60 |
}
|