mirrors/cpython
1
0
Fork 0
mirror of https://github.com/python/cpython.git synced 2025-09-26 10:19:53 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 8

bpo-43842: Fix race condition in test_logging SMTP test (GH-25436)

Browse source 
Fix a race condition in the SMTP test of test_logging. Don't close a
file descriptor (socket) from a different thread while
asyncore.loop() is polling the file descriptor.
This commit is contained in:
Victor Stinner 2021-04-16 14:32:01 +02:00 • committed by GitHub
parent 471870fc50
commit 75ec103b3a
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 9 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

7
Lib/test/test_logging.py
View file

@ -829,6 +829,7 @@ class TestSMTPServer(smtpd.SMTPServer):
self.port = self.socket.getsockname()[1] self.port = self.socket.getsockname()[1]
self._handler = handler self._handler = handler
self._thread = None self._thread = None
self._quit = False
self.poll_interval = poll_interval self.poll_interval = poll_interval
def process_message(self, peer, mailfrom, rcpttos, data): def process_message(self, peer, mailfrom, rcpttos, data):
@ -860,16 +861,18 @@ class TestSMTPServer(smtpd.SMTPServer):
:func:`select` or :func:`poll` call by :func:`select` or :func:`poll` call by
:func:`asyncore.loop`. :func:`asyncore.loop`.
""" """
asyncore.loop(poll_interval, map=self._map) while not self._quit:
asyncore.loop(poll_interval, map=self._map, count=1)
def stop(self): def stop(self):
""" """
Stop the thread by closing the server instance. Stop the thread by closing the server instance.
Wait for the server thread to terminate. Wait for the server thread to terminate.
""" """
self.close() self._quit = True
threading_helper.join_thread(self._thread) threading_helper.join_thread(self._thread)
self._thread = None self._thread = None
self.close()
asyncore.close_all(map=self._map, ignore_all=True) asyncore.close_all(map=self._map, ignore_all=True)

4
Misc/NEWS.d/next/Tests/2021-04-16-14-07-40.bpo-43842.w60GAH.rst Normal file
View file

@ -0,0 +1,4 @@
Fix a race condition in the SMTP test of test_logging. Don't close a file
descriptor (socket) from a different thread while asyncore.loop() is polling
the file descriptor.
Patch by Victor Stinner.