diff options
| author | Lasse Collin <lasse.collin@tukaani.org> | 2021-10-27 23:23:11 +0300 |
|---|---|---|
| committer | Lasse Collin <lasse.collin@tukaani.org> | 2021-10-27 23:23:11 +0300 |
| commit | f2aea1d5a504b2021bf47a238390e4f12bdd518d (patch) | |
| tree | ac89216ee1ba2d837aa78309d6e7f42df5f291b1 /src | |
| parent | xz: Avoid fchown(2) failure. (diff) | |
| download | xz-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')
| -rw-r--r-- | src/xz/file_io.c | 11 |
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: |