Hi Tomasz,

On Tue, Jan 11, 2011 at 03:53:56PM +0200, Tomasz Bursztyka wrote:
> ---
>  gsupplicant/gsupplicant.h |    7 ++++++
>  gsupplicant/supplicant.c  |   52 
> +++++++++++++++++++++++++++++++++++++++++++++
>  2 files changed, 59 insertions(+), 0 deletions(-)
> 
> diff --git a/gsupplicant/gsupplicant.h b/gsupplicant/gsupplicant.h
> index 4f507e3..05f5c35 100644
> --- a/gsupplicant/gsupplicant.h
> +++ b/gsupplicant/gsupplicant.h
> @@ -97,6 +97,12 @@ typedef enum {
>       G_SUPPLICANT_STATE_COMPLETED,
>  } GSupplicantState;
>  
> +typedef enum {
> +     G_SUPPLICANT_WPS_STATE_UNKNOWN,
> +     G_SUPPLICANT_WPS_STATE_SUCCESS,
> +     G_SUPPLICANT_WPS_STATE_FAIL,
> +} GSupplicantWpsState;
> +
>  struct _GSupplicantSSID {
>       const void *ssid;
>       unsigned int ssid_len;
> @@ -158,6 +164,7 @@ GSupplicantState 
> g_supplicant_interface_get_state(GSupplicantInterface *interfac
>  const char *g_supplicant_interface_get_wps_key(GSupplicantInterface 
> *interface);
>  const void *g_supplicant_interface_get_wps_ssid(GSupplicantInterface 
> *interface,
>                                                       unsigned int *ssid_len);
> +GSupplicantWpsState 
> g_supplicant_interface_get_wps_state(GSupplicantInterface *interface);
>  
>  /* Network API */
>  struct _GSupplicantNetwork;
> diff --git a/gsupplicant/supplicant.c b/gsupplicant/supplicant.c
> index a0d2414..653246d 100644
> --- a/gsupplicant/supplicant.c
> +++ b/gsupplicant/supplicant.c
> @@ -137,6 +137,7 @@ static GHashTable *interface_table;
>  static GHashTable *bss_mapping;
>  
>  struct _GSupplicantWpsCredentials {
> +     GSupplicantWpsState state;
This is not really a WPS credential, imho. So either rename struct
_GSupplicantWpsCredentials to struct _GSupplicantWps or include the state
separately.

Cheers,
Samuel.

-- 
Intel Open Source Technology Centre
http://oss.intel.com/
_______________________________________________
connman mailing list
connman@connman.net
http://lists.connman.net/listinfo/connman

Reply via email to