vlib: fix "<command> help"
Walk the sub_commands vector directly.
Type: fix
Fixes: 6b3f25caff262b536a5449ac5461dd041c92a655
Signed-off-by: Dave Barach <dave@barachs.net>
Change-Id: I3022b39e5a739647ec68effc6756166ce2da85b6
diff --git a/src/vlib/cli.c b/src/vlib/cli.c
index 0ecdac3..6088fc2 100644
--- a/src/vlib/cli.c
+++ b/src/vlib/cli.c
@@ -392,8 +392,6 @@
unformat_input_t sub_input;
u8 *string;
uword is_main_dispatch = cm == &vm->cli_main;
- uword value;
- u8 *key;
parent = vec_elt_at_index (cm->commands, parent_command_index);
if (is_main_dispatch && unformat (input, "help"))
@@ -428,17 +426,15 @@
else
{
vlib_cli_sub_rule_t *sr, *subs = 0;
+ vlib_cli_sub_command_t *sc;
- /* *INDENT-OFF* */
- hash_foreach_mem (key, value, c->sub_command_index_by_name,
- ({
- (void) key;
- vec_add2 (subs, sr, 1);
- sr->name = c->sub_commands[value].name;
- sr->command_index = value;
- sr->rule_index = ~0;
- }));
- /* *INDENT-ON* */
+ vec_foreach (sc, c->sub_commands)
+ {
+ vec_add2 (subs, sr, 1);
+ sr->name = sc->name;
+ sr->command_index = sc->index;
+ sr->rule_index = ~0;
+ }
vec_sort_with_function (subs, vlib_cli_cmp_rule);