202001-10-15 Jim Ingham <jingham@inghji.apple.com>

* valarith.c (value_sub): Don't pass a raw type to
value_from_pointer, it has to go through check_typedef first.

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 3c9dfd00cdf9a7dbdda56f794e30a81d80fbaf47..9eff17cf17f9239afbf137dec48a666d74524aa2 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,8 @@
+202001-10-15  Jim Ingham  <jingham@inghji.apple.com>
+
+        * valarith.c (value_sub): Don't pass a raw type to
+        value_from_pointer, it has to go through check_typedef first.
+
 2001-10-15  Andrew Cagney  <ac131313@redhat.com>
 
        * gdbarch.sh (fallbackdefault): Set to one when predefault is

diff --git a/gdb/valarith.c b/gdb/valarith.c
index e0598d08d7a47d4a4bffb02e5c14fef3a7579bf2..a9ba5c76e02e5371e7f3a85acf12ff3481bc608f 100644 (file)
--- a/gdb/valarith.c
+++ b/gdb/valarith.c
@@ -104,7 +104,7 @@ value_sub (value_ptr arg1, value_ptr arg2)
        {
          /* pointer - integer.  */
          LONGEST sz = TYPE_LENGTH (check_typedef (TYPE_TARGET_TYPE (type1)));
-         return value_from_pointer (VALUE_TYPE (arg1),
+         return value_from_pointer (type1,
                                     (value_as_pointer (arg1)
                                      - (sz * value_as_long (arg2))));
        }