From 12a83d3ef4244d354cba541ce8d3f0ded9773c89 Mon Sep 17 00:00:00 2001 From: "David Kaspar [Dee'Kej]" Date: Wed, 6 Jun 2018 19:19:11 +0200 Subject: network-scripts: setting of firewall ZONE fixed For currently unknown reason the dbus-send calls will fail to set the firewall zone for the given interface if we omit the --print-reply option... This looks like some kind of race-condition in dbus-send, since the --print-reply makes the call synchronous and slower. Hopefully this is only a temporary workaround until DBus is fixed. Resolves: #1586284 --- network-scripts/ifdown-post | 4 ++-- network-scripts/ifup-eth | 3 ++- network-scripts/ifup-post | 4 ++-- 3 files changed, 6 insertions(+), 5 deletions(-) (limited to 'network-scripts') diff --git a/network-scripts/ifdown-post b/network-scripts/ifdown-post index 83d48775..334cdaeb 100755 --- a/network-scripts/ifdown-post +++ b/network-scripts/ifdown-post @@ -53,9 +53,9 @@ if ! check_default_route ; then fi fi -# Reset firewall ZONE to "default": +# Reset firewall zone (empty ZONE means default): if [ "${REALDEVICE}" != "lo" ]; then - dbus-send --system --dest=org.fedoraproject.FirewallD1 \ + dbus-send --print-reply --system --dest=org.fedoraproject.FirewallD1 \ /org/fedoraproject/FirewallD1 \ org.fedoraproject.FirewallD1.zone.removeInterface \ string:"" string:"${DEVICE}" \ diff --git a/network-scripts/ifup-eth b/network-scripts/ifup-eth index a0988edb..932635d4 100755 --- a/network-scripts/ifup-eth +++ b/network-scripts/ifup-eth @@ -339,7 +339,8 @@ fi /etc/sysconfig/network-scripts/ifup-ipv6 ${CONFIG} if is_true "${DHCPV6C}" && [ -x /sbin/dhclient ]; then - # Assign interface into a firewalld zone so we can obtain the IPv6 via DHCPv6: + # Assign interface into a firewalld zone so we can + # obtain the IPv6 via DHCPv6 (empty ZONE means default): if [ "${REALDEVICE}" != "lo" ]; then dbus-send --print-reply --system --dest=org.fedoraproject.FirewallD1 \ /org/fedoraproject/FirewallD1 \ diff --git a/network-scripts/ifup-post b/network-scripts/ifup-post index e1c5e391..5a89cbf3 100755 --- a/network-scripts/ifup-post +++ b/network-scripts/ifup-post @@ -132,9 +132,9 @@ if [ "$2" = "boot" -a \ fi fi -# Set firewall ZONE for this device (empty means default): +# Set firewall ZONE for this device (empty ZONE means default): if [ "${REALDEVICE}" != "lo" ]; then - dbus-send --system --dest=org.fedoraproject.FirewallD1 \ + dbus-send --print-reply --system --dest=org.fedoraproject.FirewallD1 \ /org/fedoraproject/FirewallD1 \ org.fedoraproject.FirewallD1.zone.changeZoneOfInterface \ string:"${ZONE}" string:"${DEVICE}" \ -- cgit v1.2.1