diff options
| author | Oleg Nesterov <[email protected]> | 2016-01-20 14:59:58 -0800 |
|---|---|---|
| committer | Linus Torvalds <[email protected]> | 2016-01-20 17:09:18 -0800 |
| commit | 570ac9337b5c13dbf46ca6758c376e2e13e8956f (patch) | |
| tree | 361e5714890df92ffe88ea81b6125ed165511d61 | |
| parent | 7c3b00e06d731a28fc3d17ed02ba250642b15b81 (diff) | |
ptrace: task_stopped_code(ptrace => true) can't see TASK_STOPPED task
task_stopped_code()->task_is_stopped_or_traced() doesn't look right, the
traced task must never be TASK_STOPPED.
We can not add WARN_ON(task_is_stopped(p)), but this is only because
do_wait() can race with PTRACE_ATTACH from another thread.
[[email protected]: teeny cleanup]
Signed-off-by: Oleg Nesterov <[email protected]>
Cc: Andrey Ryabinin <[email protected]>
Cc: Roland McGrath <[email protected]>
Acked-by: Tejun Heo <[email protected]>
Cc: Pedro Alves <[email protected]>
Cc: Jan Kratochvil <[email protected]>
Signed-off-by: Andrew Morton <[email protected]>
Signed-off-by: Linus Torvalds <[email protected]>
| -rw-r--r-- | kernel/exit.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/kernel/exit.c b/kernel/exit.c index 07110c6020a0..b0eea830303c 100644 --- a/kernel/exit.c +++ b/kernel/exit.c @@ -1120,8 +1120,7 @@ static int wait_task_zombie(struct wait_opts *wo, struct task_struct *p) static int *task_stopped_code(struct task_struct *p, bool ptrace) { if (ptrace) { - if (task_is_stopped_or_traced(p) && - !(p->jobctl & JOBCTL_LISTENING)) + if (task_is_traced(p) && !(p->jobctl & JOBCTL_LISTENING)) return &p->exit_code; } else { if (p->signal->flags & SIGNAL_STOP_STOPPED) |