Hi,

On Thu, 2013-10-10 at 16:58 +0200, Daniel Wagner wrote:
> From: Daniel Wagner <[email protected]>
> 
> ---
>  src/service.c | 13 +++++++++++++
>  1 file changed, 13 insertions(+)
> 
> diff --git a/src/service.c b/src/service.c
> index dcd5571..3c04a59 100644
> --- a/src/service.c
> +++ b/src/service.c
> @@ -3455,6 +3455,19 @@ static bool is_ignore(struct connman_service *service)
>       if (!service->autoconnect)
>               return true;
>  
> +     if (connman_setting_get_bool("SessionAutoConnectMode")) {
> +             GSList *list;
> +
> +             for (list = service->sessions; list; list = list->next) {
> +                     struct connman_session *session = list->data;
> +
> +                     if (__connman_session_wants_connection(session))
> +                             return false;
> +             }
> +
> +             return true;
> +     }
> +
>       if (service->roaming)
>               return true;
>  

This function should probably be placed last or such. If there is no
session auto connect mode set, is_ignore() returns true even though the
rest of the tests could indicate false.


        Patrik
_______________________________________________
connman mailing list
[email protected]
https://lists.connman.net/mailman/listinfo/connman

Reply via email to