* value.c (set_internalvar): Use value_free, not xfree.

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 8c93b4aff69f395af80afc2d2f0cdec31a373715..3b97942431e4a1a957d99ca3a987d359ee85570a 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,7 @@
+2009-01-06  Tom Tromey  <tromey@redhat.com>
+
+       * value.c (set_internalvar): Use value_free, not xfree.
+

 2009-01-06  Jim Blandy  <jimb@red-bean.com>
 
        Check return values of functions declared with warn_unused_result
 2009-01-06  Jim Blandy  <jimb@red-bean.com>
 
        Check return values of functions declared with warn_unused_result

diff --git a/gdb/value.c b/gdb/value.c
index 39df98ec918ec1c55d0b342b9b1d962f12926079..1068f1d67d1f7f605300608912562ead0d5523f9 100644 (file)
--- a/gdb/value.c
+++ b/gdb/value.c
@@ -931,7 +931,7 @@ set_internalvar (struct internalvar *var, struct value *val)
      something in the value chain (i.e., before release_value is
      called), because after the error free_all_values will get called before
      long.  */
      something in the value chain (i.e., before release_value is
      called), because after the error free_all_values will get called before
      long.  */

-  xfree (var->value);
+  value_free (var->value);

   var->value = newval;
   var->endian = gdbarch_byte_order (current_gdbarch);
   release_value (newval);
   var->value = newval;
   var->endian = gdbarch_byte_order (current_gdbarch);
   release_value (newval);