Hi Patrik,

On 10/18/2013 03:50 PM, Patrik Flykt wrote:

        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.

Good catch! Thanks.

cheers,
daniel

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

Reply via email to