config: skip interface setup if interface not IFF_RUNNING
authorChristian Marangi <ansuelsmth@gmail.com>
Thu, 16 Mar 2023 22:44:43 +0000 (23:44 +0100)
committerChristian Marangi <ansuelsmth@gmail.com>
Tue, 21 Mar 2023 18:04:17 +0000 (19:04 +0100)
We currently setup odhcp service even if the interface is not running.
This is the case for bridge or specific interface that are flagged as UP
but have no carrier as nothing is connected to it.
This cause a similar error like:

Failed to send to ff02::1%br-lan (Address not available)

This is caused by the kernel assigning IPV6 address only when the
interface is set to IFF_RUNNING.
A LINK-LOCAL IPV6 address is required for odhcpd_send() to work or every
request will be rejected.

To fix this setup services only when interface is in IFF_RUNNING state.
When an interface change state, odhcpd is reloaded and the services are
correctly setup again.

Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>
Acked-by: Hans Dedecker <dedeckeh@gmail.com>
src/config.c

index 9b1f659aa630033c27cd19a3526c75ed0daf7db2..30da8794487e8f65577f14969df38d96d81a0b60 100644 (file)
@@ -1303,7 +1303,7 @@ void odhcpd_reload(void)
 
 
        avl_for_each_element_safe(&interfaces, i, avl, tmp) {
-               if (i->inuse) {
+               if (i->inuse && i->ifflags & IFF_RUNNING) {
                        /* Resolve hybrid mode */
                        if (i->dhcpv6 == MODE_HYBRID)
                                i->dhcpv6 = (master && master->dhcpv6 == MODE_RELAY) ?