diff options
author | delphij <delphij@FreeBSD.org> | 2014-07-01 22:46:39 +0000 |
---|---|---|
committer | delphij <delphij@FreeBSD.org> | 2014-07-01 22:46:39 +0000 |
commit | da50471fef8c5d34cc33e5a03f99746ed6523780 (patch) | |
tree | 71fd30debbeb3dd6fc6a25056dce59cd0314fba4 /bin/mv | |
parent | ff95e6ff7f15e80345f114ebf0db450c0e226fd8 (diff) | |
download | FreeBSD-src-da50471fef8c5d34cc33e5a03f99746ed6523780.zip FreeBSD-src-da50471fef8c5d34cc33e5a03f99746ed6523780.tar.gz |
Check if fchflags() is needed by fstat'ing before and check
the results.
Reviewed by: jilles
X-MFC-With: r267977
Diffstat (limited to 'bin/mv')
-rw-r--r-- | bin/mv/mv.c | 17 |
1 files changed, 13 insertions, 4 deletions
diff --git a/bin/mv/mv.c b/bin/mv/mv.c index ee52643..3e832ac 100644 --- a/bin/mv/mv.c +++ b/bin/mv/mv.c @@ -278,6 +278,7 @@ fastcopy(const char *from, const char *to, struct stat *sbp) static char *bp = NULL; mode_t oldmode; int nread, from_fd, to_fd; + struct stat tsb; if ((from_fd = open(from, O_RDONLY, 0)) < 0) { warn("fastcopy: open() failed (from): %s", from); @@ -336,10 +337,18 @@ err: if (unlink(to)) * if the server supports flags and we were trying to *remove* flags * on a file that we copied, i.e., that we didn't create.) */ - errno = 0; - if (fchflags(to_fd, sbp->st_flags | UF_ARCHIVE)) - if (errno != EOPNOTSUPP || ((sbp->st_flags & ~UF_ARCHIVE) != 0)) - warn("%s: set flags (was: 0%07o)", to, sbp->st_flags); + if (fstat(to_fd, &tsb) == 0) { + if ((sbp->st_flags & ~UF_ARCHIVE) != + (tsb.st_flags & ~UF_ARCHIVE)) { + if (fchflags(to_fd, + sbp->st_flags | (tsb.st_flags & UF_ARCHIVE))) + if (errno != EOPNOTSUPP || + ((sbp->st_flags & ~UF_ARCHIVE) != 0)) + warn("%s: set flags (was: 0%07o)", + to, sbp->st_flags); + } + } else + warn("%s: cannot stat", to); tval[0].tv_sec = sbp->st_atime; tval[1].tv_sec = sbp->st_mtime; |