Comment...

On 10/22/10 7:28 PM, Mark Mentovai wrote:
Always set auth_algs in hostapd.conf. For WEP, auth_algs is configurable
using a new UCI parameter, auth_algs, which may contain "open",
"shared", or both. For other encryption types, auth_algs is always set
to 1 (open).

https://dev.openwrt.org/ticket/8120

Signed-off-by: Mark Mentovai<m...@moxienet.com>

---
Index: package/hostapd/files/hostapd.sh
===================================================================
--- package/hostapd/files/hostapd.sh    (revision 23587)
+++ package/hostapd/files/hostapd.sh    (working copy)
@@ -77,7 +77,6 @@
                        append "$var" "eapol_key_index_workaround=1" "$N"
                        append "$var" "radius_acct_interim_interval=300" "$N"
                        append "$var" "ieee8021x=1" "$N"
-                       append "$var" "auth_algs=1" "$N"
                        append "$var" "wpa_key_mgmt=WPA-EAP" "$N"
                        append "$var" "wpa_group_rekey=300" "$N"
                        append "$var" "wpa_gmk_rekey=640" "$N"
@@ -110,6 +109,34 @@
                ;;
        esac
        append "$var" "wpa=$wpa" "$N"
+
+       case "$enc" in
+               *wep*)
+                       config_get auth_algs "$vif" auth_algs
+                       auth_alg_open=
+                       auth_alg_shared=
+                       for auth_alg in ${auth_algs:-open}; do
+                               case "$auth_alg" in
+                                       open)
+                                               auth_alg_open=1
+                                       ;;
+                                       shared)
+                                               auth_alg_shared=1
+                                       ;;
+                               esac
+                       done
+                       auth_alg_val=0
+                       [ -n "$auth_alg_open" ]&&  \
+                               auth_alg_val=1
+                       [ -n "$auth_alg_shared" ]&&  \
+                               auth_alg_val=$(expr "$auth_alg_val" + 2)

Just curious: why not use:

let -i auth_alg_val+=2

here instead?


+               ;;
+               *)
+                       auth_alg_val=1
+               ;;
+       esac
+       append "$var" "auth_algs=$auth_alg_val" "$N"
+
        [ -n "$crypto" ]&&  append "$var" "wpa_pairwise=$crypto" "$N"
        [ -n "$wpa_group_rekey" ]&&  append "$var" "wpa_group_rekey=$wpa_group_rekey" 
"$N"



_______________________________________________
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel

Reply via email to