diff options
author | Peter Hurley <peter@hurleysoftware.com> | 2014-10-16 14:59:50 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2014-11-05 19:26:14 -0500 |
commit | a361858da327263e04dc521ca39091d3119ccff8 (patch) | |
tree | 5efa814f84a21ab82f81f00ed3016e05ac0098d9 /drivers/tty/tty_io.c | |
parent | e218eb32f508c828dc87d0d724c70e2cf9b7866e (diff) |
tty: Update code comment in __proc_set_tty()
The session and foreground process group pid references will be
non-NULL if tiocsctty() is stealing the controlling tty from another
session (ie., arg == 1 in tiocsctty()).
Signed-off-by: Peter Hurley <peter@hurleysoftware.com>
Reviewed-by: Alan Cox <alan@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/tty/tty_io.c')
-rw-r--r-- | drivers/tty/tty_io.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/drivers/tty/tty_io.c b/drivers/tty/tty_io.c index 07275d0e30a2..9d1e247ee330 100644 --- a/drivers/tty/tty_io.c +++ b/drivers/tty/tty_io.c | |||
@@ -516,8 +516,11 @@ static void __proc_set_tty(struct tty_struct *tty) | |||
516 | { | 516 | { |
517 | unsigned long flags; | 517 | unsigned long flags; |
518 | 518 | ||
519 | /* We should not have a session or pgrp to put here but.... */ | ||
520 | spin_lock_irqsave(&tty->ctrl_lock, flags); | 519 | spin_lock_irqsave(&tty->ctrl_lock, flags); |
520 | /* | ||
521 | * The session and fg pgrp references will be non-NULL if | ||
522 | * tiocsctty() is stealing the controlling tty | ||
523 | */ | ||
521 | put_pid(tty->session); | 524 | put_pid(tty->session); |
522 | put_pid(tty->pgrp); | 525 | put_pid(tty->pgrp); |
523 | tty->pgrp = get_pid(task_pgrp(current)); | 526 | tty->pgrp = get_pid(task_pgrp(current)); |