diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 8c93b4aff6..3b97942431 100644
--- 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
diff --git a/gdb/value.c b/gdb/value.c
index 39df98ec91..1068f1d67d 100644
--- 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.  */
-  xfree (var->value);
+  value_free (var->value);
   var->value = newval;
   var->endian = gdbarch_byte_order (current_gdbarch);
   release_value (newval);