diff options
| author | Donald Sharp <sharpd@cumulusnetworks.com> | 2018-06-12 11:50:18 -0400 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2018-06-12 11:50:18 -0400 |
| commit | 81030cc3f75dc171bd4e148d721454928a05cbc9 (patch) | |
| tree | 1528d0ce65fdc86e35e5e2907ba91bd3e8dcc958 /lib/command.c | |
| parent | 2a4269ccd8f96cd2ab8c0ef9e33e3878ff0eed94 (diff) | |
| parent | 9f227e4c91f03d9ff801bf2ae2321cb22c91800d (diff) | |
Merge pull request #2399 from pacovn/Coverity_1469894_Dereference_null_return_value
lib: null return value check (Coverity 1469894)
Diffstat (limited to 'lib/command.c')
| -rw-r--r-- | lib/command.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/lib/command.c b/lib/command.c index 7df81438f2..a8e61c6bb4 100644 --- a/lib/command.c +++ b/lib/command.c @@ -1184,7 +1184,7 @@ static int handle_pipe_action(struct vty *vty, const char *cmd_in, char **cmd_out) { /* look for `|` */ - char *orig, *working, *token; + char *orig, *working, *token, *u; char *pipe = strstr(cmd_in, "| "); if (!pipe) @@ -1213,7 +1213,8 @@ static int handle_pipe_action(struct vty *vty, const char *cmd_in, goto fail; } *cmd_out = XSTRDUP(MTYPE_TMP, cmd_in); - *(strstr(*cmd_out, "|")) = '\0'; + u = *cmd_out; + strsep(&u, "|"); } else { vty_out(vty, "%% Unknown action '%s'\n", token); goto fail; |