diff options
| author | Donald Sharp <sharpd@cumulusnetworks.com> | 2017-02-10 11:06:37 -0500 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2017-02-10 11:06:37 -0500 |
| commit | c19cf154ef469b1242af3edde06260253b2fca14 (patch) | |
| tree | 657d1d89b453d16b3ce3151cf2d48bd71eb6db40 /lib/grammar_sandbox.c | |
| parent | 01024b5235fb574b7a712d1363bc14df7cd1ad7e (diff) | |
| parent | 6de469061b9933be7afaf2712cb45cb13c51a90c (diff) | |
Merge pull request #194 from opensourcerouting/coverity-lib-2
more lib/ coverity fixes
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; } |