summaryrefslogtreecommitdiffstats
path: root/drivers/tty/n_tty.c
diff options
context:
space:
mode:
authorOleg Nesterov <oleg@redhat.com>2013-01-29 20:07:41 +0100
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2013-02-04 15:02:14 -0800
commit183d95cdd834381c594d3aa801c1f9f9c0c54fa9 (patch)
tree8b24e0721242433a757f874f68ae21cf69acbb0d /drivers/tty/n_tty.c
parent4d9b109060f690f5c835130ff54165ae157b3087 (diff)
downloadlinux-183d95cdd834381c594d3aa801c1f9f9c0c54fa9.tar.gz
linux-183d95cdd834381c594d3aa801c1f9f9c0c54fa9.tar.bz2
linux-183d95cdd834381c594d3aa801c1f9f9c0c54fa9.zip
tty: set_termios/set_termiox should not return -EINTR
See https://bugzilla.redhat.com/show_bug.cgi?id=904907 read command causes bash to abort with double free or corruption (out). A simple test-case from Roman: // Compile the reproducer and send sigchld ti that process. // EINTR occurs even if SA_RESTART flag is set. void handler(int sig) { } main() { struct sigaction act; act.sa_handler = handler; act.sa_flags = SA_RESTART; sigaction (SIGCHLD, &act, 0); struct termio ttp; ioctl(0, TCGETA, &ttp); while(1) { if (ioctl(0, TCSETAW, ttp) < 0) { if (errno == EINTR) { fprintf(stderr, "BUG!"); return(1); } } } } Change set_termios/set_termiox to return -ERESTARTSYS to fix this particular problem. I didn't dare to change other EINTR's in drivers/tty/, but they look equally wrong. Reported-by: Roman Rakus <rrakus@redhat.com> Reported-by: Lingzhu Xiang <lxiang@redhat.com> Signed-off-by: Oleg Nesterov <oleg@redhat.com> Cc: Jiri Slaby <jslaby@suse.cz> Cc: stable <stable@vger.kernel.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/tty/n_tty.c')
0 files changed, 0 insertions, 0 deletions