On 12 June 2012 16:57, Kalle Valo <kvalo@xxxxxxxxxx> wrote: > Eyal Shapira <eyal@xxxxxxxxxx> writes: > >> Some Netgear APs like WNAP210 have a quirk behavior when >> configured for WEP Shared. They send an auth response with algo >> SHARED and status code NOT_SUPPORTED_AUTH_ALG (13) when being >> sent a request made with auth algo OPEN. Enable such response to >> propagate to userspace instead of discarding it so wpa_s can >> reattempt to auth with SHARED. >> >> Reported-by: Noam Shaked <noams@xxxxxx> >> Signed-off-by: Eyal Shapira <eyal@xxxxxxxxxx> > > [...] > >> - if (auth_alg != ifmgd->auth_data->algorithm || >> + if ((auth_alg != ifmgd->auth_data->algorithm && >> + status_code != WLAN_STATUS_NOT_SUPPORTED_AUTH_ALG) || > > I think a small comment in the code would be nice, like > "WLAN_STATUS_NOT_SUPPORTED_AUTH_ALG is a workaround for Netgear > WNAPxxx". > Sure. I'll add that in v2. > -- > Kalle Valo -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html