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: link
Be 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).