Hi Danny,

Le 18/10/2011 16:16, Danny Jeongseok Seo a écrit :
> ConnMan gets interface_state signals connected, associating and disconnected
> consecutively.
>
> ConnMan network state recovered, associating = true -> false, but service
> state stay CONNMAN_SERVICE_STATE_ASSOCIATION.
> User cannot make connect and even disconnect any other Wi-Fi services (in
> progress).
Ok, the symptoms sounds interesting. Actually, it would help to get the
connman and wpa_supplicant logs from that use case.

Your patch is definitely not good though, see below:
>  plugins/wifi.c |    9 +++++++--
>  1 files changed, 7 insertions(+), 2 deletions(-)
>
> diff --git a/plugins/wifi.c b/plugins/wifi.c
> index a9d7871..52fa91e 100644
> --- a/plugins/wifi.c
> +++ b/plugins/wifi.c
> @@ -935,8 +935,13 @@ static void interface_state(GSupplicantInterface
> *interface)
>  
>                 if (is_idle(wifi))
>                         break;
> -               connman_network_set_associating(network, FALSE);
> -               connman_network_set_connected(network, FALSE);
> +
> +               int err_assoc, err_conn;
If you want to insert new variables, declare them at the beginning of
the function
While compiling you should have had an error.
> +               err_assoc = connman_network_set_associating(network, FALSE);
> +               err_conn = connman_network_set_connected(network, FALSE);
> +
> +               if (err_assoc == 0 && err_conn != 0)
> +                       set_connected(network);
Hum, where does this function comes from? ^^
You cannot use the one from network.c, it's not an exported function.
Here again compilation will fail.
And anyway the logic sounds bogus here: if you get err_conn != 0 it
means that network is already set to connected so there is nothing to
achieve here.

With regards,

Tomasz
_______________________________________________
connman mailing list
[email protected]
http://lists.connman.net/listinfo/connman

Reply via email to