From: Pedro Alves Date: Tue, 5 Jun 2012 19:50:50 +0000 (+0000) Subject: 2012-06-05 Pedro Alves X-Git-Tag: sid-snapshot-20120701~281 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=8556afb4f53f99b32e75285a7eb3839e0b6911ee;p=external%2Fbinutils.git 2012-06-05 Pedro Alves PR backtrace/13866 * breakpoint.c (until_break_command): Only fetch the selected frame after decode_line_1. --- 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 + + PR backtrace/13866 + + * breakpoint.c (until_break_command): Only fetch the selected + frame after decode_line_1. + 2012-06-05 Joakim Tjernlund * 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. */