iwlwifi: do not spuriously call ieee80211_scan_completed
authorJohannes Berg <johannes.berg@intel.com>
Wed, 18 Aug 2010 16:35:22 +0000 (09:35 -0700)
committerJohn W. Linville <linville@tuxdriver.com>
Tue, 24 Aug 2010 20:32:05 +0000 (16:32 -0400)
When a scan is aborted because the corresponding
virtual interface is removed, we may still later
attempt to tell mac80211 that the scan completed.
This is obviously wrong, since we already told it
that it was aborted, so don't do that.

Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Signed-off-by: Wey-Yi Guy <wey-yi.w.guy@intel.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
drivers/net/wireless/iwlwifi/iwl-scan.c

index ac510840519c446ea001927f13f3615ebb1b7b8d..8d7fa59364feaa455ace2c23348e6cecb87be75f 100644 (file)
@@ -508,6 +508,7 @@ static void iwl_bg_scan_completed(struct work_struct *work)
        struct iwl_priv *priv =
            container_of(work, struct iwl_priv, scan_completed);
        bool internal = false;
+       bool scan_completed = false;
 
        IWL_DEBUG_SCAN(priv, "SCAN complete scan\n");
 
@@ -518,7 +519,8 @@ static void iwl_bg_scan_completed(struct work_struct *work)
                priv->is_internal_short_scan = false;
                IWL_DEBUG_SCAN(priv, "internal short scan completed\n");
                internal = true;
-       } else {
+       } else if (priv->scan_request) {
+               scan_completed = true;
                priv->scan_request = NULL;
                priv->scan_vif = NULL;
        }
@@ -549,7 +551,7 @@ static void iwl_bg_scan_completed(struct work_struct *work)
         * into driver again into functions that will attempt to take
         * mutex.
         */
-       if (!internal)
+       if (scan_completed)
                ieee80211_scan_completed(priv->hw, false);
 }