base-files: uci-defaults-new: properly quote variables
authorJo-Philipp Wich <jow@openwrt.org>
Thu, 26 Nov 2015 11:17:54 +0000 (11:17 +0000)
committerJo-Philipp Wich <jow@openwrt.org>
Thu, 26 Nov 2015 11:17:54 +0000 (11:17 +0000)
Signed-off-by: Jo-Philipp Wich <jow@openwrt.org>
SVN-Revision: 47652

package/base-files/files/lib/functions/uci-defaults-new.sh

index 120f35acdb45c4f61c95b8682f4b653be410a2d2..f795af33d7b9a299990f823427f5928c325d9a1a 100755 (executable)
@@ -10,7 +10,7 @@ json_select_array() {
        json_select "$1"
        [ $? = 0 ] && return
 
-       json_add_array $1
+       json_add_array "$1"
        json_close_array
 
        json_select "$1"
@@ -22,17 +22,17 @@ json_select_object() {
        json_select "$1"
        [ $? = 0 ] && return
 
-       json_add_object $1
+       json_add_object "$1"
        json_close_object
 
        json_select "$1"
 }
 
 _ucidef_set_interface() {
-       local name=$1
-       local iface=$2
+       local name="$1"
+       local iface="$2"
 
-       json_select_object $name
+       json_select_object "$name"
        json_add_string ifname "${iface%%.*}"
        [ "$iface" = "${iface%%.*}" ] || json_add_boolean create_vlan 1
        json_select ..
@@ -41,11 +41,11 @@ _ucidef_set_interface() {
 ucidef_set_interface_loopback()
 {
        # stub
-       local a=$1
+       local a="$1"
 }
 
 ucidef_set_interface_lan() {
-       local lan_if=$1
+       local lan_if="$1"
 
        json_select_object network
        _ucidef_set_interface lan $lan_if
@@ -53,7 +53,7 @@ ucidef_set_interface_lan() {
 }
 
 ucidef_set_interface_wan() {
-        local wan_if=$1
+        local wan_if="$1"
 
         json_select_object network
         _ucidef_set_interface wan $wan_if
@@ -61,8 +61,8 @@ ucidef_set_interface_wan() {
 }
 
 ucidef_set_interfaces_lan_wan() {
-       local lan_if=$1
-       local wan_if=$2
+       local lan_if="$1"
+       local wan_if="$2"
 
        json_select_object network
        _ucidef_set_interface lan $lan_if
@@ -71,27 +71,27 @@ ucidef_set_interfaces_lan_wan() {
 }
 
 ucidef_add_switch() {
-       local name=$1
-       local reset=$2
-       local enable=$3
+       local name="$1"
+       local reset="$2"
+       local enable="$3"
 
        json_select_object switch
 
-       json_select_object $name
-       json_add_boolean enable $enable
-       json_add_boolean reset $reset
+       json_select_object "$name"
+       json_add_boolean enable "$enable"
+       json_add_boolean reset "$reset"
        json_select ..
 
        json_select ..
 }
 
 ucidef_add_switch_attr() {
-       local name=$1
-       local key=$2
-       local val=$3
+       local name="$1"
+       local key="$2"
+       local val="$3"
 
        json_select_object switch
-       json_select_object $name
+       json_select_object "$name"
 
        case "$val" in
                [0-9]) json_add_int "$key" "$val" ;;
@@ -145,20 +145,20 @@ ucidef_add_switch_ports() {
 }
 
 ucidef_add_switch_port_attr() {
-       local name=$1
-       local port=$2
-       local key=$3
-       local val=$4
+       local name="$1"
+       local port="$2"
+       local key="$3"
+       local val="$4"
        local ports i num
 
        json_select_object switch
-       json_select_object $name
+       json_select_object "$name"
 
        json_get_keys ports ports
        json_select_array ports
 
        for i in $ports; do
-               json_select $i
+               json_select "$i"
                json_get_var num num
 
                if [ -n "$num" ] && [ $num -eq $port ]; then
@@ -181,9 +181,9 @@ ucidef_add_switch_port_attr() {
 }
 
 ucidef_add_switch_vlan() {
-       local name=$1
-       local vlan=$2
-       local ports=$3
+       local name="$1"
+       local vlan="$2"
+       local ports="$3"
        local cpu_port=''
 
        case $vlan in
@@ -193,10 +193,10 @@ ucidef_add_switch_vlan() {
        esac
 
        json_select_object switch
-       json_select_object $name
+       json_select_object "$name"
        json_select_object vlans
 
-       json_add_array $vlan
+       json_add_array "$vlan"
        for p in $ports; do
                if [ ${p%t} != $p ]; then
                        cpu_port=$p
@@ -207,24 +207,24 @@ ucidef_add_switch_vlan() {
        json_close_array
 
        json_select ..
-       [ -n "$cpu_port" ] && json_add_int cpu_port $cpu_port
+       [ -n "$cpu_port" ] && json_add_int cpu_port "$cpu_port"
        json_select ..
        json_select ..
 }
 
 ucidef_set_interface_macaddr() {
-       local network=$1
-       local macaddr=$2
+       local network="$1"
+       local macaddr="$2"
 
        json_select_object network
 
-       json_select $network
+       json_select "$network"
        [ $? -eq 0 ] || {
                json_select ..
                return
        }
 
-       json_add_string macaddr $macaddr
+       json_add_string macaddr "$macaddr"
        json_select ..
 
        json_select ..
@@ -232,33 +232,33 @@ ucidef_set_interface_macaddr() {
 
 ucidef_set_led_netdev() {
        local cfg="led_$1"
-       local name=$2
-       local sysfs=$3
-       local dev=$4
+       local name="$2"
+       local sysfs="$3"
+       local dev="$4"
 
        json_select_object led
 
-       json_select_object $1
-       json_add_string name $name
+       json_select_object "$1"
+       json_add_string name "$name"
        json_add_string type netdev
-       json_add_string sysfs $sysfs
-       json_add_string device $dev
+       json_add_string sysfs "$sysfs"
+       json_add_string device "$dev"
        json_select ..
 
        json_select ..
 }
 
 ucidef_set_led_interface() {
-       local name=$1
-       local sysfs=$2
+       local name="$1"
+       local sysfs="$2"
 
        json_select_object led
 
-       json_select_object $1
-       json_add_string name $name
+       json_select_object "$1"
+       json_add_string name "$name"
        json_add_string type interface
-       json_add_string sysfs $sysfs
-       json_add_string interface $name
+       json_add_string sysfs "$sysfs"
+       json_add_string interface "$name"
        json_select ..
 
        json_select ..
@@ -266,17 +266,17 @@ ucidef_set_led_interface() {
 
 ucidef_set_led_usbdev() {
        local cfg="led_$1"
-       local name=$2
-       local sysfs=$3
-       local dev=$4
+       local name="$2"
+       local sysfs="$3"
+       local dev="$4"
 
        json_select_object led
 
-       json_select_object $1
-       json_add_string name $name
+       json_select_object "$1"
+       json_add_string name "$name"
        json_add_string type usb
-       json_add_string sysfs $sysfs
-       json_add_string device $dev
+       json_add_string sysfs "$sysfs"
+       json_add_string device "$dev"
        json_select ..
 
        json_select ..
@@ -284,17 +284,17 @@ ucidef_set_led_usbdev() {
 
 ucidef_set_led_wlan() {
        local cfg="led_$1"
-       local name=$2
-       local sysfs=$3
-       local trigger=$4
+       local name="$2"
+       local sysfs="$3"
+       local trigger="$4"
 
        json_select_object led
 
-       json_select_object $1
-       json_add_string name $name
+       json_select_object "$1"
+       json_add_string name "$name"
        json_add_string type trigger
-       json_add_string sysfs $sysfs
-       json_add_string trigger $trigger
+       json_add_string sysfs "$sysfs"
+       json_add_string trigger "$trigger"
        json_select ..
 
        json_select ..
@@ -302,19 +302,19 @@ ucidef_set_led_wlan() {
 
 ucidef_set_led_switch() {
        local cfg="led_$1"
-       local name=$2
-       local sysfs=$3
-       local trigger=$4
-       local port_mask=$5
+       local name="$2"
+       local sysfs="$3"
+       local trigger="$4"
+       local port_mask="$5"
 
        json_select_object led
 
-       json_select_object $1
-       json_add_string name $name
+       json_select_object "$1"
+       json_add_string name "$name"
        json_add_string type switch
-       json_add_string sysfs $sysfs
-       json_add_string trigger $trigger
-       json_add_string port_mask $port_mask
+       json_add_string sysfs "$sysfs"
+       json_add_string trigger "$trigger"
+       json_add_string port_mask "$port_mask"
        json_select ..
 
        json_select ..
@@ -322,16 +322,16 @@ ucidef_set_led_switch() {
 
 ucidef_set_led_default() {
        local cfg="led_$1"
-       local name=$2
-       local sysfs=$3
-       local default=$4
+       local name="$2"
+       local sysfs="$3"
+       local default="$4"
 
        json_select_object led
 
-       json_select_object $1
-       json_add_string name $name
-       json_add_string sysfs $sysfs
-       json_add_string default $default
+       json_select_object "$1"
+       json_add_string name "$name"
+       json_add_string sysfs "$sysfs"
+       json_add_string default "$default"
        json_select ..
 
        json_select ..
@@ -339,24 +339,24 @@ ucidef_set_led_default() {
 
 ucidef_set_led_rssi() {
        local cfg="led_$1"
-       local name=$2
-       local sysfs=$3
-       local iface=$4
-       local minq=$5
-       local maxq=$6
-       local offset=$7
-       local factor=$8
+       local name="$2"
+       local sysfs="$3"
+       local iface="$4"
+       local minq="$5"
+       local maxq="$6"
+       local offset="$7"
+       local factor="$8"
 
        json_select_object led
 
        json_select_object rssi
-       json_select_object $1
-       json_add_string name $name
-       json_add_string sysfs $sysfs
-       json_add_string minq $minq
-       json_add_string maxq $maxq
-       json_add_string offset $offset
-       json_add_string factor $factor
+       json_select_object "$1"
+       json_add_string name "$name"
+       json_add_string sysfs "$sysfs"
+       json_add_string minq "$minq"
+       json_add_string maxq "$maxq"
+       json_add_string offset "$offset"
+       json_add_string factor "$factor"
        json_select ..
        json_select ..
 
@@ -372,8 +372,8 @@ ucidef_set_rssimon() {
 
        json_select_object rssi
        json_add_string type rssi
-       json_add_string dev $dev
-       json_add_string threshold $threshold
+       json_add_string dev "$dev"
+       json_add_string threshold "$threshold"
        json_select ..
 
        json_select ..