|
 |
e9ad3f |
From bccc66877af7baa95e70c4314e3016ac78c4bbc7 Mon Sep 17 00:00:00 2001
|
|
|
66253d |
From: Eric Garver <eric@garver.life>
|
|
|
66253d |
Date: Tue, 4 Feb 2020 09:12:17 -0500
|
|
|
e9ad3f |
Subject: [PATCH 02/22] RHEL only: default to AllowZoneDrifting=yes
|
|
|
66253d |
|
|
|
66253d |
---
|
|
|
e9ad3f |
config/firewalld.conf | 4 ++--
|
|
|
e9ad3f |
doc/xml/firewalld.conf.xml | 2 +-
|
|
|
e9ad3f |
doc/xml/firewalld.dbus.xml | 2 +-
|
|
|
e9ad3f |
src/firewall/config/__init__.py.in | 2 +-
|
|
|
e9ad3f |
src/tests/functions.at | 5 +++++
|
|
|
e9ad3f |
5 files changed, 10 insertions(+), 5 deletions(-)
|
|
|
66253d |
|
|
|
66253d |
diff --git a/config/firewalld.conf b/config/firewalld.conf
|
|
|
66253d |
index 532f0452212e..f791b2358ab8 100644
|
|
|
66253d |
--- a/config/firewalld.conf
|
|
|
66253d |
+++ b/config/firewalld.conf
|
|
|
66253d |
@@ -71,5 +71,5 @@ RFC3964_IPv4=yes
|
|
|
66253d |
# Note: If "yes" packets will only drift from source based zones to interface
|
|
|
66253d |
# based zones (including the default zone). Packets never drift from interface
|
|
|
66253d |
# based zones to other interfaces based zones (including the default zone).
|
|
|
66253d |
-# Possible values; "yes", "no". Defaults to "no".
|
|
|
66253d |
-AllowZoneDrifting=no
|
|
|
66253d |
+# Possible values; "yes", "no". Defaults to "yes".
|
|
|
66253d |
+AllowZoneDrifting=yes
|
|
|
66253d |
diff --git a/doc/xml/firewalld.conf.xml b/doc/xml/firewalld.conf.xml
|
|
|
66253d |
index fcfbfd2b68c1..c21ef87813bc 100644
|
|
|
66253d |
--- a/doc/xml/firewalld.conf.xml
|
|
|
66253d |
+++ b/doc/xml/firewalld.conf.xml
|
|
|
66253d |
@@ -197,7 +197,7 @@
|
|
|
66253d |
to interface based zones (including the default zone). Packets
|
|
|
66253d |
never drift from interface based zones to other interfaces
|
|
|
66253d |
based zones (including the default zone).
|
|
|
66253d |
- Valid values; "yes", "no". Defaults to "no".
|
|
|
66253d |
+ Valid values; "yes", "no". Defaults to "yes".
|
|
|
66253d |
</para>
|
|
|
66253d |
</listitem>
|
|
|
66253d |
</varlistentry>
|
|
|
66253d |
diff --git a/doc/xml/firewalld.dbus.xml b/doc/xml/firewalld.dbus.xml
|
|
|
e9ad3f |
index b75067e12c51..d68c775ee5bf 100644
|
|
|
66253d |
--- a/doc/xml/firewalld.dbus.xml
|
|
|
66253d |
+++ b/doc/xml/firewalld.dbus.xml
|
|
|
e9ad3f |
@@ -2787,7 +2787,7 @@
|
|
|
66253d |
to interface based zones (including the default zone). Packets
|
|
|
66253d |
never drift from interface based zones to other interfaces
|
|
|
66253d |
based zones (including the default zone).
|
|
|
66253d |
- Valid values; "yes", "no". Defaults to "no".
|
|
|
66253d |
+ Valid values; "yes", "no". Defaults to "yes".
|
|
|
66253d |
</para></listitem>
|
|
|
66253d |
</varlistentry>
|
|
|
66253d |
<varlistentry id="FirewallD1.config.Properties.AutomaticHelpers">
|
|
|
66253d |
diff --git a/src/firewall/config/__init__.py.in b/src/firewall/config/__init__.py.in
|
|
|
e9ad3f |
index e875e849dec1..0dec7913f694 100644
|
|
|
66253d |
--- a/src/firewall/config/__init__.py.in
|
|
|
66253d |
+++ b/src/firewall/config/__init__.py.in
|
|
|
e9ad3f |
@@ -133,4 +133,4 @@ FALLBACK_AUTOMATIC_HELPERS = "no"
|
|
|
66253d |
FALLBACK_FIREWALL_BACKEND = "nftables"
|
|
|
66253d |
FALLBACK_FLUSH_ALL_ON_RELOAD = True
|
|
|
66253d |
FALLBACK_RFC3964_IPV4 = True
|
|
|
66253d |
-FALLBACK_ALLOW_ZONE_DRIFTING = False
|
|
|
66253d |
+FALLBACK_ALLOW_ZONE_DRIFTING = True
|
|
|
66253d |
diff --git a/src/tests/functions.at b/src/tests/functions.at
|
|
|
e9ad3f |
index 6b1263b178dc..7ac28d514233 100644
|
|
|
66253d |
--- a/src/tests/functions.at
|
|
|
66253d |
+++ b/src/tests/functions.at
|
|
|
e9ad3f |
@@ -123,6 +123,11 @@ m4_define([FWD_START_TEST], [
|
|
|
e9ad3f |
dnl set the appropriate backend
|
|
|
e9ad3f |
AT_CHECK([sed -i 's/^FirewallBackend.*/FirewallBackend=FIREWALL_BACKEND/' ./firewalld.conf])
|
|
|
66253d |
|
|
|
e9ad3f |
+ dnl Expected test results assume this is set to "no", but downstream
|
|
|
e9ad3f |
+ dnl RHEL overrides it to "yes". Override it back to "no" so we don't
|
|
|
e9ad3f |
+ dnl have to fix up all the tests when bringing them from upstream.
|
|
|
e9ad3f |
+ AT_CHECK([sed -i 's/^AllowZoneDrifting.*/AllowZoneDrifting=no/' ./firewalld.conf])
|
|
|
66253d |
+
|
|
|
e9ad3f |
dnl fib matching is pretty new in nftables. Don't use rpfilter on older
|
|
|
e9ad3f |
dnl kernels.
|
|
|
e9ad3f |
m4_if(nftables, FIREWALL_BACKEND, [
|
|
|
66253d |
--
|
|
|
e9ad3f |
2.27.0
|
|
|
66253d |
|