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.
This commit is contained in:
parent
892b16cc28
commit
f2aea1d5a5
|
@ -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:
|
||||
|
|
Loading…
Reference in New Issue