diff options
| author | Petr Machata <pmachata@redhat.com> | 2013-03-05 17:10:37 +0100 |
|---|---|---|
| committer | Petr Machata <pmachata@redhat.com> | 2013-03-05 17:10:37 +0100 |
| commit | a9d6a6732c5b0b692a40ac073695332078a73738 (patch) | |
| tree | de5fbab4de02d20194df3500bb3ed5774f70aea2 | |
| parent | bc0de433c25d224cf5f4743ba4a78131267c532d (diff) | |
| download | ltrace-a9d6a6732c5b0b692a40ac073695332078a73738.tar.gz | |
Remove hacks for SPARC code
Simply continuing on SPARC is probably broken. The breakpoint isn't
disabled at that point, and continuing just hits it again.
That __ia64___ in ifdef is never defined even on an IA64 machine.
It's a typo, note the three underscores.
| -rw-r--r-- | sysdeps/linux-gnu/trace.c | 15 |
1 files changed, 4 insertions, 11 deletions
diff --git a/sysdeps/linux-gnu/trace.c b/sysdeps/linux-gnu/trace.c index e57a5ed..d10e665 100644 --- a/sysdeps/linux-gnu/trace.c +++ b/sysdeps/linux-gnu/trace.c @@ -907,18 +907,11 @@ continue_after_breakpoint(struct process *proc, struct breakpoint *sbp) if (sbp->enabled == 0) { continue_process(proc->pid); - } else { -#if defined __sparc__ || defined __ia64___ - /* we don't want to singlestep here */ + } else if (process_install_stopping_handler + (proc, sbp, NULL, NULL, NULL) < 0) { + perror("process_stopping_handler_create"); + /* Carry on not bothering to re-enable. */ continue_process(proc->pid); -#else - if (process_install_stopping_handler - (proc, sbp, NULL, NULL, NULL) < 0) { - perror("process_stopping_handler_create"); - /* Carry on not bothering to re-enable. */ - continue_process(proc->pid); - } -#endif } } |