diff options
| author | nicm <nicm> | 2017-01-15 22:00:56 +0000 |
|---|---|---|
| committer | nicm <nicm> | 2017-01-15 22:00:56 +0000 |
| commit | 52847a951802fda7a3ce36cdac77c34914b0ccca (patch) | |
| tree | 7e5ad465e719d32f2a7d9a7c3469dec6d12448a4 /cmd-confirm-before.c | |
| parent | 30548461439f02ed8327f96748035685a1a26ace (diff) | |
It is silly for cmd_list_parse to return an integer error when it could
just return NULL.
Diffstat (limited to 'cmd-confirm-before.c')
| -rw-r--r-- | cmd-confirm-before.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/cmd-confirm-before.c b/cmd-confirm-before.c index a2ffd059..e7366ec2 100644 --- a/cmd-confirm-before.c +++ b/cmd-confirm-before.c @@ -112,7 +112,8 @@ cmd_confirm_before_callback(void *data, const char *s, __unused int done) if (tolower((u_char) s[0]) != 'y' || s[1] != '\0') return (0); - if (cmd_string_parse(cdata->cmd, &cmdlist, NULL, 0, &cause) != 0) { + cmdlist = cmd_string_parse(cdata->cmd, NULL, 0, &cause); + if (cmdlist == NULL) { if (cause != NULL) { new_item = cmdq_get_callback(cmd_confirm_before_error, cause); |