]> git.ipfire.org Git - thirdparty/haproxy.git/commit
BUG/MINOR: fd: my_closefrom() on Linux could skip contiguous series of sockets
authorWilly Tarreau <w@1wt.eu>
Fri, 19 Apr 2024 14:52:32 +0000 (16:52 +0200)
committerWilly Tarreau <w@1wt.eu>
Fri, 19 Apr 2024 15:06:21 +0000 (17:06 +0200)
commit64d20fc9e0a914ad2fe03d74972b56baaf54b90f
treeb1a8b21845b77ed90bc23865f1b365a547b78e4b
parentb4734c2bd7981846142ecabe3490bbf356993b08
BUG/MINOR: fd: my_closefrom() on Linux could skip contiguous series of sockets

We got a detailed report analysis showing that our optimization consisting
in using poll() to detect already closed FDs within a 1024 range has an
issue with the case where 1024 consecutive FDs are open (hence do not show
POLLNVAL) and none of them has any activity report. In this case poll()
returns zero update and we would just skip the loop that inspects all the
FDs to close the valid ones. One visible effect is that the called programs
might occasionally see some FDs being exposed in the low range of their fd
space, possibly making the process run out of FDs when trying to open a
file for example.

Note that this is actually a fix for commit b8e602cb1b ("BUG/MINOR: fd:
make sure my_closefrom() doesn't miss some FDs") that already faced a
more common form of this problem (incomplete but non-empty FDs reported).

This can be backported up to 2.0.
src/fd.c