summaryrefslogtreecommitdiffstats
path: root/Lib/socketserver.py
diff options
context:
space:
mode:
authorAntoine Pitrou <solipsis@pitrou.net>2010-04-25 22:01:43 (GMT)
committerAntoine Pitrou <solipsis@pitrou.net>2010-04-25 22:01:43 (GMT)
commit3bcba8e2889708f646df50953cd45a4f2cf28b02 (patch)
tree4edf42c36426da21843674f41a170c1e10368877 /Lib/socketserver.py
parenta23831ff44e0178006a0039d218d60252d5bd755 (diff)
downloadcpython-3bcba8e2889708f646df50953cd45a4f2cf28b02.zip
cpython-3bcba8e2889708f646df50953cd45a4f2cf28b02.tar.gz
cpython-3bcba8e2889708f646df50953cd45a4f2cf28b02.tar.bz2
Merged revisions 80484 via svnmerge from
svn+ssh://pythondev@svn.python.org/python/trunk ........ r80484 | antoine.pitrou | 2010-04-25 23:40:32 +0200 (dim., 25 avril 2010) | 6 lines Issue #2302: Fix a race condition in SocketServer.BaseServer.shutdown, where the method could block indefinitely if called just before the event loop started running. This also fixes the occasional freezes witnessed in test_httpservers. ........
Diffstat (limited to 'Lib/socketserver.py')
-rw-r--r--Lib/socketserver.py26
1 files changed, 14 insertions, 12 deletions
diff --git a/Lib/socketserver.py b/Lib/socketserver.py
index dacec03..18aec29 100644
--- a/Lib/socketserver.py
+++ b/Lib/socketserver.py
@@ -198,7 +198,7 @@ class BaseServer:
self.server_address = server_address
self.RequestHandlerClass = RequestHandlerClass
self.__is_shut_down = threading.Event()
- self.__serving = False
+ self.__shutdown_request = False
def server_activate(self):
"""Called by constructor to activate the server.
@@ -215,17 +215,19 @@ class BaseServer:
self.timeout. If you need to do periodic tasks, do them in
another thread.
"""
- self.__serving = True
self.__is_shut_down.clear()
- while self.__serving:
- # XXX: Consider using another file descriptor or
- # connecting to the socket to wake this up instead of
- # polling. Polling reduces our responsiveness to a
- # shutdown request and wastes cpu at all other times.
- r, w, e = select.select([self], [], [], poll_interval)
- if r:
- self._handle_request_noblock()
- self.__is_shut_down.set()
+ try:
+ while not self.__shutdown_request:
+ # XXX: Consider using another file descriptor or
+ # connecting to the socket to wake this up instead of
+ # polling. Polling reduces our responsiveness to a
+ # shutdown request and wastes cpu at all other times.
+ r, w, e = select.select([self], [], [], poll_interval)
+ if self in r:
+ self._handle_request_noblock()
+ finally:
+ self.__shutdown_request = False
+ self.__is_shut_down.set()
def shutdown(self):
"""Stops the serve_forever loop.
@@ -234,7 +236,7 @@ class BaseServer:
serve_forever() is running in another thread, or it will
deadlock.
"""
- self.__serving = False
+ self.__shutdown_request = True
self.__is_shut_down.wait()
# The distinction between handling, getting, processing and