Hi Mohamed,
On Wed, Dec 08, 2010 at 04:37:14PM -0800, Mohamed Abbas wrote:
> In gsupplican defer any scanning if association in progress.
> ---
> gsupplicant/supplicant.c | 10 ++++++++++
> 1 files changed, 10 insertions(+), 0 deletions(-)
>
> diff --git a/gsupplicant/supplicant.c b/gsupplicant/supplicant.c
> index 861936e..1d98dec 100644
> --- a/gsupplicant/supplicant.c
> +++ b/gsupplicant/supplicant.c
> @@ -2007,6 +2007,16 @@ int g_supplicant_interface_scan(GSupplicantInterface
> *interface,
> if (interface->scanning == TRUE)
> return -EALREADY;
>
> + switch (interface->state) {
> + case G_SUPPLICANT_STATE_ASSOCIATING:
> + case G_SUPPLICANT_STATE_ASSOCIATED:
> + case G_SUPPLICANT_STATE_4WAY_HANDSHAKE:
> + case G_SUPPLICANT_STATE_GROUP_HANDSHAKE:
> + return -EBUSY;
So I suppose you also want to have G_SUPPLICANT_STATE_AUTHENTICATING here as
well.
> + default:
To be consistent with the rest of the ConnMan code, we need to enumerate all
state values here.
I fixed both issues and applied the patch.
Cheers,
Samuel.
--
Intel Open Source Technology Centre
http://oss.intel.com/
_______________________________________________
connman mailing list
[email protected]
http://lists.connman.net/listinfo/connman
