diff options
author | Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> | 2018-04-24 13:44:09 +0200 |
---|---|---|
committer | Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> | 2018-04-24 14:10:27 +0200 |
commit | b1c05b98bfbfd967ddeaad1c7f827153989013b8 (patch) | |
tree | c3a177237ccf0efc94e8ff7467c1eccf66390e2b /src/basic/capability-util.c | |
parent | 9169e4c7ba0cd2443fd4760b43715cb6d991c784 (diff) | |
download | systemd-b1c05b98bfbfd967ddeaad1c7f827153989013b8.tar.gz |
tree-wide: avoid assignment of r just to use in a comparison
This changes
r = ...;
if (r < 0)
to
if (... < 0)
when r will not be used again.
Diffstat (limited to 'src/basic/capability-util.c')
-rw-r--r-- | src/basic/capability-util.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/src/basic/capability-util.c b/src/basic/capability-util.c index 233d18a854..645cd322a7 100644 --- a/src/basic/capability-util.c +++ b/src/basic/capability-util.c @@ -301,8 +301,7 @@ int drop_privileges(uid_t uid, gid_t gid, uint64_t keep_capabilities) { if (prctl(PR_SET_KEEPCAPS, 1) < 0) return log_error_errno(errno, "Failed to enable keep capabilities flag: %m"); - r = setresuid(uid, uid, uid); - if (r < 0) + if (setresuid(uid, uid, uid) < 0) return log_error_errno(errno, "Failed to change user ID: %m"); if (prctl(PR_SET_KEEPCAPS, 0) < 0) |