From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 54534 invoked by alias); 21 Feb 2020 15:33:31 -0000 Mailing-List: contact gdb-cvs-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Archive: List-Post: List-Help: , Sender: gdb-cvs-owner@sourceware.org List-Subscribe: Sender: gdb-cvs-owner@sourceware.org Received: (qmail 54485 invoked by uid 9876); 21 Feb 2020 15:33:31 -0000 Date: Fri, 21 Feb 2020 15:33:00 -0000 Message-ID: <20200221153331.54483.qmail@sourceware.org> Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit From: Andrew Burgess To: gdb-cvs@sourceware.org Subject: [binutils-gdb] gdb/testsuite: Add test for case where gdb_demangle returns NULL X-Act-Checkin: binutils-gdb X-Git-Author: Andrew Burgess X-Git-Refname: refs/heads/master X-Git-Oldrev: 4f180d5396741eb65badba70cf5077b7d48f8641 X-Git-Newrev: bd360d3048e80234a8e772573c9ca3cac9c0466b X-SW-Source: 2020-02/txt/msg00215.txt.bz2 https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=bd360d3048e80234a8e772573c9ca3cac9c0466b commit bd360d3048e80234a8e772573c9ca3cac9c0466b Author: Andrew Burgess Date: Wed Feb 19 14:06:45 2020 +0000 gdb/testsuite: Add test for case where gdb_demangle returns NULL This adds a test for the commit: commit 4f180d5396741eb65badba70cf5077b7d48f8641 Date: Fri Feb 21 08:19:21 2020 -0700 Check for null result from gdb_demangle gdb/testsuite/ChangeLog: * gdb.dwarf2/cpp-linkage-name.c: New file. * gdb.dwarf2/cpp-linkage-name.exp: New file. Diff: --- gdb/testsuite/ChangeLog | 5 ++ gdb/testsuite/gdb.dwarf2/cpp-linkage-name.c | 29 ++++++++ gdb/testsuite/gdb.dwarf2/cpp-linkage-name.exp | 96 +++++++++++++++++++++++++++ 3 files changed, 130 insertions(+) diff --git a/gdb/testsuite/ChangeLog b/gdb/testsuite/ChangeLog index bafd491..6e9f94f 100644 --- a/gdb/testsuite/ChangeLog +++ b/gdb/testsuite/ChangeLog @@ -1,3 +1,8 @@ +2020-02-21 Andrew Burgess + + * gdb.dwarf2/cpp-linkage-name.c: New file. + * gdb.dwarf2/cpp-linkage-name.exp: New file. + 2020-02-21 Shahab Vahedi * lib/gdb.exp (gdb_wrapper_init): Reset diff --git a/gdb/testsuite/gdb.dwarf2/cpp-linkage-name.c b/gdb/testsuite/gdb.dwarf2/cpp-linkage-name.c new file mode 100644 index 0000000..c28dece --- /dev/null +++ b/gdb/testsuite/gdb.dwarf2/cpp-linkage-name.c @@ -0,0 +1,29 @@ +/* This testcase is part of GDB, the GNU debugger. + + Copyright 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 . */ + +volatile struct +{ + int x; + int y; +} global_var; + +int +main (void) +{ + asm ("main_label: .globl main_label"); + return global_var.x + global_var.y; +} diff --git a/gdb/testsuite/gdb.dwarf2/cpp-linkage-name.exp b/gdb/testsuite/gdb.dwarf2/cpp-linkage-name.exp new file mode 100644 index 0000000..414b43a --- /dev/null +++ b/gdb/testsuite/gdb.dwarf2/cpp-linkage-name.exp @@ -0,0 +1,96 @@ +# Copyright 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 . + +# Some compilers give anonymous structures a linkage name, and that +# linkage name doesn't demangle (within GDB calling gdb_demangle +# return NULL). At one point this caused GDB to crash due to +# dereferencing a NULL pointer. + +load_lib dwarf.exp + +# This test can only be run on targets which support DWARF-2 and use gas. +if {![dwarf2_support]} { + return 0 +} + +standard_testfile cpp-linkage-name.c cpp-linkage-name-debug.S + +# Set up the DWARF for the test. +set asm_file [standard_output_file $srcfile2] +Dwarf::assemble $asm_file { + global srcdir subdir srcfile + + lassign [function_range main ${srcdir}/${subdir}/${srcfile}] \ + main_start main_length + + cu {} { + DW_TAG_compile_unit { + {DW_AT_language @DW_LANG_C_plus_plus} + {DW_AT_name ada-linkage-name.c} + {DW_AT_comp_dir /tmp} + + } { + declare_labels a_l b_l + + a_l: DW_TAG_base_type { + {DW_AT_byte_size 4 DW_FORM_sdata} + {DW_AT_encoding @DW_ATE_signed} + {DW_AT_name int} + } + + # To expose the bug that existed at one point this + # structure must have a linkage name, but no name, and the + # linkage name is something that doesn't demangle. + b_l: DW_TAG_structure_type { + {DW_AT_byte_size 8 DW_FORM_sdata} + {DW_AT_encoding @DW_ATE_signed} + {DW_AT_linkage_name } + } { + member { + {name x} + {type :$a_l} + {data_member_location 0 data1} + } + member { + {name y} + {type :$a_l} + {data_member_location 0 data1} + } + } + DW_TAG_subprogram { + {name "main"} + {low_pc $main_start addr} + {high_pc "$main_start + $main_length" addr} + {type :$a_l} + } + DW_TAG_variable { + {type :$b_l} + {external 1 flag} + {DW_AT_name global_var} + {DW_AT_location { + DW_OP_addr [gdb_target_symbol global_var] + } SPECIAL_expr} + } + } + } +} + +if { [prepare_for_testing "failed to prepare" ${testfile} \ + [list $srcfile $asm_file] {nodebug}] } { + return -1 +} + +gdb_assert [runto_main] "run to main" +gdb_test "p global_var" " = {x = 0, y = 0}"