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.
........
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 @@
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 @@
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 @@
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