From 7977e5d2a6287c48bbfe41fdc5fc795b58e98dfe Mon Sep 17 00:00:00 2001 From: Tom Tromey Date: Thu, 27 Sep 2012 16:04:23 +0000 Subject: [PATCH] Fix https://bugzilla.redhat.com/show_bug.cgi?id=849357 * cp-valprint.c (cp_print_value_fields): Use get_vptr_fieldno. gdb/testsuite * gdb.cp/derivation.exp: Add regression test. * gdb.cp/derivation.cc (class V_base, class V_inter, class V_derived): New. (vderived): New global. --- gdb/ChangeLog | 5 +++++ gdb/cp-valprint.c | 7 +++++-- gdb/testsuite/ChangeLog | 7 +++++++ gdb/testsuite/gdb.cp/derivation.cc | 29 +++++++++++++++++++++++++++++ gdb/testsuite/gdb.cp/derivation.exp | 8 ++++++++ 5 files changed, 54 insertions(+), 2 deletions(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 8faa0298787..fe69ce6d134 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,8 @@ +2012-09-27 Tom Tromey + + Fix https://bugzilla.redhat.com/show_bug.cgi?id=849357 + * cp-valprint.c (cp_print_value_fields): Use get_vptr_fieldno. + 2012-09-27 Joel Brobecker * sol-thread.c (sol_thread_fetch_registers) diff --git a/gdb/cp-valprint.c b/gdb/cp-valprint.c index c066aa5e742..e6a99ecb873 100644 --- a/gdb/cp-valprint.c +++ b/gdb/cp-valprint.c @@ -210,7 +210,9 @@ cp_print_value_fields (struct type *type, struct type *real_type, { int statmem_obstack_initial_size = 0; int stat_array_obstack_initial_size = 0; - + struct type *vptr_basetype = NULL; + int vptr_fieldno; + if (dont_print_statmem == 0) { statmem_obstack_initial_size = @@ -225,6 +227,7 @@ cp_print_value_fields (struct type *type, struct type *real_type, } } + vptr_fieldno = get_vptr_fieldno (type, &vptr_basetype); for (i = n_baseclasses; i < len; i++) { /* If requested, skip printing of static fields. */ @@ -358,7 +361,7 @@ cp_print_value_fields (struct type *type, struct type *real_type, v, stream, recurse + 1, options); } - else if (i == TYPE_VPTR_FIELDNO (type)) + else if (i == vptr_fieldno && type == vptr_basetype) { int i_offset = offset + TYPE_FIELD_BITPOS (type, i) / 8; struct type *i_type = TYPE_FIELD_TYPE (type, i); diff --git a/gdb/testsuite/ChangeLog b/gdb/testsuite/ChangeLog index 6707751d94d..a68f69d2b57 100644 --- a/gdb/testsuite/ChangeLog +++ b/gdb/testsuite/ChangeLog @@ -1,3 +1,10 @@ +2012-09-27 Tom Tromey + + * gdb.cp/derivation.exp: Add regression test. + * gdb.cp/derivation.cc (class V_base, class V_inter, class + V_derived): New. + (vderived): New global. + 2012-09-26 Tom Tromey * gdb.dwarf2/dw2-common-block.S: New file. diff --git a/gdb/testsuite/gdb.cp/derivation.cc b/gdb/testsuite/gdb.cp/derivation.cc index 942fcd26b65..fcd57ce22ba 100644 --- a/gdb/testsuite/gdb.cp/derivation.cc +++ b/gdb/testsuite/gdb.cp/derivation.cc @@ -118,8 +118,37 @@ public: }; +class V_base +{ +public: + virtual void m(); + int base; +}; + +void +V_base::m() +{ +} + +class V_inter : public virtual V_base +{ +public: + virtual void f(); + int inter; +}; + +void +V_inter::f() +{ +} +class V_derived : public V_inter +{ +public: + double x; +}; +V_derived vderived; int A::afoo() { return 1; diff --git a/gdb/testsuite/gdb.cp/derivation.exp b/gdb/testsuite/gdb.cp/derivation.exp index b752b526e48..a4210d9d422 100644 --- a/gdb/testsuite/gdb.cp/derivation.exp +++ b/gdb/testsuite/gdb.cp/derivation.exp @@ -176,3 +176,11 @@ gdb_test_multiple "frame" "re-selected 'main' frame after inferior call" { gdb_test "print g_instance.bfoo()" "\\$\[0-9\]+ = 2" "print value of g_instance.bfoo()" gdb_test "print g_instance.cfoo()" "\\$\[0-9\]+ = 3" "print value of g_instance.cfoo()" + +# This is a regression test for a bug that caused a crash when trying +# to print the vtbl pointer. We don't care about the output so much +# here (it is tested elsewhere), just that gdb doesn't crash. We test +# "ptype" first because, before the gdb fix, that was the only code +# path calling get_vptr_fieldno. +gdb_test "ptype vderived" "type = .*" +gdb_test "print vderived" " = {.* inter = 0.*x = 0}" -- 2.30.2