[3.13] gh-124986: Fix test_no_leaking in test_subprocess on NetBSD and FreeBSD (GH-132476) (GH-132498)

On platforms where the file descriptor limit is larger than FD_SETSIZE
that test was always skipped (FreeBSD) or always failing (NetBSD).
(cherry picked from commit f7b24ffefd)

Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
This commit is contained in:
Miss Islington (bot) 2025-04-14 08:38:13 +02:00 committed by GitHub
parent f206b98c18
commit ff9198dab3
No known key found for this signature in database
GPG key ID: B5690EEEBB952194

View file

@ -40,6 +40,10 @@ try:
import grp
except ImportError:
grp = None
try:
import resource
except ImportError:
resource = None
try:
import fcntl
@ -1210,6 +1214,16 @@ class ProcessTestCase(BaseTestCase):
max_handles = 1026 # too much for most UNIX systems
else:
max_handles = 2050 # too much for (at least some) Windows setups
if resource:
# And if it is not too much, try to make it too much.
try:
soft, hard = resource.getrlimit(resource.RLIMIT_NOFILE)
if soft > 1024:
resource.setrlimit(resource.RLIMIT_NOFILE, (1024, hard))
self.addCleanup(resource.setrlimit, resource.RLIMIT_NOFILE,
(soft, hard))
except (OSError, ValueError):
pass
handles = []
tmpdir = tempfile.mkdtemp()
try:
@ -1224,7 +1238,9 @@ class ProcessTestCase(BaseTestCase):
else:
self.skipTest("failed to reach the file descriptor limit "
"(tried %d)" % max_handles)
# Close a couple of them (should be enough for a subprocess)
# Close a couple of them (should be enough for a subprocess).
# Close lower file descriptors, so select() will work.
handles.reverse()
for i in range(10):
os.close(handles.pop())
# Loop creating some subprocesses. If one of them leaks some fds,