Merge pull request #658 from gamechanger/tnt-file-handles

Close file descriptors on all subprocess calls
gamechanger · Mar 4, 2016 · 17e07e9 · 17e07e9
2 parents 8e2af4e + 75d7a16
commit 17e07e9
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/dusty/subprocess.py b/dusty/subprocess.py
@@ -33,7 +33,7 @@ def run_subprocess(fn, shell_args, demote=True, env=None, **kwargs):
         passed_env = None
     if demote:
         kwargs['preexec_fn'] = _demote_to_user(get_config_value(constants.CONFIG_MAC_USERNAME_KEY))
-    output = fn(shell_args, env=passed_env, **kwargs)
+    output = fn(shell_args, env=passed_env, close_fds=True, **kwargs)
     return output
 
 def call_demoted(shell_args, env=None, redirect_stderr=False):