[OpenWrt-Devel] [PATCH v2] wpa_supplicant: fix generating phase2 config line for WPA-EAP

[Felix Fietkau]

On 2015-12-29 04:37, Daniel Golle wrote:
> Signed-off-by: Daniel Golle <daniel at makrotopia.org>
> ---
> v2: missed the default definition two lines above, so no need to
> use shell-expansion for that then.
> 
>  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 5541e4d..75d340a 100644
> --- a/package/network/services/hostapd/files/netifd.sh
> +++ b/package/network/services/hostapd/files/netifd.sh
> @@ -630,7 +630,7 @@ wpa_supplicant_add_network() {
>  				peap|ttls)
>  					json_get_vars auth password
>  					set_default auth MSCHAPV2
> -					append network_data "phase2=\"$auth\"" "$N$T"
> +					append network_data "phase2=\"auth=$auth\"" "$N$T"
This might break existing configurations that already include the auth=
part, so when reworking this part you should detect and fix this.
Also, for EAP-TLS, phase2 needs to be autheap=TLS, which is not
supported with this change.

- Felix
_______________________________________________
openwrt-devel mailing list
openwrt-devel at lists.openwrt.org
https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel