Re: [PATCH v2 1/2] Clear WiFi requested_scan if suppl goes INACTIVE

[Dan Williams <dcbw redhat com>]

On Thu, 2016-06-16 at 15:07 -0400, Tony Espy wrote:
> It's possible for wpa_supplicant to transition to INACTIVE
> state with an outstanding requested_scan pending.  This can
> lead to a stall condition where scanning no longer occurs.
>
> Gbp-Pq: Name Clear-WiFi-requested_scan-if-suppl-goes-INACTIVE.patch
> ---
>  src/devices/wifi/nm-device-wifi.c | 4 ++++
>  1 file changed, 4 insertions(+)
>
> diff --git a/src/devices/wifi/nm-device-wifi.c b/src/devices/wifi/nm-
> device-wifi.c
> index bacd5e6..053bc07 100644
> --- a/src/devices/wifi/nm-device-wifi.c
> +++ b/src/devices/wifi/nm-device-wifi.c
> @@ -1903,6 +1903,10 @@ supplicant_iface_state_cb
> (NMSupplicantInterface *iface,
>               else
>                       _LOGI (LOGD_DEVICE | LOGD_WIFI, "supplicant
> interface keeps failing, giving up");
>               break;
> +     case NM_SUPPLICANT_INTERFACE_STATE_INACTIVE:
> +             priv->requested_scan = FALSE;
> +             nm_clear_g_source (&priv->pending_scan_id);
> +             request_wireless_scan (self, NULL);
>       default:
>               break;
>       }

LGTM.

Dan