2007-07-05 Michael Snyder <msnyder@access-company.com>

	* event-top.c (cli_command_loop): Prompt string can (and should)
	be freed after call to readline (Coverity).  Also move local var
	declarations into block where they are used.

	* tui/tui-interp.c (tui_command_loop): Prompt string can (and
	should) be freed after call to readline (Coverity).  Also move
	local var declarations into block where they are used.

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 1b617a29121cab450bebf3cfd40231512663c982..b14389c684b6d6e7bdea782598b8f4d5f389e06f 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,13 @@
+2007-07-05  Michael Snyder  <msnyder@access-company.com>
+
+       * event-top.c (cli_command_loop): Prompt string can (and should)
+       be freed after call to readline (Coverity).  Also move local var
+       declarations into block where they are used.
+
+       * tui/tui-interp.c (tui_command_loop): Prompt string can (and
+       should) be freed after call to readline (Coverity).  Also move
+       local var declarations into block where they are used.
+
 2007-07-03  Andreas Schwab  <schwab@suse.de>
 
        * linux-nat.c (linux_nat_info_proc_cmd): Fix parsing of columns in

diff --git a/gdb/event-top.c b/gdb/event-top.c
index 7765767c3dc14ab1719a07f85e1aae409ac6f55f..eecff5a063928b02940e4dd5e91d533c654a1f02 100644 (file)
--- a/gdb/event-top.c
+++ b/gdb/event-top.c
@@ -186,19 +186,20 @@ rl_callback_read_char_wrapper (gdb_client_data client_data)
 void
 cli_command_loop (void)
 {
-  int length;
-  char *a_prompt;
-  char *gdb_prompt = get_prompt ();
-
   /* If we are using readline, set things up and display the first
      prompt, otherwise just print the prompt. */
   if (async_command_editing_p)
     {
+      int length;
+      char *a_prompt;
+      char *gdb_prompt = get_prompt ();
+
       /* Tell readline what the prompt to display is and what function it
          will need to call after a whole line is read. This also displays
          the first prompt. */
-      length = strlen (PREFIX (0)) + strlen (gdb_prompt) + strlen (SUFFIX (0)) + 1;
-      a_prompt = (char *) xmalloc (length);
+      length = strlen (PREFIX (0)) 
+       + strlen (gdb_prompt) + strlen (SUFFIX (0)) + 1;
+      a_prompt = (char *) alloca (length);
       strcpy (a_prompt, PREFIX (0));
       strcat (a_prompt, gdb_prompt);
       strcat (a_prompt, SUFFIX (0));

diff --git a/gdb/tui/tui-interp.c b/gdb/tui/tui-interp.c
index a344754250ce8cca879111554c7cda8c8e9de105..292c7db95f3075faa069af6ef8b584127fb841bd 100644 (file)
--- a/gdb/tui/tui-interp.c
+++ b/gdb/tui/tui-interp.c
@@ -119,19 +119,20 @@ tui_exec (void *data, const char *command_str)
 static void
 tui_command_loop (void *data)
 {
-  int length;
-  char *a_prompt;
-  char *gdb_prompt = get_prompt ();
-
   /* If we are using readline, set things up and display the first
      prompt, otherwise just print the prompt.  */
   if (async_command_editing_p)
     {
+      int length;
+      char *a_prompt;
+      char *gdb_prompt = get_prompt ();
+
       /* Tell readline what the prompt to display is and what function
          it will need to call after a whole line is read. This also
          displays the first prompt.  */
-      length = strlen (PREFIX (0)) + strlen (gdb_prompt) + strlen (SUFFIX (0)) + 1;
-      a_prompt = (char *) xmalloc (length);
+      length = strlen (PREFIX (0)) 
+       + strlen (gdb_prompt) + strlen (SUFFIX (0)) + 1;
+      a_prompt = (char *) alloca (length);
       strcpy (a_prompt, PREFIX (0));
       strcat (a_prompt, gdb_prompt);
       strcat (a_prompt, SUFFIX (0));