restore blocking status on stdio fds before close
authordjm <djm@openbsd.org>
Wed, 19 May 2021 01:24:05 +0000 (01:24 +0000)
committerdjm <djm@openbsd.org>
Wed, 19 May 2021 01:24:05 +0000 (01:24 +0000)
commit43d5df635286bf17a0964f9867df400f13400856
tree43f0c52817f7bcb5e70d65f44227f824947f266c
parentabe06632b2f7a14b244c83fb06eb6a91a4b224fd
restore blocking status on stdio fds before close

ssh(1) needs to set file descriptors to non-blocking mode to operate
but it was not restoring the original state on exit. This could cause
problems with fds shared with other programs via the shell, e.g.

> $ cat > test.sh << _EOF
> #!/bin/sh
> {
>         ssh -Fnone -oLogLevel=verbose ::1 hostname
>         cat /usr/share/dict/words
> } | sleep 10
> _EOF
> $ ./test.sh
> Authenticated to ::1 ([::1]:22).
> Transferred: sent 2352, received 2928 bytes, in 0.1 seconds
> Bytes per second: sent 44338.9, received 55197.4
> cat: stdout: Resource temporarily unavailable

This restores the blocking status for fds 0,1,2 (stdio) before ssh(1)
abandons/closes them.

This was reported as bz3280 and GHPR246; ok dtucker@
usr.bin/ssh/channels.c
usr.bin/ssh/channels.h
usr.bin/ssh/clientloop.c
usr.bin/ssh/mux.c
usr.bin/ssh/nchan.c
usr.bin/ssh/ssh.c