From: Andrew Burgess Date: Thu, 26 Dec 2019 20:56:01 +0000 (+0000) Subject: gdb: Fix backtrace with disassemble-next-line on X-Git-Tag: gdb-9.1-release~43 X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=f7da8e5b9b16fa2e9b97c46e02fff3161562b0dc;p=thirdparty%2Fbinutils-gdb.git gdb: Fix backtrace with disassemble-next-line on In this commit: commit ec8e2b6d3051f0b4b2a8eee9917898e95046c62f Date: Fri Jun 14 23:43:00 2019 +0100 gdb: Don't allow annotations to influence what else GDB prints A change was accidentally made that moved a call to do_gdb_disassembly out of an if block guarded by 'if (source_print && sal.symtab)'. The result was that if a user has 'set disassemble-next-line on' then the backtrace would now include some disassembly of a few instructions in each frame. This change was not intentional, but was not spotted by any tests. This commit restores the old behaviour and adds a test to ensure this doesn't break again in the future. gdb/ChangeLog: * stack.c (print_frame_info): Move disassemble_next_line code inside source_print block. gdb/testsuite/ChangeLog: * gdb.base/backtrace.c: New file. * gdb.base/backtrace.exp: New file. Change-Id: I47c52a202fa74be138382646b695827940178689 --- diff --git a/gdb/ChangeLog b/gdb/ChangeLog index fdedb41aad0..37b0a773434 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,8 @@ +2020-01-06 Andrew Burgess + + * stack.c (print_frame_info): Move disassemble_next_line code + inside source_print block. + 2020-01-01 Joel Brobecker * gdbarch.sh: Update copyright year range of generated files. diff --git a/gdb/stack.c b/gdb/stack.c index bf29294f3d7..184501bb719 100644 --- a/gdb/stack.c +++ b/gdb/stack.c @@ -1153,12 +1153,12 @@ print_frame_info (const frame_print_options &fp_opts, print_source_lines (sal.symtab, sal.line, sal.line + 1, 0); } - } - /* If disassemble-next-line is set to on and there is line debug - messages, output assembly codes for next line. */ - if (disassemble_next_line == AUTO_BOOLEAN_TRUE) - do_gdb_disassembly (get_frame_arch (frame), -1, sal.pc, sal.end); + /* If disassemble-next-line is set to on and there is line debug + messages, output assembly codes for next line. */ + if (disassemble_next_line == AUTO_BOOLEAN_TRUE) + do_gdb_disassembly (get_frame_arch (frame), -1, sal.pc, sal.end); + } if (set_current_sal) { diff --git a/gdb/testsuite/ChangeLog b/gdb/testsuite/ChangeLog index 59faa1857ac..02a74b93c3f 100644 --- a/gdb/testsuite/ChangeLog +++ b/gdb/testsuite/ChangeLog @@ -1,3 +1,8 @@ +2020-01-06 Andrew Burgess + + * gdb.base/backtrace.c: New file. + * gdb.base/backtrace.exp: New file. + 2019-12-16 Simon Marchi * gdb.base/jit-reader.exp (jit_reader_test): Rename diff --git a/gdb/testsuite/gdb.base/backtrace.c b/gdb/testsuite/gdb.base/backtrace.c new file mode 100644 index 00000000000..bae8b1a8900 --- /dev/null +++ b/gdb/testsuite/gdb.base/backtrace.c @@ -0,0 +1,40 @@ +/* This testcase is part of GDB, the GNU debugger. + + Copyright 2019-2020 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 . */ + +int __attribute__((noinline,noclone)) +baz () +{ + return 0; /* Break here. */ +} + +int __attribute__((noinline,noclone)) +bar () +{ + return baz (); +} + +int __attribute__((noinline,noclone)) +foo () +{ + return bar (); +} + +int +main () +{ + return foo (); +} diff --git a/gdb/testsuite/gdb.base/backtrace.exp b/gdb/testsuite/gdb.base/backtrace.exp new file mode 100644 index 00000000000..17e3b61fa3f --- /dev/null +++ b/gdb/testsuite/gdb.base/backtrace.exp @@ -0,0 +1,48 @@ +# Copyright 2019-2020 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 . + +# A place for miscellaneous tests related to backtrace. + +standard_testfile + +if { [prepare_for_testing "failed to prepare" $testfile $srcfile] } { + return -1 +} + +if ![runto_main] then { + fail "can't run to main" + return 0 +} + +# Run to the breakpoint at return. +gdb_breakpoint [gdb_get_line_number "Break here."] +gdb_continue_to_breakpoint "Break here." + +# Backtrace with the default options. +gdb_test "bt" \ + [multi_line \ + "#0\[ \t\]*baz \\(\\) at \[^\r\n\]+" \ + "#1\[ \t\]*$hex in bar \\(\\) at \[^\r\n\]+" \ + "#2\[ \t\]*$hex in foo \\(\\) at \[^\r\n\]+" \ + "#3\[ \t\]*$hex in main \\(\\) at \[^\r\n\]+" ] + +# Backtrace with 'set disassemble-next-line on'. This shouldn't make +# any difference to the backtrace. +gdb_test "with disassemble-next-line on -- bt" \ + [multi_line \ + "#0\[ \t\]*baz \\(\\) at \[^\r\n\]+" \ + "#1\[ \t\]*$hex in bar \\(\\) at \[^\r\n\]+" \ + "#2\[ \t\]*$hex in foo \\(\\) at \[^\r\n\]+" \ + "#3\[ \t\]*$hex in main \\(\\) at \[^\r\n\]+" ]