crash printing optimized out variant type

Assuming the following declarations:

   type Discriminants_Record (A : Integer; B : Boolean) is record
      C : Float;
   end record;
   Z : Discriminants_Record := (A => 1, B => False, C => 2.0);

If variable Z is not used, and the compiler optimizes it out,
GDB would crash as follow:

    (gdb) print Z
    /[...]/gdb/valops.c:1121: internal-error: Unexpected lazy value type.

This is because the ada-lang module forgot to set the optimized_out
flag in the value returned by ada_evaluate_subexp during the value's
"fixing" process.  Later on, when trying to print the resulting value,
GDB finds that the value is still lazily allocated, and thus tries to
fetch it. But this is not allowed for not_lval values, hence the internal
error.

gdb/ChangeLog:

        * ada-lang.c (coerce_unspec_val_to_type): Make sure that
        the optimized_out flag is preserved.

gdb/testsuite/ChangeLog:

        * gdb.ada/optim_drec: New testcase.
This commit is contained in:
Joel Brobecker 2012-09-10 22:18:24 +00:00
parent 5152ff9039
commit 2fa15f237d
5 changed files with 69 additions and 0 deletions

View File

@ -1,3 +1,8 @@
2012-09-10 Joel Brobecker <brobecker@adacore.com>
* ada-lang.c (coerce_unspec_val_to_type): Make sure that
the optimized_out flag is preserved.
2012-09-10 Anthony Green <green@moxielogic.com>
* moxie-tdep.c (moxie_analyze_prologue): Update for function

View File

@ -581,6 +581,7 @@ coerce_unspec_val_to_type (struct value *val, struct type *type)
set_value_bitsize (result, value_bitsize (val));
set_value_bitpos (result, value_bitpos (val));
set_value_address (result, value_address (val));
set_value_optimized_out (result, value_optimized_out (val));
return result;
}
}

View File

@ -1,3 +1,7 @@
2012-09-10 Joel Brobecker <brobecker@adacore.com>
* gdb.ada/optim_drec: New testcase.
2012-09-10 Doug Evans <dje@google.com>
* boards/fission.exp: Explicitly mark "board" as not remote.

View File

@ -0,0 +1,34 @@
# Copyright 2012 Free Software Foundation, Inc.
#
# This program is free software; you can redistribute it and/or modify
# it under the terms of the GNU General Public License as published by
# the Free Software Foundation; either version 3 of the License, or
# (at your option) any later version.
#
# This program is distributed in the hope that it will be useful,
# but WITHOUT ANY WARRANTY; without even the implied warranty of
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
# GNU General Public License for more details.
#
# You should have received a copy of the GNU General Public License
# along with this program. If not, see <http://www.gnu.org/licenses/>.
load_lib "ada.exp"
standard_ada_testfile foo
if {[gdb_compile_ada "${srcfile}" "${binfile}" executable [list debug ]] != "" } {
return -1
}
clean_restart ${testfile}
if ![runto "foo" ] then {
perror "Couldn't run ${testfile}"
return
}
gdb_test "print z" \
"= (\\(a => .*, b => .*, c => .*\\)|<optimized out>)"

View File

@ -0,0 +1,25 @@
-- Copyright 2012 Free Software Foundation, Inc.
--
-- This program is free software; you can redistribute it and/or modify
-- it under the terms of the GNU General Public License as published by
-- the Free Software Foundation; either version 3 of the License, or
-- (at your option) any later version.
--
-- This program is distributed in the hope that it will be useful,
-- but WITHOUT ANY WARRANTY; without even the implied warranty of
-- MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
-- GNU General Public License for more details.
--
-- You should have received a copy of the GNU General Public License
-- along with this program. If not, see <http://www.gnu.org/licenses/>.
procedure Foo is
type Discriminants_Record (A : Integer; B : Boolean) is record
C : Float;
end record;
-- The following variable is unused on purpose, and might be
-- optimized out by the compiler.
Z : Discriminants_Record := (A => 1, B => False, C => 2.0);
begin
null;
end Foo;