From 2212045f9b51cbcd91e2d44fa7bb69a73c21d12e Mon Sep 17 00:00:00 2001 From: Jonatan Schlag Date: Wed, 16 Aug 2017 08:25:13 +0200 Subject: [PATCH] network fix parameter passing when using "" When we use "" on the command line to pass a value with spaces the argument was broken when passing it to the next function. Now the argument is kept as one string with spaces Fixes: #11438 Signed-off-by: Jonatan Schlag Signed-off-by: Michael Tremer --- src/functions/functions.bonding | 4 +- src/functions/functions.bridge | 4 +- src/functions/functions.cli.firewall | 16 +- src/functions/functions.device | 8 +- src/functions/functions.dhcpd | 38 ++--- src/functions/functions.distro | 4 +- src/functions/functions.dns | 6 +- src/functions/functions.editor | 2 +- src/functions/functions.firewall | 4 +- src/functions/functions.firewall-zones | 6 +- src/functions/functions.he | 6 +- src/functions/functions.hook | 8 +- src/functions/functions.hostapd | 12 +- src/functions/functions.http | 6 +- src/functions/functions.interrupts | 2 +- src/functions/functions.ip | 4 +- src/functions/functions.ip-tunnel | 12 +- src/functions/functions.ipsec | 50 +++--- src/functions/functions.iptables | 12 +- src/functions/functions.ipv4 | 10 +- src/functions/functions.ipv6 | 18 +-- src/functions/functions.list | 20 +-- src/functions/functions.modem | 6 +- src/functions/functions.ports | 16 +- src/functions/functions.ppp | 34 ++-- src/functions/functions.pppoe-server | 14 +- src/functions/functions.route | 64 ++++---- src/functions/functions.settings | 20 +-- src/functions/functions.util | 10 +- src/functions/functions.vpn | 4 +- src/functions/functions.vpn-security-policies | 16 +- src/functions/functions.wireless | 12 +- src/functions/functions.wpa_supplicant | 8 +- src/functions/functions.zone | 56 +++---- src/header-config | 2 +- src/header-port | 8 +- src/header-zone | 10 +- src/hooks/configs/dhcp | 2 +- src/hooks/configs/ipv4-static | 4 +- src/hooks/configs/ipv6-auto | 4 +- src/hooks/configs/ipv6-static | 2 +- src/hooks/configs/pppoe-server | 10 +- src/hooks/ports/batman-adv | 10 +- src/hooks/ports/bonding | 14 +- src/hooks/ports/dummy | 4 +- src/hooks/ports/vlan | 8 +- src/hooks/ports/wireless-adhoc | 16 +- src/hooks/ports/wireless-ap | 18 +-- src/hooks/zones/6to4-tunnel | 14 +- src/hooks/zones/bridge | 4 +- src/hooks/zones/modem | 20 +-- src/hooks/zones/pppoe | 12 +- src/hooks/zones/wireless | 10 +- src/network | 152 +++++++++--------- 54 files changed, 418 insertions(+), 418 deletions(-) diff --git a/src/functions/functions.bonding b/src/functions/functions.bonding index 490ac2e3..afa33067 100644 --- a/src/functions/functions.bonding +++ b/src/functions/functions.bonding @@ -35,10 +35,10 @@ bonding_create() { while [ $# -gt 0 ]; do case "${1}" in --address=*) - address=$(cli_get_val ${1}) + address=$(cli_get_val "${1}") ;; --mode=*) - mode=$(cli_get_val ${1}) + mode=$(cli_get_val "${1}") ;; *) error "Unrecognized argument: ${1}" diff --git a/src/functions/functions.bridge b/src/functions/functions.bridge index 5b817fbb..38f51ed3 100644 --- a/src/functions/functions.bridge +++ b/src/functions/functions.bridge @@ -30,10 +30,10 @@ bridge_create() { while [ $# -gt 0 ]; do case "${1}" in --address=*) - address=$(cli_get_val ${1}) + address=$(cli_get_val "${1}") ;; --mtu=*) - mtu=$(cli_get_val ${1}) + mtu=$(cli_get_val "${1}") ;; *) error "Unrecognized argument: ${1}" diff --git a/src/functions/functions.cli.firewall b/src/functions/functions.cli.firewall index ffd3a657..74cb6526 100644 --- a/src/functions/functions.cli.firewall +++ b/src/functions/functions.cli.firewall @@ -58,7 +58,7 @@ firewall_cli() { ;; zone) - firewall_cli_zone $@ + firewall_cli_zone "$@" ;; ""|help|--help|-h) @@ -81,7 +81,7 @@ firewall_cli_panic() { assert isset protocol shift - if cli_help_requested $@; then + if cli_help_requested "$@"; then cli_show_man firewall-panic exit ${EXIT_OK} fi @@ -104,7 +104,7 @@ firewall_cli_panic() { } firewall_cli_settings() { - if cli_help_requested $@; then + if cli_help_requested "$@"; then cli_show_man firewall-settings exit ${EXIT_OK} fi @@ -122,7 +122,7 @@ firewall_cli_zone() { assert isset protocol shift - if cli_help_requested $@; then + if cli_help_requested "$@"; then cli_show_man firewall-zone exit ${EXIT_OK} fi @@ -143,15 +143,15 @@ firewall_cli_zone() { # Process the given action. case "${action}" in edit) - firewall_cli_zone_edit ${zone} $@ + firewall_cli_zone_edit ${zone} "$@" ;; status|"") - firewall_cli_zone_status ${zone} $@ + firewall_cli_zone_status ${zone} "$@" ;; # Print the raw configuration settings. show) - firewall_zone_print ${zone} $@ + firewall_zone_print ${zone} "$@" exit ${EXIT_ERROR} ;; @@ -168,7 +168,7 @@ firewall_cli_zone() { case "${action}" in reset) - firewall_zone_reset $@ + firewall_zone_reset "$@" exit $? ;; diff --git a/src/functions/functions.device b/src/functions/functions.device index 897f812d..12bf203f 100644 --- a/src/functions/functions.device +++ b/src/functions/functions.device @@ -320,11 +320,11 @@ device_get_phy() { } device_is_phy() { - phy_exists $@ + phy_exists "$@" } device_is_serial() { - serial_exists $@ + serial_exists "$@" } # Returns true if a device is a tun device @@ -540,7 +540,7 @@ device_set_promisc() { # Check if the device is free device_is_free() { - ! device_is_used $@ + ! device_is_used "$@" } # Check if the device is used @@ -775,7 +775,7 @@ device_identify() { seconds="$(cli_get_val "${arg}")" ;; esac - done <<< "$(args $@)" + done <<< "$(args "$@")" assert isinteger seconds diff --git a/src/functions/functions.dhcpd b/src/functions/functions.dhcpd index 7d32c394..3b1214fb 100644 --- a/src/functions/functions.dhcpd +++ b/src/functions/functions.dhcpd @@ -118,7 +118,7 @@ dhcpd_service() { } dhcpd_start() { - local services=$(dhcpd_service $@) + local services=$(dhcpd_service "$@") local service for service in ${services}; do @@ -127,7 +127,7 @@ dhcpd_start() { } dhcpd_stop() { - local services=$(dhcpd_service $@) + local services=$(dhcpd_service "$@") local service for service in ${services}; do @@ -138,7 +138,7 @@ dhcpd_stop() { dhcpd_restart() { # DHCP does not support a reload, so # we retsart it. - local services=$(dhcpd_service $@) + local services=$(dhcpd_service "$@") local service for service in ${services}; do @@ -147,11 +147,11 @@ dhcpd_restart() { } dhcpd_reload() { - dhcpd_restart $@ + dhcpd_restart "$@" } dhcpd_enable() { - local services=$(dhcpd_service $@) + local services=$(dhcpd_service "$@") local service for service in ${services}; do @@ -160,7 +160,7 @@ dhcpd_enable() { } dhcpd_disable() { - local services=$(dhcpd_service $@) + local services=$(dhcpd_service "$@") local service for service in ${services}; do @@ -181,10 +181,10 @@ dhcpd_edit() { case "${proto}" in ipv6) - _dhcpd_edit_ipv6 $@ || return $? + _dhcpd_edit_ipv6 "$@" || return $? ;; ipv4) - _dhcpd_edit_ipv4 $@ || return $? + _dhcpd_edit_ipv4 "$@" || return $? ;; esac @@ -197,7 +197,7 @@ _dhcpd_edit_ipv4() { while [ $# -gt 0 ]; do case "${1}" in --authoritative=*) - val=$(cli_get_val ${1}) + val=$(cli_get_val "${1}") if enabled val; then AUTHORITATIVE="true" @@ -206,7 +206,7 @@ _dhcpd_edit_ipv4() { fi ;; --default-lease-time=*) - local val=$(cli_get_val ${1}) + local val=$(cli_get_val "${1}") DEFAULT_LEASE_TIME=$(parse_time ${val}) if ! isinteger DEFAULT_LEASE_TIME; then @@ -215,7 +215,7 @@ _dhcpd_edit_ipv4() { fi ;; --max-lease-time=*) - local val=$(cli_get_val ${1}) + local val=$(cli_get_val "${1}") MAX_LEASE_TIME=$(parse_time ${val}) if ! isinteger MAX_LEASE_TIME; then @@ -224,7 +224,7 @@ _dhcpd_edit_ipv4() { fi ;; --min-lease-time=*) - local val=$(cli_get_val ${1}) + local val=$(cli_get_val "${1}") MIN_LEASE_TIME=$(parse_time ${val}) if isset MIN_LEASE_TIME; then @@ -252,7 +252,7 @@ _dhcpd_edit_ipv6() { while [ $# -gt 0 ]; do case "${1}" in --preferred-lifetime=*) - local val=$(cli_get_val ${1}) + local val=$(cli_get_val "${1}") PREFERRED_LIFETIME=$(parse_time ${val}) if ! isinteger PREFERRED_LIFETIME; then @@ -261,7 +261,7 @@ _dhcpd_edit_ipv6() { fi ;; --valid-lifetime=*) - local val=$(cli_get_val ${1}) + local val=$(cli_get_val "${1}") VALID_LIFETIME=$(parse_time ${val}) if ! isinteger VALID_LIFETIME; then @@ -512,7 +512,7 @@ dhcpd_subnet_match() { } dhcpd_subnet_exists() { - dhcpd_subnet_match $@ + dhcpd_subnet_match "$@" } dhcpd_subnet_new() { @@ -520,7 +520,7 @@ dhcpd_subnet_new() { assert isset proto shift - dhcpd_subnet_edit ${proto} "new" $@ + dhcpd_subnet_edit ${proto} "new" "$@" } dhcpd_subnet_edit() { @@ -557,7 +557,7 @@ dhcpd_subnet_edit() { case "${proto},${mode},${1}" in # Common options ipv6,new,*:*/*|ipv4,new,*.*.*.*/*) - local subnet="$(cli_get_val ${1})" + local subnet="$(cli_get_val "${1}")" ADDRESS="$(ip_split_prefix ${subnet})" PREFIX="$(ip_get_prefix ${subnet})" @@ -617,7 +617,7 @@ dhcpd_subnet_edit() { # IPv4 options ipv4,*,--routers=*) - ROUTERS=$(cli_get_val ${1}) + ROUTERS=$(cli_get_val "${1}") ;; *) @@ -948,7 +948,7 @@ dhcpd_subnet_settings() { } dhcpd_subnet_options_file() { - local path=$(dhcpd_subnet_path $@) + local path=$(dhcpd_subnet_path "$@") assert isset path print "${path}/options" diff --git a/src/functions/functions.distro b/src/functions/functions.distro index 4dc831b8..6978233f 100644 --- a/src/functions/functions.distro +++ b/src/functions/functions.distro @@ -26,8 +26,8 @@ __os_release() { local line k v while read line; do - k=$(cli_get_key ${line}) - v=$(cli_get_val ${line}) + k=$(cli_get_key "${line}") + v=$(cli_get_val "${line}") if [ "${what}" = "${k}" ]; then echo "${v//\"/}" diff --git a/src/functions/functions.dns b/src/functions/functions.dns index a54feecd..e7c3e471 100644 --- a/src/functions/functions.dns +++ b/src/functions/functions.dns @@ -199,13 +199,13 @@ dns_server_parse_line() { while read -r arg; do case "${arg}" in server=*) - server=$(cli_get_val ${arg}) + server=$(cli_get_val "${arg}") ;; priority=*) - priority=$(cli_get_val ${arg}) + priority=$(cli_get_val "${arg}") ;; esac - done <<< "$(args $@)" + done <<< "$(args "$@")" # The server address must be set. isset server || return ${EXIT_ERROR} diff --git a/src/functions/functions.editor b/src/functions/functions.editor index f362a6fe..6edac62a 100644 --- a/src/functions/functions.editor +++ b/src/functions/functions.editor @@ -121,7 +121,7 @@ editor_check() { shift # Execute the check function - "${check_func}" $@ + "${check_func}" "$@" local ret="${?}" case "${ret}" in diff --git a/src/functions/functions.firewall b/src/functions/functions.firewall index a503c620..58a2aa4e 100644 --- a/src/functions/functions.firewall +++ b/src/functions/functions.firewall @@ -740,9 +740,9 @@ _firewall_parse_rule_line() { return ${EXIT_ERROR} fi - val=$(cli_get_val ${arg}) + val=$(cli_get_val "${arg}") assign "${key}" "${val}" - done <<< "$(args $@)" + done <<< "$(args "$@")" # action must always be set. if ! isset action; then diff --git a/src/functions/functions.firewall-zones b/src/functions/functions.firewall-zones index a72bb8e0..6b06fdf4 100644 --- a/src/functions/functions.firewall-zones +++ b/src/functions/functions.firewall-zones @@ -44,7 +44,7 @@ firewall_zone_config() { } firewall_zone_exists() { - local file=$(firewall_zone_config $@) + local file=$(firewall_zone_config "$@") [ -r "${file}" ] && return ${EXIT_TRUE} || return ${EXIT_FALSE} } @@ -106,10 +106,10 @@ firewall_zone_edit() { while [ $# -gt 0 ]; do case "${1}" in --masquerade4=*) - MASQUERADE4=$(cli_get_val ${1}) + MASQUERADE4=$(cli_get_val "${1}") ;; --policy=*) - POLICY=$(cli_get_val ${1}) + POLICY=$(cli_get_val "${1}") ;; *) warning "Unknown option: ${1}" diff --git a/src/functions/functions.he b/src/functions/functions.he index 205170cd..0a73585f 100644 --- a/src/functions/functions.he +++ b/src/functions/functions.he @@ -27,13 +27,13 @@ he_tunnelbroker_endpoint_update() { while [ $# -gt 0 ]; do case "${1}" in --username=*) - username="$(cli_get_val ${1})" + username="$(cli_get_val "${1}")" ;; --password=*) - password="$(cli_get_val ${1})" + password="$(cli_get_val "${1}")" ;; --tunnel-id=*) - tunnel_id="$(cli_get_val ${1})" + tunnel_id="$(cli_get_val "${1}")" ;; esac shift diff --git a/src/functions/functions.hook b/src/functions/functions.hook index b8049db5..601baee5 100644 --- a/src/functions/functions.hook +++ b/src/functions/functions.hook @@ -155,11 +155,11 @@ config_get_hook() { } hook_zone_exists() { - hook_exists zone $@ + hook_exists zone "$@" } hook_zone_exec() { - hook_exec zone $@ + hook_exec zone "$@" } hook_zone_get_all() { @@ -167,11 +167,11 @@ hook_zone_get_all() { } hook_config_exists() { - hook_exists config $@ + hook_exists config "$@" } hook_config_exec() { - hook_exec config $@ + hook_exec config "$@" } hook_config_get_all() { diff --git a/src/functions/functions.hostapd b/src/functions/functions.hostapd index f7bf2fa5..86458289 100644 --- a/src/functions/functions.hostapd +++ b/src/functions/functions.hostapd @@ -44,13 +44,13 @@ hostapd_config_write() { while [ $# -gt 0 ]; do case "${1}" in --broadcast-ssid=*) - broadcast_ssid=$(cli_get_val ${1}) + broadcast_ssid=$(cli_get_val "${1}") ;; --channel=*) - channel=$(cli_get_val ${1}) + channel=$(cli_get_val "${1}") ;; --encryption=*) - encryption=$(cli_get_val ${1}) + encryption=$(cli_get_val "${1}") ;; --ieee80211d=*) local val="$(cli_get_val "${1}")" @@ -61,13 +61,13 @@ hostapd_config_write() { fi ;; --key=*) - key=$(cli_get_val ${1}) + key=$(cli_get_val "${1}") ;; --mode=*) - mode=$(cli_get_val ${1}) + mode=$(cli_get_val "${1}") ;; --ssid=*) - ssid=$(cli_get_val ${1}) + ssid=$(cli_get_val "${1}") ;; --wmm=*) local val="$(cli_get_val "${1}")" diff --git a/src/functions/functions.http b/src/functions/functions.http index 693f9f87..f33aeb64 100644 --- a/src/functions/functions.http +++ b/src/functions/functions.http @@ -34,10 +34,10 @@ http_GET() { while [ $# -gt 0 ]; do case "${1}" in --username=*) - username="$(cli_get_val ${1})" + username="$(cli_get_val "${1}")" ;; --password=*) - password="$(cli_get_val ${1})" + password="$(cli_get_val "${1}")" ;; *) break @@ -46,7 +46,7 @@ http_GET() { shift done - local url="$(cli_get_val ${1})" + local url="$(cli_get_val "${1}")" shift # Add credentials to the URL. diff --git a/src/functions/functions.interrupts b/src/functions/functions.interrupts index 4e089d35..3bd2c01e 100644 --- a/src/functions/functions.interrupts +++ b/src/functions/functions.interrupts @@ -99,7 +99,7 @@ __bitmap_to_processor_ids() { } __processor_id_to_bitmap() { - hex $(( 1 << $@ )) + hex $(( 1 << "$@" )) } interrupt_set_smp_affinity() { diff --git a/src/functions/functions.ip b/src/functions/functions.ip index d6fd76dc..3b43da77 100644 --- a/src/functions/functions.ip +++ b/src/functions/functions.ip @@ -114,13 +114,13 @@ ip_prefix_is_valid() { } ip_get_network() { - inetcalc -n $@ && return ${EXIT_OK} || return ${EXIT_ERROR} + inetcalc -n "$@" && return ${EXIT_OK} || return ${EXIT_ERROR} } ip_network_is_subset_of() { assert [ $# -eq 2 ] - inetcalc -s $@ && return ${EXIT_TRUE} || return ${EXIT_FALSE} + inetcalc -s "$@" && return ${EXIT_TRUE} || return ${EXIT_FALSE} } ip_address_add() { diff --git a/src/functions/functions.ip-tunnel b/src/functions/functions.ip-tunnel index 0e1115b2..cb25edda 100644 --- a/src/functions/functions.ip-tunnel +++ b/src/functions/functions.ip-tunnel @@ -37,24 +37,24 @@ ip_tunnel_add() { while [ $# -gt 0 ]; do case "${1}" in --mode=*) - mode="$(cli_get_val ${1})" + mode="$(cli_get_val "${1}")" ;; --ttl=*) - ttl="$(cli_get_val ${1})" + ttl="$(cli_get_val "${1}")" ;; --remote-address=*) - remote_address="$(cli_get_val ${1})" + remote_address="$(cli_get_val "${1}")" ;; --local-address=*) - local_address="$(cli_get_val ${1})" + local_address="$(cli_get_val "${1}")" ;; # Keys for VTI --ikey=*) - ikey="$(cli_get_val ${1})" + ikey="$(cli_get_val "${1}")" ;; --okey=*) - okey="$(cli_get_val ${1})" + okey="$(cli_get_val "${1}")" ;; esac shift diff --git a/src/functions/functions.ipsec b/src/functions/functions.ipsec index 72838363..038ed879 100644 --- a/src/functions/functions.ipsec +++ b/src/functions/functions.ipsec @@ -65,10 +65,10 @@ cli_ipsec() { case "${action}" in connection) - cli_ipsec_connection $@ + cli_ipsec_connection "$@" ;; pool) - cli_ipsec_pool $@ + cli_ipsec_pool "$@" ;; *) error "Unrecognized argument: ${action}" @@ -86,7 +86,7 @@ cli_ipsec_connection() { case "${key}" in authentication|down|disable|dpd|enable|inactivity_timeout|local|mode|peer|pool|remote|security_policy|start_action|up) - ipsec_connection_${key} ${connection} $@ + ipsec_connection_${key} ${connection} "$@" ;; show) cli_ipsec_connection_show "${connection}" @@ -103,10 +103,10 @@ cli_ipsec_connection() { case "${action}" in new) - ipsec_connection_new $@ + ipsec_connection_new "$@" ;; destroy) - cli_ipsec_connection_destroy $@ + cli_ipsec_connection_destroy "$@" ;; ""|*) if [ -n "${action}" ]; then @@ -127,7 +127,7 @@ cli_ipsec_pool() { case "${key}" in dns_server|network) - ipsec_pool_${key} ${pool} $@ + ipsec_pool_${key} ${pool} "$@" ;; show) cli_ipsec_pool_show "${pool}" @@ -144,10 +144,10 @@ cli_ipsec_pool() { case "${action}" in new) - ipsec_pool_new $@ + ipsec_pool_new "$@" ;; destroy) - ipsec_pool_destroy $@ + ipsec_pool_destroy "$@" ;; ""|*) if [ -n "${action}" ]; then @@ -363,7 +363,7 @@ ipsec_connection_read_config() { if [ $# -eq 0 ] && [ -n "${IPSEC_CONNECTION_CONFIG_SETTINGS}" ]; then list_append args ${IPSEC_CONNECTION_CONFIG_SETTINGS} else - list_append args $@ + list_append args "$@" fi local path="${NETWORK_IPSEC_CONNS_DIR}/${connection}/settings" @@ -477,10 +477,10 @@ ipsec_connection_authentication() { case ${cmd} in mode) - ipsec_connection_authentication_mode "${connection}" $@ + ipsec_connection_authentication_mode "${connection}" "$@" ;; pre-shared-key) - ipsec_connection_authentication_psk "${connection}" $@ + ipsec_connection_authentication_psk "${connection}" "$@" ;; *) log ERROR "Unrecognized argument: ${cmd}" @@ -574,13 +574,13 @@ ipsec_connection_dpd() { case ${cmd} in action) - ipsec_connection_dpd_action "${connection}" $@ + ipsec_connection_dpd_action "${connection}" "$@" ;; delay) - ipsec_connection_dpd_delay "${connection}" $@ + ipsec_connection_dpd_delay "${connection}" "$@" ;; timeout) - ipsec_connection_dpd_timeout "${connection}" $@ + ipsec_connection_dpd_timeout "${connection}" "$@" ;; *) log ERROR "Unrecognized argument: ${cmd}" @@ -621,7 +621,7 @@ ipsec_connection_dpd_delay() { local value=$@ if ! isinteger value; then - value=$(parse_time $@) + value=$(parse_time "$@") if [ ! $? -eq 0 ]; then log ERROR "Parsing the passed time was not sucessful please check the passed values." return ${EXIT_ERROR} @@ -653,7 +653,7 @@ ipsec_connection_dpd_timeout() { local value=$@ if ! isinteger value; then - value=$(parse_time $@) + value=$(parse_time "$@") if [ ! $? -eq 0 ]; then log ERROR "Parsing the passed time was not sucessful please check the passed values." return ${EXIT_ERROR} @@ -686,13 +686,13 @@ ipsec_connection_local() { case ${cmd} in address) - ipsec_connection_local_address "${connection}" $@ + ipsec_connection_local_address "${connection}" "$@" ;; id) - ipsec_connection_id "${connection}" "LOCAL" $@ + ipsec_connection_id "${connection}" "LOCAL" "$@" ;; prefix) - ipsec_connection_prefix "${connection}" "LOCAL" $@ + ipsec_connection_prefix "${connection}" "LOCAL" "$@" ;; *) log ERROR "Unrecognized argument: ${cmd}" @@ -1003,11 +1003,11 @@ ipsec_connection_remote() { case ${cmd} in id) - ipsec_connection_id "${connection}" "REMOTE" $@ + ipsec_connection_id "${connection}" "REMOTE" "$@" ;; prefix) - ipsec_connection_prefix "${connection}" "REMOTE" $@ + ipsec_connection_prefix "${connection}" "REMOTE" "$@" ;; *) log ERROR "Unrecognized argument: ${cmd}" @@ -1030,7 +1030,7 @@ ipsec_connection_inactivity_timeout() { local value=$@ if ! isinteger value; then - value=$(parse_time $@) + value=$(parse_time "$@") if [ ! $? -eq 0 ]; then log ERROR "Parsing the passed time was not sucessful please check the passed values." return ${EXIT_ERROR} @@ -1195,7 +1195,7 @@ ipsec_connection_new() { # Function that deletes based on the passed parameters one ore more vpn security policies ipsec_connection_destroy() { local connection - for connection in $@; do + for connection in "$@"; do if ! ipsec_connection_exists "${connection}"; then log ERROR "The VPN IPsec connection ${connection} does not exist." continue @@ -1598,7 +1598,7 @@ ipsec_pool_read_config() { if [ $# -eq 0 ] && [ -n "${IPSEC_POOL_CONFIG_SETTINGS}" ]; then list_append args ${IPSEC_POOL_CONFIG_SETTINGS} else - list_append args $@ + list_append args "$@" fi local path="${NETWORK_IPSEC_POOLS_DIR}/${pool}/settings" @@ -1679,7 +1679,7 @@ ipsec_pool_new() { # one ore more vpn ipsec pools ipsec_pool_destroy() { local pool - for pool in $@; do + for pool in "$@"; do if ! ipsec_pool_exists "${pool}"; then log ERROR "The VPN IPsec pool ${pool} does not exist." continue diff --git a/src/functions/functions.iptables b/src/functions/functions.iptables index 86627a42..821eb608 100644 --- a/src/functions/functions.iptables +++ b/src/functions/functions.iptables @@ -116,7 +116,7 @@ iptables_chain_create() { shift ;; --policy=*) - policy="$(cli_get_val ${1})" + policy="$(cli_get_val "${1}")" ;; -*) log WARNING "Unrecognized argument: ${1}" @@ -369,7 +369,7 @@ iptables_dump() { while [ $# -gt 0 ]; do case "${1}" in --log-facility=*) - log_facility="$(cli_get_val ${1})" + log_facility="$(cli_get_val "${1}")" ;; *) log WARNING "Unrecognized argument: ${1}" @@ -441,11 +441,11 @@ IPTABLES_MULTIPORT=1 IPTABLES_PORTRANGE=2 _iptables_port_range() { - grep -q ":" <<< $@ + grep -q ":" <<< "$@" } _iptables_port_multiport() { - grep -q "," <<< $@ + grep -q "," <<< "$@" } _iptables_port() { @@ -461,7 +461,7 @@ _iptables_port() { iptables_source_port() { [ -z "$@" ] && return local type - type=$(_iptables_port $@) + type=$(_iptables_port "$@") if [ "$type" = "$IPTABLES_MULTIPORT" ]; then echo "-m multiport --source-ports $@" else @@ -472,7 +472,7 @@ iptables_source_port() { iptables_destination_port() { [ -z "$@" ] && return local type - type=$(_iptables_port $@) + type=$(_iptables_port "$@") if [ "$type" = "$IPTABLES_MULTIPORT" ]; then echo "-m multiport --destination-ports $@" else diff --git a/src/functions/functions.ipv4 b/src/functions/functions.ipv4 index e893f5ec..d19be518 100644 --- a/src/functions/functions.ipv4 +++ b/src/functions/functions.ipv4 @@ -260,11 +260,11 @@ ipv4_netmask2prefix() { } ipv4_get_network() { - ip_get_network $@ + ip_get_network "$@" } ipv4_get_broadcast() { - inetcalc -4 -b $@ && return ${EXIT_OK} || return ${EXIT_ERROR} + inetcalc -4 -b "$@" && return ${EXIT_OK} || return ${EXIT_ERROR} } ipv4_encode() { @@ -315,15 +315,15 @@ ipv4_addr_gt() { } ipv4_addr_ge() { - ipv4_addr_eq $@ || ipv4_addr_gt $@ + ipv4_addr_eq "$@" || ipv4_addr_gt "$@" } ipv4_addr_lt() { - ! ipv4_addr_eq $@ && ! ipv4_addr_gt $@ + ! ipv4_addr_eq "$@" && ! ipv4_addr_gt "$@" } ipv4_addr_le() { - ipv4_addr_eq $@ || ! ipv4_addr_gt $@ + ipv4_addr_eq "$@" || ! ipv4_addr_gt "$@" } ipv4_range() { diff --git a/src/functions/functions.ipv6 b/src/functions/functions.ipv6 index e160fb31..50c2b9ef 100644 --- a/src/functions/functions.ipv6 +++ b/src/functions/functions.ipv6 @@ -64,7 +64,7 @@ ipv6_device_forwarding_enable() { accept_ra=2 ;; esac - done <<< "$(args $@)" + done <<< "$(args "$@")" sysctl_set "net.ipv6.conf.${device}.forwarding" 1 @@ -191,7 +191,7 @@ ipv6_address_add() { wait_for_dad="false" ;; esac - done <<< "$(args $@)" + done <<< "$(args "$@")" local cmd="ip addr add ${address} dev ${device} scope ${scope}" @@ -280,7 +280,7 @@ ipv6_address_change_lifetime() { valid_lft="$(cli_get_val "${arg}")" ;; esac - done <<< "$(args $@)" + done <<< "$(args "$@")" local cmd="ip -6 addr change ${address} dev ${device} scope global" @@ -385,7 +385,7 @@ ipv6_device_get_addresses() { scope="$(cli_get_val "${arg}")" ;; esac - done <<< "$(args $@)" + done <<< "$(args "$@")" local cmd="ip -o addr show dev ${device}" if isset scope; then @@ -411,7 +411,7 @@ ipv6_device_get_addresses() { } ipv6_format() { - inetcalc -6 -f $@ + inetcalc -6 -f "$@" } ipv6_addr_eq() { @@ -435,17 +435,17 @@ ipv6_addr_gt() { } ipv6_addr_ge() { - ipv6_addr_eq $@ || ipv6_addr_gt $@ + ipv6_addr_eq "$@" || ipv6_addr_gt "$@" } ipv6_addr_lt() { - ! ipv6_addr_eq $@ && ! ipv6_addr_gt $@ + ! ipv6_addr_eq "$@" && ! ipv6_addr_gt "$@" } ipv6_addr_le() { - ipv6_addr_eq $@ || ! ipv6_addr_gt $@ + ipv6_addr_eq "$@" || ! ipv6_addr_gt "$@" } ipv6_get_network() { - ip_get_network $@ + ip_get_network "$@" } diff --git a/src/functions/functions.list b/src/functions/functions.list index 7e96cdf6..9ca13b97 100644 --- a/src/functions/functions.list +++ b/src/functions/functions.list @@ -29,7 +29,7 @@ list_append() { shift local arg - for arg in $@; do + for arg in "$@"; do list_append_one "${list}" "${arg}" done } @@ -60,7 +60,7 @@ list_append_unique() { local ret=${EXIT_ERROR} local arg - for arg in $@; do + for arg in "$@"; do if ! list_match ${arg} ${!list}; then list_append_one ${list} "${arg}" ret=${EXIT_OK} @@ -83,7 +83,7 @@ list_remove() { local _list k for k in ${!list}; do - if list_match ${k} $@; then + if list_match ${k} "$@"; then ret=${EXIT_OK} continue fi @@ -98,7 +98,7 @@ list_remove() { list_sort() { local i - for i in $@; do + for i in "$@"; do print "${i}" done | sort | tr '\n' ' ' print @@ -106,7 +106,7 @@ list_sort() { list_unique() { local items item - for item in $@; do + for item in "$@"; do # Check if the item has already been processed. list_match "${item}" ${items} && continue @@ -120,7 +120,7 @@ list_match() { shift local i - for i in $@; do + for i in "$@"; do [ "${match}" = "${i}" ] && return ${EXIT_OK} done @@ -137,7 +137,7 @@ list_length() { local length=0 local i - for i in $@; do + for i in "$@"; do length=$(( ${length} + 1 )) done @@ -152,7 +152,7 @@ list_count() { local counter=0 local arg - for arg in $@; do + for arg in "$@"; do if [ "${arg}" = "${what}" ]; then counter=$(( ${counter} + 1 )) fi @@ -177,7 +177,7 @@ list_join() { list_reverse() { local reversed arg - for arg in $@; do + for arg in "$@"; do reversed="${arg} ${reversed}" done @@ -187,7 +187,7 @@ list_reverse() { list_head() { local arg - for arg in $@; do + for arg in "$@"; do print "${arg}" return ${EXIT_OK} done diff --git a/src/functions/functions.modem b/src/functions/functions.modem index 90def3f3..a2b01ca5 100644 --- a/src/functions/functions.modem +++ b/src/functions/functions.modem @@ -34,10 +34,10 @@ modem_chat() { while [ $# -gt 0 ]; do case "${1}" in --timeout=*) - timeout=$(cli_get_val ${1}) + timeout=$(cli_get_val "${1}") ;; --answer=*) - answer=$(cli_get_val ${1}) + answer=$(cli_get_val "${1}") ;; --quiet) quiet="true" @@ -204,7 +204,7 @@ modem_sim_unlocked() { } modem_sim_locked() { - modem_sim_unlocked $@ && return ${EXIT_FALSE} || return ${EXIT_TRUE} + modem_sim_unlocked "$@" && return ${EXIT_FALSE} || return ${EXIT_TRUE} } modem_sim_unlock() { diff --git a/src/functions/functions.ports b/src/functions/functions.ports index f7ea1a0b..5defe016 100644 --- a/src/functions/functions.ports +++ b/src/functions/functions.ports @@ -149,7 +149,7 @@ port_is_attached() { } port_is_up() { - device_is_up $@ + device_is_up "$@" } port_new() { @@ -161,7 +161,7 @@ port_new() { return ${EXIT_ERROR} fi - hook_exec port "${hook}" new $@ + hook_exec port "${hook}" new "$@" } port_destroy() { @@ -203,7 +203,7 @@ port_destroy() { } port_create() { - port_cmd "create" $@ + port_cmd "create" "$@" } port_remove() { @@ -234,7 +234,7 @@ port_restart() { } port_edit() { - port_cmd edit $@ + port_cmd edit "$@" } port_up() { @@ -252,11 +252,11 @@ port_up() { } port_down() { - port_cmd down $@ + port_cmd down "$@" } port_status() { - port_cmd status $@ + port_cmd status "$@" } port_cmd() { @@ -275,7 +275,7 @@ port_cmd() { return ${EXIT_ERROR} fi - hook_exec port ${hook} ${cmd} ${port} $@ + hook_exec port ${hook} ${cmd} ${port} "$@" } ports_get() { @@ -414,7 +414,7 @@ ports_lowest_address() { } port_identify() { - device_identify $@ + device_identify "$@" } port_get_color() { diff --git a/src/functions/functions.ppp b/src/functions/functions.ppp index 0840467c..b114585f 100644 --- a/src/functions/functions.ppp +++ b/src/functions/functions.ppp @@ -399,18 +399,18 @@ pppd_write_config() { while [ $# -gt 0 ]; do case "${1}" in --auth=*) - auth=$(cli_get_val ${1}) + auth=$(cli_get_val "${1}") ;; --baudrate=*) - baudrate=$(cli_get_val ${1}) + baudrate=$(cli_get_val "${1}") assert isoneof baudrate ${SERIAL_BAUDRATES} ;; --connect-command=*) - connect_cmd=$(cli_get_val ${1}) + connect_cmd=$(cli_get_val "${1}") ;; # Enable or disable the use of the default asyncmap. --default-asyncmap=*) - value=$(cli_get_val ${1}) + value=$(cli_get_val "${1}") if enabled value; then default_asyncmap="true" else @@ -419,15 +419,15 @@ pppd_write_config() { ;; # The name of the created ppp interface. --interface=*) - interface=$(cli_get_val ${1}) + interface=$(cli_get_val "${1}") ;; # IPv6 --ipv6=*) - ipv6="$(cli_get_val ${1})" + ipv6="$(cli_get_val "${1}")" ;; # LCP echo failure. --lcr-echo-failure=*) - lcr_echo_failure=$(cli_get_val ${1}) + lcr_echo_failure=$(cli_get_val "${1}") if ! isinteger ${lcr_echo_failure}; then error "--lcr-echo-failure= requires a number" @@ -436,7 +436,7 @@ pppd_write_config() { ;; # LCP echo interval. --lcr-echo-interval=*) - lcr_echo_interval=$(cli_get_val ${1}) + lcr_echo_interval=$(cli_get_val "${1}") if ! isinteger ${lcr_echo_failure}; then error "--lcr-echo-interval= requires a number" @@ -445,23 +445,23 @@ pppd_write_config() { ;; # Maximum Transmission Unit --mtu=*) - mtu=$(cli_get_val ${1}) + mtu=$(cli_get_val "${1}") ;; # Maximum Receive Unit --mru=*) - mru=$(cli_get_val ${1}) + mru=$(cli_get_val "${1}") ;; --password=*) - password=$(cli_get_val ${1}) + password=$(cli_get_val "${1}") ;; --plugin=*) - plugin=$(cli_get_val ${1}) + plugin=$(cli_get_val "${1}") ;; --plugin-options=*) - plugin_options=$(cli_get_val ${1}) + plugin_options=$(cli_get_val "${1}") ;; --pty=*) - pty=$(cli_get_val ${1}) + pty=$(cli_get_val "${1}") ;; # Refused authentication methods --refuse=*) @@ -470,13 +470,13 @@ pppd_write_config() { ;; # Sets if the modem is a serial device. --serial=*) - serial=$(cli_get_val ${1}) + serial=$(cli_get_val "${1}") ;; --serial-device=*) - serial_device=$(cli_get_val ${1}) + serial_device=$(cli_get_val "${1}") ;; --username=*) - username=$(cli_get_val ${1}) + username=$(cli_get_val "${1}") ;; *) log WARNING "Unhandled argument: ${1}" diff --git a/src/functions/functions.pppoe-server b/src/functions/functions.pppoe-server index 2db74331..5d65d24e 100644 --- a/src/functions/functions.pppoe-server +++ b/src/functions/functions.pppoe-server @@ -70,30 +70,30 @@ pppoe_server_options() { while [ $# -gt 0 ]; do case "${1}" in --auth=*) - auth=$(cli_get_val ${1}) + auth=$(cli_get_val "${1}") ;; --default-asyncmap=*) - default_asyncmap=$(cli_get_val ${1}) + default_asyncmap=$(cli_get_val "${1}") ;; --disable-ipv6) ipv6="false" ;; --dns-servers=*) - dns_servers="$(cli_get_val ${1})" + dns_servers="$(cli_get_val "${1}")" ;; --lcp-echo-failure=*) - lcp_echo_failure=$(cli_get_val ${1}) + lcp_echo_failure=$(cli_get_val "${1}") assert isinteger ${lcp_echo_failure} ;; --lcp-echo-interval=*) - lcp_echo_interval=$(cli_get_val ${1}) + lcp_echo_interval=$(cli_get_val "${1}") assert isinteger ${lcp_echo_interval} ;; --proxyarp=*) - proxyarp=$(cli_get_val ${1}) + proxyarp=$(cli_get_val "${1}") ;; --require-auth=*) - required_auths="${required_auths} $(cli_get_val ${1})" + required_auths="${required_auths} $(cli_get_val "${1}")" ;; esac shift diff --git a/src/functions/functions.route b/src/functions/functions.route index 6a2c1fa5..7ca4f59d 100644 --- a/src/functions/functions.route +++ b/src/functions/functions.route @@ -23,7 +23,7 @@ # cli_route() { - if cli_help_requested $@; then + if cli_help_requested "$@"; then cli_show_man network-route exit ${EXIT_OK} fi @@ -33,7 +33,7 @@ cli_route() { case "${action}" in static) - cli_route_static $@ + cli_route_static "$@" ;; *) error "Unrecognized action: ${action}" @@ -47,7 +47,7 @@ cli_route() { } cli_route_static() { - if cli_help_requested $@; then + if cli_help_requested "$@"; then cli_show_man network-route-static exit ${EXIT_OK} fi @@ -58,20 +58,20 @@ cli_route_static() { case "${action}" in # Add a new route. add) - route_add $@ + route_add "$@" ;; # Remove an existing route. remove) - route_remove $@ + route_remove "$@" ;; # List all routes. list) - route_list $@ + route_list "$@" return ${EXIT_OK} ;; # Reload all routes reload) - route_apply $@ + route_apply "$@" ;; *) error "Unrecognized action: ${action}" @@ -93,7 +93,7 @@ route_add() { while [ $# -gt 0 ]; do case "${1}" in --gateway=*) - gateway=$(cli_get_val ${1}) + gateway=$(cli_get_val "${1}") ;; --unreachable) unreachable="true" @@ -105,7 +105,7 @@ route_add() { blackhole="true" ;; --mtu=*) - mtu=$(cli_get_val ${1}) + mtu=$(cli_get_val "${1}") ;; *) if isset network; then @@ -217,7 +217,7 @@ route_remove() { local _network local error=${EXIT_OK} - for _network in $@; do + for _network in "$@"; do # Validate input if ! ip_net_is_valid ${_network} && ! ip_is_valid ${_network}; then error "Invalid IP address or network: ${_network}" @@ -260,7 +260,7 @@ route_list() { while [ $# -gt 0 ]; do case "${1}" in --protocol=*) - protocol=$(cli_get_val ${1}) + protocol=$(cli_get_val "${1}") ;; *) warning "Unrecognized argument: ${1}" @@ -335,25 +335,25 @@ route_parse_line() { while read arg; do case "${arg}" in network=*) - network=$(cli_get_val ${arg}) + network=$(cli_get_val "${arg}") ;; gateway=*) - gateway=$(cli_get_val ${arg}) + gateway=$(cli_get_val "${arg}") ;; unreachable=*) - unreachable=$(cli_get_val ${arg}) + unreachable=$(cli_get_val "${arg}") ;; prohibit=*) - prohibit=$(cli_get_val ${arg}) + prohibit=$(cli_get_val "${arg}") ;; blackhole=*) - blackhole=$(cli_get_val ${arg}) + blackhole=$(cli_get_val "${arg}") ;; mtu=*) - mtu=$(cli_get_val ${arg}) + mtu=$(cli_get_val "${arg}") ;; esac - done <<< "$(args $@)" + done <<< "$(args "$@")" ### Check if all values are correctly set. @@ -443,19 +443,19 @@ route_entry_add() { while [ $# -gt 0 ]; do case "${1}" in --gateway=*) - gateway=$(cli_get_val ${1}) + gateway=$(cli_get_val "${1}") ;; --table=*) - table=$(cli_get_val ${1}) + table=$(cli_get_val "${1}") ;; --type=*) - type=$(cli_get_val ${1}) + type=$(cli_get_val "${1}") ;; --proto=*) - proto=$(cli_get_val ${1}) + proto=$(cli_get_val "${1}") ;; --mtu=*) - mtu=$(cli_get_val ${1}) + mtu=$(cli_get_val "${1}") ;; *) if isset network; then @@ -574,7 +574,7 @@ route_table_flush() { while [ $# -gt 0 ]; do case "${1}" in --protocol=*) - protocol=$(cli_get_val ${1}) + protocol=$(cli_get_val "${1}") ;; *) table="${1}" @@ -621,7 +621,7 @@ route_table_exists() { while [ $# -gt 0 ]; do case "${1}" in --id=*) - _id=$(cli_get_val ${1}) + _id=$(cli_get_val "${1}") ;; *) _table=${1} @@ -653,13 +653,13 @@ route_rule_add() { while [ $# -gt 0 ]; do case "${1}" in --lookup=*) - lookup=$(cli_get_val ${1}) + lookup=$(cli_get_val "${1}") ;; --priority=*) - priority=$(cli_get_val ${1}) + priority=$(cli_get_val "${1}") ;; --protocol=*) - protocols=$(cli_get_val ${1}) + protocols=$(cli_get_val "${1}") assert isoneof protocols ${IP_SUPPORTED_PROTOCOLS} ;; @@ -717,16 +717,16 @@ route_rule_exists() { while [ $# -gt 0 ]; do case "${1}" in --from=*) - from=$(cli_get_val ${1}) + from=$(cli_get_val "${1}") ;; --lookup=*) - lookup=$(cli_get_val ${1}) + lookup=$(cli_get_val "${1}") ;; --priority=*) - prio=$(cli_get_val ${1}) + prio=$(cli_get_val "${1}") ;; --protocol=*) - proto=$(cli_get_val ${1}) + proto=$(cli_get_val "${1}") ;; *) warning "Unrecognized argument: ${1}" diff --git a/src/functions/functions.settings b/src/functions/functions.settings index a7d94640..becf3f0f 100644 --- a/src/functions/functions.settings +++ b/src/functions/functions.settings @@ -37,7 +37,7 @@ settings_read() { list_append valid_keys "${arg}" ;; esac - done <<< "$(args $@)" + done <<< "$(args "$@")" if [ -d "${file}" ]; then error "Not a configuration file: '${file}'" @@ -58,7 +58,7 @@ settings_read() { list_match ${key} ${valid_keys} || continue fi - val=$(cli_get_val ${line}) + val=$(cli_get_val "${line}") val=$(settings_strip ${val}) # Assign variable. @@ -99,7 +99,7 @@ settings_read_array() { fi fi - val=$(cli_get_val ${line}) + val=$(cli_get_val "${line}") val=$(settings_strip ${val}) # Assign variable. @@ -146,7 +146,7 @@ settings_write() { ;; esac shift - done <<< "$(args $@)" + done <<< "$(args "$@")" # Check if all values to be written are sane if isset check_func && ! settings_check "${check_func}"; then @@ -159,7 +159,7 @@ settings_write() { > ${settings_file} local param - for param in $(list_sort $@); do + for param in $(list_sort "$@"); do echo "${param}=\"${!param}\"" >> ${settings_file} done } @@ -179,7 +179,7 @@ settings_remove() { settings_print() { local param - for param in $(list_sort $@); do + for param in $(list_sort "$@"); do printf "%-32s = %s\n" "${param}" "${!param}" done } @@ -219,8 +219,8 @@ settings_set() { while [ $# -gt 0 ]; do case "${1}" in *=*) - local key=$(cli_get_key ${1}) - local val=$(cli_get_val ${1}) + local key=$(cli_get_key "${1}") + local val=$(cli_get_val "${1}") log INFO "Setting configuration option '${key}=${val}'". @@ -258,8 +258,8 @@ network_settings_set() { case "${arg}" in *=*) - local key=$(cli_get_key ${arg}) - local val=$(cli_get_val ${arg}) + local key=$(cli_get_key "${arg}") + local val=$(cli_get_val "${arg}") case "${key}" in DNS_RANDOMIZE|DNS_SEARCH_DOMAIN|DNS_USE_LOCAL_RESOLVER) diff --git a/src/functions/functions.util b/src/functions/functions.util index 687d008f..24e3e66b 100644 --- a/src/functions/functions.util +++ b/src/functions/functions.util @@ -134,7 +134,7 @@ parse_time() { local ret=0 local arg - for arg in $@; do + for arg in "$@"; do local unit case "${arg}" in @@ -495,7 +495,7 @@ cmd() { } cmd_quiet() { - cmd $@ &>/dev/null + cmd "$@" &>/dev/null } cmd_exec() { @@ -515,7 +515,7 @@ cmd_not_implemented() { # Executes the given command in background cmd_background() { - cmd_quiet $@ & + cmd_quiet "$@" & } # Prints the PID of the process that was started last @@ -531,7 +531,7 @@ cmd_background_result() { # Increase security of the read command read() { - builtin read -r $@ + builtin read -r "$@" } seq() { @@ -557,7 +557,7 @@ count() { } which() { - type -P $@ + type -P "$@" } # Prints the number of seconds since epoch. diff --git a/src/functions/functions.vpn b/src/functions/functions.vpn index 6d62f17d..c004397e 100644 --- a/src/functions/functions.vpn +++ b/src/functions/functions.vpn @@ -25,10 +25,10 @@ cli_vpn() { case "${action}" in ipsec) - cli_ipsec $@ + cli_ipsec "$@" ;; security-policies) - cli_vpn_security_policies $@ + cli_vpn_security_policies "$@" ;; *) error "Unrecognized argument: ${action}" diff --git a/src/functions/functions.vpn-security-policies b/src/functions/functions.vpn-security-policies index d5b43b0c..14386ecc 100644 --- a/src/functions/functions.vpn-security-policies +++ b/src/functions/functions.vpn-security-policies @@ -269,13 +269,13 @@ cli_vpn_security_policies() { case "${key}" in cipher|compression|integrity|lifetime|pfs|show) - vpn_security_policies_${key} ${security_policy} $@ + vpn_security_policies_${key} ${security_policy} "$@" ;; group-type) - vpn_security_policies_group_type ${security_policy} $@ + vpn_security_policies_group_type ${security_policy} "$@" ;; key-exchange) - vpn_security_policies_key_exchange ${security_policy} $@ + vpn_security_policies_key_exchange ${security_policy} "$@" ;; *) error "Unrecognized argument: ${key}" @@ -288,10 +288,10 @@ cli_vpn_security_policies() { case "${action}" in new) - vpn_security_policies_new $@ + vpn_security_policies_new "$@" ;; destroy) - vpn_security_policies_destroy $@ + vpn_security_policies_destroy "$@" ;; ""|*) if [ -n "${action}" ]; then @@ -419,7 +419,7 @@ vpn_security_policies_read_config() { if [ $# -eq 0 ] && [ -n "${VPN_SECURITY_POLICIES_CONFIG_SETTINGS}" ]; then list_append args ${VPN_SECURITY_POLICIES_CONFIG_SETTINGS} else - list_append args $@ + list_append args "$@" fi local path="$(vpn_security_policies_path ${name})" @@ -875,7 +875,7 @@ vpn_security_policies_lifetime(){ fi if ! isinteger value; then - value=$(parse_time $@) + value=$(parse_time "$@") if [ ! $? -eq 0 ]; then log ERROR "Parsing the passed time was not sucessful please check the passed values." return ${EXIT_ERROR} @@ -974,7 +974,7 @@ vpn_security_policies_new() { # Function that deletes based on the passed parameters one ore more vpn security policies vpn_security_policies_destroy() { local name - for name in $@; do + for name in "$@"; do if ! vpn_security_policy_exists ${name}; then log ERROR "The vpn security policy ${name} does not exist." continue diff --git a/src/functions/functions.wireless b/src/functions/functions.wireless index ad6a4377..57377e1a 100644 --- a/src/functions/functions.wireless +++ b/src/functions/functions.wireless @@ -38,17 +38,17 @@ wireless_create() { while [ $# -gt 0 ]; do case "${1}" in --address=*) - address=$(cli_get_val ${1}) + address=$(cli_get_val "${1}") ;; --channel=*) channel=$(cli_get_val "${1}") ;; --phy=*) - phy=$(cli_get_val ${1}) + phy=$(cli_get_val "${1}") phy=$(phy_get ${phy}) ;; --type=*) - type=$(cli_get_val ${1}) + type=$(cli_get_val "${1}") # ap --> __ap [ "${type}" = "ap" ] && type="__ap" @@ -322,13 +322,13 @@ wireless_ibss_join() { while [ $# -gt 0 ]; do case "${1}" in --bssid=*) - bssid="$(cli_get_val ${1})" + bssid="$(cli_get_val "${1}")" ;; --essid=*) - essid="$(cli_get_val ${1})" + essid="$(cli_get_val "${1}")" ;; --channel=*) - local channel="$(cli_get_val ${1})" + local channel="$(cli_get_val "${1}")" # Save the frequency of the channel instead # of the channel itself. diff --git a/src/functions/functions.wpa_supplicant b/src/functions/functions.wpa_supplicant index f8aed22a..e7f36771 100644 --- a/src/functions/functions.wpa_supplicant +++ b/src/functions/functions.wpa_supplicant @@ -36,19 +36,19 @@ wpa_supplicant_config_write() { for arg in "$@"; do case "${arg}" in --ap-scan=*) - ap_scan=$(cli_get_val ${arg}) + ap_scan=$(cli_get_val "${arg}") ;; --mode=*) - mode=$(cli_get_val ${arg}) + mode=$(cli_get_val "${arg}") # Empty signals no encryption. isset mode || mode="NONE" ;; --ssid=*) - ssid=$(cli_get_val ${arg}) + ssid=$(cli_get_val "${arg}") ;; --key=*) - key=$(cli_get_val ${arg}) + key=$(cli_get_val "${arg}") ;; *) error "Unrecognized argument: ${arg}" diff --git a/src/functions/functions.zone b/src/functions/functions.zone index b79127a2..e9e2dd44 100644 --- a/src/functions/functions.zone +++ b/src/functions/functions.zone @@ -210,7 +210,7 @@ zone_cmd() { local hook="$(zone_get_hook ${zone})" assert isset hook - hook_exec zone "${hook}" "${cmd}" "${zone}" $@ + hook_exec zone "${hook}" "${cmd}" "${zone}" "$@" } zone_new() { @@ -238,7 +238,7 @@ zone_new() { # Create directories for configs and ports mkdir -p $(zone_dir ${zone})/{configs,ports} - hook_zone_exec "${hook}" "new" "${zone}" $@ + hook_zone_exec "${hook}" "new" "${zone}" "$@" local ret=$? # Maybe the zone new hook did not exit correctly. @@ -275,7 +275,7 @@ zone_edit() { return ${EXIT_ERROR} fi - hook_zone_exec ${hook} edit ${zone} $@ + hook_zone_exec ${hook} edit ${zone} "$@" } zone_rename() { @@ -360,7 +360,7 @@ zone_up() { zone_db ${zone} starting - hook_zone_exec ${hook} up ${zone} $@ + hook_zone_exec ${hook} up ${zone} "$@" zone_db ${zone} started @@ -390,7 +390,7 @@ zone_down() { zone_db ${zone} stopping - hook_zone_exec ${hook} down ${zone} $@ + hook_zone_exec ${hook} down ${zone} "$@" zone_db ${zone} stopped @@ -437,7 +437,7 @@ zone_identify() { local port for port in $(zone_get_ports "${zone}"); do # Identify all the ports - port_identify "${port}" --background $@ + port_identify "${port}" --background "$@" # Save the PIDs of the subprocesses list_append pids "$(cmd_background_get_pid)" @@ -716,7 +716,7 @@ zone_show() { zones_show() { local zone - for zone in $(zones_get $@); do + for zone in $(zones_get "$@"); do zone_show ${zone} done } @@ -956,23 +956,23 @@ zone_port_cmd() { assert isset hook # Dispatch command to hook - hook_zone_exec "${hook}" "${cmd}" "${zone}" "${port}" $@ + hook_zone_exec "${hook}" "${cmd}" "${zone}" "${port}" "$@" } zone_port_create() { - zone_port_cmd "port_create" $@ + zone_port_cmd "port_create" "$@" } zone_port_remove() { - zone_port_cmd "port_remove" $@ + zone_port_cmd "port_remove" "$@" } zone_port_up() { - zone_port_cmd "port_up" $@ + zone_port_cmd "port_up" "$@" } zone_port_down() { - zone_port_cmd "port_down" $@ + zone_port_cmd "port_down" "$@" } # The next two functions automagically bring up and down @@ -1013,7 +1013,7 @@ zone_port_stop() { } zone_port_status() { - zone_port_cmd "port_status" $@ + zone_port_cmd "port_status" "$@" } zone_ports_cmd() { @@ -1029,28 +1029,28 @@ zone_ports_cmd() { local port for port in $(zone_get_ports ${zone}); do - hook_zone_exec "${hook}" "${cmd}" "${zone}" "${port}" $@ + hook_zone_exec "${hook}" "${cmd}" "${zone}" "${port}" "$@" done } zone_ports_create() { - zone_ports_cmd "port_create" $@ + zone_ports_cmd "port_create" "$@" } zone_ports_remove() { - zone_ports_cmd "port_remove" $@ + zone_ports_cmd "port_remove" "$@" } zone_ports_up() { - zone_ports_cmd "port_up" $@ + zone_ports_cmd "port_up" "$@" } zone_ports_down() { - zone_ports_cmd "port_down" $@ + zone_ports_cmd "port_down" "$@" } zone_ports_status() { - zone_ports_cmd "port_status" $@ + zone_ports_cmd "port_status" "$@" } zone_configs_cmd() { @@ -1067,20 +1067,20 @@ zone_configs_cmd() { local config_hook="$(zone_config_get_hook "${zone}" "${config}")" assert isset config_hook - hook_config_exec "${config_hook}" "${cmd}" "${zone}" "${config}" $@ + hook_config_exec "${config_hook}" "${cmd}" "${zone}" "${config}" "$@" done } zone_configs_up() { - zone_configs_cmd "up" $@ + zone_configs_cmd "up" "$@" } zone_configs_down() { - zone_configs_cmd "down" $@ + zone_configs_cmd "down" "$@" } zone_configs_status() { - zone_configs_cmd "status" $@ + zone_configs_cmd "status" "$@" } zone_configs_list() { @@ -1234,7 +1234,7 @@ zone_config_get_hook_from_id() { } zone_has_ip() { - device_has_ip $@ + device_has_ip "$@" } zone_db() { @@ -1256,7 +1256,7 @@ zone_is_up() { } zone_is_down() { - ! zone_is_up $@ + ! zone_is_up "$@" } zone_get_supported_port_hooks() { @@ -1288,7 +1288,7 @@ zone_settings_read() { if [ $# -eq 0 ] && [ -n "${HOOK_SETTINGS}" ]; then list_append args ${HOOK_SETTINGS} else - list_append args $@ + list_append args "$@" fi # Save the HOOK variable. @@ -1357,7 +1357,7 @@ zone_config_settings_read() { if [ $# -eq 0 ] && [ -n "${HOOK_CONFIG_SETTINGS}" ]; then list_append args ${HOOK_CONFIG_SETTINGS} else - list_append args $@ + list_append args "$@" fi local path="$(zone_dir "${zone}")/configs/${config}" @@ -1415,7 +1415,7 @@ zone_port_settings_read() { if [ $# -eq 0 ] && [ -n "${HOOK_PORT_SETTINGS}" ]; then list_append args ${HOOK_PORT_SETTINGS} else - list_append args $@ + list_append args "$@" fi local path="$(zone_dir "${zone}")/ports/${port}" diff --git a/src/header-config b/src/header-config index 6341a22b..ec85a70e 100644 --- a/src/header-config +++ b/src/header-config @@ -55,7 +55,7 @@ hook_edit() { return ${EXIT_ERROR} fi - if ! hook_parse_cmdline $@; then + if ! hook_parse_cmdline "$@"; then # Return an error if the parsing of the cmd line fails return ${EXIT_ERROR} fi diff --git a/src/header-port b/src/header-port index 9fd2503a..68cb62de 100644 --- a/src/header-port +++ b/src/header-port @@ -66,7 +66,7 @@ hook_default_edit() { port_settings_read "${port}" ${HOOK_SETTINGS} - if ! hook_parse_cmdline $@; then + if ! hook_parse_cmdline "$@"; then return ${EXIT_ERROR} fi @@ -76,7 +76,7 @@ hook_default_edit() { } hook_edit() { - hook_default_edit $@ + hook_default_edit "$@" } # Returns a list of all children of this port @@ -132,7 +132,7 @@ hook_default_up() { # Depends on the port existing hook_up() { - hook_default_up $@ + hook_default_up "$@" } hook_default_down() { @@ -151,5 +151,5 @@ hook_default_down() { } hook_down() { - hook_default_down $@ + hook_default_down "$@" } diff --git a/src/header-zone b/src/header-zone index 189fbc38..ead4a32c 100644 --- a/src/header-zone +++ b/src/header-zone @@ -34,7 +34,7 @@ hook_new() { assert isset zone shift - if ! hook_parse_cmdline $@; then + if ! hook_parse_cmdline "$@"; then return ${EXIT_ERROR} fi @@ -56,7 +56,7 @@ hook_edit() { return ${EXIT_ERROR} fi - if ! hook_parse_cmdline $@; then + if ! hook_parse_cmdline "$@"; then return ${EXIT_ERROR} fi @@ -112,7 +112,7 @@ hook_port() { local ret case "${action}" in add|create|edit|rem|show) - hook_port_${action} "${zone}" $@ + hook_port_${action} "${zone}" "$@" ret=$? ;; *) @@ -150,7 +150,7 @@ hook_default_port_create() { } hook_port_create() { - hook_default_port_create $@ + hook_default_port_create "$@" } hook_default_port_remove() { @@ -163,7 +163,7 @@ hook_default_port_remove() { } hook_port_remove() { - hook_default_port_remove $@ + hook_default_port_remove "$@" } hook_port_up() { diff --git a/src/hooks/configs/dhcp b/src/hooks/configs/dhcp index 7f6780b5..f22507c6 100644 --- a/src/hooks/configs/dhcp +++ b/src/hooks/configs/dhcp @@ -72,7 +72,7 @@ hook_new() { return ${EXIT_ERROR} fi - if ! hook_parse_cmdline $@; then + if ! hook_parse_cmdline "$@"; then # Return an error if the parsing of the cmd line fails return ${EXIT_ERROR} fi diff --git a/src/hooks/configs/ipv4-static b/src/hooks/configs/ipv4-static index ef74991e..7aea0b97 100644 --- a/src/hooks/configs/ipv4-static +++ b/src/hooks/configs/ipv4-static @@ -87,7 +87,7 @@ hook_parse_cmdline() { exit ${EXIT_CONF_ERROR} ;; esac - done <<< "$(args $@)" + done <<< "$(args "$@")" if ! isset ADDRESS; then error "You need to provide an IPv4 address" @@ -115,7 +115,7 @@ hook_new() { assert zone_exists "${zone}" - if ! hook_parse_cmdline $@; then + if ! hook_parse_cmdline "$@"; then # Return an error if the parsing of the cmd line fails return ${EXIT_ERROR} fi diff --git a/src/hooks/configs/ipv6-auto b/src/hooks/configs/ipv6-auto index 375e585d..bf1003d1 100644 --- a/src/hooks/configs/ipv6-auto +++ b/src/hooks/configs/ipv6-auto @@ -45,7 +45,7 @@ hook_parse_cmdline() { fi ;; esac - done <<< "$(args $@)" + done <<< "$(args "$@")" } hook_new() { @@ -57,7 +57,7 @@ hook_new() { return ${EXIT_ERROR} fi - if ! hook_parse_cmdline $@; then + if ! hook_parse_cmdline "$@"; then # Return an error if the parsing of the cmd line fails return ${EXIT_ERROR} fi diff --git a/src/hooks/configs/ipv6-static b/src/hooks/configs/ipv6-static index c41401cb..273c201a 100644 --- a/src/hooks/configs/ipv6-static +++ b/src/hooks/configs/ipv6-static @@ -66,7 +66,7 @@ hook_new() { local zone=${1} shift - if ! hook_parse_cmdline $@; then + if ! hook_parse_cmdline "$@"; then # Return an error if the parsing of the cmd line fails return ${EXIT_ERROR} fi diff --git a/src/hooks/configs/pppoe-server b/src/hooks/configs/pppoe-server index b4d25389..7021be2a 100644 --- a/src/hooks/configs/pppoe-server +++ b/src/hooks/configs/pppoe-server @@ -65,24 +65,24 @@ hook_parse_cmdline() { done ;; --max-sessions=*) - MAX_SESSIONS=$(cli_get_val ${1}) + MAX_SESSIONS=$(cli_get_val "${1}") if ! isinteger ${MAX_SESSIONS} || ! [ ${MAX_SESSIONS} -ge 0 ]; then error "Invalid value for '--max-session'. This value must be an integer greate or eqal zero." exit ${EXIT_ERROR} fi ;; --mtu=*) - MTU=$(cli_get_val ${1}) + MTU=$(cli_get_val "${1}") if ! mtu_is_valid "ipv4" ${MTU}; then error "Invalid value for '--mtu'. Cannot be larger then 9000 or smaller than 576" exit ${EXIT_ERROR} fi ;; --service-name=*) - SERVICE_NAME=$(cli_get_val ${1}) + SERVICE_NAME=$(cli_get_val "${1}") ;; --subnet=*) - SUBNET=$(cli_get_val ${1}) + SUBNET=$(cli_get_val "${1}") if ! ipv4_net_is_valid "${SUBNET}"; then error "Invalid IPv4 Subnet ${SUBNET}." exit ${EXIT_ERROR} @@ -105,7 +105,7 @@ hook_new() { return ${EXIT_ERROR} fi - if ! hook_parse_cmdline $@; then + if ! hook_parse_cmdline "$@"; then # Return an error if the parsing of the cmd line fails return ${EXIT_ERROR} fi diff --git a/src/hooks/ports/batman-adv b/src/hooks/ports/batman-adv index 4f162019..b304f4fe 100644 --- a/src/hooks/ports/batman-adv +++ b/src/hooks/ports/batman-adv @@ -37,10 +37,10 @@ hook_new() { while [ $# -gt 0 ]; do case "${1}" in --address=*) - ADDRESS="$(cli_get_val ${1})" + ADDRESS="$(cli_get_val "${1}")" ;; --slaves=*) - SLAVES="$(cli_get_val ${1})" + SLAVES="$(cli_get_val "${1}")" ;; *) warning "Ignoring unknown argument '${1}'" @@ -67,13 +67,13 @@ hook_edit() { while [ $# -gt 0 ]; do case "${1}" in --address=*) - ADDRESS="$(cli_get_val ${1})" + ADDRESS="$(cli_get_val "${1}")" ;; --add-slave=*) - SLAVES="${SLAVES} $(cli_get_val ${1})" + SLAVES="${SLAVES} $(cli_get_val "${1}")" ;; --del-slave=*) - local slave="$(cli_get_val ${1})" + local slave="$(cli_get_val "${1}")" local s slaves for s in ${SLAVES}; do diff --git a/src/hooks/ports/bonding b/src/hooks/ports/bonding index e268186c..36bcf6c7 100644 --- a/src/hooks/ports/bonding +++ b/src/hooks/ports/bonding @@ -39,16 +39,16 @@ hook_parse_cmdline() { while [ $# -gt 0 ]; do case "${1}" in --address=*) - ADDRESS=$(cli_get_val ${1}) + ADDRESS=$(cli_get_val "${1}") ;; --miimon=*) - MIIMON=$(cli_get_val ${1}) + MIIMON=$(cli_get_val "${1}") ;; --mode=*) - MODE=$(cli_get_val ${1}) + MODE=$(cli_get_val "${1}") ;; +*) - local slave=$(cli_get_val ${1:1}) + local slave=$(cli_get_val "${1:1}") if port_exists "${slave}"; then if list_match "${slave}" ${SLAVES}; then @@ -61,7 +61,7 @@ hook_parse_cmdline() { fi ;; -*) - local slave=$(cli_get_val ${1:1}) + local slave=$(cli_get_val "${1:1}") if ! list_remove SLAVES "${slave}"; then warning "Port ${slave} is not a slave of this bonding device" fi @@ -84,7 +84,7 @@ hook_parse_cmdline() { } hook_new() { - if ! hook_parse_cmdline $@; then + if ! hook_parse_cmdline "$@"; then return ${EXIT_ERROR} fi @@ -106,7 +106,7 @@ hook_new() { hook_edit() { local port=${1} - if ! hook_default_edit $@; then + if ! hook_default_edit "$@"; then return ${EXIT_ERROR} fi diff --git a/src/hooks/ports/dummy b/src/hooks/ports/dummy index 7e6f5d24..30de50e9 100644 --- a/src/hooks/ports/dummy +++ b/src/hooks/ports/dummy @@ -31,7 +31,7 @@ hook_parse_cmdline() { while [ $# -gt 0 ]; do case "${1}" in --address=*) - ADDRESS=$(cli_get_val ${1}) + ADDRESS=$(cli_get_val "${1}") ;; *) warning "Unknown argument '${1}'" @@ -53,7 +53,7 @@ hook_parse_cmdline() { } hook_new() { - if ! hook_parse_cmdline $@; then + if ! hook_parse_cmdline "$@"; then return ${EXIT_ERROR} fi diff --git a/src/hooks/ports/vlan b/src/hooks/ports/vlan index 34436148..2231de11 100644 --- a/src/hooks/ports/vlan +++ b/src/hooks/ports/vlan @@ -51,13 +51,13 @@ hook_new() { while [ $# -gt 0 ]; do case "${1}" in --parent-device=*) - PARENT_DEVICE=$(cli_get_val ${1}) + PARENT_DEVICE=$(cli_get_val "${1}") ;; --address=*) - ADDRESS=$(cli_get_val ${1}) + ADDRESS=$(cli_get_val "${1}") ;; --tag=*) - TAG=$(cli_get_val ${1}) + TAG=$(cli_get_val "${1}") ;; *) warning "Unknown argument '${1}'" @@ -83,7 +83,7 @@ hook_edit() { while [ $# -gt 0 ]; do case "${1}" in --address=*) - ADDRESS=$(cli_get_val ${1}) + ADDRESS=$(cli_get_val "${1}") ;; *) warning "Unknown argument '${1}'" diff --git a/src/hooks/ports/wireless-adhoc b/src/hooks/ports/wireless-adhoc index 09ab4028..d97bf551 100644 --- a/src/hooks/ports/wireless-adhoc +++ b/src/hooks/ports/wireless-adhoc @@ -45,22 +45,22 @@ hook_new() { while [ $# -gt 0 ]; do case "${1}" in --address=*) - ADDRESS=$(cli_get_val ${1}) + ADDRESS=$(cli_get_val "${1}") ;; --bssid=*) - BSSID=$(cli_get_val ${1}) + BSSID=$(cli_get_val "${1}") ;; --channel=*) - CHANNEL=$(cli_get_val ${1}) + CHANNEL=$(cli_get_val "${1}") ;; --mtu=*) MTU="$(cli_get_val "${1}")" ;; --phy=*) - PHY=$(cli_get_val ${1}) + PHY=$(cli_get_val "${1}") ;; --ssid=*) - SSID=$(cli_get_val ${1}) + SSID=$(cli_get_val "${1}") ;; *) warning "Ignoring unknown argument '${1}'" @@ -91,16 +91,16 @@ hook_edit() { while [ $# -gt 0 ]; do case "${1}" in --bssid=*) - BSSID=$(cli_get_val ${1}) + BSSID=$(cli_get_val "${1}") ;; --channel=*) - CHANNEL=$(cli_get_val ${1}) + CHANNEL=$(cli_get_val "${1}") ;; --mtu=*) MTU="$(cli_get_val "${1}")" ;; --ssid=*) - SSID=$(cli_get_val ${1}) + SSID=$(cli_get_val "${1}") ;; *) warning "Unknown argument '${1}'" diff --git a/src/hooks/ports/wireless-ap b/src/hooks/ports/wireless-ap index 9c0375cc..4b31a8b2 100644 --- a/src/hooks/ports/wireless-ap +++ b/src/hooks/ports/wireless-ap @@ -59,28 +59,28 @@ hook_parse_cmdline() { while [ $# -gt 0 ]; do case "${1}" in --broadcast-ssid=*) - BROADCAST_SSID=$(cli_get_val ${1}) + BROADCAST_SSID=$(cli_get_val "${1}") ;; --channel=*) - CHANNEL=$(cli_get_val ${1}) + CHANNEL=$(cli_get_val "${1}") ;; --encryption=*) - ENCRYPTION=$(cli_get_val ${1}) + ENCRYPTION=$(cli_get_val "${1}") ;; --key=*) - KEY=$(cli_get_val ${1}) + KEY=$(cli_get_val "${1}") ;; --mac=*) - ADDRESS=$(cli_get_val ${1}) + ADDRESS=$(cli_get_val "${1}") ;; --mode=*) - MODE=$(cli_get_val ${1}) + MODE=$(cli_get_val "${1}") ;; --phy=*) - PHY=$(cli_get_val ${1}) + PHY=$(cli_get_val "${1}") ;; --ssid=*) - SSID=$(cli_get_val ${1}) + SSID=$(cli_get_val "${1}") ;; *) warning "Ignoring unknown argument '${1}'" @@ -98,7 +98,7 @@ hook_edit() { local port=${1} assert isset port - if ! hook_default_edit $@; then + if ! hook_default_edit "$@"; then return ${EXIT_ERROR} fi diff --git a/src/hooks/zones/6to4-tunnel b/src/hooks/zones/6to4-tunnel index f5507414..bf20872a 100644 --- a/src/hooks/zones/6to4-tunnel +++ b/src/hooks/zones/6to4-tunnel @@ -64,16 +64,16 @@ hook_parse_cmdline() { while [ $# -gt 0 ]; do case "${1}" in --server-address=*) - SERVER_ADDRESS=$(cli_get_val ${1}) + SERVER_ADDRESS=$(cli_get_val "${1}") ;; --local-ipv4-address=*) - LOCAL_ADDRESS=$(cli_get_val ${1}) + LOCAL_ADDRESS=$(cli_get_val "${1}") ;; --local-ipv6-address=*) - LOCAL_ADDRESS6=$(cli_get_val ${1}) + LOCAL_ADDRESS6=$(cli_get_val "${1}") ;; --auto-update-endpoint=*) - local val="$(cli_get_val ${1})" + local val="$(cli_get_val "${1}")" if enabled val; then AUTO_UPDATE_ENDPOINT="true" @@ -82,13 +82,13 @@ hook_parse_cmdline() { fi ;; --tunnel-id=*) - TUNNEL_ID="$(cli_get_val ${1})" + TUNNEL_ID="$(cli_get_val "${1}")" ;; --username=*) - USERNAME="$(cli_get_val ${1})" + USERNAME="$(cli_get_val "${1}")" ;; --password=*) - PASSWORD="$(cli_get_val ${1})" + PASSWORD="$(cli_get_val "${1}")" ;; *) echo "Unknown option: ${1}" >&2 diff --git a/src/hooks/zones/bridge b/src/hooks/zones/bridge index 245f4c1a..e90111ca 100644 --- a/src/hooks/zones/bridge +++ b/src/hooks/zones/bridge @@ -268,7 +268,7 @@ hook_port_attach() { PRIORITY="$(cli_get_val "${arg}")" ;; esac - done <<< "$(args $@)" + done <<< "$(args "$@")" if ! zone_port_settings_write "${zone}" "${port}"; then exit ${EXIT_ERROR} @@ -294,7 +294,7 @@ hook_port_detach() { } hook_port_edit() { - hook_port_attach $@ + hook_port_attach "$@" } hook_port_up() { diff --git a/src/hooks/zones/modem b/src/hooks/zones/modem index ff21d455..1b4c3c02 100644 --- a/src/hooks/zones/modem +++ b/src/hooks/zones/modem @@ -94,39 +94,39 @@ hook_parse_cmdline() { while [ $# -gt 0 ]; do case "${1}" in --apn=*) - APN=$(cli_get_val ${1}) + APN=$(cli_get_val "${1}") ;; --auth=*) - AUTH=$(cli_get_val ${1}) + AUTH=$(cli_get_val "${1}") ;; --baudrate=*) - BAUDRATE=$(cli_get_val ${1}) + BAUDRATE=$(cli_get_val "${1}") assert isoneif "${BAUDRATE}" ${SERIAL_BAUDRATES} ;; --device=*) - DEVICE=$(cli_get_val ${1}) + DEVICE=$(cli_get_val "${1}") ;; --imsi=*) IMSI="$(cli_get_val "${1}")" ;; --monitor-device=*) - MONITOR_DEVICE=$(cli_get_val ${1}) + MONITOR_DEVICE=$(cli_get_val "${1}") ;; --mtu=*) - MTU=$(cli_get_val ${1}) + MTU=$(cli_get_val "${1}") assert isinteger ${MTU} ;; --password=*) - PASSWORD=$(cli_get_val ${1}) + PASSWORD=$(cli_get_val "${1}") ;; --phone-number=*) - PHONE_NUMBER=$(cli_get_val ${1}) + PHONE_NUMBER=$(cli_get_val "${1}") ;; --pin=*) - PIN=$(cli_get_val ${1}) + PIN=$(cli_get_val "${1}") ;; --username=*) - USERNAME=$(cli_get_val ${1}) + USERNAME=$(cli_get_val "${1}") ;; *) echo "Unknown argument: ${1}" >&2 diff --git a/src/hooks/zones/pppoe b/src/hooks/zones/pppoe index 5b44b8c3..021d89b2 100644 --- a/src/hooks/zones/pppoe +++ b/src/hooks/zones/pppoe @@ -64,10 +64,10 @@ hook_parse_cmdline() { while [ $# -gt 0 ]; do case "${1}" in --access-concentrator=*) - ACCESS_CONCENTRATOR=$(cli_get_val ${1}) + ACCESS_CONCENTRATOR=$(cli_get_val "${1}") ;; --auth=*) - AUTH=$(cli_get_val ${1}) + AUTH=$(cli_get_val "${1}") ;; --ipv6=*) local value="$(cli_get_val "${1}")" @@ -78,19 +78,19 @@ hook_parse_cmdline() { fi ;; --mtu=*) - MTU=$(cli_get_val ${1}) + MTU=$(cli_get_val "${1}") ;; --password=*) - PASSWORD=$(cli_get_val ${1}) + PASSWORD=$(cli_get_val "${1}") ;; --prefix-delegation=*) PREFIX_DELEGATION="$(cli_get_bool "${1}")" ;; --service-name=*) - SERVICE_NAME=$(cli_get_val ${1}) + SERVICE_NAME=$(cli_get_val "${1}") ;; --username=*) - USERNAME=$(cli_get_val ${1}) + USERNAME=$(cli_get_val "${1}") ;; *) warning "Unknown argument: ${1}" >&2 diff --git a/src/hooks/zones/wireless b/src/hooks/zones/wireless index bd202f2a..0bb4aa81 100644 --- a/src/hooks/zones/wireless +++ b/src/hooks/zones/wireless @@ -48,19 +48,19 @@ hook_parse_cmdline() { while [ $# -gt 0 ]; do case "${1}" in --phy=*|--parent-device=*) - PHY=$(cli_get_val ${1}) + PHY=$(cli_get_val "${1}") ;; --encryption-mode=*) - ENCRYPTION_MODE=$(cli_get_val ${1}) + ENCRYPTION_MODE=$(cli_get_val "${1}") ;; --address=*) - ADDRESS=$(cli_get_val ${1}) + ADDRESS=$(cli_get_val "${1}") ;; --ssid=*) - SSID=$(cli_get_val ${1}) + SSID=$(cli_get_val "${1}") ;; --key=*) - KEY=$(cli_get_val ${1}) + KEY=$(cli_get_val "${1}") ;; *) warning "Unrecognized option: ${1}" diff --git a/src/network b/src/network index 711cf75f..bc4210c0 100644 --- a/src/network +++ b/src/network @@ -39,13 +39,13 @@ done network_settings_read cli_settings() { - if cli_help_requested $@; then + if cli_help_requested "$@"; then cli_show_man network-settings exit ${EXIT_OK} fi if [ -n "${1}" ]; then - network_settings_set $@ + network_settings_set "$@" network_settings_write else network_settings_print @@ -53,7 +53,7 @@ cli_settings() { } cli_device() { - if cli_help_requested $@; then + if cli_help_requested "$@"; then cli_show_man network-device exit ${EXIT_OK} fi @@ -63,7 +63,7 @@ cli_device() { case "${action}" in list) - cli_device_list $@ + cli_device_list "$@" ;; *) local device="${action}" @@ -79,22 +79,22 @@ cli_device() { case "${action}" in discover) - cli_device_discover ${device} $@ + cli_device_discover ${device} "$@" ;; identify) - device_identify "${device}" $@ + device_identify "${device}" "$@" ;; monitor) - cli_device_monitor "${device}" $@ + cli_device_monitor "${device}" "$@" ;; status) cli_device_status ${device} ;; unlock) - cli_device_serial_unlock ${device} $@ + cli_device_serial_unlock ${device} "$@" ;; ussd) - cli_device_send_ussd_command "${device}" $@ + cli_device_send_ussd_command "${device}" "$@" ;; *) cli_show_man network-device @@ -339,7 +339,7 @@ cli_device_discover() { } cli_device_serial_unlock() { - if cli_help_requested $@; then + if cli_help_requested "$@"; then cli_show_man network-device exit ${EXIT_OK} fi @@ -482,7 +482,7 @@ cli_device_list() { } cli_hostname() { - if cli_help_requested $@; then + if cli_help_requested "$@"; then cli_show_man network exit ${EXIT_OK} fi @@ -501,7 +501,7 @@ cli_hostname() { } cli_port() { - if cli_help_requested $@; then + if cli_help_requested "$@"; then cli_show_man network-port exit ${EXIT_OK} fi @@ -516,13 +516,13 @@ cli_port() { case "${action}" in edit|create|remove|up|down|status|identify) - port_${action} "${port}" $@ + port_${action} "${port}" "$@" ;; color) - color_cli "port" "${port}" $@ + color_cli "port" "${port}" "$@" ;; description) - description_cli "port" "${port}" $@ + description_cli "port" "${port}" "$@" ;; *) error "Unrecognized argument: ${action}" @@ -535,7 +535,7 @@ cli_port() { case "${action}" in new|destroy) - port_${action} $@ + port_${action} "$@" ;; *) error "Unrecognized argument: ${action}" @@ -546,7 +546,7 @@ cli_port() { } cli_zone() { - if cli_help_requested $@; then + if cli_help_requested "$@"; then cli_show_man network-zone exit ${EXIT_OK} fi @@ -574,19 +574,19 @@ cli_zone() { case "${action}" in port) - cli_zone_port "${zone}" $@ + cli_zone_port "${zone}" "$@" ;; rename) - cli_zone_rename "${zone}" $@ + cli_zone_rename "${zone}" "$@" ;; config|disable|down|edit|enable|identify|status|up) - zone_${action} ${zone} $@ + zone_${action} ${zone} "$@" ;; color) - color_cli "zone" "${zone}" $@ + color_cli "zone" "${zone}" "$@" ;; description) - description_cli "zone" ${zone} $@ + description_cli "zone" ${zone} "$@" ;; *) error "Unrecognized argument: ${action}" @@ -600,10 +600,10 @@ cli_zone() { case "${action}" in new) - cli_zone_new $@ + cli_zone_new "$@" ;; destroy) - cli_zone_destroy $@ + cli_zone_destroy "$@" ;; ""|*) if [ -n "${action}" ]; then @@ -619,19 +619,19 @@ cli_zone() { } cli_zone_new() { - if cli_help_requested $@ || [ $# -lt 2 ]; then + if cli_help_requested "$@" || [ $# -lt 2 ]; then cli_show_man network-zone-new exit ${EXIT_OK} fi - zone_new $@ + zone_new "$@" } # Removes a zone either immediately, if it is currently down, # or adds a tag that the removal will be done when the zone # is brought down the next time. cli_zone_destroy() { - if cli_help_requested $@; then + if cli_help_requested "$@"; then cli_show_man network-zone exit ${EXIT_OK} fi @@ -651,7 +651,7 @@ cli_zone_destroy() { } cli_zone_port() { - if cli_help_requested $@; then + if cli_help_requested "$@"; then cli_show_man network-zone-port exit ${EXIT_OK} fi @@ -666,7 +666,7 @@ cli_zone_port() { case "${action}" in edit) - zone_port_edit "${zone}" "${port}" $@ + zone_port_edit "${zone}" "${port}" "$@" ;; *) error "Unrecognised argument: ${action}" @@ -679,10 +679,10 @@ cli_zone_port() { case "${action}" in attach) - zone_port_attach "${zone}" $@ + zone_port_attach "${zone}" "$@" ;; detach) - zone_port_detach "${zone}" $@ + zone_port_detach "${zone}" "$@" ;; *) error "Unrecognised argument: ${action}" @@ -695,7 +695,7 @@ cli_zone_port() { } cli_zone_rename() { - if cli_help_requested $@; then + if cli_help_requested "$@"; then cli_show_man network-zone exit ${EXIT_OK} fi @@ -739,7 +739,7 @@ cli_list_hooks() { local type=${1} shift - if cli_help_requested $@; then + if cli_help_requested "$@"; then cli_show_man network-zone exit ${EXIT_OK} fi @@ -759,7 +759,7 @@ cli_dhcpd() { local proto=${1} shift - if cli_help_requested $@; then + if cli_help_requested "$@"; then cli_show_man network-dhcp exit ${EXIT_OK} fi @@ -769,7 +769,7 @@ cli_dhcpd() { case "${action}" in edit) - dhcpd_edit ${proto} $@ + dhcpd_edit ${proto} "$@" ;; start) dhcpd_start ${proto} @@ -787,10 +787,10 @@ cli_dhcpd() { dhcpd_reload ${proto} ;; subnet) - cli_dhcpd_subnet ${proto} $@ + cli_dhcpd_subnet ${proto} "$@" ;; show|"") - cli_dhcpd_show ${proto} $@ + cli_dhcpd_show ${proto} "$@" ;; *) error "Unrecognized action: ${action}" @@ -879,7 +879,7 @@ cli_dhcpd_subnet() { local proto=${1} shift - if cli_help_requested $@; then + if cli_help_requested "$@"; then cli_show_man network-dhcp-subnet exit ${EXIT_OK} fi @@ -889,10 +889,10 @@ cli_dhcpd_subnet() { case "${action}" in new) - dhcpd_subnet_new ${proto} $@ + dhcpd_subnet_new ${proto} "$@" ;; remove) - dhcpd_subnet_remove ${proto} $@ + dhcpd_subnet_remove ${proto} "$@" ;; *:*/*|*.*.*.*/*) local subnet=${action} @@ -908,7 +908,7 @@ cli_dhcpd_subnet() { case "${action}" in edit) - dhcpd_subnet_edit ${proto} ${subnet} $@ + dhcpd_subnet_edit ${proto} ${subnet} "$@" local ret=$? if [ ${ret} -eq ${EXIT_OK} ]; then @@ -917,15 +917,15 @@ cli_dhcpd_subnet() { exit ${ret} ;; range) - cli_dhcpd_subnet_range ${proto} ${subnet} $@ + cli_dhcpd_subnet_range ${proto} ${subnet} "$@" exit $? ;; show) - cli_dhcpd_subnet_show ${proto} ${subnet} $@ + cli_dhcpd_subnet_show ${proto} ${subnet} "$@" exit $? ;; options) - cli_dhcpd_subnet_options ${proto} ${subnet} $@ + cli_dhcpd_subnet_options ${proto} ${subnet} "$@" exit $? ;; *) @@ -962,10 +962,10 @@ cli_dhcpd_subnet_range() { case "${action}" in new) - dhcpd_subnet_range_new ${proto} ${subnet} $@ || exit ${EXIT_ERROR} + dhcpd_subnet_range_new ${proto} ${subnet} "$@" || exit ${EXIT_ERROR} ;; remove) - dhcpd_subnet_range_remove ${proto} ${subnet} $@ || exit ${EXIT_ERROR} + dhcpd_subnet_range_remove ${proto} ${subnet} "$@" || exit ${EXIT_ERROR} ;; *) error "Unrecognized action: ${action}" @@ -1044,7 +1044,7 @@ cli_dhcpd_subnet_options() { case "${1}" in *=*) key=$(cli_get_key ${1}) - val=$(cli_get_val ${1}) + val=$(cli_get_val "${1}") dhcpd_subnet_option_set ${proto} ${subnet} ${key} ${val} esac @@ -1053,12 +1053,12 @@ cli_dhcpd_subnet_options() { } cli_start() { - if cli_help_requested $@; then + if cli_help_requested "$@"; then cli_show_man network exit ${EXIT_OK} fi - local zones=$(zones_get $@) + local zones=$(zones_get "$@") local zone for zone in ${zones}; do @@ -1069,12 +1069,12 @@ cli_start() { } cli_stop() { - if cli_help_requested $@; then + if cli_help_requested "$@"; then cli_show_man network exit ${EXIT_OK} fi - local zones=$(zones_get $@) + local zones=$(zones_get "$@") local zone for zone in ${zones}; do @@ -1085,21 +1085,21 @@ cli_stop() { } cli_restart() { - if cli_help_requested $@; then + if cli_help_requested "$@"; then cli_show_man network exit ${EXIT_OK} fi - cli_stop $@ + cli_stop "$@" # Give the system some time to calm down sleep ${TIMEOUT_RESTART} - cli_start $@ + cli_start "$@" } cli_status() { - if cli_help_requested $@; then + if cli_help_requested "$@"; then cli_show_man network exit ${EXIT_OK} fi @@ -1110,7 +1110,7 @@ cli_status() { local log_disable_stdout=${LOG_DISABLE_STDOUT} LOG_DISABLE_STDOUT="true" - local zones=$(zones_get $@) + local zones=$(zones_get "$@") local zone for zone in ${zones}; do @@ -1122,7 +1122,7 @@ cli_status() { } cli_reset() { - if cli_help_requested $@; then + if cli_help_requested "$@"; then cli_show_man network exit ${EXIT_OK} fi @@ -1219,7 +1219,7 @@ cli_help() { } cli_dns_server() { - if cli_help_requested $@; then + if cli_help_requested "$@"; then cli_show_man network-dns-server exit ${EXIT_OK} fi @@ -1282,22 +1282,22 @@ cli_raw() { db_dump ;; ipsec-connection-exists) - ipsec_connection_exists $@ + ipsec_connection_exists "$@" ;; list-devices) device_list ;; list-dhcpd-ranges-of-subnet) - dhcpd_subnet_range_list $@ + dhcpd_subnet_range_list "$@" ;; list-dhcpd-settings) - dhcpd_global_settings_list $@ + dhcpd_global_settings_list "$@" ;; list-dhcpd-subnets) - dhcpd_subnet_list $@ + dhcpd_subnet_list "$@" ;; list-dhcpd-subnet-options) - dhcpd_subnet_options_list $@ + dhcpd_subnet_options_list "$@" ;; list-dns-servers) dns_server_list @@ -1306,7 +1306,7 @@ cli_raw() { port_list_free ;; list-hooks) - hook_list $@ + hook_list "$@" ;; list-ipsec-connections) ipsec_list_connections @@ -1315,7 +1315,7 @@ cli_raw() { port_list ;; list-ports-of-zone) - zone_get_ports $@ + zone_get_ports "$@" ;; list-vpn-security-policies-all) vpn_security_policies_list_all @@ -1330,22 +1330,22 @@ cli_raw() { zones_get_next_free ;; list-zone-config-ids) - zone_config_list_ids $@ + zone_config_list_ids "$@" ;; list-zone-config-hids) - zone_config_list_hids $@ + zone_config_list_hids "$@" ;; vpn-security-policy-exists) - vpn_security_policy_exists $@ + vpn_security_policy_exists "$@" ;; zone-name-is-valid) - zone_name_is_valid $@ + zone_name_is_valid "$@" ;; zone-config-id-is-valid) - zone_config_id_is_valid $@ + zone_config_id_is_valid "$@" ;; zone-config-hid-is-valid) - zone_config_hid_is_valid $@ + zone_config_hid_is_valid "$@" ;; *) error "No such command: ${cmd}" @@ -1363,25 +1363,25 @@ case "${action}" in ;; settings|hostname|port|device|zone|start|stop|restart|status|reset|route|vpn) - cli_${action} $@ + cli_${action} "$@" ;; # DHCP server configuration (automatically detects which protocol to use). dhcpv6|dhcpv4) - cli_dhcpd ${action/dhcp/ip} $@ + cli_dhcpd ${action/dhcp/ip} "$@" ;; # DNS server configuration. dns-server) - cli_dns_server $@ + cli_dns_server "$@" ;; ""|help|--help|-h) - cli_help $@ + cli_help "$@" ;; raw) - cli_raw $@ + cli_raw "$@" ;; *) -- 2.39.2