lib: Fix command execution npe

Caller may pass NULL if it does not care about what
command was matched.

Signed-off-by: Quentin Young <qlyoung@cumulusnetworks.com>

diff --git a/lib/command.c b/lib/command.c
index a74e967c94b6fa8b8d55f9b7dc52c184827ce041..9d68f33989f034bd399b60e0399bf3193770d751 100644 (file)
--- a/lib/command.c
+++ b/lib/command.c
@@ -698,8 +698,13 @@ cmd_execute_command_real (vector vline,
 {
   struct list *argv_list;
   enum matcher_rv status;
+  struct cmd_element *matched_element = NULL;
+
   struct graph *cmdgraph = cmd_node_graph (cmdvec, vty->node);
-  status = command_match (cmdgraph, vline, &argv_list, cmd);
+  status = command_match (cmdgraph, vline, &argv_list, &matched_element);
+
+  if (cmd)
+    *cmd = matched_element;
 
   // if matcher error, return corresponding CMD_ERR
   if (MATCHER_ERROR(status))
@@ -724,10 +729,10 @@ cmd_execute_command_real (vector vline,
   int argc = argv_list->count;
 
   int ret;
-  if ((*cmd)->daemon)
+  if (matched_element->daemon)
     ret = CMD_SUCCESS_DAEMON;
   else
-    ret = (*cmd)->func (*cmd, vty, argc, argv);
+    ret = matched_element->func (matched_element, vty, argc, argv);
 
   // delete list and cmd_token's in it
   list_delete (argv_list);