[openwrt/openwrt] netifd: support DHCP sendopts as list options

LEDE Commits lede-commits at lists.infradead.org
Tue Feb 27 04:37:18 PST 2018


dedeckeh pushed a commit to openwrt/openwrt.git, branch master:
https://git.lede-project.org/3bd2e195ecdcf77482a983451067ad725771de4b

commit 3bd2e195ecdcf77482a983451067ad725771de4b
Author: Hans Dedecker <dedeckeh at gmail.com>
AuthorDate: Sat Feb 24 16:52:38 2018 +0100

    netifd: support DHCP sendopts as list options
    
    Support config in the form of ....
    	add_list sendopts=router:10.10.10.2
    	add_list sendopts=nissrv:20.20.20.2
    	add_list sendopts=0x7D:abba
    
    This allows to configure sendopts having white spaces as option value
    
    Signed-off-by: Hans Dedecker <dedeckeh at gmail.com>
---
 package/network/config/netifd/Makefile                       |  2 +-
 package/network/config/netifd/files/lib/netifd/proto/dhcp.sh | 12 +++++++-----
 2 files changed, 8 insertions(+), 6 deletions(-)

diff --git a/package/network/config/netifd/Makefile b/package/network/config/netifd/Makefile
index d2210e7..d62b60a 100644
--- a/package/network/config/netifd/Makefile
+++ b/package/network/config/netifd/Makefile
@@ -1,7 +1,7 @@
 include $(TOPDIR)/rules.mk
 
 PKG_NAME:=netifd
-PKG_RELEASE:=3
+PKG_RELEASE:=4
 
 PKG_SOURCE_PROTO:=git
 PKG_SOURCE_URL=$(PROJECT_GIT)/project/netifd.git
diff --git a/package/network/config/netifd/files/lib/netifd/proto/dhcp.sh b/package/network/config/netifd/files/lib/netifd/proto/dhcp.sh
index 385cfc3..afcfba9 100755
--- a/package/network/config/netifd/files/lib/netifd/proto/dhcp.sh
+++ b/package/network/config/netifd/files/lib/netifd/proto/dhcp.sh
@@ -16,7 +16,7 @@ proto_dhcp_init_config() {
 	proto_config_add_string 'reqopts:list(string)'
 	proto_config_add_boolean 'defaultreqopts:bool'
 	proto_config_add_string iface6rd
-	proto_config_add_string sendopts
+	proto_config_add_array 'sendopts:list(string)'
 	proto_config_add_boolean delegate
 	proto_config_add_string zone6rd
 	proto_config_add_string zone
@@ -25,21 +25,23 @@ proto_dhcp_init_config() {
 	proto_config_add_boolean classlessroute
 }
 
+proto_dhcp_add_sendopts() {
+	[ -n "$1" ] && append "$3" "-x $1"
+}
+
 proto_dhcp_setup() {
 	local config="$1"
 	local iface="$2"
 
 	local ipaddr hostname clientid vendorid broadcast release reqopts defaultreqopts iface6rd sendopts delegate zone6rd zone mtu6rd customroutes classlessroute
-	json_get_vars ipaddr hostname clientid vendorid broadcast release reqopts defaultreqopts iface6rd sendopts delegate zone6rd zone mtu6rd customroutes classlessroute
+	json_get_vars ipaddr hostname clientid vendorid broadcast release reqopts defaultreqopts iface6rd delegate zone6rd zone mtu6rd customroutes classlessroute
 
 	local opt dhcpopts
 	for opt in $reqopts; do
 		append dhcpopts "-O $opt"
 	done
 
-	for opt in $sendopts; do
-		append dhcpopts "-x $opt"
-	done
+	json_for_each_item proto_dhcp_add_sendopts sendopts dhcpopts
 
 	[ -z "$hostname" ] && hostname="$(cat /proc/sys/kernel/hostname)"
 	[ "$defaultreqopts" = 0 ] && defaultreqopts="-o" || defaultreqopts=



More information about the lede-commits mailing list