From fb60dd2ae69bc27a6f53ba1a360e1e052be4633f Mon Sep 17 00:00:00 2001 From: John Crispin Date: Fri, 13 Mar 2015 08:38:35 +0000 Subject: [PATCH] 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 SVN-Revision: 44744 --- package/network/services/dnsmasq/files/dnsmasq.init | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/package/network/services/dnsmasq/files/dnsmasq.init b/package/network/services/dnsmasq/files/dnsmasq.init index 40ec584c1b4f..fb118d5756a9 100644 --- a/package/network/services/dnsmasq/files/dnsmasq.init +++ b/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 -- 2.30.2