Merge pull request #3679 from DavidLin1577/patch-11

[finsh]Fixed a bug may cause stackover flow
2020-06-13 16:23:40 +08:00 · 2020-06-13 16:23:40 +08:00 · b04fdd89ed
parent d3c270412d 5bb38bb4e8
commit b04fdd89ed
1 changed files with 5 additions and 0 deletions
--- a/components/finsh/msh_file.c
+++ b/components/finsh/msh_file.c
@ -101,6 +101,11 @@ int msh_exec_script(const char *cmd_line, int size)
        int length;

        line_buf = (char *) rt_malloc(RT_CONSOLEBUF_SIZE);
+        if (line_buf == RT_NULL) 
+        {
+            close(fd);
+            return -RT_ENOMEM;
+        }

        /* read line by line and then exec it */
        do