diff options
| author | Mohamed Bouhaouel <mohamed.bouhaouel@intel.com> | 2023-06-30 10:10:15 +0200 |
|---|---|---|
| committer | Tom Tromey <tromey@adacore.com> | 2023-09-19 06:56:53 -0600 |
| commit | 093da43d2adb4497dfec8afbb4eeaf2425668fdd (patch) | |
| tree | 17a7c274bf741c4888997021519c2ccf0e17da02 /gdb/breakpoint.h | |
| parent | 12f567bcb640285b0afab74e2ac8d471a175722d (diff) | |
| download | gdb-093da43d2adb4497dfec8afbb4eeaf2425668fdd.zip gdb-093da43d2adb4497dfec8afbb4eeaf2425668fdd.tar.gz gdb-093da43d2adb4497dfec8afbb4eeaf2425668fdd.tar.bz2 | |
gdb, breakpoint: add a destructor to the watchpoint struct
Make sure to unlink the related breakpoint when the watchpoint instance
is deleted. This prevents having a wp-related breakpoint that is
linked to a NULL watchpoint (e.g. the watchpoint instance is being
deleted when the 'watch' command fails). With the below scenario,
having such a left out breakpoint will lead to a GDB hang, and this
is due to an infinite loop when deleting all inferior breakpoints.
Scenario:
(gdb) set can-use-hw-watchpoints 0
(gdb) awatch <SCOPE VAR>
Can't set read/access watchpoint when hardware watchpoints are disabled.
(gdb) rwatch <SCOPE VAR>
Can't set read/access watchpoint when hardware watchpoints are disabled.
(gdb) <continue the program until the end>
>> HANG <<
Signed-off-by: Mohamed Bouhaouel <mohamed.bouhaouel@intel.com>
Reviewed-by: Bruno Larsen <blarsen@redhat.com>
Diffstat (limited to 'gdb/breakpoint.h')
| -rw-r--r-- | gdb/breakpoint.h | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/gdb/breakpoint.h b/gdb/breakpoint.h index 1a73d08..e75efc9 100644 --- a/gdb/breakpoint.h +++ b/gdb/breakpoint.h @@ -1000,6 +1000,9 @@ struct watchpoint : public breakpoint void print_recreate (struct ui_file *fp) const override; bool explains_signal (enum gdb_signal) override; + /* Destructor for WATCHPOINT. */ + ~watchpoint (); + /* String form of exp to use for displaying to the user (malloc'd), or NULL if none. */ gdb::unique_xmalloc_ptr<char> exp_string; |