diff options
| author | Sebastian Andrzej Siewior <bigeasy@linutronix.de> | 2012-08-07 20:12:30 +0400 |
|---|---|---|
| committer | Oleg Nesterov <oleg@redhat.com> | 2012-08-28 20:21:16 +0400 |
| commit | 8bd874456e2ec49b9e64372ddc89a6f88901d184 (patch) | |
| tree | fe918f684d04e88c7ba55c5ba56ee4ce7f3c7b74 /kernel/events | |
| parent | 61e1d394984110e2e76f25572d5b1b5d48796751 (diff) | |
| download | linux-8bd874456e2ec49b9e64372ddc89a6f88901d184.tar.xz | |
uprobes: Remove check for uprobe variable in handle_swbp()
by the time we get here (after we pass cleanup_ret) uprobe is always is
set. If it is NULL we leave very early in the code.
Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
Signed-off-by: Oleg Nesterov <oleg@redhat.com>
Diffstat (limited to 'kernel/events')
| -rw-r--r-- | kernel/events/uprobes.c | 16 |
1 files changed, 7 insertions, 9 deletions
diff --git a/kernel/events/uprobes.c b/kernel/events/uprobes.c index 7cff24c60dd7..0cefde276641 100644 --- a/kernel/events/uprobes.c +++ b/kernel/events/uprobes.c @@ -1516,17 +1516,15 @@ cleanup_ret: utask->active_uprobe = NULL; utask->state = UTASK_RUNNING; } - if (uprobe) { - if (!(uprobe->flags & UPROBE_SKIP_SSTEP)) + if (!(uprobe->flags & UPROBE_SKIP_SSTEP)) - /* - * cannot singlestep; cannot skip instruction; - * re-execute the instruction. - */ - instruction_pointer_set(regs, bp_vaddr); + /* + * cannot singlestep; cannot skip instruction; + * re-execute the instruction. + */ + instruction_pointer_set(regs, bp_vaddr); - put_uprobe(uprobe); - } + put_uprobe(uprobe); } /* |