diff options
author | Rob Landley <rob@landley.net> | 2013-01-16 06:57:44 -0600 |
---|---|---|
committer | Rob Landley <rob@landley.net> | 2013-01-16 06:57:44 -0600 |
commit | 6be5ac6a9287f4c48daf60d36eae44f04d7f9a80 (patch) | |
tree | a05685219498613a76a3f82517f38612ebbb42c7 /toys/posix/cp.c | |
parent | ae9243aa425458eda4e3e0b8bc7dd8f19fc8113a (diff) | |
download | toybox-6be5ac6a9287f4c48daf60d36eae44f04d7f9a80.tar.gz |
Make "sudo cp -rp /dev/null blah" work. Still not happy with it, fchmodat(AT_SYMLINK_NOFOLLOW) doesn't work (there's a glibc bug open for this. It's really a missing kernel syscall, but glibc fails without ever making any syscall if you feed it that flag, which isn't helpful).
Diffstat (limited to 'toys/posix/cp.c')
-rw-r--r-- | toys/posix/cp.c | 29 |
1 files changed, 21 insertions, 8 deletions
diff --git a/toys/posix/cp.c b/toys/posix/cp.c index d9a0d29c..55f4fe71 100644 --- a/toys/posix/cp.c +++ b/toys/posix/cp.c @@ -97,6 +97,8 @@ int cp_node(struct dirtree *try) // Loop for -f retry after unlink do { + // directory, hardlink, symlink, mknod (char, block, fifo, socket), file + // Copy directory if (S_ISDIR(try->st.st_mode)) { @@ -116,7 +118,7 @@ int cp_node(struct dirtree *try) // that what we open _is_ a directory rather than something else. if (!mkdirat(cfd, catch, try->st.st_mode | 0200) || errno == EEXIST) - if (-1 != (try->extra = openat(cfd, catch, 0))) + if (-1 != (try->extra = openat(cfd, catch, O_NOFOLLOW))) if (!fstat(try->extra, &st2)) if (S_ISDIR(st2.st_mode)) return DIRTREE_COMEAGAIN; @@ -135,9 +137,10 @@ int cp_node(struct dirtree *try) if (i < 1 || i >= sizeof(toybuf)) break; else if (!symlinkat(toybuf, cfd, catch)) err = 0; // block, char, fifo, socket - } else if (!mknodat(cfd, catch, try->st.st_mode, try->st.st_dev)) - if (!(flags & (FLAG_a|FLAG_p)) - || -1 != (fdout = openat(cfd, catch, O_RDONLY))) err = 0; + } else if (!mknodat(cfd, catch, try->st.st_mode, try->st.st_rdev)) { + err = 0; + if (flags & (FLAG_a|FLAG_p)) fdout = AT_FDCWD; + } // Copy contents of file. } else { @@ -165,14 +168,24 @@ int cp_node(struct dirtree *try) // Inability to set these isn't fatal, some require root access. - fchown(fdout, try->st.st_uid, try->st.st_gid); times[0] = try->st.st_atim; times[1] = try->st.st_mtim; - futimens(fdout, times); - fchmod(fdout, try->st.st_mode); + + // If we can't get a filehandle to the actual object, use racy functions + if (fdout == AT_FDCWD) { + if (fchownat(cfd, catch, try->st.st_uid, try->st.st_gid, + AT_SYMLINK_NOFOLLOW) + || utimensat(cfd, catch, times, AT_SYMLINK_NOFOLLOW) + || fchmodat(cfd, catch, try->st.st_mode&07777, 0)) + err = "%s"; + } else { + if (fchown(fdout, try->st.st_uid, try->st.st_gid) + || futimens(fdout, times) || fchmod(fdout, try->st.st_mode&07777)) + err = "%s"; + } } - xclose(fdout); + if (fdout != AT_FDCWD) xclose(fdout); } if (err) perror_msg(err, catch); |