aboutsummaryrefslogtreecommitdiff
path: root/src/xz
diff options
context:
space:
mode:
authorLasse Collin <lasse.collin@tukaani.org>2021-10-27 23:23:11 +0300
committerLasse Collin <lasse.collin@tukaani.org>2021-10-27 23:23:11 +0300
commitf2aea1d5a504b2021bf47a238390e4f12bdd518d (patch)
treeac89216ee1ba2d837aa78309d6e7f42df5f291b1 /src/xz
parentxz: Avoid fchown(2) failure. (diff)
downloadxz-f2aea1d5a504b2021bf47a238390e4f12bdd518d.tar.xz
xz: Change the coding style of the previous commit.
It isn't any better now but it's consistent with the rest of the code base.
Diffstat (limited to 'src/xz')
-rw-r--r--src/xz/file_io.c11
1 files changed, 6 insertions, 5 deletions
diff --git a/src/xz/file_io.c b/src/xz/file_io.c
index 1602443e..6fa847e2 100644
--- a/src/xz/file_io.c
+++ b/src/xz/file_io.c
@@ -369,12 +369,13 @@ io_copy_attrs(const file_pair *pair)
mode_t mode;
// With BSD semantics the new dest file may have a group that
- // does not belong to the user. If the src file has the same gid
- // nothing has to be done. Nevertheless OpenBSD fchown(2) fails
- // in this case which seems to be POSIX compliant. As there is
+ // does not belong to the user. If the src file has the same gid
+ // nothing has to be done. Nevertheless OpenBSD fchown(2) fails
+ // in this case which seems to be POSIX compliant. As there is
// nothing to do, skip the system call.
- if (pair->dest_st.st_gid != pair->src_st.st_gid &&
- fchown(pair->dest_fd, (uid_t)(-1), pair->src_st.st_gid)) {
+ if (pair->dest_st.st_gid != pair->src_st.st_gid
+ && fchown(pair->dest_fd, (uid_t)(-1),
+ pair->src_st.st_gid)) {
message_warning(_("%s: Cannot set the file group: %s"),
pair->dest_name, strerror(errno));
// We can still safely copy some additional permissions: