commit 2fe8aba3a09b3e2a64ee1861d8eeb389efaf1eb0
author Paul Jakma <paul.jakma@sun.com> Fri May 12 23:22:01 2006 +0000
committer Paul Jakma <paul.jakma@sun.com> Fri May 12 23:22:01 2006 +0000
tree f2c18e4f6c17430fccf12efdc76a9e9e6041e2ef
parent d2519962b71b512eb5fd55a378f8ef748d17c30f

[lib] CID #39, fix leak in error path, vty_describe_command

2006-05-12 Paul Jakma <paul.jakma@sun.com>

	* vty.c: (vty_describe_command) fix leak of describe vector in
	  error path, CID #39.

lib/ChangeLog

diff --git a/lib/ChangeLog b/lib/ChangeLog
index dc9999b..4672747 100644
--- a/lib/ChangeLog
+++ b/lib/ChangeLog
@@ -4,6 +4,8 @@
 	  of prefix and typestr strings, Coverity CID #3.
 	* command.c: (cmd_complete_command_real) Fix leak of cmd_vector
 	  in error case, Coverity CID #37.
+	* vty.c: (vty_describe_command) fix leak of describe vector in
+	  error path, CID #39.
 
 2006-03-30 Paul Jakma <paul.jakma@sun.com>
 

lib/vty.c

diff --git a/lib/vty.c b/lib/vty.c
index 206af06..98f6494 100644
--- a/lib/vty.c
+++ b/lib/vty.c
@@ -985,18 +985,12 @@
   switch (ret)
     {
     case CMD_ERR_AMBIGUOUS:
-      cmd_free_strvec (vline);
       vty_out (vty, "%% Ambiguous command.%s", VTY_NEWLINE);
-      vty_prompt (vty);
-      vty_redraw_line (vty);
-      return;
+      goto out;
       break;
     case CMD_ERR_NO_MATCH:
-      cmd_free_strvec (vline);
       vty_out (vty, "%% There is no matched command.%s", VTY_NEWLINE);
-      vty_prompt (vty);
-      vty_redraw_line (vty);
-      return;
+      goto out;
       break;
     }  
 
@@ -1066,6 +1060,7 @@
 	vty_describe_fold (vty, width, desc_width, desc);
     }
 
+out:
   cmd_free_strvec (vline);
   vector_free (describe);