config: respect ignore uci option
authorJo-Philipp Wich <jo@mein.io>
Tue, 3 Jan 2017 13:09:08 +0000 (14:09 +0100)
committerHans Dedecker <dedeckeh@gmail.com>
Thu, 5 Jan 2017 09:36:17 +0000 (10:36 +0100)
The odhcpd documentation currently implies that "option ignore 1" in a section
of type "dhcp" will disable any services on the referenced interface while the
code actually ignores the setting when initializing referenced interfaces.

This commonly leads to situations where users think that "option disabled"
will prevent any server activity on a given interface while in reality odhcpd
effectively acts as rogue DHCPv6 server in its default configuration.

To fully inhibit any activity, one has to specify both "option ignore 1" as
well as "option ra disabled", "option ndp disabled" and
"option dhcpv6 disabled" which is highly counter intuitive at best.

A previous commit (5f425ed Respect interface "ignore" settings as documented.)
attempted to address the problem by observing the value of i->ignore instead
of unconditionally enabling all protocols but broke the ability to use DHCPv6
relay mode while having DHCPv4 disabled.

This patch changes ...

 - the enabling of services on an interface to use the value of the ignore
   option as default choice instead of hardcoding "true"

 - the options "ra", "ndp", "dhcpv4" and "dhcpv6" to take precedence over the
   default "ignore" state to allow selectively enabling services on ignored
   interfaces

 - the README to clarify the fact that "ra", "ndp", "dhcpv4" and "dhcpv6" can
   now override "option ignore"

Signed-off-by: Jo-Philipp Wich <jo@mein.io>
Acked-by: Karl Palsson <karlp@etactica.com>
Tested-by: Stijn Tintel <stijn@linux-ipv6.be>
Tested-by: Hans Dedecker <dedeckeh@gmail.com>
README
src/config.c

diff --git a/README b/README
index e14f8227169480db4a3450697f36b1cf6516284c..4ae5ef2859740bba91a6e122787e392fd634d291 100644 (file)
--- a/README
+++ b/README
@@ -70,7 +70,9 @@ Option                Type    Default                 Description
 interface      string  <name of UCI section>   logical OpenWrt interface
 ifname         string  <resolved from logical> physical network interface
 networkid      string  same as ifname          compat. alias for ifname
-ignore         bool    0                       ignore this interface
+ignore         bool    0                       do not serve this interface
+                                               unless overridden by ra, ndp,
+                                               dhcpv4 or dhcpv6 options
 master         bool    0                       is a master interface
                                                for relaying
 
index e3bf78a96d55e7879fbf9b639eea3bebad2e9868..ba513757202acac8830842a60b86a55d8b791f2f 100644 (file)
@@ -721,10 +721,10 @@ void odhcpd_reload(void)
                                i->ndp = (master && master->ndp == RELAYD_RELAY) ?
                                                RELAYD_RELAY : RELAYD_DISABLED;
 
-                       setup_router_interface(i, true);
-                       setup_dhcpv6_interface(i, true);
-                       setup_ndp_interface(i, true);
-                       setup_dhcpv4_interface(i, true);
+                       setup_router_interface(i, !i->ignore || i->ra != RELAYD_DISABLED);
+                       setup_dhcpv6_interface(i, !i->ignore || i->dhcpv6 != RELAYD_DISABLED);
+                       setup_ndp_interface(i, !i->ignore || i->ndp != RELAYD_DISABLED);
+                       setup_dhcpv4_interface(i, !i->ignore || i->dhcpv4 != RELAYD_DISABLED);
                } else {
                        close_interface(i);
                }