On 10.02.2015 16:56, Patrik Flykt wrote:

>>      if (service) {
>>              connman_service_create_ip6config(service, network->index);
>>  
>> +            network->connecting = false;
>> +
>> +            connman_network_set_associating(network, false);
>> +
>>              __connman_service_ipconfig_indicate_state(service,
>>                                      CONNMAN_SERVICE_STATE_CONFIGURATION,
>>                                      CONNMAN_IPCONFIG_TYPE_IPV6);
> 
> We're not connected yet at this point, network->connecting can go false
> starting with dhcpv6_callback().

The network->connected is set to true already on set_connected(),
meaning it's done before connecting is changed to false (which is done
on autoconf_ipv6_set()).

Therefore doing network->connecting = false here does not do anything as
it is already false. Will update the PR.

Br,
Pasi




_______________________________________________
connman mailing list
connman@connman.net
https://lists.connman.net/mailman/listinfo/connman

Reply via email to