diff options
| author | David Lamparter <equinox@opensourcerouting.org> | 2017-02-10 15:04:06 +0100 |
|---|---|---|
| committer | David Lamparter <equinox@opensourcerouting.org> | 2017-02-10 15:04:06 +0100 |
| commit | 148781210fbcb1b26d5df678a400fa5dd8549e42 (patch) | |
| tree | 78d57b0b68c54d029e47fdc834aacb0bc3815715 /lib/grammar_sandbox.c | |
| parent | 01024b5235fb574b7a712d1363bc14df7cd1ad7e (diff) | |
lib: fix remaining coverity issues
Reported-by: Coverity
Signed-off-by: David Lamparter <equinox@opensourcerouting.org>
Diffstat (limited to 'lib/grammar_sandbox.c')
| -rw-r--r-- | lib/grammar_sandbox.c | 16 |
1 files changed, 14 insertions, 2 deletions
diff --git a/lib/grammar_sandbox.c b/lib/grammar_sandbox.c index 315bd4d59c..08f012f8ac 100644 --- a/lib/grammar_sandbox.c +++ b/lib/grammar_sandbox.c @@ -86,6 +86,11 @@ DEFUN (grammar_test_complete, return CMD_SUCCESS; vector command = cmd_make_strvec (cmdstr); + if (!command) + { + XFREE (MTYPE_TMP, cmdstr); + return CMD_SUCCESS; + } // generate completions of user input struct list *completions; @@ -121,7 +126,7 @@ DEFUN (grammar_test_complete, // free resources list_delete (completions); cmd_free_strvec (command); - free (cmdstr); + XFREE (MTYPE_TMP, cmdstr); return CMD_SUCCESS; } @@ -138,7 +143,14 @@ DEFUN (grammar_test_match, return CMD_SUCCESS; char *cmdstr = argv_concat(argv, argc, idx_command); + if (!cmdstr) + return CMD_SUCCESS; vector command = cmd_make_strvec (cmdstr); + if (!command) + { + XFREE (MTYPE_TMP, cmdstr); + return CMD_SUCCESS; + } struct list *argvv = NULL; const struct cmd_element *element = NULL; @@ -177,7 +189,7 @@ DEFUN (grammar_test_match, // free resources cmd_free_strvec (command); - free (cmdstr); + XFREE (MTYPE_TMP, cmdstr); return CMD_SUCCESS; } |