* varobj.c (varobj_set_display_format): Use xfree.

	* tracepoint.c (stringify_collection_list): Use xfree.
	* remote-fileio.c (remote_fileio_reset): Use xfree.
	* mipsread.c (read_alphacoff_dynamic_symtab): Use xfree.
	* dfp.c (decimal_from_floating): Use xfree, xstrprintf.  Don't use
	asprintf.
	* cp-support.c (mangled_name_to_comp): Use xfree.

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index bd96697..897c3ac 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,13 @@
+2008-09-13  Tom Tromey  <tromey@redhat.com>
+
+       * varobj.c (varobj_set_display_format): Use xfree.
+       * tracepoint.c (stringify_collection_list): Use xfree.
+       * remote-fileio.c (remote_fileio_reset): Use xfree.
+       * mipsread.c (read_alphacoff_dynamic_symtab): Use xfree.
+       * dfp.c (decimal_from_floating): Use xfree, xstrprintf.  Don't use
+       asprintf.
+       * cp-support.c (mangled_name_to_comp): Use xfree.
+
 2008-09-13  Joel Brobecker  <brobecker@adacore.com>
 
        * ada-lang.c (remove_extra_symbols): Remove stub symbols if

diff --git a/gdb/cp-support.c b/gdb/cp-support.c
index df48f60..f50d8fd 100644 (file)
--- a/gdb/cp-support.c
+++ b/gdb/cp-support.c
@@ -167,7 +167,7 @@ mangled_name_to_comp (const char *mangled_name, int options,
 
   if (ret == NULL)
     {
-      free (demangled_name);
+      xfree (demangled_name);
       return NULL;
     }
 

diff --git a/gdb/dfp.c b/gdb/dfp.c
index 9816d27..bbaf9aa 100644 (file)
--- a/gdb/dfp.c
+++ b/gdb/dfp.c
@@ -235,16 +235,12 @@ void
 decimal_from_floating (struct value *from, gdb_byte *to, int len)
 {
   char *buffer;
-  int ret;
 
-  ret = asprintf (&buffer, "%.30" DOUBLEST_PRINT_FORMAT,
-                 value_as_double (from));
-  if (ret < 0)
-    error (_("Error in memory allocation for conversion to decimal float."));
+  buffer = xstrprintf ("%.30" DOUBLEST_PRINT_FORMAT, value_as_double (from));
 
   decimal_from_string (to, len, buffer);
 
-  free (buffer);
+  xfree (buffer);
 }
 
 /* Converts a decimal float of LEN bytes to a double value.  */

diff --git a/gdb/mipsread.c b/gdb/mipsread.c
index fdd8634..1e88f81 100644 (file)
--- a/gdb/mipsread.c
+++ b/gdb/mipsread.c
@@ -217,13 +217,13 @@ read_alphacoff_dynamic_symtab (struct section_offsets *section_offsets,
   dyninfo_secsize = bfd_get_section_size (si.dyninfo_sect);
   got_secsize = bfd_get_section_size (si.got_sect);
   sym_secptr = xmalloc (sym_secsize);
-  cleanups = make_cleanup (free, sym_secptr);
+  cleanups = make_cleanup (xfree, sym_secptr);
   str_secptr = xmalloc (str_secsize);
-  make_cleanup (free, str_secptr);
+  make_cleanup (xfree, str_secptr);
   dyninfo_secptr = xmalloc (dyninfo_secsize);
-  make_cleanup (free, dyninfo_secptr);
+  make_cleanup (xfree, dyninfo_secptr);
   got_secptr = xmalloc (got_secsize);
-  make_cleanup (free, got_secptr);
+  make_cleanup (xfree, got_secptr);
 
   if (!bfd_get_section_contents (abfd, si.sym_sect, sym_secptr,
                                 (file_ptr) 0, sym_secsize))

diff --git a/gdb/remote-fileio.c b/gdb/remote-fileio.c
index 93665c9..05a78be 100644 (file)
--- a/gdb/remote-fileio.c
+++ b/gdb/remote-fileio.c
@@ -1398,7 +1398,7 @@ remote_fileio_reset (void)
     }
   if (remote_fio_data.fd_map)
     {
-      free (remote_fio_data.fd_map);
+      xfree (remote_fio_data.fd_map);
       remote_fio_data.fd_map = NULL;
       remote_fio_data.fd_map_size = 0;
     }

diff --git a/gdb/tracepoint.c b/gdb/tracepoint.c
index 690eba3..4f5c56a 100644 (file)
--- a/gdb/tracepoint.c
+++ b/gdb/tracepoint.c
@@ -1452,7 +1452,7 @@ stringify_collection_list (struct collection_list *list, char *string)
 
   if (ndx == 0)
     {
-      free (str_list);
+      xfree (str_list);
       return NULL;
     }
   else

diff --git a/gdb/varobj.c b/gdb/varobj.c
index 35d876f..5b44eef 100644 (file)
--- a/gdb/varobj.c
+++ b/gdb/varobj.c
@@ -693,7 +693,7 @@ varobj_set_display_format (struct varobj *var,
   if (varobj_value_is_changeable_p (var) 
       && var->value && !value_lazy (var->value))
     {
-      free (var->print_value);
+      xfree (var->print_value);
       var->print_value = value_get_print_value (var->value, var->format);
     }