vtysh: properly fix memory leak

Commit 44f12f20 fixed the memory leak in the wrong way and introduced a
"uninitialized variable" warning.

Signed-off-by: Renato Westphal <renato@opensourcerouting.org>
This commit is contained in:
Renato Westphal 2017-10-25 07:43:32 -02:00
parent b5b583aa8d
commit 80c872f699

View File

@ -807,24 +807,26 @@ static int vtysh_rl_describe(void)
fprintf(stdout, "\n");
describe = cmd_describe_command(vline, vty, &ret);
/* Ambiguous and no match error. */
switch (ret) {
case CMD_ERR_AMBIGUOUS:
cmd_free_strvec(vline);
vector_free(describe);
fprintf(stdout, "%% Ambiguous command.\n");
rl_on_new_line();
return 0;
break;
case CMD_ERR_NO_MATCH:
cmd_free_strvec(vline);
vector_free(describe);
fprintf(stdout, "%% There is no matched command.\n");
rl_on_new_line();
return 0;
break;
}
describe = cmd_describe_command(vline, vty, &ret);
/* Get width of command string. */
width = 0;
for (i = 0; i < vector_active(describe); i++)