public inbox for gdb-cvs@sourceware.org help / color / mirror / Atom feed
From: Simon Marchi <simark@sourceware.org> To: gdb-cvs@sourceware.org Subject: [binutils-gdb] gdb/testsuite: remove gdb.python/pretty-print-call-by-hand.exp Date: Thu, 24 Mar 2022 14:25:20 +0000 (GMT) [thread overview] Message-ID: <20220324142520.623333858C2C@sourceware.org> (raw) https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=ba7789a316f78e163f03f9206a61d34652f7e060 commit ba7789a316f78e163f03f9206a61d34652f7e060 Author: Simon Marchi <simon.marchi@efficios.com> Date: Thu Mar 24 10:21:10 2022 -0400 gdb/testsuite: remove gdb.python/pretty-print-call-by-hand.exp This test was added without a corresponding fix, with some setup_kfails. However, it results in UNRESOLVED results when GDB is built with ASan. ERROR: GDB process no longer exists GDB process exited with wait status 1946871 exp7 0 1 UNRESOLVED: gdb.python/pretty-print-call-by-hand.exp: frame print: backtrace test (PRMS gdb/28856) Remove the test from the tree, I'll attach it to the Bugzilla bug instead [1]. [1] https://sourceware.org/bugzilla/show_bug.cgi?id=28856 Change-Id: Id95d8949fb8742874bd12aeac758aa4d7564d678 Diff: --- .../gdb.python/pretty-print-call-by-hand.c | 53 --------- .../gdb.python/pretty-print-call-by-hand.exp | 127 --------------------- .../gdb.python/pretty-print-call-by-hand.py | 45 -------- 3 files changed, 225 deletions(-) diff --git a/gdb/testsuite/gdb.python/pretty-print-call-by-hand.c b/gdb/testsuite/gdb.python/pretty-print-call-by-hand.c deleted file mode 100644 index 3be5675b096..00000000000 --- a/gdb/testsuite/gdb.python/pretty-print-call-by-hand.c +++ /dev/null @@ -1,53 +0,0 @@ -/* This testcase is part of GDB, the GNU debugger. - - Copyright 2022 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/>. */ - -struct mytype -{ - char *x; -}; - -void -rec (int i) -{ - if (i <= 0) - return; - rec (i-1); -} - -int -f () -{ - rec (100); - return 2; -} - -void -g (struct mytype mt, int depth) -{ - if (depth <= 0) - return; /* TAG: final frame */ - g (mt, depth - 1); /* TAG: first frame */ -} - -int -main () -{ - struct mytype mt; - mt.x = "hello world"; - g (mt, 10); /* TAG: outside the frame */ - return 0; -} diff --git a/gdb/testsuite/gdb.python/pretty-print-call-by-hand.exp b/gdb/testsuite/gdb.python/pretty-print-call-by-hand.exp deleted file mode 100644 index 0a6d014cf12..00000000000 --- a/gdb/testsuite/gdb.python/pretty-print-call-by-hand.exp +++ /dev/null @@ -1,127 +0,0 @@ -# Copyright (C) 2022 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/>. - -# This file is part of the GDB testsuite. It tests a pretty printer that -# calls an inferior function by hand, triggering a Use-after-Free bug -# (PR gdb/28856). - -load_lib gdb-python.exp - -standard_testfile - -# gdb needs to be started here for skip_python_tests to work. -# prepare_for_testing could be used instead, but it could compile the program -# unnecessarily, so starting GDB like this is preferable. -gdb_start - -# Skip all tests if Python scripting is not enabled. -if { [skip_python_tests] } { continue } - -if { [prepare_for_testing "failed to prepare" $testfile $srcfile debug] } { - untested "failed to compile" - return -1 -} - -# This proc restarts GDB, makes the inferior reach the desired spot - marked -# by a comment in the .c file - and turns on the pretty printer for testing. -# Starting with a new GDB is important because the test may crash GDB. The -# return values are here to avoid us trying to test the pretty printer if -# there was a problem getting to main. -proc start_test { breakpoint_comment } { - global srcdir subdir testfile binfile - - # Start with a fresh gdb. - # This is important because the test can crash GDB. - - clean_restart ${binfile} - - if { ![runto_main] } then { - untested "couldn't run to breakpoint" - return -1 - } - - # Let GDB get to the return line. - gdb_breakpoint [gdb_get_line_number ${breakpoint_comment} ${testfile}.c ] - gdb_continue_to_breakpoint ${breakpoint_comment} ".*" - - gdb_test_no_output "set print pretty on" "starting to pretty print" - - set remote_python_file [gdb_remote_download host ${srcdir}/${subdir}/${testfile}.py] - gdb_test_no_output "source ${remote_python_file}" "load python file" - - return 0 -} - -# Testing the backtrace command. -with_test_prefix "frame print" { - if { [start_test "TAG: final frame"] == 0 } { - setup_kfail gdb/28856 "*-*-*" - gdb_test "backtrace -frame-arguments all" [multi_line \ - "#0 .*g \\(mt=mytype is .*\\).*"\ - "#1 .*g \\(mt=mytype is .*\\).*"\ - "#2 .*g \\(mt=mytype is .*\\).*"\ - "#3 .*g \\(mt=mytype is .*\\).*"\ - "#4 .*g \\(mt=mytype is .*\\).*"\ - "#5 .*g \\(mt=mytype is .*\\).*"\ - "#6 .*g \\(mt=mytype is .*\\).*"\ - "#7 .*g \\(mt=mytype is .*\\).*"\ - "#8 .*g \\(mt=mytype is .*\\).*"\ - "#9 .*g \\(mt=mytype is .*\\).*"\ - "#10 .*g \\(mt=mytype is .*\\).*"\ - "#11 .*main \\(\\).*"] \ - "backtrace test" - } -} -# Testing the down command. -with_test_prefix "frame movement down" { - if { [start_test "TAG: first frame"] == 0 } { - gdb_test "up" [multi_line "#1 .*in main \\(\\) at .*" ".*outside the frame.*"] - setup_kfail gdb/28856 "*-*-*" - gdb_test "down" [multi_line "#0\\s+g \\(mt=mytype is .*\\).*" ".*first frame.*"] - } -} - -# Testing the up command. -with_test_prefix "frame movement up" { - if { [start_test "TAG: final frame"] == 0 } { - setup_kfail gdb/28856 "*-*-*" - gdb_test "up" [multi_line "#1 .*in g \\(mt=mytype is .*\\).*" ".*first frame.*"] - } -} - -# Testing the finish command. -with_test_prefix "frame exit through finish" { - if { [start_test "TAG: final frame"] == 0 } { - setup_kfail gdb/28856 "*-*-*" - gdb_test "finish" [multi_line "Run till exit from #0 .*" ".* g \\(mt=mytype is .*\\, depth=1\\).*" ".*first frame.*"] - } -} - -# Testing the step command. -with_test_prefix "frame enter through step" { - if { [start_test "TAG: outside the frame"] == 0 } { - setup_kfail gdb/28856 "*-*-*" - gdb_test "step" [multi_line "g \\(mt=mytype is .*\\, depth=10\\).*" "41.*if \\(depth \\<= 0\\)"] - } -} - -# Testing the continue command. -with_test_prefix "frame enter through continue" { - if { [start_test "TAG: outside the frame"] == 0 } { - setup_kfail gdb/28856 "*-*-*" - gdb_breakpoint [gdb_get_line_number "TAG: first frame" ${testfile}.c ] - gdb_continue_to_breakpoint "TAG: first frame" ".*TAG: first frame.*" - } -} diff --git a/gdb/testsuite/gdb.python/pretty-print-call-by-hand.py b/gdb/testsuite/gdb.python/pretty-print-call-by-hand.py deleted file mode 100644 index 5343af831a7..00000000000 --- a/gdb/testsuite/gdb.python/pretty-print-call-by-hand.py +++ /dev/null @@ -1,45 +0,0 @@ -# Copyright (C) 2022 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/>. - - -class MytypePrinter: - """pretty print my type""" - - def __init__(self, val): - self.val = val - - def to_string(self): - calls = gdb.parse_and_eval("f()") - return "mytype is %s" % self.val["x"] - - -def ec_lookup_function(val): - typ = val.type - if typ.code == gdb.TYPE_CODE_REF: - typ = typ.target() - if str(typ) == "struct mytype": - return MytypePrinter(val) - return None - - -def disable_lookup_function(): - ec_lookup_function.enabled = False - - -def enable_lookup_function(): - ec_lookup_function.enabled = True - - -gdb.pretty_printers.append(ec_lookup_function)
reply other threads:[~2022-03-24 14:25 UTC|newest] Thread overview: [no followups] expand[flat|nested] mbox.gz Atom feed
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20220324142520.623333858C2C@sourceware.org \ --to=simark@sourceware.org \ --cc=gdb-cvs@sourceware.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).