From: Felix Fietkau <nbd@openwrt.org>
Date: Sun, 16 Dec 2012 13:23:37 +0000 (+0000)
Subject: wpa_supplicant.sh: always use parameters from the current section
X-Git-Tag: reboot~11942
X-Git-Url: http://git.lede-project.org./?a=commitdiff_plain;h=056d75049d44f94ec356a2bb9e9718aaadbd35d5;p=openwrt%2Fstaging%2Fblogic.git

wpa_supplicant.sh: always use parameters from the current section

Using variables from the outer scope unnecessarily complicates the code and
leads to issues.

This patch fixes the bug when having an "adhoc" wifi-iface section before a
"sta" section prevents wpa_supplicant from using the key specified in the
corresponding section as it tries to use the "adhoc" key instead (1 by
default).

Signed-off-by: Paul Fertser <fercerpav@gmail.com>

SVN-Revision: 34716
---

diff --git a/package/network/services/hostapd/files/wpa_supplicant.sh b/package/network/services/hostapd/files/wpa_supplicant.sh
index 928243e42509..127c5a70f511 100644
--- a/package/network/services/hostapd/files/wpa_supplicant.sh
+++ b/package/network/services/hostapd/files/wpa_supplicant.sh
@@ -9,13 +9,8 @@ wpa_supplicant_setup_vif() {
 	local scan_ssid="1"
 	[ -n "$4" ] && freq="frequency=$4"
 
-	# make sure we have the encryption type and the psk
-	[ -n "$enc" ] || {
-		config_get enc "$vif" encryption
-	}
-	[ -n "$key" ] || {
-		config_get key "$vif" key
-	}
+	config_get enc "$vif" encryption
+	config_get key "$vif" key
 
 	local net_cfg bridge
 	config_get bridge "$vif" bridge