improve SOCK_NONBLOCK/SOCK_CLOEXEC fallback code
authorRich Felker <dalias@aerifal.cx>
Mon, 5 Nov 2012 19:30:15 +0000 (14:30 -0500)
committerRich Felker <dalias@aerifal.cx>
Mon, 5 Nov 2012 19:30:15 +0000 (14:30 -0500)
commite8a0b27a7101f5b9939de83df3d6d8b606c5678b
tree7545ed6491dff15ae0d71d38f778876b5e64d14f
parent32d6d77e544dbb1d9c44cacfc2ea0cbfe5ccfdef
improve SOCK_NONBLOCK/SOCK_CLOEXEC fallback code

checking for EINVAL should be sufficient, but qemu user emulation
returns EPROTONOSUPPORT in some of the failure cases, and it seems
conceivable that other kernels doing linux-emulation could make the
same mistake. since DNS lookups and other important code might break
if the fallback does not get invoked, be extra careful and check for
either error.

note that it's important NOT to perform the fallback code on other
errors such as resource-exhaustion cases, since the fallback is not
atomic and will lead to file-descriptor leaks in multi-threaded
programs that use exec. the fallback code is only "safe" to run when
the initial failure is caused by the application's choice of
arguments, not the system state.
src/network/socket.c