Changeset b7be230 in mainline for uspace/app/bdsh/cmds/modules/cp/cp.c

Timestamp:

2009-01-22T17:11:54Z (15 years ago)

Author:

Tim Post <echo@…>

Branches:

lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

bffd91f

Parents:

bf226890

Message:

Stop using assert() in modules/builtins as abort() will crash the entire shell.
Handle the case and report the issue, leaving the user able to reproduce it
without rebooting.

File:

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

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

@@ -35 +35 @@
 #include <string.h>
 #include <fcntl.h>
-#include <assert.h>
 #include "config.h"
 #include "util.h"
@@ -125 +124 @@
                         res -= bytes;
                 } while (res > 0);
-                assert(res == 0);
+
+                /* TODO: re-insert assert() once this is stand alone,
+                 * removed as abort() exits the entire shell
+                 */
+                if (res != 0) {
+                        printf("\n%d more bytes than actually exist were copied\n", res);
+                        goto err;
+                }
         }
 
         if (bytes < 0) {
 err:
-                printf("Error copying %s, (%d)\n", src, bytes);
+                printf("\nError copying %s, (%d)\n", src, bytes);
                 copied = bytes;
         }