diff options
author | Alan Cox <alan@lxorguk.ukuu.org.uk> | 2006-09-25 23:33:08 -0700 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-09-26 08:49:09 -0700 |
commit | b1fc0b1f21c4082d24d1f456a846b4fa7d16a70b (patch) | |
tree | a2568be1bfd435ffcc398357577e8556b34a6fb9 /arch/um/kernel | |
parent | 75e29b18d9a46bf3193278e92dc95609a8cca2ab (diff) | |
download | blackbird-op-linux-b1fc0b1f21c4082d24d1f456a846b4fa7d16a70b.tar.gz blackbird-op-linux-b1fc0b1f21c4082d24d1f456a846b4fa7d16a70b.zip |
[PATCH] UML: tty locking
Ensure current->signal->tty doesn't get freed during log_exec().
Signed-off-by: Alan Cox <alan@redhat.com>
Acked-by: Jeff Dike <jdike@addtoit.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'arch/um/kernel')
-rw-r--r-- | arch/um/kernel/exec.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/arch/um/kernel/exec.c b/arch/um/kernel/exec.c index fc38a6d5906d..0561c43b4685 100644 --- a/arch/um/kernel/exec.c +++ b/arch/um/kernel/exec.c @@ -41,9 +41,11 @@ static long execve1(char *file, char __user * __user *argv, long error; #ifdef CONFIG_TTY_LOG - task_lock(current); + mutex_lock(&tty_mutex); + task_lock(current); /* FIXME: is this needed ? */ log_exec(argv, current->signal->tty); task_unlock(current); + mutex_unlock(&tty_mutex); #endif error = do_execve(file, argv, env, ¤t->thread.regs); if (error == 0){ |