Commit 4f9bfbb1 authored by David Spinadel's avatar David Spinadel Committed by John W. Linville

iwlwifi: disable default wildcard ssid scan

iwl_fill_probe_request has used to add a wildcard ssid IE to
any probe request template, now it's disabled and it will
send wildcard ssid only for full scan. Instead, the highest
priority ssid is set to the template.
Due to adding high priority SSID to the template, it reduce
IE len, but since we had only 260 bytes for IEs before changing
allocation size to be dynamic, now we should have a bit more room
for IEs.
Signed-off-by: default avatarDavid Spinadel <david.spinadel@intel.com>
Reviewed-by: default avatarEmmanuel Grumbach <emmanuel.grumbach@intel.com>
Signed-off-by: default avatarJohannes Berg <johannes.berg@intel.com>
Signed-off-by: default avatarJohn W. Linville <linville@tuxdriver.com>
parent 0ed46287
...@@ -223,8 +223,8 @@ int iwlagn_mac_setup_register(struct iwl_priv *priv, ...@@ -223,8 +223,8 @@ int iwlagn_mac_setup_register(struct iwl_priv *priv,
hw->wiphy->flags &= ~WIPHY_FLAG_PS_ON_BY_DEFAULT; hw->wiphy->flags &= ~WIPHY_FLAG_PS_ON_BY_DEFAULT;
hw->wiphy->max_scan_ssids = PROBE_OPTION_MAX; hw->wiphy->max_scan_ssids = PROBE_OPTION_MAX;
/* we create the 802.11 header and a zero-length SSID element */ /* we create the 802.11 header and a max-length SSID element */
hw->wiphy->max_scan_ie_len = capa->max_probe_length - 24 - 2; hw->wiphy->max_scan_ie_len = capa->max_probe_length - 24 - 34;
/* /*
* We don't use all queues: 4 and 9 are unused and any * We don't use all queues: 4 and 9 are unused and any
......
...@@ -617,7 +617,8 @@ static int iwl_get_channels_for_scan(struct iwl_priv *priv, ...@@ -617,7 +617,8 @@ static int iwl_get_channels_for_scan(struct iwl_priv *priv,
*/ */
static u16 iwl_fill_probe_req(struct ieee80211_mgmt *frame, const u8 *ta, static u16 iwl_fill_probe_req(struct ieee80211_mgmt *frame, const u8 *ta,
const u8 *ies, int ie_len, int left) const u8 *ies, int ie_len, const u8 *ssid,
u8 ssid_len, int left)
{ {
int len = 0; int len = 0;
u8 *pos = NULL; u8 *pos = NULL;
...@@ -639,14 +640,18 @@ static u16 iwl_fill_probe_req(struct ieee80211_mgmt *frame, const u8 *ta, ...@@ -639,14 +640,18 @@ static u16 iwl_fill_probe_req(struct ieee80211_mgmt *frame, const u8 *ta,
/* ...next IE... */ /* ...next IE... */
pos = &frame->u.probe_req.variable[0]; pos = &frame->u.probe_req.variable[0];
/* fill in our indirect SSID IE */ /* fill in our SSID IE */
left -= 2; left -= ssid_len + 2;
if (left < 0) if (left < 0)
return 0; return 0;
*pos++ = WLAN_EID_SSID; *pos++ = WLAN_EID_SSID;
*pos++ = 0; *pos++ = ssid_len;
if (ssid && ssid_len) {
memcpy(pos, ssid, ssid_len);
pos += ssid_len;
}
len += 2; len += ssid_len + 2;
if (WARN_ON(left < ie_len)) if (WARN_ON(left < ie_len))
return len; return len;
...@@ -683,6 +688,8 @@ static int iwlagn_request_scan(struct iwl_priv *priv, struct ieee80211_vif *vif) ...@@ -683,6 +688,8 @@ static int iwlagn_request_scan(struct iwl_priv *priv, struct ieee80211_vif *vif)
int scan_cmd_size = sizeof(struct iwl_scan_cmd) + int scan_cmd_size = sizeof(struct iwl_scan_cmd) +
MAX_SCAN_CHANNEL * sizeof(struct iwl_scan_channel) + MAX_SCAN_CHANNEL * sizeof(struct iwl_scan_channel) +
priv->fw->ucode_capa.max_probe_length; priv->fw->ucode_capa.max_probe_length;
const u8 *ssid = NULL;
u8 ssid_len = 0;
if (WARN_ON_ONCE(priv->scan_request && if (WARN_ON_ONCE(priv->scan_request &&
priv->scan_request->n_channels > MAX_SCAN_CHANNEL)) priv->scan_request->n_channels > MAX_SCAN_CHANNEL))
...@@ -753,10 +760,14 @@ static int iwlagn_request_scan(struct iwl_priv *priv, struct ieee80211_vif *vif) ...@@ -753,10 +760,14 @@ static int iwlagn_request_scan(struct iwl_priv *priv, struct ieee80211_vif *vif)
if (priv->scan_request->n_ssids) { if (priv->scan_request->n_ssids) {
int i, p = 0; int i, p = 0;
IWL_DEBUG_SCAN(priv, "Kicking off active scan\n"); IWL_DEBUG_SCAN(priv, "Kicking off active scan\n");
for (i = 0; i < priv->scan_request->n_ssids; i++) { /*
/* always does wildcard anyway */ * The highest priority SSID is inserted to the
if (!priv->scan_request->ssids[i].ssid_len) * probe request template.
continue; */
ssid_len = priv->scan_request->ssids[0].ssid_len;
ssid = priv->scan_request->ssids[0].ssid;
for (i = 1; i < priv->scan_request->n_ssids; i++) {
scan->direct_scan[p].id = WLAN_EID_SSID; scan->direct_scan[p].id = WLAN_EID_SSID;
scan->direct_scan[p].len = scan->direct_scan[p].len =
priv->scan_request->ssids[i].ssid_len; priv->scan_request->ssids[i].ssid_len;
...@@ -890,6 +901,7 @@ static int iwlagn_request_scan(struct iwl_priv *priv, struct ieee80211_vif *vif) ...@@ -890,6 +901,7 @@ static int iwlagn_request_scan(struct iwl_priv *priv, struct ieee80211_vif *vif)
vif->addr, vif->addr,
priv->scan_request->ie, priv->scan_request->ie,
priv->scan_request->ie_len, priv->scan_request->ie_len,
ssid, ssid_len,
scan_cmd_size - sizeof(*scan)); scan_cmd_size - sizeof(*scan));
break; break;
case IWL_SCAN_RADIO_RESET: case IWL_SCAN_RADIO_RESET:
...@@ -898,6 +910,7 @@ static int iwlagn_request_scan(struct iwl_priv *priv, struct ieee80211_vif *vif) ...@@ -898,6 +910,7 @@ static int iwlagn_request_scan(struct iwl_priv *priv, struct ieee80211_vif *vif)
cmd_len = iwl_fill_probe_req( cmd_len = iwl_fill_probe_req(
(struct ieee80211_mgmt *)scan->data, (struct ieee80211_mgmt *)scan->data,
iwl_bcast_addr, NULL, 0, iwl_bcast_addr, NULL, 0,
NULL, 0,
scan_cmd_size - sizeof(*scan)); scan_cmd_size - sizeof(*scan));
break; break;
default: default:
......
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