diff options
author | Eric W. Biederman <ebiederm@xmission.com> | 2006-03-28 19:10:58 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-03-28 21:36:40 -0500 |
commit | fef23e7fbb11a0a78cd61935f7056bc2b237995a (patch) | |
tree | a6ff460b3a5d11d01c0532de561eb81183e056c2 /kernel/exit.c | |
parent | ca9ba4471c1203bb6e759b76e83167fec54fe590 (diff) |
[PATCH] exec: allow init to exec from any thread.
After looking at the problem of init calling exec some more I figured out
an easy way to make the code work.
The actual symptom without out this patch is that all threads will die
except pid == 1, and the thread calling exec. The thread calling exec will
wait forever for pid == 1 to die.
Since pid == 1 does not install a handler for SIGKILL it will never die.
This modifies the tests for init from current->pid == 1 to the equivalent
current == child_reaper. And then it causes exec in the ugly case to
modify child_reaper.
The only weird symptom is that you wind up with an init process that
doesn't have the oldest start time on the box.
Signed-off-by: Eric W. Biederman <ebiederm@xmission.com>
Cc: Oleg Nesterov <oleg@tv-sign.ru>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'kernel/exit.c')
-rw-r--r-- | kernel/exit.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/kernel/exit.c b/kernel/exit.c index a8c7efc7a681..223a8802b665 100644 --- a/kernel/exit.c +++ b/kernel/exit.c | |||
@@ -807,7 +807,7 @@ fastcall NORET_TYPE void do_exit(long code) | |||
807 | panic("Aiee, killing interrupt handler!"); | 807 | panic("Aiee, killing interrupt handler!"); |
808 | if (unlikely(!tsk->pid)) | 808 | if (unlikely(!tsk->pid)) |
809 | panic("Attempted to kill the idle task!"); | 809 | panic("Attempted to kill the idle task!"); |
810 | if (unlikely(tsk->pid == 1)) | 810 | if (unlikely(tsk == child_reaper)) |
811 | panic("Attempted to kill init!"); | 811 | panic("Attempted to kill init!"); |
812 | 812 | ||
813 | if (unlikely(current->ptrace & PT_TRACE_EXIT)) { | 813 | if (unlikely(current->ptrace & PT_TRACE_EXIT)) { |