diff options
author | Oleg Nesterov <oleg@tv-sign.ru> | 2006-02-15 22:50:10 +0300 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-02-15 11:05:43 -0800 |
commit | 5ecfbae093f0c37311e89b29bfc0c9d586eace87 (patch) | |
tree | eabd0a145af64e26c900578c95175ab313828661 /fs | |
parent | dadac81b1b86196fcc48fb87620403c4a7174f06 (diff) | |
download | op-kernel-dev-5ecfbae093f0c37311e89b29bfc0c9d586eace87.zip op-kernel-dev-5ecfbae093f0c37311e89b29bfc0c9d586eace87.tar.gz |
[PATCH] fix zap_thread's ptrace related problems
1. The tracee can go from ptrace_stop() to do_signal_stop()
after __ptrace_unlink(p).
2. It is unsafe to __ptrace_unlink(p) while p->parent may wait
for tasklist_lock in ptrace_detach().
Signed-off-by: Oleg Nesterov <oleg@tv-sign.ru>
Cc: Roland McGrath <roland@redhat.com>
Cc: Ingo Molnar <mingo@elte.hu>
Cc: Christoph Hellwig <hch@lst.de>
Cc: Eric W. Biederman <ebiederm@xmission.com>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'fs')
-rw-r--r-- | fs/exec.c | 2 |
1 files changed, 1 insertions, 1 deletions
@@ -1403,7 +1403,7 @@ static void zap_threads (struct mm_struct *mm) do_each_thread(g,p) { if (mm == p->mm && p != tsk && p->ptrace && p->parent->mm == mm) { - __ptrace_unlink(p); + __ptrace_detach(p, 0); } } while_each_thread(g,p); write_unlock_irq(&tasklist_lock); |