summaryrefslogtreecommitdiffstats
path: root/Lib/multiprocessing
diff options
context:
space:
mode:
authorVictor Stinner <vstinner@python.org>2021-04-16 17:42:34 (GMT)
committerGitHub <noreply@github.com>2021-04-16 17:42:34 (GMT)
commit7c29ae1f0585378dba4d220a2c0fb5dd49fdab3e (patch)
treeb02839b2fdec7e81707bd28bdbba5cb8715174c1 /Lib/multiprocessing
parent62ec63864822de1dd1933225584e8503ac40c1f9 (diff)
downloadcpython-7c29ae1f0585378dba4d220a2c0fb5dd49fdab3e.zip
cpython-7c29ae1f0585378dba4d220a2c0fb5dd49fdab3e.tar.gz
cpython-7c29ae1f0585378dba4d220a2c0fb5dd49fdab3e.tar.bz2
bpo-43867: multiprocessing Server catchs SystemExit (GH-25441)
The multiprocessing Server class now explicitly catchs SystemExit and closes the client connection in this case. It happens when the Server.serve_client() method reachs the end of file (EOF).
Diffstat (limited to 'Lib/multiprocessing')
-rw-r--r--Lib/multiprocessing/managers.py20
1 files changed, 14 insertions, 6 deletions
diff --git a/Lib/multiprocessing/managers.py b/Lib/multiprocessing/managers.py
index 0eb16c6..6ee9521 100644
--- a/Lib/multiprocessing/managers.py
+++ b/Lib/multiprocessing/managers.py
@@ -192,11 +192,8 @@ class Server(object):
t.daemon = True
t.start()
- def handle_request(self, c):
- '''
- Handle a new connection
- '''
- funcname = result = request = None
+ def _handle_request(self, c):
+ request = None
try:
connection.deliver_challenge(c, self.authkey)
connection.answer_challenge(c, self.authkey)
@@ -213,6 +210,7 @@ class Server(object):
msg = ('#TRACEBACK', format_exc())
else:
msg = ('#RETURN', result)
+
try:
c.send(msg)
except Exception as e:
@@ -224,7 +222,17 @@ class Server(object):
util.info(' ... request was %r', request)
util.info(' ... exception was %r', e)
- c.close()
+ def handle_request(self, conn):
+ '''
+ Handle a new connection
+ '''
+ try:
+ self._handle_request(conn)
+ except SystemExit:
+ # Server.serve_client() calls sys.exit(0) on EOF
+ pass
+ finally:
+ conn.close()
def serve_client(self, conn):
'''