diff options
-rw-r--r-- | NEWS | 4 | ||||
-rw-r--r-- | src/cp.c | 16 | ||||
-rwxr-xr-x | tests/cp/cp-parents.sh | 6 |
3 files changed, 21 insertions, 5 deletions
@@ -4,6 +4,10 @@ GNU coreutils NEWS -*- outline -*- ** Bug fixes + cp --parents again succeeds to preserve mode for absolute directories. + Previously it would have failed with a "No such file or directory" error. + [bug introduced in coreutils-9.1] + cksum again diagnoses read errors in its default CRC32 mode. [bug introduced in coreutils-9.0] @@ -296,15 +296,19 @@ regular file.\n\ when done. */ static bool -re_protect (char const *const_dst_name, int dst_dirfd, char const *dst_relname, +re_protect (char const *const_dst_name, int dst_dirfd, char const *dst_fullname, struct dir_attr *attr_list, const struct cp_options *x) { struct dir_attr *p; char *dst_name; /* A copy of CONST_DST_NAME we can change. */ - char *src_name; /* The source name in 'dst_name'. */ + char *src_name; /* The relative source name in 'dst_name'. */ + char *full_src_name; /* The full source name in 'dst_name'. */ ASSIGN_STRDUPA (dst_name, const_dst_name); - src_name = dst_name + (dst_relname - const_dst_name); + full_src_name = dst_name + (dst_fullname - const_dst_name); + src_name = full_src_name; + while (*src_name == '/') + src_name++; for (p = attr_list; p; p = p->next) { @@ -347,7 +351,7 @@ re_protect (char const *const_dst_name, int dst_dirfd, char const *dst_relname, if (x->preserve_mode) { - if (copy_acl (src_name, -1, dst_name, -1, p->st.st_mode) != 0) + if (copy_acl (full_src_name, -1, dst_name, -1, p->st.st_mode) != 0) return false; } else if (p->restore_mode) @@ -687,6 +691,7 @@ do_copy (int n_files, char **file, char const *target_directory, bool parent_exists = true; /* True if dir_name (dst_name) exists. */ struct dir_attr *attr_list; char *arg_in_concat = NULL; + char *full_arg_in_concat = NULL; char *arg = file[i]; /* Trailing slashes are meaningful (i.e., maybe worth preserving) @@ -719,6 +724,7 @@ do_copy (int n_files, char **file, char const *target_directory, (x->verbose ? "%s -> %s\n" : NULL), &attr_list, &new_dst, x)); + full_arg_in_concat = arg_in_concat; while (*arg_in_concat == '/') arg_in_concat++; } @@ -747,7 +753,7 @@ do_copy (int n_files, char **file, char const *target_directory, new_dst, x, ©_into_self, NULL); if (parents_option) - ok &= re_protect (dst_name, target_dirfd, arg_in_concat, + ok &= re_protect (dst_name, target_dirfd, full_arg_in_concat, attr_list, x); } diff --git a/tests/cp/cp-parents.sh b/tests/cp/cp-parents.sh index 20963eace..9437504c2 100755 --- a/tests/cp/cp-parents.sh +++ b/tests/cp/cp-parents.sh @@ -66,4 +66,10 @@ p=$(ls -ld g/sym/b/c|cut -b-10); case $p in drwxr-xr-x);; *) fail=1;; esac cp --parents --no-preserve=mode np/b/file np_dest/ || fail=1 p=$(ls -ld np_dest/np|cut -b-10); case $p in drwxr-xr-x);; *) fail=1;; esac +# coreutils 9.1-9.3 inclusive would fail to copy acls for absolute dirs +mkdir dest || framework_failure_ +if test -f /bin/ls; then + cp -t dest --parents -p /bin/ls || fail=1 +fi + Exit $fail |