diff options
author | Pedro Alves <palves@redhat.com> | 2012-06-05 19:50:50 +0000 |
---|---|---|
committer | Pedro Alves <palves@redhat.com> | 2012-06-05 19:50:50 +0000 |
commit | 8556afb4f53f99b32e75285a7eb3839e0b6911ee (patch) | |
tree | 030a5b9fee25f396c3263a2cd6f4298f99d59334 /gdb | |
parent | 91cc9324e0bda6b53a4ccd548cc87f98d4b31a2d (diff) | |
download | binutils-8556afb4f53f99b32e75285a7eb3839e0b6911ee.zip binutils-8556afb4f53f99b32e75285a7eb3839e0b6911ee.tar.gz binutils-8556afb4f53f99b32e75285a7eb3839e0b6911ee.tar.bz2 |
2012-06-05 Pedro Alves <palves@redhat.com>
PR backtrace/13866
* breakpoint.c (until_break_command): Only fetch the selected
frame after decode_line_1.
Diffstat (limited to 'gdb')
-rw-r--r-- | gdb/ChangeLog | 7 | ||||
-rw-r--r-- | gdb/breakpoint.c | 19 |
2 files changed, 20 insertions, 6 deletions
diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 2bb9d81..5c5f8c5 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,10 @@ +2012-06-05 Pedro Alves <palves@redhat.com> + + PR backtrace/13866 + + * breakpoint.c (until_break_command): Only fetch the selected + frame after decode_line_1. + 2012-06-05 Joakim Tjernlund <Joakim.Tjernlund@transmode.se> * solib-svr4.c (enable_break): Don't fallback to setting the solib diff --git a/gdb/breakpoint.c b/gdb/breakpoint.c index 5cc1f64..12db39b 100644 --- a/gdb/breakpoint.c +++ b/gdb/breakpoint.c @@ -10815,10 +10815,10 @@ until_break_command (char *arg, int from_tty, int anywhere) { struct symtabs_and_lines sals; struct symtab_and_line sal; - struct frame_info *frame = get_selected_frame (NULL); - struct gdbarch *frame_gdbarch = get_frame_arch (frame); - struct frame_id stack_frame_id = get_stack_frame_id (frame); - struct frame_id caller_frame_id = frame_unwind_caller_id (frame); + struct frame_info *frame; + struct gdbarch *frame_gdbarch; + struct frame_id stack_frame_id; + struct frame_id caller_frame_id; struct breakpoint *breakpoint; struct breakpoint *breakpoint2 = NULL; struct cleanup *old_chain; @@ -10854,8 +10854,15 @@ until_break_command (char *arg, int from_tty, int anywhere) old_chain = make_cleanup (null_cleanup, NULL); - /* Installing a breakpoint invalidates the frame chain (as it may - need to switch threads), so do any frame handling first. */ + /* Note linespec handling above invalidates the frame chain. + Installing a breakpoint also invalidates the frame chain (as it + may need to switch threads), so do any frame handling before + that. */ + + frame = get_selected_frame (NULL); + frame_gdbarch = get_frame_arch (frame); + stack_frame_id = get_stack_frame_id (frame); + caller_frame_id = frame_unwind_caller_id (frame); /* Keep within the current frame, or in frames called by the current one. */ |