diff options
author | Michal Schmidt <mschmidt@redhat.com> | 2014-11-28 19:29:59 +0100 |
---|---|---|
committer | Michal Schmidt <mschmidt@redhat.com> | 2014-11-28 19:49:27 +0100 |
commit | 56f64d95763a799ba4475daf44d8e9f72a1bd474 (patch) | |
tree | 4c38253c718dc1972b811fa7c01ebfa3c2b7776c /src/run/run.c | |
parent | 895b3a7b44fe7ca2f260986be2a877ff56a72718 (diff) |
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.
Diffstat (limited to 'src/run/run.c')
-rw-r--r-- | src/run/run.c | 10 |
1 files changed, 5 insertions, 5 deletions
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; } |