diff options
author | Zac Medico <zmedico@gentoo.org> | 2012-02-14 04:29:02 -0800 |
---|---|---|
committer | Zac Medico <zmedico@gentoo.org> | 2012-02-14 04:29:02 -0800 |
commit | a75341bf3a66f75edd68d1a8bc5efdb51c0c0740 (patch) | |
tree | 731f6f1bb4840d83b128d4401b052a8377818961 /pym/portage/process.py | |
parent | fe3960b69c326bc779bdf5ec34d56630b3e188ae (diff) | |
download | portage-a75341bf3a66f75edd68d1a8bc5efdb51c0c0740.tar.gz portage-a75341bf3a66f75edd68d1a8bc5efdb51c0c0740.tar.bz2 portage-a75341bf3a66f75edd68d1a8bc5efdb51c0c0740.zip |
After python fork, don't close fds for PyPy 1.8.
If we close all open file descriptors after a fork, with PyPy 1.8 it
triggers "[Errno 9] Bad file descriptor" later in the subprocess.
Apparently it is holding references to file descriptors and closing
them after they've already been closed and re-opened for other
purposes. As a workaround, we don't close the file descriptors, so
that they won't be re-used and therefore we won't be vulnerable to this
kind of interference.
The obvious caveat of not closing the fds is that the subprocess can
hold locks that belonged to the parent process, even after the parent
process has released the locks. Hopefully this won't be a major problem
though, since the subprocess has to exit at release the lock
eventually, when the EbuildFetcher or _MergeProcess task is complete.
Diffstat (limited to 'pym/portage/process.py')
-rw-r--r-- | pym/portage/process.py | 20 |
1 files changed, 11 insertions, 9 deletions
diff --git a/pym/portage/process.py b/pym/portage/process.py index 47b0a2147..e7313abc3 100644 --- a/pym/portage/process.py +++ b/pym/portage/process.py @@ -386,7 +386,7 @@ def _exec(binary, mycommand, opt_name, fd_pipes, env, gid, groups, uid, umask, # And switch to the new process. os.execve(binary, myargs, env) -def _setup_pipes(fd_pipes): +def _setup_pipes(fd_pipes, close_fds=True): """Setup pipes for a forked process.""" my_fds = {} # To protect from cases where direct assignment could @@ -397,14 +397,16 @@ def _setup_pipes(fd_pipes): # Then assign them to what they should be. for fd in my_fds: os.dup2(my_fds[fd], fd) - # Then close _all_ fds that haven't been explicitly - # requested to be kept open. - for fd in get_open_fds(): - if fd not in my_fds: - try: - os.close(fd) - except OSError: - pass + + if close_fds: + # Then close _all_ fds that haven't been explicitly + # requested to be kept open. + for fd in get_open_fds(): + if fd not in my_fds: + try: + os.close(fd) + except OSError: + pass def find_binary(binary): """ |