diff options
| -rw-r--r-- | src/basic/fs-util.c | 17 | ||||
| -rw-r--r-- | src/tmpfiles/tmpfiles.c | 11 |
2 files changed, 18 insertions, 10 deletions
diff --git a/src/basic/fs-util.c b/src/basic/fs-util.c index 34a2260783..461bc04e50 100644 --- a/src/basic/fs-util.c +++ b/src/basic/fs-util.c @@ -229,6 +229,7 @@ int chmod_and_chown(const char *path, mode_t mode, uid_t uid, gid_t gid) { int fchmod_and_chown(int fd, mode_t mode, uid_t uid, gid_t gid) { bool do_chown, do_chmod; struct stat st; + int r; /* Change ownership and access mode of the specified fd. Tries to do so safely, ensuring that at no * point in time the access mode is above the old access mode under the old ownership or the new @@ -259,18 +260,22 @@ int fchmod_and_chown(int fd, mode_t mode, uid_t uid, gid_t gid) { if (do_chown && do_chmod) { mode_t minimal = st.st_mode & mode; /* the subset of the old and the new mask */ - if (((minimal ^ st.st_mode) & 07777) != 0) - if (fchmod_opath(fd, minimal & 07777) < 0) - return -errno; + if (((minimal ^ st.st_mode) & 07777) != 0) { + r = fchmod_opath(fd, minimal & 07777); + if (r < 0) + return r; + } } if (do_chown) if (fchownat(fd, "", uid, gid, AT_EMPTY_PATH) < 0) return -errno; - if (do_chmod) - if (fchmod_opath(fd, mode & 07777) < 0) - return -errno; + if (do_chmod) { + r = fchmod_opath(fd, mode & 07777); + if (r < 0) + return r; + } return do_chown || do_chmod; } diff --git a/src/tmpfiles/tmpfiles.c b/src/tmpfiles/tmpfiles.c index 2404e36bf2..2db3773fcb 100644 --- a/src/tmpfiles/tmpfiles.c +++ b/src/tmpfiles/tmpfiles.c @@ -773,6 +773,7 @@ static int fd_set_perms(Item *i, int fd, const char *path, const struct stat *st struct stat stbuf; mode_t new_mode; bool do_chown; + int r; assert(i); assert(fd); @@ -818,8 +819,9 @@ static int fd_set_perms(Item *i, int fd, const char *path, const struct stat *st log_debug("\"%s\" matches temporary mode %o already.", path, m); else { log_debug("Temporarily changing \"%s\" to mode %o.", path, m); - if (fchmod_opath(fd, m) < 0) - return log_error_errno(errno, "fchmod() of %s failed: %m", path); + r = fchmod_opath(fd, m); + if (r < 0) + return log_error_errno(r, "fchmod() of %s failed: %m", path); } } } @@ -850,8 +852,9 @@ static int fd_set_perms(Item *i, int fd, const char *path, const struct stat *st log_debug("\"%s\" matches mode %o already.", path, new_mode); else { log_debug("Changing \"%s\" to mode %o.", path, new_mode); - if (fchmod_opath(fd, new_mode) < 0) - return log_error_errno(errno, "fchmod() of %s failed: %m", path); + r = fchmod_opath(fd, new_mode); + if (r < 0) + return log_error_errno(r, "fchmod() of %s failed: %m", path); } } } |