From 56f64d95763a799ba4475daf44d8e9f72a1bd474 Mon Sep 17 00:00:00 2001 From: Michal Schmidt Date: Fri, 28 Nov 2014 19:29:59 +0100 Subject: treewide: use log_*_errno whenever %m is in the format string If the format string contains %m, clearly errno must have a meaningful value, so we might as well use log_*_errno to have ERRNO= logged. Using: find . -name '*.[ch]' | xargs sed -r -i -e \ 's/log_(debug|info|notice|warning|error|emergency)\((".*%m.*")/log_\1_errno(errno, \2/' Plus some whitespace, linewrap, and indent adjustments. --- src/run/run.c | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) (limited to 'src/run') diff --git a/src/run/run.c b/src/run/run.c index b96517a99a..d61f8db350 100644 --- a/src/run/run.c +++ b/src/run/run.c @@ -489,7 +489,7 @@ static int start_transient_scope( if (arg_nice_set) { if (setpriority(PRIO_PROCESS, 0, arg_nice) < 0) { - log_error("Failed to set nice level: %m"); + log_error_errno(errno, "Failed to set nice level: %m"); return -errno; } } @@ -502,7 +502,7 @@ static int start_transient_scope( return log_error_errno(r, "Failed to resolve group %s: %m", arg_exec_group); if (setresgid(gid, gid, gid) < 0) { - log_error("Failed to change GID to " GID_FMT ": %m", gid); + log_error_errno(errno, "Failed to change GID to " GID_FMT ": %m", gid); return -errno; } } @@ -534,13 +534,13 @@ static int start_transient_scope( if (!arg_exec_group) { if (setresgid(gid, gid, gid) < 0) { - log_error("Failed to change GID to " GID_FMT ": %m", gid); + log_error_errno(errno, "Failed to change GID to " GID_FMT ": %m", gid); return -errno; } } if (setresuid(uid, uid, uid) < 0) { - log_error("Failed to change UID to " UID_FMT ": %m", uid); + log_error_errno(errno, "Failed to change UID to " UID_FMT ": %m", uid); return -errno; } } @@ -552,7 +552,7 @@ static int start_transient_scope( log_info("Running as unit %s.", name); execvpe(argv[0], argv, env); - log_error("Failed to execute: %m"); + log_error_errno(errno, "Failed to execute: %m"); return -errno; } -- cgit v1.2.3-54-g00ecf