]> cloudbase.mooo.com Git - z180-stamp.git/blobdiff - avr/cli.c
cli_run_command(): Move local copy of command from heap to stack.
[z180-stamp.git] / avr / cli.c
index 52c85db1b4b17c1f192f008730e655cb6f986d2c..f11e932666a4b1d7d10f20061f8e933faef99c41 100644 (file)
--- a/avr/cli.c
+++ b/avr/cli.c
@@ -1,5 +1,5 @@
 /*
- * (C) Copyright 2014 Leo C. <erbl259-lmu@yahoo.de>
+ * (C) Copyright 2014-2016 Leo C. <erbl259-lmu@yahoo.de>
  *
  * (C) Copyright 2000
  * Wolfgang Denk, DENX Software Engineering, wd@denx.de.
@@ -8,9 +8,10 @@
  * (C) Copyright 2005
  * JinHua Luo, GuangDong Linux Center, <luo.jinhua@gd-linux.com>
  *
- * SPDX-License-Identifier:    GPL-2.0+
+ * SPDX-License-Identifier:    GPL-2.0
  */
 
+#include "cli.h"
 #include "common.h"
 
 #include <string.h>
 
 #include "config.h"
 #include "command.h"
-#include "xmalloc.h"
 #include "debug.h"
 #include "env.h"
 #include "cli_readline.h"
 #include "con-utils.h"
-#include "cli.h"
+
+
+/* FIXME: Quoting problems */
 
 #define DEBUG_PARSER   0       /* set to 1 to debug */
 
 #define debug_parser(fmt, args...)             \
        debug_cond(DEBUG_PARSER, fmt, ##args)
 
+
+static int_least8_t exec_flags;
+#define OPT_XTRACE             1
+#define OPT_VERBOSE            2
+
+static int_least8_t command_level;
+
+static void cli_trace_cmd(int_fast8_t level, int argc, char *argv[])
+{
+       while (level-- > 0)
+               putchar('+');
+       for (int_fast8_t i = 0; i < argc; i++)
+               printf_P(PSTR(" %s"), argv[i]);
+       putchar('\n');
+}
+
+
+
+
 static int cli_parse_line(char *line, char *argv[])
 {
        uint_fast8_t state = 0;
@@ -41,12 +62,10 @@ static int cli_parse_line(char *line, char *argv[])
 
        debug_parser("%s: \"%s\"\n", __func__, line);
 
-       for (outp = inp = line, quote = '\0' ;
-                       nargs < CONFIG_SYS_MAXARGS && (c = *inp) != '\0';
-                       inp++) {
+       for (outp = inp = line, quote = '\0'; (c = *inp) != '\0'; inp++) {
 
                switch (state) {
-               case 0:
+               case 0:                                         /* before arg string, waiting for arg start */
                        if (isblank(c))
                                continue;
 
@@ -55,7 +74,7 @@ static int cli_parse_line(char *line, char *argv[])
                        state = 1;
                        /* fall thru */
 
-               case 1:
+               case 1:                                         /* in arg string, waiting for end of arg string */
                        if (c == '\\') {
                                ++state;
                                continue;
@@ -71,7 +90,7 @@ static int cli_parse_line(char *line, char *argv[])
                        }
                        break;
 
-               case 3:
+               case 3:                                         /* in quote */
                        if (c == '\\' && quote == '\"') {
                                ++state;
                                continue;
@@ -82,12 +101,17 @@ static int cli_parse_line(char *line, char *argv[])
                        }
                        break;
 
-               case 2:
+               case 2:                                         /* waiting for next char */
                case 4:
                        --state;
                        break;
 
                }
+
+               if (nargs > CONFIG_SYS_MAXARGS) {
+                       --nargs;
+                       break;
+               }
                *outp++ = c;
        }
 
@@ -116,11 +140,16 @@ void append_char(uint_fast8_t pass, char **p, char c)
 }
 
 static
-char *process_macros(char *input, char *output)
+char *process_macros(char *input)
 {
-       char c, prev, *inp, *outp;
+       char c, prev, *inp;
+       char *output = NULL;
+       char *outp = NULL;
        const char *varname = NULL;
 
+       debug_parser("[PROCESS_MACROS] INPUT len %d: \"%s\"\n",
+                                       strlen(input), input);
+
        for(uint_fast8_t pass = 0; pass < 2; pass++)
        {
                uint_fast8_t state = 0;
@@ -129,19 +158,18 @@ char *process_macros(char *input, char *output)
                /* 2 = waiting for '}' */
                /* 3 = waiting for ''' */
 
-               if (pass == 0) {
+               if (pass > 0) {
+                       size_t outputlen = outp - (char *) NULL;
+                       output = (char *) malloc(outputlen);
+                       if (output == NULL) {
+                               printf_P(PSTR("** Can't process command: Out of memory! **\n"));
+                               return NULL;
+                       }
                        outp = output;
-               } else {
-                       int outputlen = outp - output;
-                       outp = xrealloc(output, outputlen);
-                       output = outp;
                }
 
                inp = input;
 
-               debug_parser("[PROCESS_MACROS] INPUT len %d: \"%s\"\n",
-                                               strlen(inp), inp);
-
                for (prev = '\0'; (c = *inp++) != '\0'; prev = c) {
 
 
@@ -171,7 +199,7 @@ char *process_macros(char *input, char *output)
                                if (c == '}') {
                                        /* Terminate variable name */
                                        *(inp-1) = '\0';
-                                       const char *envval = getenv(varname);
+                                       const char *envval = getenv_str(varname);
                                        *(inp-1) = '}';
                                        /* Copy into the line if it exists */
                                        if (envval != NULL)
@@ -204,7 +232,7 @@ char *process_macros(char *input, char *output)
  * WARNING:
  *
  * We must create a temporary copy of the command since the command we get
- * may be the result from getenv(), which returns a pointer directly to
+ * may be the result from getenv_str(), which returns a pointer directly to
  * the environment data, which may change magicly when the command we run
  * creates or modifies environment variables (like "bootp" does).
  *
@@ -216,7 +244,7 @@ char *process_macros(char *input, char *output)
  */
 static int cli_run_command(const char *cmd, int flag)
 {
-       char *cmdbuf;                   /* working copy of cmd          */
+       char cmdbuf[strlen(cmd) + 1];                   /* working copy of cmd          */
        char *token;                    /* start of token in cmdbuf     */
        char *sep;                      /* end of token (separator) in cmdbuf */
        char *finaltoken = NULL;        /* token after macro expansion  */
@@ -226,25 +254,35 @@ static int cli_run_command(const char *cmd, int flag)
        uint_fast8_t inquotes, repeatable = 1;
        int rc = 0;
 
+       exec_flags = 0;
+       char *optenv = getenv_str(PSTR("cli"));
+       if (optenv) {
+               if (strstr_P(optenv, PSTR("verbose")) != NULL)
+                       exec_flags |= OPT_VERBOSE;
+               if (strstr_P(optenv, PSTR("xtrace")) != NULL)
+                       exec_flags |= OPT_XTRACE;
+       }
+
        debug_parser("[RUN_COMMAND] cmd[%p]=\"%s\"\n",
                        cmd, cmd ? cmd : "NULL");
 
-       clear_ctrlc();          /* forget any previous Control C */
+       if (exec_flags & OPT_VERBOSE)
+               printf_P(PSTR("%s\n"), cmd, cmd ? cmd : "");
 
        if (!cmd || !*cmd)
                return -1;      /* empty command */
 
-       cmdbuf = strdup(cmd);
-       if (!cmdbuf)
-               return -1;      /* not enough memory */
+       clear_ctrlc();          /* forget any previous Control C */
 
-       str = cmdbuf;
+       str = strcpy(cmdbuf, cmd);
+       //str = cmdbuf;
+       ++command_level;
 
        /* Process separators and check for invalid
         * repeatable commands
         */
 
-       debug_parser("[PROCESS_SEPARATORS] %s\n", cmd);
+       debug_parser("[PROCESS_SEPARATORS] \"%s\"\n", cmd);
        while (*str) {
                /*
                 * Find separator, or string end
@@ -252,40 +290,57 @@ static int cli_run_command(const char *cmd, int flag)
                 */
                for (inquotes = 0, sep = str; *sep; sep++) {
                        if ((*sep == '\'') &&
-                           (*(sep - 1) != '\\'))
+                           (sep != str) &&                                     /* past string start */
+                           (*(sep - 1) != '\\'))                       /* and NOT escaped */
                                inquotes = !inquotes;
 
                        if (!inquotes &&
-                           (*sep == ';' || *sep == '\n') &&    /* separator            */
-                           (sep != str) &&                     /* past string start    */
-                           (*(sep - 1) != '\\'))               /* and NOT escaped */
+                           (*sep == ';' || *sep == '\n'        /* separator */
+                                       || *sep == '#') &&                      /*   or start of comment */
+                           ((sep == str) ||                            /* string start */
+                           (*(sep - 1) != '\\')))                      /*   or NOT escaped */
                                break;
                }
 
-               /*
-                * Limit the token to data between separators
-                */
+               /* no more commands after unescaped '#' token */
+               if (*sep == '#')
+                       *sep = '\0';
+
+               /* Limit the token to data between separators */
                token = str;
                if (*sep) {
-                       str = sep + 1;  /* start of command for next pass */
+                       str = sep + 1;          /* start of command for next pass */
                        *sep = '\0';
                } else {
-                       str = sep;      /* no more commands for next pass */
+                       str = sep;                      /* no more commands for next pass */
                }
                debug_parser("token: \"%s\"\n", token);
 
                /* find macros in this token and replace them */
-               finaltoken = process_macros(token, finaltoken);
+               finaltoken = process_macros(token);
+               if (finaltoken == NULL) {
+                       rc = -1;        /* no command at all */
+                       continue;
+               }
 
                /* Extract arguments */
                argc = cli_parse_line(finaltoken, argv);
                if (argc == 0) {
                        rc = -1;        /* no command at all */
+                       free(finaltoken);
                        continue;
                }
 
-               if (cmd_process(flag, argc, argv, &repeatable) != CMD_RET_SUCCESS)
+               if (exec_flags & OPT_XTRACE)
+                       cli_trace_cmd(command_level, argc, argv);
+
+               rc = cmd_process(flag, argc, argv, &repeatable);
+               free(finaltoken);
+               if (rc != CMD_RET_SUCCESS) {
+                       if (exec_flags & OPT_VERBOSE)
+                               printf_P(PSTR("Command failed, result=%d\n"), rc);
                        rc = -1;
+               }
 
                /* Did the user stop this? */
                if (had_ctrlc()) {
@@ -294,8 +349,7 @@ static int cli_run_command(const char *cmd, int flag)
                }
        }
 
-       free(cmdbuf);
-       free(finaltoken);
+       --command_level;
 
        return rc ? rc : repeatable;
 }
@@ -309,7 +363,7 @@ static int cli_run_command_list(const char *cmd)
 
 
 /*
- * Run a command using the selected parser.
+ * Run a command.
  *
  * @param cmd  Command to run
  * @param flag Execution flags (CMD_FLAG_...)
@@ -357,7 +411,7 @@ void cli_loop(void)
        int rc = 1;
 
        for (;;) {
-               len = cli_readline(PSTR(CONFIG_SYS_PROMPT));
+               len = cli_readline(lastcommand ? PSTR(CONFIG_SYS_PROMPT_REPEAT) : PSTR(CONFIG_SYS_PROMPT), 1);
 
                flag = 0;       /* assume no special flags for now */
                if (len > 0) {
@@ -366,9 +420,10 @@ void cli_loop(void)
                } else if (len == 0)
                        flag |= CMD_FLAG_REPEAT;
 
-               if (len == -1)
-                       my_puts_P(PSTR("<INTERRUPT>\n"));
-               else
+               if (len == -1) {
+                       if (exec_flags & OPT_VERBOSE)
+                               my_puts_P(PSTR("<INTERRUPT>\n"));
+               } else
                        rc = run_command_repeatable(lastcommand, flag);
 
                if (rc <= 0) {
@@ -378,27 +433,3 @@ void cli_loop(void)
                }
        }
 }
-
-
-command_ret_t do_run(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
-{
-       int i;
-       (void) cmdtp;
-
-       if (argc < 2)
-               return CMD_RET_USAGE;
-
-       for (i = 1; i < argc; ++i) {
-               char *arg;
-
-               arg = getenv(argv[i]);
-               if (arg == NULL) {
-                       printf_P(PSTR("## Error: \"%s\" not defined\n"), argv[i]);
-                       return CMD_RET_FAILURE;
-               }
-
-               if (run_command(arg, flag) != 0)
-                       return CMD_RET_FAILURE;
-       }
-       return CMD_RET_SUCCESS;
-}