Merge pull request #1533 from dhozac/fireball-leaking-socket

Keep fireball from spreading its socket around
This commit is contained in:
Michael DeHaan 2012-11-06 06:43:56 -08:00
commit ac6f67b592

View file

@ -151,7 +151,7 @@ def command(data):
return dict(failed=True, msg='internal error: tmp_path is required') return dict(failed=True, msg='internal error: tmp_path is required')
log("executing: %s" % data['cmd']) log("executing: %s" % data['cmd'])
p = subprocess.Popen(data['cmd'], shell=True, stdout=subprocess.PIPE) p = subprocess.Popen(data['cmd'], shell=True, stdout=subprocess.PIPE, close_fds=True)
(stdout, stderr) = p.communicate() (stdout, stderr) = p.communicate()
if stdout is None: if stdout is None:
stdout = '' stdout = ''