Try to make indent formatting less horrible
[oweals/busybox.git] / libbb / copy_file.c
index ea05c9b8e2a6f4e9a9d8fcbf5118d8ebf759cf8d..5808ca48cd16ad339a7d3b6252e74f3a6b9d00a9 100644 (file)
@@ -36,30 +36,30 @@ int copy_file(const char *source, const char *dest, int flags)
 {
        struct stat source_stat;
        struct stat dest_stat;
-       int dest_exists = 1;
+       int dest_exists = 0;
        int status = 0;
 
        if ((!(flags & FILEUTILS_DEREFERENCE) &&
                        lstat(source, &source_stat) < 0) ||
                        ((flags & FILEUTILS_DEREFERENCE) &&
                         stat(source, &source_stat) < 0)) {
-               perror_msg("%s", source);
+               bb_perror_msg("%s", source);
                return -1;
        }
 
-       if (stat(dest, &dest_stat) < 0) {
+       if (lstat(dest, &dest_stat) < 0) {
                if (errno != ENOENT) {
-                       perror_msg("unable to stat `%s'", dest);
+                       bb_perror_msg("unable to stat `%s'", dest);
                        return -1;
                }
-               dest_exists = 0;
-       }
-
-       if (dest_exists && source_stat.st_rdev == dest_stat.st_rdev &&
+       } else {
+               if (source_stat.st_dev == dest_stat.st_dev &&
                        source_stat.st_ino == dest_stat.st_ino) {
-               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;
        }
+               dest_exists = 1;
+       }
 
        if (S_ISDIR(source_stat.st_mode)) {
                DIR *dp;
@@ -67,14 +67,14 @@ int copy_file(const char *source, const char *dest, int flags)
                mode_t saved_umask = 0;
 
                if (!(flags & FILEUTILS_RECUR)) {
-                       error_msg("%s: omitting directory", source);
+                       bb_error_msg("%s: omitting directory", source);
                        return -1;
                }
 
                /* Create DEST.  */
                if (dest_exists) {
                        if (!S_ISDIR(dest_stat.st_mode)) {
-                               error_msg("`%s' is not a directory", dest);
+                               bb_error_msg("`%s' is not a directory", dest);
                                return -1;
                        }
                } else {
@@ -88,7 +88,7 @@ int copy_file(const char *source, const char *dest, int flags)
 
                        if (mkdir(dest, mode) < 0) {
                                umask(saved_umask);
-                               perror_msg("cannot create directory `%s'", dest);
+                               bb_perror_msg("cannot create directory `%s'", dest);
                                return -1;
                        }
 
@@ -97,7 +97,7 @@ int copy_file(const char *source, const char *dest, int flags)
 
                /* Recursively copy files in SOURCE.  */
                if ((dp = opendir(source)) == NULL) {
-                       perror_msg("unable to open directory `%s'", source);
+                       bb_perror_msg("unable to open directory `%s'", source);
                        status = -1;
                        goto end;
                }
@@ -105,28 +105,21 @@ int copy_file(const char *source, const char *dest, int flags)
                while ((d = readdir(dp)) != NULL) {
                        char *new_source, *new_dest;
 
-                       if (strcmp(d->d_name, ".") == 0 ||
-                                       strcmp(d->d_name, "..") == 0)
+                       new_source = concat_subpath_file(source, d->d_name);
+                       if(new_source == NULL)
                                continue;
-
-                       new_source = concat_path_file(source, d->d_name);
                        new_dest = concat_path_file(dest, d->d_name);
                        if (copy_file(new_source, new_dest, flags) < 0)
                                status = -1;
                        free(new_source);
                        free(new_dest);
                }
-
-               /* ??? What if an error occurs in readdir?  */
-
-               if (closedir(dp) < 0) {
-                       perror_msg("unable to close directory `%s'", source);
-                       status = -1;
-               }
+               /* closedir have only EBADF error, but "dp" not changes */
+               closedir(dp);
 
                if (!dest_exists &&
                                chmod(dest, source_stat.st_mode & ~saved_umask) < 0) {
-                       perror_msg("unable to change permissions of `%s'", dest);
+                       bb_perror_msg("unable to change permissions of `%s'", dest);
                        status = -1;
                }
        } else if (S_ISREG(source_stat.st_mode)) {
@@ -137,7 +130,7 @@ int copy_file(const char *source, const char *dest, int flags)
                if (!(flags & FILEUTILS_DEREFERENCE) &&
                                is_in_ino_dev_hashtable(&source_stat, &link_name)) {
                        if (link(link_name, dest) < 0) {
-                               perror_msg("unable to link `%s'", dest);
+                               bb_perror_msg("unable to link `%s'", dest);
                                return -1;
                        }
 
@@ -145,15 +138,14 @@ int copy_file(const char *source, const char *dest, int flags)
                }
 #endif
 
-               if ((sfp = fopen(source, "r")) == NULL) {
-                       perror_msg("unable to open `%s'", source);
+               if ((sfp = bb_wfopen(source, "r")) == NULL) {
                        return -1;
                }
 
                if (dest_exists) {
                        if (flags & FILEUTILS_INTERACTIVE) {
-                               fprintf(stderr, "%s: overwrite `%s'? ", applet_name, dest);
-                               if (!ask_confirmation()) {
+                               fprintf(stderr, "%s: overwrite `%s'? ", bb_applet_name, dest);
+                               if (!bb_ask_confirmation()) {
                                        fclose (sfp);
                                        return 0;
                                }
@@ -161,13 +153,13 @@ int copy_file(const char *source, const char *dest, int flags)
 
                        if ((dfp = fopen(dest, "w")) == NULL) {
                                if (!(flags & FILEUTILS_FORCE)) {
-                                       perror_msg("unable to open `%s'", dest);
+                                       bb_perror_msg("unable to open `%s'", dest);
                                        fclose (sfp);
                                        return -1;
                                }
 
                                if (unlink(dest) < 0) {
-                                       perror_msg("unable to remove `%s'", dest);
+                                       bb_perror_msg("unable to remove `%s'", dest);
                                        fclose (sfp);
                                        return -1;
                                }
@@ -183,39 +175,56 @@ int copy_file(const char *source, const char *dest, int flags)
                                        (dfp = fdopen(fd, "w")) == NULL) {
                                if (fd >= 0)
                                        close(fd);
-                               perror_msg("unable to open `%s'", dest);
+                               bb_perror_msg("unable to open `%s'", dest);
                                fclose (sfp);
                                return -1;
                        }
                }
 
-               if (copy_file_chunk(sfp, dfp, -1) < 0)
+               if (bb_copyfd(fileno(sfp), fileno(dfp), 0) == -1)
                        status = -1;
 
                if (fclose(dfp) < 0) {
-                       perror_msg("unable to close `%s'", dest);
+                       bb_perror_msg("unable to close `%s'", dest);
                        status = -1;
                }
 
                if (fclose(sfp) < 0) {
-                       perror_msg("unable to close `%s'", source);
+                       bb_perror_msg("unable to close `%s'", source);
                        status = -1;
                }
-       } else if (S_ISBLK(source_stat.st_mode) || S_ISCHR(source_stat.st_mode) ||
-                       S_ISSOCK(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)) {
+
+               if (dest_exists &&
+                      ((flags & FILEUTILS_FORCE) == 0 || unlink(dest) < 0)) {
+                               bb_perror_msg("unable to remove `%s'", dest);
+                               return -1;
+
+                       }
+       } else {
+               bb_error_msg("internal error: unrecognized file type");
+               return -1;
+               }
+       if (S_ISBLK(source_stat.st_mode) || S_ISCHR(source_stat.st_mode) ||
+           S_ISSOCK(source_stat.st_mode)) {
                if (mknod(dest, source_stat.st_mode, source_stat.st_rdev) < 0) {
-                       perror_msg("unable to create `%s'", dest);
+                       bb_perror_msg("unable to create `%s'", dest);
                        return -1;
                }
        } else if (S_ISFIFO(source_stat.st_mode)) {
                if (mkfifo(dest, source_stat.st_mode) < 0) {
-                       perror_msg("cannot create fifo `%s'", dest);
+                       bb_perror_msg("cannot create fifo `%s'", dest);
                        return -1;
                }
        } else if (S_ISLNK(source_stat.st_mode)) {
-               char *lpath = xreadlink(source);
+               char *lpath;
+
+               lpath = xreadlink(source);
                if (symlink(lpath, dest) < 0) {
-                       perror_msg("cannot create symlink `%s'", dest);
+                       bb_perror_msg("cannot create symlink `%s'", dest);
                        return -1;
                }
                free(lpath);
@@ -223,7 +232,7 @@ int copy_file(const char *source, const char *dest, int flags)
 #if (__GLIBC__ >= 2) && (__GLIBC_MINOR__ >= 1)
                if (flags & FILEUTILS_PRESERVE_STATUS)
                        if (lchown(dest, source_stat.st_uid, source_stat.st_gid) < 0)
-                               perror_msg("unable to preserve ownership of `%s'", dest);
+                               bb_perror_msg("unable to preserve ownership of `%s'", dest);
 #endif
 
 #ifdef CONFIG_FEATURE_PRESERVE_HARDLINKS
@@ -231,9 +240,6 @@ int copy_file(const char *source, const char *dest, int flags)
 #endif
 
                return 0;
-       } else {
-               error_msg("internal error: unrecognized file type");
-               return -1;
        }
 
 #ifdef CONFIG_FEATURE_PRESERVE_HARDLINKS
@@ -248,13 +254,13 @@ end:
                times.actime = source_stat.st_atime;
                times.modtime = source_stat.st_mtime;
                if (utime(dest, &times) < 0)
-                       perror_msg("unable to preserve times of `%s'", dest);
+                       bb_perror_msg("unable to preserve times of `%s'", dest);
                if (chown(dest, source_stat.st_uid, source_stat.st_gid) < 0) {
                        source_stat.st_mode &= ~(S_ISUID | S_ISGID);
-                       perror_msg("unable to preserve ownership of `%s'", dest);
+                       bb_perror_msg("unable to preserve ownership of `%s'", dest);
                }
                if (chmod(dest, source_stat.st_mode) < 0)
-                       perror_msg("unable to preserve permissions of `%s'", dest);
+                       bb_perror_msg("unable to preserve permissions of `%s'", dest);
        }
 
        return status;