On Wed, Jun 08, 2011 at 02:55:01PM +0300, Kalle Valo wrote: > - /* Translate data to cfg80211 mgmt format */ > - if (ar->arWmi) ^^^^^^^^^ You removed this check on purpose? Is this only NULL when the scan is canceled? > - wmi_iterate_nodes(ar->arWmi, ar6k_cfg80211_scan_node, ar->wdev->wiphy); > + if (!ar->scan_request) > + return; > + > + if ((status == A_ECANCELED) || (status == A_EBUSY)) { > + cfg80211_scan_done(ar->scan_request, true); > + goto out; > + } > + > + /* Translate data to cfg80211 mgmt format */ > + wmi_iterate_nodes(ar->arWmi, ar6k_cfg80211_scan_node, ar->wdev->wiphy); regards, dan carpenter -- 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