emo/busybox
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

cp: add support for -s, -l. Fix free(nonmalloc) bug.

Browse Source 
Add doc on POSIX's rules on -i and -f (insane!).
ln: make "ln dangling_symlink new_link" work.
This commit is contained in:
Denis Vlasenko 2006-10-21 23:40:20 +00:00
parent 8d73c35916
commit f24e1f40e0
6 changed files with 190 additions and 161 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

72
coreutils/cp.c
View File

@ -7,8 +7,6 @@
* Licensed under GPL v2 or later, see file LICENSE in this tarball for details. * Licensed under GPL v2 or later, see file LICENSE in this tarball for details.
*/ */
/* BB_AUDIT SUSv3 defects - unsupported options -H, -L, and -P. */
/* BB_AUDIT GNU defects - only extension options supported are -a and -d. */
/* http://www.opengroup.org/onlinepubs/007904975/utilities/cp.html */ /* http://www.opengroup.org/onlinepubs/007904975/utilities/cp.html */
/* Mar 16, 2003 Manuel Novoa III (mjn3@codepoet.org) /* Mar 16, 2003 Manuel Novoa III (mjn3@codepoet.org)
@ -16,15 +14,6 @@
* Size reduction. * Size reduction.
*/ */
#include <sys/types.h>
#include <sys/stat.h>
#include <unistd.h>
#include <fcntl.h>
#include <utime.h>
#include <errno.h>
#include <dirent.h>
#include <stdlib.h>
#include <assert.h>
#include "busybox.h" #include "busybox.h"
#include "libcoreutils/coreutils.h" #include "libcoreutils/coreutils.h"
@ -38,31 +27,26 @@ int cp_main(int argc, char **argv)
int d_flags; int d_flags;
int flags; int flags;
int status = 0; int status = 0;
enum {
OPT_a = 1 << (sizeof(FILEUTILS_CP_OPTSTR)-1),
OPT_r = 1 << (sizeof(FILEUTILS_CP_OPTSTR)),
OPT_P = 1 << (sizeof(FILEUTILS_CP_OPTSTR)+1),
OPT_H = 1 << (sizeof(FILEUTILS_CP_OPTSTR)+2),
OPT_L = 1 << (sizeof(FILEUTILS_CP_OPTSTR)+3),
};
flags = getopt32(argc, argv, "pdRfiarPHL"); // Soft- and hardlinking don't mix
// -P and -d are the same (-P is POSIX, -d is GNU)
if (flags & 32) { // -r and -R are the same
flags |= (FILEUTILS_PRESERVE_STATUS | FILEUTILS_RECUR | FILEUTILS_DEREFERENCE); // -a = -pdR
} opt_complementary = "?:l--s:s--l:Pd:rR:apdR";
if (flags & 64) { flags = getopt32(argc, argv, FILEUTILS_CP_OPTSTR "arPHL");
/* Make -r a synonym for -R,
* -r was marked as obsolete in SUSv3, but is included for compatibility
*/
flags |= FILEUTILS_RECUR;
}
if (flags & 128) {
/* Make -P a synonym for -d,
* -d is the GNU option while -P is the POSIX 2003 option
*/
flags |= FILEUTILS_DEREFERENCE;
}
/* Default behavior of cp is to dereference, so we don't have to do /* Default behavior of cp is to dereference, so we don't have to do
* anything special when we are given -L. * anything special when we are given -L.
* The behavior of -H is *almost* like -L, but not quite, so let's * The behavior of -H is *almost* like -L, but not quite, so let's
* just ignore it too for fun. * just ignore it too for fun.
if (flags & 256 || flags & 512) { if (flags & OPT_L) ...
; if (flags & OPT_H) ... // deref command-line params only
}
*/ */
flags ^= FILEUTILS_DEREFERENCE; /* The sense of this flag was reversed. */ flags ^= FILEUTILS_DEREFERENCE; /* The sense of this flag was reversed. */
@ -78,33 +62,31 @@ int cp_main(int argc, char **argv)
if (optind + 2 == argc) { if (optind + 2 == argc) {
s_flags = cp_mv_stat2(*argv, &source_stat, s_flags = cp_mv_stat2(*argv, &source_stat,
(flags & FILEUTILS_DEREFERENCE) ? stat : lstat); (flags & FILEUTILS_DEREFERENCE) ? stat : lstat);
if ((s_flags < 0) || ((d_flags = cp_mv_stat(last, &dest_stat)) < 0)) { if (s_flags < 0)
exit(EXIT_FAILURE); return EXIT_FAILURE;
} d_flags = cp_mv_stat(last, &dest_stat);
if (d_flags < 0)
return EXIT_FAILURE;
/* ...if neither is a directory or... */ /* ...if neither is a directory or... */
if ( !((s_flags | d_flags) & 2) || if ( !((s_flags | d_flags) & 2) ||
/* ...recursing, the 1st is a directory, and the 2nd doesn't exist... */ /* ...recursing, the 1st is a directory, and the 2nd doesn't exist... */
/* ((flags & FILEUTILS_RECUR) && (s_flags & 2) && !d_flags) */ ((flags & FILEUTILS_RECUR) && (s_flags & 2) && !d_flags)
/* Simplify the above since FILEUTILS_RECUR >> 1 == 2. */
((((flags & FILEUTILS_RECUR) >> 1) & s_flags) && !d_flags)
) { ) {
/* ...do a simple copy. */ /* ...do a simple copy. */
dest = last; dest = xstrdup(last);
goto DO_COPY; /* Note: optind+2==argc implies argv[1]==last below. */ goto DO_COPY; /* Note: optind+2==argc implies argv[1]==last below. */
} }
} }
do { do {
dest = concat_path_file(last, bb_get_last_path_component(*argv)); dest = concat_path_file(last, bb_get_last_path_component(*argv));
DO_COPY: DO_COPY:
if (copy_file(*argv, dest, flags) < 0) { if (copy_file(*argv, dest, flags) < 0) {
status = 1; status = 1;
} }
if (*++argv == last) { free((void*)dest);
break; } while (*++argv != last);
}
free((void *) dest);
} while (1);
exit(status); return status;
} }

4
coreutils/libcoreutils/cp_mv_stat.c
View File

@ -20,8 +20,6 @@
* *
*/ */
#include <errno.h>
#include <sys/stat.h>
#include "libbb.h" #include "libbb.h"
#include "coreutils.h" #include "coreutils.h"
@ -29,7 +27,7 @@ int cp_mv_stat2(const char *fn, struct stat *fn_stat, stat_func sf)
{ {
if (sf(fn, fn_stat) < 0) { if (sf(fn, fn_stat) < 0) {
if (errno != ENOENT) { if (errno != ENOENT) {
bb_perror_msg("unable to stat `%s'", fn); bb_perror_msg("unable to stat '%s'", fn);
return -1; return -1;
} }
return 0; return 0;

43
coreutils/ln.c
View File

@ -49,36 +49,39 @@ int ln_main(int argc, char **argv)
src = last; src = last;
if (is_directory(src, if (is_directory(src,
(flag & LN_NODEREFERENCE) ^ LN_NODEREFERENCE, (flag & LN_NODEREFERENCE) ^ LN_NODEREFERENCE,
NULL)) { NULL)) {
src_name = xstrdup(*argv); src_name = xstrdup(*argv);
src = concat_path_file(src, bb_get_last_path_component(src_name)); src = concat_path_file(src, bb_get_last_path_component(src_name));
free(src_name); free(src_name);
src_name = src; src_name = src;
} }
if (!(flag & LN_SYMLINK) && stat(*argv, &statbuf)) { if (!(flag & LN_SYMLINK) && stat(*argv, &statbuf)) {
bb_perror_msg("%s", *argv); // coreutils: "ln dangling_symlink new_hardlink" works
status = EXIT_FAILURE; if (lstat(*argv, &statbuf) || !S_ISLNK(statbuf.st_mode)) {
free(src_name); bb_perror_msg("%s", *argv);
continue; status = EXIT_FAILURE;
free(src_name);
continue;
}
} }
if (flag & LN_BACKUP) { if (flag & LN_BACKUP) {
char *backup; char *backup;
backup = xasprintf("%s%s", src, suffix); backup = xasprintf("%s%s", src, suffix);
if (rename(src, backup) < 0 && errno != ENOENT) { if (rename(src, backup) < 0 && errno != ENOENT) {
bb_perror_msg("%s", src); bb_perror_msg("%s", src);
status = EXIT_FAILURE; status = EXIT_FAILURE;
free(backup);
continue;
}
free(backup); free(backup);
/* continue;
* When the source and dest are both hard links to the same }
* inode, a rename may succeed even though nothing happened. free(backup);
* Therefore, always unlink(). /*
*/ * When the source and dest are both hard links to the same
unlink(src); * inode, a rename may succeed even though nothing happened.
* Therefore, always unlink().
*/
unlink(src);
} else if (flag & LN_FORCE) { } else if (flag & LN_FORCE) {
unlink(src); unlink(src);
} }

13
coreutils/mv.c
View File

@ -57,7 +57,8 @@ int mv_main(int argc, char **argv)
argv += optind; argv += optind;
if (optind + 2 == argc) { if (optind + 2 == argc) {
if ((dest_exists = cp_mv_stat(last, &dest_stat)) < 0) { dest_exists = cp_mv_stat(last, &dest_stat);
if (dest_exists < 0) {
return 1; return 1;
} }
@ -69,8 +70,8 @@ int mv_main(int argc, char **argv)
do { do {
dest = concat_path_file(last, bb_get_last_path_component(*argv)); dest = concat_path_file(last, bb_get_last_path_component(*argv));
dest_exists = cp_mv_stat(dest, &dest_stat);
if ((dest_exists = cp_mv_stat(dest, &dest_stat)) < 0) { if (dest_exists < 0) {
goto RET_1; goto RET_1;
} }
@ -79,7 +80,7 @@ DO_MOVE:
if (dest_exists && !(flags & OPT_FILEUTILS_FORCE) && if (dest_exists && !(flags & OPT_FILEUTILS_FORCE) &&
((access(dest, W_OK) < 0 && isatty(0)) || ((access(dest, W_OK) < 0 && isatty(0)) ||
(flags & OPT_FILEUTILS_INTERACTIVE))) { (flags & OPT_FILEUTILS_INTERACTIVE))) {
if (fprintf(stderr, "mv: overwrite `%s'? ", dest) < 0) { if (fprintf(stderr, "mv: overwrite '%s'? ", dest) < 0) {
goto RET_1; /* Ouch! fprintf failed! */ goto RET_1; /* Ouch! fprintf failed! */
} }
if (!bb_ask_confirmation()) { if (!bb_ask_confirmation()) {
@ -92,7 +93,7 @@ DO_MOVE:
if (errno != EXDEV || if (errno != EXDEV ||
(source_exists = cp_mv_stat(*argv, &source_stat)) < 1) { (source_exists = cp_mv_stat(*argv, &source_stat)) < 1) {
bb_perror_msg("unable to rename `%s'", *argv); bb_perror_msg("cannot rename '%s'", *argv);
} else { } else {
if (dest_exists) { if (dest_exists) {
if (dest_exists == 3) { if (dest_exists == 3) {
@ -107,7 +108,7 @@ DO_MOVE:
} }
} }
if (unlink(dest) < 0) { if (unlink(dest) < 0) {
bb_perror_msg("cannot remove `%s'", dest); bb_perror_msg("cannot remove '%s'", dest);
goto RET_1; goto RET_1;
} }
} }

5
include/libbb.h
View File

@ -473,8 +473,11 @@ enum { /* DO NOT CHANGE THESE VALUES! cp.c depends on them. */
FILEUTILS_DEREFERENCE = 2, FILEUTILS_DEREFERENCE = 2,
FILEUTILS_RECUR = 4, FILEUTILS_RECUR = 4,
FILEUTILS_FORCE = 8, FILEUTILS_FORCE = 8,
FILEUTILS_INTERACTIVE = 16 FILEUTILS_INTERACTIVE = 0x10,
FILEUTILS_MAKE_HARDLINK = 0x20,
FILEUTILS_MAKE_SOFTLINK = 0x40,
}; };
#define FILEUTILS_CP_OPTSTR "pdRfils"
extern const char *applet_name; extern const char *applet_name;

214
libbb/copy_file.c
View File

@ -10,31 +10,53 @@
#include "libbb.h" #include "libbb.h"
static int retry_overwrite(const char *dest, int flags)
{
if (!(flags & (FILEUTILS_FORCE|FILEUTILS_INTERACTIVE))) {
fprintf(stderr, "'%s' exists\n", dest);
return -1;
}
if (flags & FILEUTILS_INTERACTIVE) {
fprintf(stderr, "%s: overwrite '%s'? ", applet_name, dest);
if (!bb_ask_confirmation())
return 0; // not allowed to overwrite
}
if (unlink(dest) < 0) {
bb_perror_msg("cannot remove '%s'", dest);
return -1; // error
}
return 1; // ok (to try again)
}
int copy_file(const char *source, const char *dest, int flags) int copy_file(const char *source, const char *dest, int flags)
{ {
struct stat source_stat; struct stat source_stat;
struct stat dest_stat; struct stat dest_stat;
int dest_exists = 0;
int status = 0; int status = 0;
signed char dest_exists = 0;
signed char ovr;
if ((!(flags & FILEUTILS_DEREFERENCE) && #define FLAGS_DEREF (flags & FILEUTILS_DEREFERENCE)
lstat(source, &source_stat) < 0) ||
((flags & FILEUTILS_DEREFERENCE) && if ((FLAGS_DEREF ? stat : lstat)(source, &source_stat) < 0) {
stat(source, &source_stat) < 0)) { // This may be a dangling symlink.
bb_perror_msg("%s", source); // Making [sym]links to dangling symlinks works, so...
if (flags & (FILEUTILS_MAKE_SOFTLINK|FILEUTILS_MAKE_HARDLINK))
goto make_links;
bb_perror_msg("cannot stat '%s'", source);
return -1; return -1;
} }
if (lstat(dest, &dest_stat) < 0) { if (lstat(dest, &dest_stat) < 0) {
if (errno != ENOENT) { if (errno != ENOENT) {
bb_perror_msg("unable to stat `%s'", dest); bb_perror_msg("cannot stat '%s'", dest);
return -1; return -1;
} }
} else { } else {
if (source_stat.st_dev == dest_stat.st_dev && if (source_stat.st_dev == dest_stat.st_dev
source_stat.st_ino == dest_stat.st_ino) && source_stat.st_ino == dest_stat.st_ino
{ ) {
bb_error_msg("`%s' and `%s' are the same file", source, dest); bb_error_msg("'%s' and '%s' are the same file", source, dest);
return -1; return -1;
} }
dest_exists = 1; dest_exists = 1;
@ -46,14 +68,14 @@ int copy_file(const char *source, const char *dest, int flags)
mode_t saved_umask = 0; mode_t saved_umask = 0;
if (!(flags & FILEUTILS_RECUR)) { if (!(flags & FILEUTILS_RECUR)) {
bb_error_msg("%s: omitting directory", source); bb_error_msg("omitting directory '%s'", source);
return -1; return -1;
} }
/* Create DEST. */ /* Create DEST. */
if (dest_exists) { if (dest_exists) {
if (!S_ISDIR(dest_stat.st_mode)) { if (!S_ISDIR(dest_stat.st_mode)) {
bb_error_msg("`%s' is not a directory", dest); bb_error_msg("target '%s' is not a directory", dest);
return -1; return -1;
} }
} else { } else {
@ -67,7 +89,7 @@ int copy_file(const char *source, const char *dest, int flags)
if (mkdir(dest, mode) < 0) { if (mkdir(dest, mode) < 0) {
umask(saved_umask); umask(saved_umask);
bb_perror_msg("cannot create directory `%s'", dest); bb_perror_msg("cannot create directory '%s'", dest);
return -1; return -1;
} }
@ -75,7 +97,8 @@ int copy_file(const char *source, const char *dest, int flags)
} }
/* Recursively copy files in SOURCE. */ /* Recursively copy files in SOURCE. */
if ((dp = opendir(source)) == NULL) { dp = opendir(source);
if (dp == NULL) {
status = -1; status = -1;
goto preserve_status; goto preserve_status;
} }
@ -84,7 +107,7 @@ int copy_file(const char *source, const char *dest, int flags)
char *new_source, *new_dest; char *new_source, *new_dest;
new_source = concat_subpath_file(source, d->d_name); new_source = concat_subpath_file(source, d->d_name);
if(new_source == NULL) if (new_source == NULL)
continue; continue;
new_dest = concat_path_file(dest, d->d_name); new_dest = concat_path_file(dest, d->d_name);
if (copy_file(new_source, new_dest, flags) < 0) if (copy_file(new_source, new_dest, flags) < 0)
@ -92,103 +115,119 @@ int copy_file(const char *source, const char *dest, int flags)
free(new_source); free(new_source);
free(new_dest); free(new_dest);
} }
/* closedir have only EBADF error, but "dp" not changes */
closedir(dp); closedir(dp);
if (!dest_exists && if (!dest_exists
chmod(dest, source_stat.st_mode & ~saved_umask) < 0) { && chmod(dest, source_stat.st_mode & ~saved_umask) < 0
bb_perror_msg("unable to change permissions of `%s'", dest); ) {
bb_perror_msg("cannot change permissions of '%s'", dest);
status = -1; status = -1;
} }
} else if (S_ISREG(source_stat.st_mode) ||
(S_ISLNK(source_stat.st_mode) && (flags & FILEUTILS_DEREFERENCE))) } else if (flags & (FILEUTILS_MAKE_SOFTLINK|FILEUTILS_MAKE_HARDLINK)) {
{ int (*lf)(const char *oldpath, const char *newpath);
make_links:
// Hmm... maybe
// if (DEREF && MAKE_SOFTLINK) source = realpath(source) ?
// (but realpath returns NULL on dangling symlinks...)
lf = (flags & FILEUTILS_MAKE_SOFTLINK) ? symlink : link;
if (lf(source, dest) < 0) {
ovr = retry_overwrite(dest, flags);
if (ovr <= 0)
return ovr;
if (lf(source, dest) < 0) {
bb_perror_msg("cannot create link '%s'", dest);
return -1;
}
}
return 0;
} else if (S_ISREG(source_stat.st_mode)
// Huh? DEREF uses stat, which never returns links IIRC...
|| (FLAGS_DEREF && S_ISLNK(source_stat.st_mode))
) {
int src_fd; int src_fd;
int dst_fd; int dst_fd;
if (ENABLE_FEATURE_PRESERVE_HARDLINKS) { if (ENABLE_FEATURE_PRESERVE_HARDLINKS) {
char *link_name; char *link_name;
if (!(flags & FILEUTILS_DEREFERENCE) && if (!FLAGS_DEREF
is_in_ino_dev_hashtable(&source_stat, &link_name)) { && is_in_ino_dev_hashtable(&source_stat, &link_name)
) {
if (link(link_name, dest) < 0) { if (link(link_name, dest) < 0) {
bb_perror_msg("unable to link `%s'", dest); ovr = retry_overwrite(dest, flags);
return -1; if (ovr <= 0)
return ovr;
if (link(link_name, dest) < 0) {
bb_perror_msg("cannot create link '%s'", dest);
return -1;
}
} }
return 0; return 0;
} }
// TODO: probably is_in_.. and add_to_...
// can be combined: find_or_add_...
add_to_ino_dev_hashtable(&source_stat, dest); add_to_ino_dev_hashtable(&source_stat, dest);
} }
src_fd = open(source, O_RDONLY); src_fd = open(source, O_RDONLY);
if (src_fd == -1) { if (src_fd == -1) {
bb_perror_msg("unable to open `%s'", source); bb_perror_msg("cannot open '%s'", source);
return(-1); return -1;
} }
if (dest_exists) { // POSIX: if exists and -i, ask (w/o -i assume yes).
if (flags & FILEUTILS_INTERACTIVE) { // Then open w/o EXCL.
fprintf(stderr, "%s: overwrite `%s'? ", applet_name, dest); // If open still fails and -f, try unlink, then try open again.
if (!bb_ask_confirmation()) { // Result: a mess:
close (src_fd); // If dest is a softlink, we overwrite softlink's destination!
return 0; // (or fail, if it points to dir/nonexistent location/etc).
} // This is strange, but POSIX-correct.
} // coreutils cp has --remove-destination to override this...
dst_fd = open(dest, (flags & FILEUTILS_INTERACTIVE)
dst_fd = open(dest, O_WRONLY|O_TRUNC); ? O_WRONLY|O_CREAT|O_TRUNC|O_EXCL
if (dst_fd == -1) { : O_WRONLY|O_CREAT|O_TRUNC, source_stat.st_mode);
if (!(flags & FILEUTILS_FORCE)) { if (dst_fd == -1) {
bb_perror_msg("unable to open `%s'", dest); // We would not do POSIX insanity. -i asks,
close(src_fd); // then _unlinks_ the offender. Presto.
return -1; // Or else we will end up having 3 open()s!
} ovr = retry_overwrite(dest, flags);
if (ovr <= 0) {
if (unlink(dest) < 0) {
bb_perror_msg("unable to remove `%s'", dest);
close(src_fd);
return -1;
}
goto dest_removed;
}
} else {
dest_removed:
dst_fd = open(dest, O_WRONLY|O_CREAT, source_stat.st_mode);
if (dst_fd == -1) {
bb_perror_msg("unable to open `%s'", dest);
close(src_fd); close(src_fd);
return(-1); return ovr;
}
dst_fd = open(dest, O_WRONLY|O_CREAT|O_TRUNC, source_stat.st_mode);
if (dst_fd == -1) {
bb_perror_msg("cannot open '%s'", dest);
close(src_fd);
return -1;
} }
} }
if (bb_copyfd_eof(src_fd, dst_fd) == -1) if (bb_copyfd_eof(src_fd, dst_fd) == -1)
status = -1; status = -1;
if (close(dst_fd) < 0) { if (close(dst_fd) < 0) {
bb_perror_msg("unable to close `%s'", dest); bb_perror_msg("cannot close '%s'", dest);
status = -1; status = -1;
} }
if (close(src_fd) < 0) { if (close(src_fd) < 0) {
bb_perror_msg("unable to close `%s'", source); bb_perror_msg("cannot close '%s'", source);
status = -1; status = -1;
} }
} else if (S_ISBLK(source_stat.st_mode) || S_ISCHR(source_stat.st_mode) ||
S_ISSOCK(source_stat.st_mode) || S_ISFIFO(source_stat.st_mode) ||
S_ISLNK(source_stat.st_mode)) {
} else if (S_ISBLK(source_stat.st_mode) || S_ISCHR(source_stat.st_mode)
|| S_ISSOCK(source_stat.st_mode) || S_ISFIFO(source_stat.st_mode)
|| S_ISLNK(source_stat.st_mode)
) {
// We are lazy here, a bit lax with races...
if (dest_exists) { if (dest_exists) {
if((flags & FILEUTILS_FORCE) == 0) { ovr = retry_overwrite(dest, flags);
fprintf(stderr, "`%s' exists\n", dest); if (ovr <= 0)
return -1; return ovr;
}
if(unlink(dest) < 0) {
bb_perror_msg("unable to remove `%s'", dest);
return -1;
}
} }
if (S_ISFIFO(source_stat.st_mode)) { if (S_ISFIFO(source_stat.st_mode)) {
if (mkfifo(dest, source_stat.st_mode) < 0) { if (mkfifo(dest, source_stat.st_mode) < 0) {
bb_perror_msg("cannot create fifo `%s'", dest); bb_perror_msg("cannot create fifo '%s'", dest);
return -1; return -1;
} }
} else if (S_ISLNK(source_stat.st_mode)) { } else if (S_ISLNK(source_stat.st_mode)) {
@ -196,20 +235,21 @@ dest_removed:
lpath = xreadlink(source); lpath = xreadlink(source);
if (symlink(lpath, dest) < 0) { if (symlink(lpath, dest) < 0) {
bb_perror_msg("cannot create symlink `%s'", dest); bb_perror_msg("cannot create symlink '%s'", dest);
free(lpath);
return -1; return -1;
} }
free(lpath); free(lpath);
if (flags & FILEUTILS_PRESERVE_STATUS) if (flags & FILEUTILS_PRESERVE_STATUS)
if (lchown(dest, source_stat.st_uid, source_stat.st_gid) < 0) if (lchown(dest, source_stat.st_uid, source_stat.st_gid) < 0)
bb_perror_msg("unable to preserve ownership of `%s'", dest); bb_perror_msg("cannot preserve %s of '%s'", "ownership", dest);
return 0; return 0;
} else { } else {
if (mknod(dest, source_stat.st_mode, source_stat.st_rdev) < 0) { if (mknod(dest, source_stat.st_mode, source_stat.st_rdev) < 0) {
bb_perror_msg("unable to create `%s'", dest); bb_perror_msg("cannot create '%s'", dest);
return -1; return -1;
} }
} }
@ -218,22 +258,24 @@ dest_removed:
return -1; return -1;
} }
preserve_status: preserve_status:
if (flags & FILEUTILS_PRESERVE_STATUS) { if (flags & FILEUTILS_PRESERVE_STATUS
/* Cannot happen: */
/* && !(flags & (FILEUTILS_MAKE_SOFTLINK|FILEUTILS_MAKE_HARDLINK)) */
) {
struct utimbuf times; struct utimbuf times;
char *msg="unable to preserve %s of `%s'";
times.actime = source_stat.st_atime; times.actime = source_stat.st_atime;
times.modtime = source_stat.st_mtime; times.modtime = source_stat.st_mtime;
if (utime(dest, &times) < 0) if (utime(dest, &times) < 0)
bb_perror_msg(msg, "times", dest); bb_perror_msg("cannot preserve %s of '%s'", "times", dest);
if (chown(dest, source_stat.st_uid, source_stat.st_gid) < 0) { if (chown(dest, source_stat.st_uid, source_stat.st_gid) < 0) {
source_stat.st_mode &= ~(S_ISUID | S_ISGID); source_stat.st_mode &= ~(S_ISUID | S_ISGID);
bb_perror_msg(msg, "ownership", dest); bb_perror_msg("cannot preserve %s of '%s'", "ownership", dest);
} }
if (chmod(dest, source_stat.st_mode) < 0) if (chmod(dest, source_stat.st_mode) < 0)
bb_perror_msg(msg, "permissions", dest); bb_perror_msg("cannot preserve %s of '%s'", "permissions", dest);
} }
return status; return status;