Fabian Deutsch has uploaded a new change for review. Change subject: firewall: No shell when cmd is a list ......................................................................
firewall: No shell when cmd is a list Change-Id: I778271bb878f78d3834976b0476b5c8a42c5041f Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1007417 Signed-off-by: Fabian Deutsch <[email protected]> --- M src/ovirt/node/utils/firewall.py 1 file changed, 1 insertion(+), 1 deletion(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-node refs/changes/07/19207/1 diff --git a/src/ovirt/node/utils/firewall.py b/src/ovirt/node/utils/firewall.py index 960b3f2..5617cfa 100644 --- a/src/ovirt/node/utils/firewall.py +++ b/src/ovirt/node/utils/firewall.py @@ -49,7 +49,7 @@ def setup_iptables(port, proto): cmd = ["iptables", "-I", "INPUT", "1", "-p", proto, "--dport", port, "-j", "ACCEPT"] - process.check_call(cmd, shell=True) + process.check_call(cmd) def setup_firewalld(port, proto): -- To view, visit http://gerrit.ovirt.org/19207 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I778271bb878f78d3834976b0476b5c8a42c5041f Gerrit-PatchSet: 1 Gerrit-Project: ovirt-node Gerrit-Branch: node-3.0 Gerrit-Owner: Fabian Deutsch <[email protected]> _______________________________________________ node-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/node-patches
