mirror of
https://github.com/AuxXxilium/linux_dsm_epyc7002.git
synced 2024-12-03 01:26:41 +07:00
iwlwifi: Send broadcast probe request only when asked to
When running directed active scans we currently end up sending both the SSID probe requests and an additional broadcast one. This is due to the fact that we always leave the probe request template SSID IE length to 0. Instead we should set it to the first SSID to scan, and fill the direct_scan array with the remaining SSIDs to scan for. This way we only send what we've been asked to: a broadcast probe request when no directed scan is requested, and directed probe requests otherwise. Signed-off-by: Samuel Ortiz <sameo@linux.intel.com> Signed-off-by: Reinette Chatre <reinette.chatre@intel.com> Signed-off-by: John W. Linville <linville@tuxdriver.com>
This commit is contained in:
parent
ed56a3f15a
commit
21b2d8bd2f
@ -2631,7 +2631,7 @@ static int iwl_mac_setup_register(struct iwl_priv *priv)
|
||||
*/
|
||||
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 + 1;
|
||||
/* we create the 802.11 header and a zero-length SSID element */
|
||||
hw->wiphy->max_scan_ie_len = IWL_MAX_PROBE_REQUEST - 24 - 2;
|
||||
|
||||
|
@ -651,9 +651,20 @@ u16 iwl_fill_probe_req(struct iwl_priv *priv, struct ieee80211_mgmt *frame,
|
||||
if (left < 0)
|
||||
return 0;
|
||||
*pos++ = WLAN_EID_SSID;
|
||||
*pos++ = 0;
|
||||
if (!priv->is_internal_short_scan &&
|
||||
priv->scan_request->n_ssids) {
|
||||
struct cfg80211_ssid *ssid =
|
||||
priv->scan_request->ssids;
|
||||
|
||||
len += 2;
|
||||
/* Broadcast if ssid_len is 0 */
|
||||
*pos++ = ssid->ssid_len;
|
||||
memcpy(pos, ssid->ssid, ssid->ssid_len);
|
||||
pos += ssid->ssid_len;
|
||||
len += 2 + ssid->ssid_len;
|
||||
} else {
|
||||
*pos++ = 0;
|
||||
len += 2;
|
||||
}
|
||||
|
||||
if (WARN_ON(left < ie_len))
|
||||
return len;
|
||||
@ -782,20 +793,26 @@ static void iwl_bg_request_scan(struct work_struct *data)
|
||||
if (priv->is_internal_short_scan) {
|
||||
IWL_DEBUG_SCAN(priv, "Start internal passive scan.\n");
|
||||
} else if (priv->scan_request->n_ssids) {
|
||||
int i, p = 0;
|
||||
IWL_DEBUG_SCAN(priv, "Kicking off active scan\n");
|
||||
for (i = 0; i < priv->scan_request->n_ssids; i++) {
|
||||
/* always does wildcard anyway */
|
||||
if (!priv->scan_request->ssids[i].ssid_len)
|
||||
continue;
|
||||
scan->direct_scan[p].id = WLAN_EID_SSID;
|
||||
scan->direct_scan[p].len =
|
||||
priv->scan_request->ssids[i].ssid_len;
|
||||
memcpy(scan->direct_scan[p].ssid,
|
||||
priv->scan_request->ssids[i].ssid,
|
||||
priv->scan_request->ssids[i].ssid_len);
|
||||
n_probes++;
|
||||
p++;
|
||||
/*
|
||||
* The first SSID to scan is stuffed into the probe request
|
||||
* template and the remaining ones are handled through the
|
||||
* direct_scan array.
|
||||
*/
|
||||
if (priv->scan_request->n_ssids > 1) {
|
||||
int i, p = 0;
|
||||
for (i = 1; i < priv->scan_request->n_ssids; i++) {
|
||||
if (!priv->scan_request->ssids[i].ssid_len)
|
||||
continue;
|
||||
scan->direct_scan[p].id = WLAN_EID_SSID;
|
||||
scan->direct_scan[p].len =
|
||||
priv->scan_request->ssids[i].ssid_len;
|
||||
memcpy(scan->direct_scan[p].ssid,
|
||||
priv->scan_request->ssids[i].ssid,
|
||||
priv->scan_request->ssids[i].ssid_len);
|
||||
n_probes++;
|
||||
p++;
|
||||
}
|
||||
}
|
||||
is_active = true;
|
||||
} else
|
||||
|
Loading…
Reference in New Issue
Block a user