Changeset 1569a9b in mainline for uspace/app/bdsh/input.c

Timestamp:

2017-12-24T16:40:33Z (6 years ago)

Author:

GitHub <noreply@…>

Branches:

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

Children:

48fd597

Parents:

a1026da

git-author:

Jiří Zárevúcky <zarevucky.jiri@…> (2017-12-24 16:40:33)

git-committer:

GitHub <noreply@…> (2017-12-24 16:40:33)

Message:

The "not-so-obvious" error handling tweaks. (#9)

Some more changes to enable type-checking with errno_t. Some of the "fixes" here don't quite feel right, but a proper solution would be a more elaborate refactoring of the surrounding code.

File:

• uspace/app/bdsh/input.c (modified) (3 diffs)

uspace/app/bdsh/input.c

@@ -73 +73 @@
         
         char *cmd[WORD_MAX];
-        int rc = 0;
+        int rc = EOK;
         tokenizer_t tok;
         unsigned int i, pipe_count, processed_pipes;
@@ -82 +82 @@
         if (usr->line == NULL) {
                 free(tokens_buf);
-                return CL_EFAIL;
+                return EINVAL;
         }
 
@@ -198 +198 @@
         }
 
-        rc = run_command(cmd, usr, &new_iostate);
+        if (run_command(cmd, usr, &new_iostate) == 0) {
+                rc = EOK;
+        } else {
+                rc = EINVAL;
+        }
         
 finit_with_files: