diff options
| author | Denis Vlasenko <vda.linux@googlemail.com> | 2008-07-12 11:22:19 +0000 |
|---|---|---|
| committer | Denis Vlasenko <vda.linux@googlemail.com> | 2008-07-12 11:22:19 +0000 |
| commit | 13afb2acdae8a63c0d7851e97de592f9a4996eea (patch) | |
| tree | 773e168def38b9ae7f3364e14ff8d31a0f7c788b /coreutils/mv.c | |
| parent | a87045ce37e01971bd40219293afdacd4c2317e2 (diff) | |
| download | busybox-13afb2acdae8a63c0d7851e97de592f9a4996eea.tar.gz | |
mv: fix a case when we move dangling symlink
Diffstat (limited to 'coreutils/mv.c')
| -rw-r--r-- | coreutils/mv.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/coreutils/mv.c b/coreutils/mv.c index 5611ecd01..be10b030a 100644 --- a/coreutils/mv.c +++ b/coreutils/mv.c @@ -26,16 +26,13 @@ static const char mv_longopts[] ALIGN1 = #define OPT_FILEUTILS_FORCE 1 #define OPT_FILEUTILS_INTERACTIVE 2 -static const char fmt[] ALIGN1 = - "cannot overwrite %sdirectory with %sdirectory"; - int mv_main(int argc, char **argv) MAIN_EXTERNALLY_VISIBLE; int mv_main(int argc, char **argv) { struct stat dest_stat; const char *last; const char *dest; - unsigned long flags; + unsigned flags; int dest_exists; int status = 0; int copy_flag = 0; @@ -57,7 +54,7 @@ int mv_main(int argc, char **argv) return EXIT_FAILURE; } - if (!(dest_exists & 2)) { + if (!(dest_exists & 2)) { /* last is not a directory */ dest = last; goto DO_MOVE; } @@ -88,10 +85,13 @@ int mv_main(int argc, char **argv) int source_exists; if (errno != EXDEV - || (source_exists = cp_mv_stat(*argv, &source_stat)) < 1 + || (source_exists = cp_mv_stat2(*argv, &source_stat, lstat)) < 1 ) { bb_perror_msg("cannot rename '%s'", *argv); } else { + static const char fmt[] ALIGN1 = + "cannot overwrite %sdirectory with %sdirectory"; + if (dest_exists) { if (dest_exists == 3) { if (source_exists != 3) { |