On Fri, 2014-08-08 at 14:02 +0200, Richard Röjfors wrote:
> > case CONNMAN_SERVICE_STATE_UNKNOWN:
> > case CONNMAN_SERVICE_STATE_IDLE:
> > - if (service->state == CONNMAN_SERVICE_STATE_FAILURE)
> > - return -EINVAL;
> > - break;
> > case CONNMAN_SERVICE_STATE_ASSOCIATION:
> > break;
>
> This will also cause CONNMAN_SERVICE_STATE_UNKNOWN to fall through,
> was that intended?
Even though nothing sets the state to CONNMAN_SERVICE_STATE_UNKNOWN via
this function, we should not let that fall through. I'll fix on Monday.
Thanks,
Patrik
_______________________________________________
connman mailing list
[email protected]
https://lists.connman.net/mailman/listinfo/connman
