mirror of
https://github.com/freebsd/freebsd-src
synced 2024-07-08 20:26:31 +00:00
cp: Improved conformance when copying directories.
* When copying a directory, if the destination exists and is not a directory, we would previously emit an error message and exit. The correct behavior according to POSIX is to emit an error message and continue without descending further into the source directory. * When copying a directory, if the destination does not exist and we fail to create it, we would previously emit an error message and exit. The correct behavior according to POSIX is to emit an error message and continue. Whether to descend further into the source directory is explicitly left unspecified; GNU cp does not, which seems to me to be the safer and less surprising option, so let's not either. MFC after: 1 week Sponsored by: Klara, Inc. Reviewed by: kevans Differential Revision: https://reviews.freebsd.org/D44577
This commit is contained in:
parent
aaa1806f68
commit
dd286b0dc1
27
bin/cp/cp.c
27
bin/cp/cp.c
|
@ -506,9 +506,13 @@ copy(char *argv[], enum op type, int fts_options, struct stat *root_stat)
|
|||
* umask blocks owner writes, we fail.
|
||||
*/
|
||||
if (dne) {
|
||||
if (mkdir(to.p_path,
|
||||
curr->fts_statp->st_mode | S_IRWXU) < 0)
|
||||
err(1, "%s", to.p_path);
|
||||
mode = curr->fts_statp->st_mode | S_IRWXU;
|
||||
if (mkdir(to.p_path, mode) != 0) {
|
||||
warn("%s", to.p_path);
|
||||
(void)fts_set(ftsp, curr, FTS_SKIP);
|
||||
badcp = rval = 1;
|
||||
break;
|
||||
}
|
||||
/*
|
||||
* First DNE with a NULL root_stat is the root
|
||||
* path, so set root_stat. We can't really
|
||||
|
@ -517,14 +521,19 @@ copy(char *argv[], enum op type, int fts_options, struct stat *root_stat)
|
|||
* first directory we created and use that.
|
||||
*/
|
||||
if (root_stat == NULL &&
|
||||
stat(to.p_path, &created_root_stat) == -1) {
|
||||
err(1, "stat");
|
||||
} else if (root_stat == NULL) {
|
||||
root_stat = &created_root_stat;
|
||||
stat(to.p_path, &created_root_stat) != 0) {
|
||||
warn("%s", to.p_path);
|
||||
(void)fts_set(ftsp, curr, FTS_SKIP);
|
||||
badcp = rval = 1;
|
||||
break;
|
||||
}
|
||||
if (root_stat == NULL)
|
||||
root_stat = &created_root_stat;
|
||||
} else if (!S_ISDIR(to_stat.st_mode)) {
|
||||
errno = ENOTDIR;
|
||||
err(1, "%s", to.p_path);
|
||||
warnc(ENOTDIR, "%s", to.p_path);
|
||||
(void)fts_set(ftsp, curr, FTS_SKIP);
|
||||
badcp = rval = 1;
|
||||
break;
|
||||
}
|
||||
/*
|
||||
* Arrange to correct directory attributes later
|
||||
|
|
Loading…
Reference in New Issue
Block a user