diff options
author | Richard Purdie <richard.purdie@linuxfoundation.org> | 2017-07-28 15:42:50 +0100 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2017-07-30 08:43:36 +0100 |
commit | 07cee9dea1ad3408a49e4b9d9f5318474ddfa60b (patch) | |
tree | 716bc34d7de4192bfcdaeac63a7e5dc9888145fb /bitbake | |
parent | b2c39835bb61687220abbef72e4f69a272cf8afb (diff) | |
download | openembedded-core-contrib-07cee9dea1ad3408a49e4b9d9f5318474ddfa60b.tar.gz |
bitbake: process: Don't leak open pipes upon reconnection
If we reconnect to the server, stop leaking pipes and clean up
after ourselves.
(Bitbake rev: f41e4e971e807157be68cf4496580494b8b60643)
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'bitbake')
-rw-r--r-- | bitbake/lib/bb/server/process.py | 19 |
1 files changed, 18 insertions, 1 deletions
diff --git a/bitbake/lib/bb/server/process.py b/bitbake/lib/bb/server/process.py index 6106c07380..3530bdc07e 100644 --- a/bitbake/lib/bb/server/process.py +++ b/bitbake/lib/bb/server/process.py @@ -420,7 +420,11 @@ def connectProcessServer(sockname, featureset): finally: os.chdir(cwd) + readfd = writefd = readfd1 = writefd1 = readfd2 = writefd2 = None + eq = command_chan_recv = command_chan = None + try: + # Send an fd for the remote to write events to readfd, writefd = os.pipe() eq = BBUIEventQueue(readfd) @@ -435,9 +439,22 @@ def connectProcessServer(sockname, featureset): server_connection = BitBakeProcessServerConnection(command_chan, command_chan_recv, eq, sock) + # Close the ends of the pipes we won't use + for i in [writefd, readfd1, writefd2]: + os.close(i) + server_connection.connection.updateFeatureSet(featureset) - except: + except (Exception, SystemExit) as e: + if command_chan_recv: + command_chan_recv.close() + if command_chan: + command_chan.close() + for i in [writefd, readfd1, writefd2]: + try: + os.close(i) + except OSError: + pass sock.close() raise |