Changes in uspace/app/bdsh/cmds/modules/cp/cp.c [1569a9b:582a0b8] in mainline

File:

• uspace/app/bdsh/cmds/modules/cp/cp.c (modified) (20 diffs)

uspace/app/bdsh/cmds/modules/cp/cp.c

@@ -28 +28 @@
 
 #include <errno.h>
-#include <str_error.h>
 #include <stdio.h>
 #include <stdlib.h>
@@ -67 +66 @@
 } dentry_type_t;
 
-static int copy_file(const char *src, const char *dest,
+static int64_t copy_file(const char *src, const char *dest,
     size_t blen, int vb);
 
@@ -176 +175 @@
 }
 
-static int do_copy(const char *src, const char *dest,
+static int64_t do_copy(const char *src, const char *dest,
     size_t blen, int vb, int recursive, int force, int interactive)
 {
-        int rc = EOK;
+        int r = -1;
         char dest_path[PATH_MAX];
         char src_path[PATH_MAX];
@@ -218 +217 @@
                                 printf("The dest directory %s does not exists",
                                     dest_path);
-                                rc = ENOENT;
                                 goto exit;
                         }
@@ -226 +224 @@
                         printf("Cannot overwrite existing directory %s\n",
                             dest_path);
-                        rc = EEXIST;
                         goto exit;
                 } else if (dest_type == TYPE_FILE) {
@@ -236 +233 @@
                          */
                         if (force && !interactive) {
-                                rc = vfs_unlink_path(dest_path);
-                                if (rc != EOK) {
+                                if (vfs_unlink_path(dest_path) != EOK) {
                                         printf("Unable to remove %s\n",
                                             dest_path);
@@ -248 +244 @@
                                 if (overwrite) {
                                         printf("Overwriting file: %s\n", dest_path);
-                                        rc = vfs_unlink_path(dest_path);
-                                        if (rc != EOK) {
+                                        if (vfs_unlink_path(dest_path) != EOK) {
                                                 printf("Unable to remove %s\n", dest_path);
                                                 goto exit;
@@ -255 +250 @@
                                 } else {
                                         printf("Not overwriting file: %s\n", dest_path);
-                                        rc = EOK;
+                                        r = 0;
                                         goto exit;
                                 }
                         } else {
                                 printf("File already exists: %s\n", dest_path);
-                                rc = EEXIST;
                                 goto exit;
                         }
@@ -266 +260 @@
 
                 /* call copy_file and exit */
-                if (copy_file(src, dest_path, blen, vb) < 0) {
-                        rc = EIO;
-                }
+                r = (copy_file(src, dest_path, blen, vb) < 0);
 
         } else if (src_type == TYPE_DIR) {
@@ -276 +268 @@
                         printf("Cannot copy the %s directory without the "
                             "-r option\n", src);
-                        rc = EINVAL;
                         goto exit;
                 } else if (dest_type == TYPE_FILE) {
                         printf("Cannot overwrite a file with a directory\n");
-                        rc = EEXIST;
                         goto exit;
                 }
@@ -303 +293 @@
                                 merge_paths(dest_path, PATH_MAX, src_dirname);
 
-                                rc = vfs_link_path(dest_path, KIND_DIRECTORY,
-                                    NULL);
-                                if (rc != EOK) {
+                                if (vfs_link_path(dest_path, KIND_DIRECTORY,
+                                    NULL) != EOK) {
                                         printf("Unable to create "
                                             "dest directory %s\n", dest_path);
@@ -319 +308 @@
                          * e.g. cp -r /src /data/new_dir_src
                          */
-                        rc = vfs_link_path(dest_path, KIND_DIRECTORY, NULL);
-                        if (rc != EOK) {
+                        if (vfs_link_path(dest_path, KIND_DIRECTORY,
+                            NULL) != EOK) {
                                 printf("Unable to create "
                                     "dest directory %s\n", dest_path);
@@ -332 +321 @@
                         /* Something strange is happening... */
                         printf("Unable to open src %s directory\n", src);
-                        rc = ENOENT;
                         goto exit;
                 }
@@ -360 +348 @@
                                 printf("Cannot copy a directory "
                                     "into itself\n");
-                                rc = EEXIST;
                                 goto exit;
                         }
@@ -368 +355 @@
 
                         /* Recursively call do_copy() */
-                        rc = do_copy(src_dent, dest_dent, blen, vb, recursive,
+                        r = do_copy(src_dent, dest_dent, blen, vb, recursive,
                             force, interactive);
-                        if (rc != EOK)
+                        if (r)
                                 goto exit;
 
@@ -380 +367 @@
         if (dir)
                 closedir(dir);
-        return rc;
-}
-
-static int copy_file(const char *src, const char *dest,
+        return r;
+}
+
+static int64_t copy_file(const char *src, const char *dest,
         size_t blen, int vb)
 {
-        int fd1, fd2;
-        size_t rbytes, wbytes;
-        int rc;
+        int fd1, fd2, bytes;
         off64_t total;
+        int64_t copied = 0;
         char *buff = NULL;
         aoff64_t posr = 0, posw = 0;
@@ -397 +383 @@
                 printf("Copying %s to %s\n", src, dest);
 
-        rc = vfs_lookup_open(src, WALK_REGULAR, MODE_READ, &fd1);
-        if (rc != EOK) {
+        fd1 = vfs_lookup_open(src, WALK_REGULAR, MODE_READ);
+        if (fd1 < 0) {
                 printf("Unable to open source file %s\n", src);
                 return -1;
         }
 
-        rc = vfs_lookup_open(dest, WALK_REGULAR | WALK_MAY_CREATE, MODE_WRITE, &fd2);
-        if (rc != EOK) {
+        fd2 = vfs_lookup_open(dest, WALK_REGULAR | WALK_MAY_CREATE, MODE_WRITE);
+        if (fd2 < 0) {
                 printf("Unable to open destination file %s\n", dest);
                 vfs_put(fd1);
@@ -424 +410 @@
                 printf("Unable to allocate enough memory to read %s\n",
                     src);
-                rc = ENOMEM;
+                copied = -1;
                 goto out;
         }
 
-        while ((rc = vfs_read(fd1, &posr, buff, blen, &rbytes)) == EOK &&
-            rbytes > 0) {
-                if ((rc = vfs_write(fd2, &posw, buff, rbytes, &wbytes)) != EOK)
-                        break;
-        }
-
-        if (rc != EOK) {
-                printf("\nError copying %s: %s\n", src, str_error(rc));
-                return -1;
+        while ((bytes = vfs_read(fd1, &posr, buff, blen)) > 0) {
+                if ((bytes = vfs_write(fd2, &posw, buff, bytes)) < 0)
+                        break;
+                copied += bytes;
+        }
+
+        if (bytes < 0) {
+                printf("\nError copying %s, (%d)\n", src, bytes);
+                copied = bytes;
         }
 
@@ -444 +430 @@
         if (buff)
                 free(buff);
-        if (rc != EOK) {
-                return -1;
-        } else {
-                return 0;
-        }
+        return copied;
 }
 
@@ -479 +461 @@
         int force = 0, interactive = 0;
         int c, opt_ind;
-        int ret;
+        int64_t ret;
 
         con = console_init(stdin, stdout);