2004-06-26 Andrew Cagney <cagney@gnu.org>
authorAndrew Cagney <cagney@redhat.com>
Mon, 28 Jun 2004 23:59:29 +0000 (23:59 +0000)
committerAndrew Cagney <cagney@redhat.com>
Mon, 28 Jun 2004 23:59:29 +0000 (23:59 +0000)
* xcoffsolib.c (xcoff_solib_address): Replace xasprintf with
xstrprintf.
* varobj.c (varobj_gen_name, create_child, c_name_of_child)
(c_value_of_variable): Ditto.
* utils.c (internal_vproblem): Ditto.
* solib-aix5.c (build_so_list_from_mapfile): Ditto.
* remote.c (add_packet_config_cmd): Ditto.
* remote-rdp.c (rdp_set_command_line): Ditto.
* regcache.c (regcache_dump): Ditto.
* frv-tdep.c (new_variant, new_variant): Ditto.
* fbsd-proc.c (child_pid_to_exec_file): Ditto.
(fbsd_find_memory_regions): Ditto.
* breakpoint.c (create_thread_event_breakpoint)
(create_breakpoints): Ditto.
* aix-thread.c (aix_thread_pid_to_str): Ditto.
* ada-lang.c (is_package_name): Ditto.  Also delete xmalloc call.

Index: doc/ChangeLog
2004-06-26  Andrew Cagney  <cagney@gnu.org>

* gdbint.texinfo (Coding): Replace xasprintf with xstrprintf.

15 files changed:
gdb/ChangeLog
gdb/ada-lang.c
gdb/aix-thread.c
gdb/breakpoint.c
gdb/doc/ChangeLog
gdb/doc/gdbint.texinfo
gdb/fbsd-proc.c
gdb/frv-tdep.c
gdb/regcache.c
gdb/remote-rdp.c
gdb/remote.c
gdb/solib-aix5.c
gdb/utils.c
gdb/varobj.c
gdb/xcoffsolib.c

index ae7e0b5dbeef83a08ef97e2531489baeb1941510..51d6b3647f654ebbbe6d69c7aaaebb4406f3dac5 100644 (file)
@@ -1,3 +1,22 @@
+2004-06-28  Andrew Cagney  <cagney@gnu.org>
+
+       * xcoffsolib.c (xcoff_solib_address): Replace xasprintf with
+       xstrprintf.
+       * varobj.c (varobj_gen_name, create_child, c_name_of_child)
+       (c_value_of_variable): Ditto.
+       * utils.c (internal_vproblem): Ditto.
+       * solib-aix5.c (build_so_list_from_mapfile): Ditto.
+       * remote.c (add_packet_config_cmd): Ditto.
+       * remote-rdp.c (rdp_set_command_line): Ditto.
+       * regcache.c (regcache_dump): Ditto.
+       * frv-tdep.c (new_variant, new_variant): Ditto.
+       * fbsd-proc.c (child_pid_to_exec_file): Ditto.
+       (fbsd_find_memory_regions): Ditto.
+       * breakpoint.c (create_thread_event_breakpoint) 
+       (create_breakpoints): Ditto.
+       * aix-thread.c (aix_thread_pid_to_str): Ditto.
+       * ada-lang.c (is_package_name): Ditto.  Also delete xmalloc call.
+
 2004-06-28  Joel Brobecker  <brobecker@gnat.com>
 
        * ada-lang.c: Re-indent file, with some massaging to help indent
index 57f373331956f1f6b57139949d3969e616f4bffe..6ec5044c01e31bd81d53ce4310e97d9c97772d3f 100644 (file)
@@ -4369,8 +4369,7 @@ is_package_name (const char *name)
   if (strstr (name, "__") != NULL)
     return 0;
 
-  fun_name = (char *) alloca (strlen (name) + 5 + 1);
-  xasprintf (&fun_name, "_ada_%s", name);
+  fun_name = xstrprintf ("_ada_%s", name);
 
   return (standard_lookup (fun_name, NULL, VAR_DOMAIN) == NULL);
 }
index 05493ae0f9885eb7b9c22083df0f64735fc3e866..e5e8a9aac4ea4fbd3ab70d6d0daaaa0a9c3736c5 100644 (file)
@@ -1629,10 +1629,10 @@ aix_thread_pid_to_str (ptid_t ptid)
     return base_target.to_pid_to_str (ptid);
 
   /* Free previous return value; a new one will be allocated by
-     xasprintf().  */
+     xstrprintf().  */
   xfree (ret);
 
-  xasprintf (&ret, "Thread %ld", ptid_get_tid (ptid));
+  ret = xstrprintf ("Thread %ld", ptid_get_tid (ptid));
   return ret;
 }
 
index 336264d5efd11ac97313aed75adc32979bc559e4..018f464653177adc762117c1d3e9c63e0b0df2b6 100644 (file)
@@ -4258,7 +4258,7 @@ create_thread_event_breakpoint (CORE_ADDR address)
   
   b->enable_state = bp_enabled;
   /* addr_string has to be used or breakpoint_re_set will delete me.  */
-  xasprintf (&b->addr_string, "*0x%s", paddr (b->loc->address));
+  b->addr_string = xstrprintf ("*0x%s", paddr (b->loc->address));
 
   return b;
 }
@@ -4902,7 +4902,7 @@ create_breakpoints (struct symtabs_and_lines sals, char **addr_string,
        else
          /* addr_string has to be used or breakpoint_re_set will delete
             me.  */
-         xasprintf (&b->addr_string, "*0x%s", paddr (b->loc->address));
+         b->addr_string = xstrprintf ("*0x%s", paddr (b->loc->address));
        b->cond_string = cond_string[i];
        b->ignore_count = ignore_count;
        b->enable_state = bp_enabled;
index 4ab61047b41ab848ba4b7a6f88ae15f1b5043766..5a0b81076063003c40dfbea88bdf48abc25c3e91 100644 (file)
@@ -1,3 +1,7 @@
+2004-06-26  Andrew Cagney  <cagney@gnu.org>
+
+       * gdbint.texinfo (Coding): Replace xasprintf with xstrprintf.
+
 2004-06-20  Andrew Cagney  <cagney@gnu.org>
 
        * gdbint.texinfo (Target Architecture Definition): Deprecate
index 5d1ba7f32707aac8a59a8191b5a72954c76828fc..a3821b59b037cf6577f4b423cb23dd333ea40eb5 100644 (file)
@@ -4942,7 +4942,7 @@ allocation of small temporary values (such as strings).
 restrict the memory being allocated to no more than a few kilobytes.}
 
 @value{GDBN} uses the string function @code{xstrdup} and the print
-function @code{xasprintf}.
+function @code{xstrprintf}.
 
 @emph{Pragmatics: @code{asprintf} and @code{strdup} can fail.  Print
 functions such as @code{sprintf} are very prone to buffer overflow
index 8eed4ccb2cd376e9f7ea7ec05171ce426c0cf6a4..f021d275b9aa16d1de6f8b9a9aa9feb7c32d88d3 100644 (file)
@@ -38,7 +38,7 @@ child_pid_to_exec_file (int pid)
   char *path;
   char *buf;
 
-  xasprintf (&path, "/proc/%d/file", pid);
+  path = xstrprintf ("/proc/%d/file", pid);
   buf = xcalloc (MAXPATHLEN, sizeof (char));
   make_cleanup (xfree, path);
   make_cleanup (xfree, buf);
@@ -82,7 +82,7 @@ fbsd_find_memory_regions (int (*func) (CORE_ADDR, unsigned long,
   char protection[4];
   int read, write, exec;
 
-  xasprintf (&mapfilename, "/proc/%ld/map", (long) pid);
+  mapfilename = xstrprintf ("/proc/%ld/map", (long) pid);
   mapfile = fopen (mapfilename, "r");
   if (mapfile == NULL)
     error ("Couldn't open %s\n", mapfilename);
index 77f137452a08799b894e0d2ae0855be197964c5c..9b7524c2bb0394c241b1a6aff1d202367cfa20ef 100644 (file)
@@ -197,7 +197,7 @@ new_variant (void)
   for (r = acc0_regnum; r <= acc7_regnum; r++)
     {
       char *buf;
-      xasprintf (&buf, "acc%d", r - acc0_regnum);
+      buf = xstrprintf ("acc%d", r - acc0_regnum);
       var->register_names[r] = buf;
     }
 
@@ -210,7 +210,7 @@ new_variant (void)
   for (r = accg0_regnum; r <= accg7_regnum; r++)
     {
       char *buf;
-      xasprintf (&buf, "accg%d", r - accg0_regnum);
+      buf = xstrprintf ("accg%d", r - accg0_regnum);
       var->register_names[r] = buf;
     }
 
index 4058a986071fd4a05a3bbe9556712f0751592b16..ceeaca63fe17ae4a376a64fc4332e32398c61014 100644 (file)
@@ -1572,7 +1572,7 @@ regcache_dump (struct regcache *regcache, struct ui_file *file,
                char *n;
                if (!footnote_register_type_name_null)
                  footnote_register_type_name_null = ++footnote_nr;
-               xasprintf (&n, "*%d", footnote_register_type_name_null);
+               n = xstrprintf ("*%d", footnote_register_type_name_null);
                make_cleanup (xfree, n);
                t = n;
              }
index 68e69326f2ea5d024e4c3114d5fe22f97fd81a88..53604678f20b0569df5a794531a84cf9198400f0 100644 (file)
@@ -721,7 +721,7 @@ rdp_set_command_line (char *command, char *args)
   if (commandline != NULL)
     xfree (commandline);
 
-  xasprintf (&commandline, "%s %s", command, args);
+  commandline = xstrprintf ("%s %s", command, args);
 }
 
 static void
index 962a58db49bff27d90f95e2bebdb679b38c178c1..f754f8048ddd08bd80bbf84727de9957a0371e5e 100644 (file)
@@ -651,12 +651,12 @@ add_packet_config_cmd (struct packet_config *config,
   config->title = title;
   config->detect = AUTO_BOOLEAN_AUTO;
   config->support = PACKET_SUPPORT_UNKNOWN;
-  xasprintf (&set_doc, "Set use of remote protocol `%s' (%s) packet",
-            name, title);
-  xasprintf (&show_doc, "Show current use of remote protocol `%s' (%s) packet",
-            name, title);
+  set_doc = xstrprintf ("Set use of remote protocol `%s' (%s) packet",
+                       name, title);
+  show_doc = xstrprintf ("Show current use of remote protocol `%s' (%s) packet",
+                        name, title);
   /* set/show TITLE-packet {auto,on,off} */
-  xasprintf (&cmd_name, "%s-packet", title);
+  cmd_name = xstrprintf ("%s-packet", title);
   add_setshow_auto_boolean_cmd (cmd_name, class_obscure,
                                &config->detect, set_doc, show_doc,
                                set_func, show_func,
@@ -665,7 +665,7 @@ add_packet_config_cmd (struct packet_config *config,
   if (legacy)
     {
       char *legacy_name;
-      xasprintf (&legacy_name, "%s-packet", name);
+      legacy_name = xstrprintf ("%s-packet", name);
       add_alias_cmd (legacy_name, cmd_name, class_obscure, 0,
                     set_remote_list);
       add_alias_cmd (legacy_name, cmd_name, class_obscure, 0,
index 1a8c9dfeea94d8ae575f0e25eb25decb2803bd82..54b1199b92a4dc01a0aaa1c2d5c25aa75b28c992 100644 (file)
@@ -193,7 +193,7 @@ build_so_list_from_mapfile (int pid, long match_mask, long match_val)
 
     /* Open the map file */
 
-    xasprintf (&map_pathname, "/proc/%d/map", pid);
+    map_pathname = xstrprintf ("/proc/%d/map", pid);
     map_fd = open (map_pathname, O_RDONLY);
     xfree (map_pathname);
     if (map_fd < 0)
index 504890ebf154137963c6d1d3de0242ce20b25dd1..f1c2fe443934a36463126a307b3aa96697879516 100644 (file)
@@ -753,7 +753,7 @@ internal_vproblem (struct internal_problem *problem,
   {
     char *msg;
     xvasprintf (&msg, fmt, ap);
-    xasprintf (&reason, "\
+    reason = xstrprintf ("\
 %s:%d: %s: %s\n\
 A problem internal to GDB has been detected,\n\
 further debugging may prove unreliable.", file, line, problem->name, msg);
index c662518c82fa439f273bedfa6c03b91f608428b6..ea4fa0118c124b2e6d92621a77c37956adc21c04 100644 (file)
@@ -549,7 +549,7 @@ varobj_gen_name (void)
 
   /* generate a name for this object */
   id++;
-  xasprintf (&obj_name, "var%d", id);
+  obj_name = xstrprintf ("var%d", id);
 
   return obj_name;
 }
@@ -1254,7 +1254,7 @@ create_child (struct varobj *parent, int index, char *name)
     child->error = 1;
   child->parent = parent;
   child->root = parent->root;
-  xasprintf (&childs_name, "%s.%s", parent->obj_name, name);
+  childs_name = xstrprintf ("%s.%s", parent->obj_name, name);
   child->obj_name = childs_name;
   install_variable (child);
 
@@ -1837,7 +1837,7 @@ c_name_of_child (struct varobj *parent, int index)
   switch (TYPE_CODE (type))
     {
     case TYPE_CODE_ARRAY:
-      xasprintf (&name, "%d", index);
+      name = xstrprintf ("%d", index);
       break;
 
     case TYPE_CODE_STRUCT:
@@ -1856,7 +1856,7 @@ c_name_of_child (struct varobj *parent, int index)
          break;
 
        default:
-         xasprintf (&name, "*%s", parent->name);
+         name = xstrprintf ("*%s", parent->name);
          break;
        }
       break;
@@ -2070,7 +2070,7 @@ c_value_of_variable (struct varobj *var)
     case TYPE_CODE_ARRAY:
       {
        char *number;
-       xasprintf (&number, "[%d]", var->num_children);
+       number = xstrprintf ("[%d]", var->num_children);
        return (number);
       }
       /* break; */
index 99d2cc8e8f7faa783e5c9433881b549cf686eec2..62426c92fb07398e70bde869e7bb137d63a0fa5c 100644 (file)
@@ -48,11 +48,11 @@ xcoff_solib_address (CORE_ADDR addr)
     if (vp->tstart <= addr && addr < vp->tend)
       {
        xfree (buffer);
-       xasprintf (&buffer, "%s%s%s%s",
-                           vp->name,
-                           *vp->member ? "(" : "",
-                           vp->member,
-                           *vp->member ? ")" : "");
+       buffer = xstrprintf ("%s%s%s%s",
+                            vp->name,
+                            *vp->member ? "(" : "",
+                            vp->member,
+                            *vp->member ? ")" : "");
        return buffer;
       }
   return NULL;