From: Pedro Alves Date: Mon, 30 Mar 2015 15:39:56 +0000 (+0100) Subject: infrun.c:resume: currently_stepping after clearing stepped_breakpoint X-Git-Tag: users/hjl/linux/release/2.25.51.0.2~2^2~18^2 X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=6b403daae993621542c249c00ca1e6d3cfef1aa4;p=thirdparty%2Fbinutils-gdb.git infrun.c:resume: currently_stepping after clearing stepped_breakpoint My all-stop-on-top-of-non-stop series manages to shows regressions due to this latent bug. currently_stepping returns true if stepped_breakpoint is set. Obviously we should clear it before checking currently_stepping, not after. Tested on x86_64 Fedora 20. gdb/ChangeLog: 2015-04-01 Pedro Alves * infrun.c (resume): Check currently_stepping after clearing stepped_breakpoint, not before. --- diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 7f9cc431fb2..b85b43723c1 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,8 @@ +2015-04-01 Pedro Alves + + * infrun.c (resume): Check currently_stepping after clearing + stepped_breakpoint, not before. + 2015-04-01 Pedro Alves * infrun.c (print_target_wait_results): Print all the ptid diff --git a/gdb/infrun.c b/gdb/infrun.c index 4b38fce8593..f5faa0a22af 100644 --- a/gdb/infrun.c +++ b/gdb/infrun.c @@ -2084,12 +2084,15 @@ resume (enum gdb_signal sig) This can decay from a step to a continue, if e.g., we need to implement single-stepping with breakpoints (software single-step). */ - int step = currently_stepping (tp); + int step; tp->stepped_breakpoint = 0; QUIT; + /* Depends on stepped_breakpoint. */ + step = currently_stepping (tp); + if (current_inferior ()->waiting_for_vfork_done) { /* Don't try to single-step a vfork parent that is waiting for