Browse Source

dnsmasq: Make parameters optional in dhcpboot config

The --dhcp-boot option of dnsmasq does not require servername and serveraddress
arguments if the builtin tftp server is used.

Signed-off-by: Stefan Tomanek <[email protected]>

SVN-Revision: 44744
John Crispin 11 years ago
parent
commit
fb60dd2ae6
1 changed files with 3 additions and 4 deletions
  1. 3 4
      package/network/services/dnsmasq/files/dnsmasq.init

+ 3 - 4
package/network/services/dnsmasq/files/dnsmasq.init

@@ -364,12 +364,11 @@ dhcp_boot_add() {
 	[ -n "$filename" ] || return 0
 
 	config_get servername "$cfg" servername
-	[ -n "$servername" ] || return 0
-
 	config_get serveraddress "$cfg" serveraddress
-	[ -n "$serveraddress" ] || return 0
 
-	xappend "--dhcp-boot=${networkid:+net:$networkid,}$filename,$servername,$serveraddress"
+	[ -n "$serveraddress" -a ! -n "$servername" ] && return 0
+
+	xappend "--dhcp-boot=${networkid:+net:$networkid,}${filename}${servername:+,$servername}${serveraddress:+,$serveraddress}"
 
 	config_get_bool force "$cfg" force 0