summaryrefslogtreecommitdiff
path: root/include/linux/tty.h
diff options
context:
space:
mode:
authorJann Horn <jannh@google.com>2020-12-03 02:25:04 +0100
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2020-12-04 17:38:48 +0100
commit54ffccbf053b5b6ca4f6e45094b942fab92a25fc (patch)
treeb30e76730582ec6960a35741dea3023040afbaf0 /include/linux/tty.h
parent418baf2c28f3473039f2f7377760bd8f6897ae18 (diff)
tty: Fix ->pgrp locking in tiocspgrp()
tiocspgrp() takes two tty_struct pointers: One to the tty that userspace passed to ioctl() (`tty`) and one to the TTY being changed (`real_tty`). These pointers are different when ioctl() is called with a master fd. To properly lock real_tty->pgrp, we must take real_tty->ctrl_lock. This bug makes it possible for racing ioctl(TIOCSPGRP, ...) calls on both sides of a PTY pair to corrupt the refcount of `struct pid`, leading to use-after-free errors. Fixes: 47f86834bbd4 ("redo locking of tty->pgrp") CC: stable@kernel.org Signed-off-by: Jann Horn <jannh@google.com> Reviewed-by: Jiri Slaby <jirislaby@kernel.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'include/linux/tty.h')
0 files changed, 0 insertions, 0 deletions