On 09/07/15 13:03, Tomasz Bursztyka wrote:
> Hi Slava,
>
>> It only does the job for the networks that have been connected:
>>
>>      if (network->connected)
>>          set_disconnected(network);
>
> Why not adding there a test on
> network->associating/network->connecting as well?
>


Because __connman_service_remove_from_network() is invoked from two
places. Besides, most of what set_disconnected() is doing is irrelevant
in case if the network hasn't been connected.

Regards,
-Slava


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


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

Reply via email to