Commit 5f81ff5a authored by Jussi Kivilinna's avatar Jussi Kivilinna Committed by John W. Linville

rndis_wlan: ignore OID_802_11_ADD_KEY triggered media connect indications

Setting WPA keys with OID_802_11_ADD_KEY sometimes trigger
instant media connect indication. These indications are extranous and
should be ignored, as otherwise driver would send reassociation event to
userspace which in this case is not needed.
Signed-off-by: default avatarJussi Kivilinna <jussi.kivilinna@mbnet.fi>
Signed-off-by: default avatarJohn W. Linville <linville@tuxdriver.com>
parent 64e368bf
...@@ -470,6 +470,7 @@ struct rndis_wlan_private { ...@@ -470,6 +470,7 @@ struct rndis_wlan_private {
int radio_on; int radio_on;
int infra_mode; int infra_mode;
struct ndis_80211_ssid essid; struct ndis_80211_ssid essid;
__le32 current_command_oid;
/* encryption stuff */ /* encryption stuff */
int encr_tx_key_index; int encr_tx_key_index;
...@@ -665,7 +666,9 @@ static int rndis_query_oid(struct usbnet *dev, __le32 oid, void *data, int *len) ...@@ -665,7 +666,9 @@ static int rndis_query_oid(struct usbnet *dev, __le32 oid, void *data, int *len)
u.get->msg_len = cpu_to_le32(sizeof *u.get); u.get->msg_len = cpu_to_le32(sizeof *u.get);
u.get->oid = oid; u.get->oid = oid;
priv->current_command_oid = oid;
ret = rndis_command(dev, u.header, buflen); ret = rndis_command(dev, u.header, buflen);
priv->current_command_oid = 0;
if (ret < 0) if (ret < 0)
devdbg(dev, "rndis_query_oid(%s): rndis_command() failed, %d " devdbg(dev, "rndis_query_oid(%s): rndis_command() failed, %d "
"(%08x)", oid_to_string(oid), ret, "(%08x)", oid_to_string(oid), ret,
...@@ -725,7 +728,9 @@ static int rndis_set_oid(struct usbnet *dev, __le32 oid, void *data, int len) ...@@ -725,7 +728,9 @@ static int rndis_set_oid(struct usbnet *dev, __le32 oid, void *data, int len)
u.set->handle = cpu_to_le32(0); u.set->handle = cpu_to_le32(0);
memcpy(u.buf + sizeof(*u.set), data, len); memcpy(u.buf + sizeof(*u.set), data, len);
priv->current_command_oid = oid;
ret = rndis_command(dev, u.header, buflen); ret = rndis_command(dev, u.header, buflen);
priv->current_command_oid = 0;
if (ret < 0) if (ret < 0)
devdbg(dev, "rndis_set_oid(%s): rndis_command() failed, %d " devdbg(dev, "rndis_set_oid(%s): rndis_command() failed, %d "
"(%08x)", oid_to_string(oid), ret, "(%08x)", oid_to_string(oid), ret,
...@@ -760,6 +765,7 @@ static int rndis_reset(struct usbnet *usbdev) ...@@ -760,6 +765,7 @@ static int rndis_reset(struct usbnet *usbdev)
memset(reset, 0, sizeof(*reset)); memset(reset, 0, sizeof(*reset));
reset->msg_type = RNDIS_MSG_RESET; reset->msg_type = RNDIS_MSG_RESET;
reset->msg_len = cpu_to_le32(sizeof(*reset)); reset->msg_len = cpu_to_le32(sizeof(*reset));
priv->current_command_oid = 0;
ret = rndis_command(usbdev, (void *)reset, CONTROL_BUFFER_SIZE); ret = rndis_command(usbdev, (void *)reset, CONTROL_BUFFER_SIZE);
mutex_unlock(&priv->command_lock); mutex_unlock(&priv->command_lock);
...@@ -2558,6 +2564,17 @@ static void rndis_wlan_indication(struct usbnet *usbdev, void *ind, int buflen) ...@@ -2558,6 +2564,17 @@ static void rndis_wlan_indication(struct usbnet *usbdev, void *ind, int buflen)
switch (msg->status) { switch (msg->status) {
case RNDIS_STATUS_MEDIA_CONNECT: case RNDIS_STATUS_MEDIA_CONNECT:
if (priv->current_command_oid == OID_802_11_ADD_KEY) {
/* OID_802_11_ADD_KEY causes sometimes extra
* "media connect" indications which confuses driver
* and userspace to think that device is
* roaming/reassociating when it isn't.
*/
devdbg(usbdev, "ignored OID_802_11_ADD_KEY triggered "
"'media connect'");
return;
}
usbnet_pause_rx(usbdev); usbnet_pause_rx(usbdev);
devinfo(usbdev, "media connect"); devinfo(usbdev, "media connect");
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment