* ben <b...@gowasabi.net> [30.06.2015 06:48]:
> Signed-off-by: ben <b...@gowasabi.net>
> ---
>  package/network/services/hostapd/files/netifd.sh | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/package/network/services/hostapd/files/netifd.sh 
> b/package/network/services/hostapd/files/netifd.sh
> index 23d2e7e..f148544 100644
> --- a/package/network/services/hostapd/files/netifd.sh
> +++ b/package/network/services/hostapd/files/netifd.sh
> @@ -632,7 +632,7 @@ wpa_supplicant_add_network() {
>                       ;;
>               esac
>       }
> -     local beacon_int brates mrate
> +     local brates mrate
>       [ -n "$bssid" ] && append network_data "bssid=$bssid" "$N$T"
>       [ -n "$beacon_int" ] && append network_data "beacon_int=$beacon_int" 
> "$N$T"

this is a bug only for busybox 1.22.1
http://lists.busybox.net/pipermail/busybox/2015-April/082780.html

beside this:
we should take more care in the shellscripts, so that local vars
are 'lowercase' and global vars are 'UPPERCASE'. thus it should read e.g.:

hostapd_prepare_device_config()
{
  ...
  local country country_ie beacon_int doth require_mode
  json_get_vars country country_ie beacon_int doth require_mode
  ...
}

are doesnt this work and 'json_get_vars' is always global?

bye, bastian
_______________________________________________
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel

Reply via email to