On 01/19/2015, 06:49 PM, Peter Hurley wrote:
> Commit 26df6d13406d1a5 ("tty: Add EXTPROC support for LINEMODE")
> allows a process which has opened a pty master to send _any_ signal
> to the process group of the pty slave. Although potentially
> exploitable by a malicious program running a setuid program on
> a pty slave, it's unknown if this exploit currently exists.
>
> Limit to signals actually used.
>
> Cc: Theodore Ts'o <[email protected]>
> Cc: Howard Chu <[email protected]>
> Cc: One Thousand Gnomes <[email protected]>
> Cc: Jiri Slaby <[email protected]>
> Cc: <[email protected]> # 2.6.36+
> Signed-off-by: Peter Hurley <[email protected]>
> ---
> drivers/tty/pty.c | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/drivers/tty/pty.c b/drivers/tty/pty.c
> index a9d256d..46366f0 100644
> --- a/drivers/tty/pty.c
> +++ b/drivers/tty/pty.c
> @@ -210,6 +210,9 @@ static int pty_signal(struct tty_struct *tty, int sig)
> {
> struct pid *pgrp;
>
> + if (sig != SIGINT || sig != SIGQUIT || sig != SIGTSTP)
> + return -EINVAL;
s/||/\&\&/ :)
--
js
suse labs
--
To unsubscribe from this list: send the line "unsubscribe stable" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html