[node-patches] Change in ovirt-node[master]: Add subprocess wrapper to close open fds

mburns at redhat.com mburns at redhat.com
Wed Jun 13 22:40:08 UTC 2012


Michael Burns has submitted this change and it was merged.

Change subject: Add subprocess wrapper to close open fds
......................................................................


Add subprocess wrapper to close open fds

Open fds need to be closed when spawning a new shell, otherwise these
leaked fds will raise SELinux AVC denials.

rhbz#520345

Change-Id: I9044d63f8f26243487dddc276da3eb1b15fc75c0
Signed-off-by: Fabian Deutsch <fabiand at fedoraproject.org>
---
M scripts/collectd.py
M scripts/install.py
M scripts/iscsi.py
M scripts/network.py
M scripts/ovirt-config-setup.py
M scripts/ovirtfunctions.py
M scripts/rhn.py
M scripts/snmp.py
M scripts/storage.py
9 files changed, 63 insertions(+), 56 deletions(-)

Approvals:
  Michael Burns: Verified; Looks good to me, approved


--
To view, visit http://gerrit.ovirt.org/5241
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I9044d63f8f26243487dddc276da3eb1b15fc75c0
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-node
Gerrit-Branch: master
Gerrit-Owner: Fabian Deutsch <fabiand at fedoraproject.org>
Gerrit-Reviewer: Fabian Deutsch <fabiand at fedoraproject.org>
Gerrit-Reviewer: Michael Burns <mburns at redhat.com>



More information about the node-patches mailing list