Fix missing default route at boot

board/aarch64/bananapi-bpi-r3/rootfs/usr/share/product/bananapi,bpi-r3/etc/factory-config.cfg

@@ -21,21 +21,21 @@
       {
         "name": "radio0",
         "class": "infix-hardware:wifi",
-	"infix-hardware:wifi-radio": {
-	  "country-code": "DE",
-	  "band": "2.4GHz",
-	  "channel": "auto"
-	 }
-     },
-     {
-	"name": "radio1",
-	"class": "infix-hardware:wifi",
-	"infix-hardware:wifi-radio": {
-	  "country-code": "DE",
+        "infix-hardware:wifi-radio": {
+          "country-code": "DE",
+          "band": "2.4GHz",
+          "channel": "auto"
+        }
+      },
+      {
+        "name": "radio1",
+        "class": "infix-hardware:wifi",
+        "infix-hardware:wifi-radio": {
+          "country-code": "DE",
           "band": "5GHz",
           "channel": "auto"
+        }
       }
-    }
     ]
   },
   "ietf-interfaces:interfaces": {
@@ -130,7 +130,8 @@
                 "id": "domain"
               },
               {
-                "id": "hostname"
+                "id": "hostname",
+                "value": "auto"
               },
               {
                 "id": "dns-server"
@@ -179,7 +180,7 @@
             }
           }
         },
-	"infix-interfaces:bridge-port": {
+        "infix-interfaces:bridge-port": {
           "bridge": "br0"
         }
       },
@@ -195,7 +196,7 @@
             }
           }
         },
-       "infix-interfaces:bridge-port": {
+        "infix-interfaces:bridge-port": {
           "bridge": "br0"
         }
       }

board/aarch64/bananapi-bpi-r3/rootfs/usr/share/product/bananapi,bpi-r3mini/etc/factory-config.cfg

@@ -21,21 +21,21 @@
       {
         "name": "radio0",
         "class": "infix-hardware:wifi",
-	"infix-hardware:wifi-radio": {
-	  "country-code": "DE",
-	  "band": "2.4GHz",
-	  "channel": "auto"
-	 }
-     },
-     {
-	"name": "radio1",
-	"class": "infix-hardware:wifi",
-	"infix-hardware:wifi-radio": {
-	  "country-code": "DE",
+        "infix-hardware:wifi-radio": {
+          "country-code": "DE",
+          "band": "2.4GHz",
+          "channel": "auto"
+        }
+      },
+      {
+        "name": "radio1",
+        "class": "infix-hardware:wifi",
+        "infix-hardware:wifi-radio": {
+          "country-code": "DE",
           "band": "5GHz",
           "channel": "auto"
+        }
       }
-    }
     ]
   },
   "ietf-interfaces:interfaces": {
@@ -96,7 +96,8 @@
                 "id": "domain"
               },
               {
-                "id": "hostname"
+                "id": "hostname",
+                "value": "auto"
               },
               {
                 "id": "dns-server"
@@ -145,7 +146,7 @@
             }
           }
         },
-	"infix-interfaces:bridge-port": {
+        "infix-interfaces:bridge-port": {
           "bridge": "br0"
         }
       },
@@ -161,7 +162,7 @@
             }
           }
         },
-       "infix-interfaces:bridge-port": {
+        "infix-interfaces:bridge-port": {
           "bridge": "br0"
         }
       }

board/aarch64/bananapi-bpi-r64/rootfs/usr/share/product/bananapi,bpi-r64/etc/factory-config.cfg

@@ -14,11 +14,11 @@
       {
         "name": "radio0",
         "class": "infix-hardware:wifi",
-	"infix-hardware:wifi-radio": {
-	  "country-code": "DE",
-	  "band": "2.4GHz",
-	  "channel": "auto"
-	}
+        "infix-hardware:wifi-radio": {
+          "country-code": "DE",
+          "band": "2.4GHz",
+          "channel": "auto"
+        }
       }
     ]
   },
@@ -104,7 +104,8 @@
                 "id": "domain"
               },
               {
-                "id": "hostname"
+                "id": "hostname",
+                "value": "auto"
               },
               {
                 "id": "dns-server"
@@ -153,7 +154,7 @@
             }
           }
         },
-	"infix-interfaces:bridge-port": {
+        "infix-interfaces:bridge-port": {
           "bridge": "br0"
         }
       }

board/aarch64/friendlyarm-nanopi-r2s/rootfs/usr/share/product/friendlyarm,nanopi-r2s/etc/factory-config.cfg

@@ -71,7 +71,8 @@
                 "id": "domain"
               },
               {
-                "id": "hostname"
+                "id": "hostname",
+                "value": "auto"
               },
               {
                 "id": "dns-server"

board/aarch64/raspberrypi-rpi64/rootfs/usr/share/product/raspberrypi,4-model-b/etc/factory-config.cfg

@@ -62,7 +62,8 @@
                 "id": "domain"
               },
               {
-                "id": "hostname"
+                "id": "hostname",
+                "value": "auto"
               },
               {
                 "id": "dns-server"

board/aarch64/raspberrypi-rpi64/rootfs/usr/share/product/raspberrypi,400/etc/factory-config.cfg

@@ -76,7 +76,8 @@
                 "id": "domain"
               },
               {
-                "id": "hostname"
+                "id": "hostname",
+                "value": "auto"
               },
               {
                 "id": "dns-server"

board/arm/raspberrypi-rpi2/rootfs/usr/share/product/raspberrypi,2-model-b/etc/factory-config.cfg

@@ -54,7 +54,8 @@
                 "id": "domain"
               },
               {
-                "id": "hostname"
+                "id": "hostname",
+                "value": "auto"
               },
               {
                 "id": "dns-server"

board/common/rootfs/etc/dnsmasq.conf

@@ -2,6 +2,7 @@
 # managed by openresolv.  DHCP lease, VPN tunnel establishment,
 # and similar events feed servers and configuration to dnsmasq.
 domain-needed
+dhcp-authoritative
 
 # Allow configuration and cache clear over D-Bus
 enable-dbus
@@ -10,8 +11,6 @@ enable-dbus
 #dhcp-option=option:netmask
 #dhcp-option=28			# option:broadcast
 #dhcp-option=option:domain-name
-dhcp-option=option:router
-dhcp-option=option:dns-server
 dhcp-option=12			# option:hostname
 
 # Generated by openresolv

board/common/rootfs/usr/share/udhcpc/default.script

@@ -7,7 +7,7 @@ ACTION="$1"
 IP_CACHE="/var/lib/misc/${interface}.cache"
 RESOLV_CONF="/run/resolvconf/interfaces/${interface}.conf"
 NTPFILE="/run/chrony/dhcp-sources.d/${interface}.sources"
-NAME="/etc/netd/conf.d/${interface}-dhcp.conf"
+NAME="/etc/net.d/${interface}-dhcp.conf"
 NEXT="${NAME}+"
 
 [ -n "$broadcast" ] && BROADCAST="broadcast $broadcast"
@@ -118,17 +118,13 @@ set_dhcp_routes()
     # Reduce changes needed by comparing with previous route(s)
     cmp -s "$NAME" "$NEXT" && return
     mv "$NEXT" "$NAME"
-
-    initctl reload netd
 }
 
 clr_dhcp_routes()
 {
     log "deleting DHCP routes"
     [ -f "$NAME" ] || return
     rm "$NAME"
-
-    initctl reload netd
 }
 
 clr_dhcp_addresses()

doc/ChangeLog.md

@@ -32,6 +32,18 @@ All notable changes to the project are documented in this file.
 
 ### Fixes
 
+- Fix #1438: default route from DHCP client not set at boot, regression
+  introduced in v26.02.0
+- Fix DHCP client not sending hostname to server (e.g. for `dnsmasq` lease
+  registration).  All board factory configs had the hostname option without
+  `value: auto`, so `udhcpc` only requested the option instead of sending it.
+  Setting up a DHCP client manually was not affected.  Also, the hostname
+  format string (e.g. `rpi-%m`) was not expanded to the actual system hostname
+  before being passed to `udhcpc`
+- Fix dnsmasq logging spurious "Ignoring duplicate dhcp-option" warnings for
+  options 3 (router) and 6 (dns-server) on every DHCP transaction.  Add
+  `dhcp-authoritative` so clients with unexpired leases are not NAKed after
+  a router crash or factory reset that clears the lease database
 - Fix #1387: `infix.local` now resolves to exactly one device per LAN.  Previously
   all Infix devices claimed both `hostname.local` and `infix.local`, causing avahi
   to append `#2`, `#3` suffixes to the shared alias on busy networks.  Assignment

package/netd/Config.in

@@ -2,9 +2,10 @@ config BR2_PACKAGE_NETD
 	bool "netd"
 	select BR2_PACKAGE_LIBITE
 	select BR2_PACKAGE_LIBCONFUSE
+	select BR2_PACKAGE_LIBEV
 	help
 	  Network route daemon. Manages static routes and RIP routing.
-	  Reads configuration from /etc/netd/conf.d/*.conf.
+	  Reads configuration from /etc/net.d/*.conf.
 
 	  With FRR: Full routing via gRPC, vtysh, or frr.conf.
 	  Without FRR: Standalone Linux backend via rtnetlink.

package/netd/netd.mk

@@ -4,13 +4,13 @@
 #
 ################################################################################
 
-NETD_VERSION = 1.0
+NETD_VERSION = 1.1.0
 NETD_SITE_METHOD = local
 NETD_SITE = $(BR2_EXTERNAL_INFIX_PATH)/src/netd
 NETD_LICENSE = BSD-3-Clause
 NETD_LICENSE_FILES = LICENSE
 NETD_REDISTRIBUTE = NO
-NETD_DEPENDENCIES = libite libconfuse jansson
+NETD_DEPENDENCIES = libite libconfuse jansson libev
 NETD_AUTORECONF = YES
 
 NETD_CONF_ENV = CFLAGS="$(INFIX_CFLAGS)"

src/confd/src/dhcp-common.c

@@ -1,6 +1,7 @@
 /* SPDX-License-Identifier: BSD-3-Clause */
 
 #include <ctype.h>
+#include <limits.h>
 #include <pwd.h>
 #include <sys/utsname.h>
 #include <sys/sysinfo.h>
@@ -70,21 +71,20 @@ int dhcp_option_lookup(const struct lyd_node *id)
 
 char *dhcp_hostname(struct lyd_node *cfg, char *str, size_t len)
 {
+	char hostname[HOST_NAME_MAX + 1] = { 0 };
 	struct lyd_node *node;
-	const char *hostname;
-	char *ptr;
+	const char *fmt;
 
 	node = lydx_get_xpathf(cfg, "/ietf-system:system/hostname");
 	if (!node)
 		return NULL;
 
-	hostname = lyd_get_value(node);
-	if (!hostname || hostname[0] == 0)
+	fmt = lyd_get_value(node);
+	if (!fmt || !fmt[0])
 		return NULL;
 
-	ptr = strchr(hostname, '.');
-	if (ptr)
-		*ptr = 0;
+	if (hostnamefmt(&confd, fmt, hostname, sizeof(hostname), NULL, 0) || !hostname[0])
+		return NULL;
 
 	snprintf(str, len, "-x hostname:%s ", hostname);
 
@@ -262,10 +262,12 @@ int ifchange_cand_infer_dhcp(sr_session_ctx_t *session, const char *xpath)
 
 	/* Check if options already exist */
 	err = srx_nitems(session, &cnt, "%s/option", path);
-	if (err || cnt) {
-		ERROR("%s(): no %s/options err %d cnt %zu", __func__, path, err, cnt);
+	if (err) {
+		ERROR("%s(): failed querying %s/option: %d", __func__, path, err);
 		goto out;
 	}
+	if (cnt)
+		goto out;
 
 	/* Infer options based on IPv4 or IPv6 */
 	if (strstr(path, ":ipv4/"))

src/confd/src/routing.c

@@ -8,7 +8,7 @@
 
 #define XPATH_BASE_ "/ietf-routing:routing/control-plane-protocols/control-plane-protocol"
 #define XPATH_OSPF_ XPATH_BASE_ "/ietf-ospf:ospf"
-#define NETD_CONF "/etc/netd/conf.d/confd.conf"
+#define NETD_CONF "/etc/net.d/confd.conf"
 #define NETD_CONF_NEXT NETD_CONF "+"
 #define NETD_CONF_PREV NETD_CONF "-"
 #define OSPFD_CONF "/etc/frr/ospfd.conf"

src/netd/Makefile.am

@@ -6,7 +6,7 @@ netd_SOURCES        = src/netd.c src/netd.h src/config.c src/config.h
 netd_CPPFLAGS       = -D_DEFAULT_SOURCE -D_GNU_SOURCE -I$(srcdir)/src
 netd_CFLAGS         = -W -Wall -Wextra
 netd_CFLAGS        += $(libite_CFLAGS) $(libconfuse_CFLAGS) $(jansson_CFLAGS)
-netd_LDADD          = $(libite_LIBS) $(libconfuse_LIBS) $(jansson_LIBS)
+netd_LDADD          = $(libite_LIBS) $(libconfuse_LIBS) $(jansson_LIBS) $(EV_LIBS)
 
 # Backend selection: FRR gRPC, FRR frr.conf, or Linux kernel
 if HAVE_FRR_GRPC

src/netd/README.md

@@ -60,7 +60,7 @@ make install
 
 netd uses [libconfuse](https://github.com/martinh/libconfuse) for configuration parsing, providing a clean and structured format.
 
-Configuration files are placed in `/etc/netd/conf.d/` with the `.conf` extension. Files are processed in alphabetical order.
+Configuration files are placed in `/etc/net.d/` with the `.conf` extension. Files are processed in alphabetical order.
 
 ### Configuration Format
 
@@ -219,10 +219,10 @@ rip {
 
 ### Configuration Files
 
-Configuration files must be placed in `/etc/netd/conf.d/` with the `.conf` extension:
+Configuration files must be placed in `/etc/net.d/` with the `.conf` extension:
 
 ```bash
-/etc/netd/conf.d/
+/etc/net.d/
 ├── 10-static.conf      # Static routes
 ├── 20-rip.conf         # RIP configuration
 └── 99-local.conf       # Local overrides
@@ -256,7 +256,7 @@ netd validates configuration on reload. Check syslog for errors.
 
 ```
 ┌─────────┐
-│  confd  │ Writes /etc/netd/conf.d/confd.conf
+│  confd  │ Writes /etc/net.d/confd.conf
 └────┬────┘
      │ SIGHUP
      ▼