Only try to connect to available networks
[connman] / plugins / wifi.c
index f175494..6dafad4 100644 (file)
 #include "inet.h"
 #include "supplicant.h"
 
+#define CLEANUP_TIMEOUT   8    /* in seconds */
+#define INACTIVE_TIMEOUT  12   /* in seconds */
+
 struct wifi_data {
-       GStaticMutex mutex;
-       GSList *list;
+       GSList *current;
+       GSList *pending;
+       guint cleanup_timer;
+       guint inactive_timer;
        gchar *identifier;
+       gboolean connected;
 };
 
 static int network_probe(struct connman_element *element)
@@ -61,35 +67,44 @@ static void network_remove(struct connman_element *element)
 
 static int network_enable(struct connman_element *element)
 {
-       char *identifier, *passphrase = NULL;
+       struct connman_element *device = element->parent;
+       char *name, *security = NULL, *passphrase = NULL;
        unsigned char *ssid;
        int ssid_len;
 
        DBG("element %p name %s", element, element->name);
 
        if (connman_element_get_static_property(element,
-                                       "WiFi.Name", &identifier) == FALSE)
+                                               "Name", &name) == FALSE)
                return -EIO;
 
        if (connman_element_get_static_array_property(element,
                                "WiFi.SSID", &ssid, &ssid_len) == FALSE)
                return -EIO;
 
-       if (element->parent != NULL) {
-               struct wifi_data *data = connman_element_get_data(element->parent);
+       if (device != NULL) {
+               struct wifi_data *data = connman_element_get_data(device);
 
                if (data != NULL) {
+                       if (data->connected == TRUE)
+                               return -EBUSY;
+
                        g_free(data->identifier);
-                       data->identifier = g_strdup(identifier);
+                       data->identifier = g_strdup(name);
                }
        }
 
        connman_element_get_value(element,
+                       CONNMAN_PROPERTY_ID_WIFI_SECURITY, &security);
+
+       connman_element_get_value(element,
                        CONNMAN_PROPERTY_ID_WIFI_PASSPHRASE, &passphrase);
 
-       DBG("identifier %s passhprase %s", identifier, passphrase);
+       DBG("name %s security %s passhprase %s",
+                                       name, security, passphrase);
 
-       if (__supplicant_connect(element, ssid, ssid_len, passphrase) < 0)
+       if (__supplicant_connect(element, ssid, ssid_len,
+                                               security, passphrase) < 0)
                connman_error("Failed to initiate connect");
 
        return 0;
@@ -116,53 +131,173 @@ static struct connman_driver network_driver = {
        .disable        = network_disable,
 };
 
-static struct connman_element *find_element(struct wifi_data *data,
-                                               const char *identifier)
+static struct connman_element *find_current_element(struct wifi_data *data,
+                                                       const char *identifier)
+{
+       GSList *list;
+
+       for (list = data->current; list; list = list->next) {
+               struct connman_element *element = list->data;
+
+               if (connman_element_match_static_property(element,
+                                       "Name", &identifier) == TRUE)
+                       return element;
+       }
+
+       return NULL;
+}
+
+static struct connman_element *find_pending_element(struct wifi_data *data,
+                                                       const char *identifier)
 {
        GSList *list;
 
-       for (list = data->list; list; list = list->next) {
+       for (list = data->pending; list; list = list->next) {
                struct connman_element *element = list->data;
 
                if (connman_element_match_static_property(element,
-                                       "WiFi.Name", &identifier) == TRUE)
+                                       "Name", &identifier) == TRUE)
                        return element;
        }
 
        return NULL;
 }
 
-static void state_change(struct connman_element *parent,
+static gboolean inactive_scan(gpointer user_data)
+{
+       struct connman_element *device = user_data;
+       struct wifi_data *data = connman_element_get_data(device);
+
+       DBG("");
+
+       if (data->cleanup_timer > 0) {
+               g_source_remove(data->cleanup_timer);
+               data->cleanup_timer = 0;
+       }
+
+       __supplicant_scan(device);
+
+       data->inactive_timer = 0;
+
+       return FALSE;
+}
+
+static void connect_known_networks(struct connman_element *device)
+{
+       struct wifi_data *data = connman_element_get_data(device);
+       GSList *list;
+
+       DBG("");
+
+       if (data->inactive_timer > 0) {
+               g_source_remove(data->inactive_timer);
+               data->inactive_timer = 0;
+       }
+
+       for (list = data->current; list; list = list->next) {
+               struct connman_element *element = list->data;
+
+               if (element->policy == CONNMAN_ELEMENT_POLICY_AUTO &&
+                                               element->remember == TRUE &&
+                                               element->available == TRUE) {
+                       if (network_enable(element) == 0)
+                               return;
+               }
+       }
+
+       data->inactive_timer = g_timeout_add_seconds(INACTIVE_TIMEOUT,
+                                                       inactive_scan, device);
+}
+
+static void state_change(struct connman_element *device,
                                                enum supplicant_state state)
 {
-       struct wifi_data *data = connman_element_get_data(parent);
+       struct wifi_data *data = connman_element_get_data(device);
        struct connman_element *element;
 
        DBG("state %d", state);
 
-       if (data->identifier == NULL)
+       if (data == NULL)
                return;
 
-       element = find_element(data, data->identifier);
+       if (data->identifier == NULL)
+               goto reconnect;
+
+       element = find_current_element(data, data->identifier);
        if (element == NULL)
-               return;
+               goto reconnect;
 
        if (state == STATE_COMPLETED) {
                struct connman_element *dhcp;
 
+               data->connected = TRUE;
+               connman_element_set_enabled(element, TRUE);
+
                dhcp = connman_element_create(NULL);
 
                dhcp->type = CONNMAN_ELEMENT_TYPE_DHCP;
                dhcp->index = element->index;
 
                connman_element_register(dhcp, element);
+       } else if (state == STATE_INACTIVE || state == STATE_DISCONNECTED) {
+               data->connected = FALSE;
+               connman_element_set_enabled(element, FALSE);
+
+               connman_element_unregister_children(element);
+       }
+
+reconnect:
+       if (state == STATE_INACTIVE || state == STATE_DISCONNECTED) {
+               data->connected = FALSE;
+               connect_known_networks(device);
+       }
+}
+
+static gboolean cleanup_pending(gpointer user_data)
+{
+       struct wifi_data *data = user_data;
+       GSList *list;
+
+       DBG("");
+
+       for (list = data->pending; list; list = list->next) {
+               struct connman_element *element = list->data;
+
+               DBG("element %p name %s", element, element->name);
+
+               connman_element_unregister(element);
+               connman_element_unref(element);
        }
+
+       g_slist_free(data->pending);
+       data->pending = NULL;
+
+       data->cleanup_timer = 0;
+
+       return FALSE;
+}
+
+static void clear_results(struct connman_element *device)
+{
+       struct wifi_data *data = connman_element_get_data(device);
+
+       DBG("pending %d", g_slist_length(data->pending));
+       DBG("current %d", g_slist_length(data->current));
+
+       data->pending = data->current;
+       data->current = NULL;
+
+       if (data->cleanup_timer > 0)
+               return;
+
+       data->cleanup_timer = g_timeout_add_seconds(CLEANUP_TIMEOUT,
+                                                       cleanup_pending, data);
 }
 
-static void scan_result(struct connman_element *parent,
+static void scan_result(struct connman_element *device,
                                        struct supplicant_network *network)
 {
-       struct wifi_data *data = connman_element_get_data(parent);
+       struct wifi_data *data = connman_element_get_data(device);
        struct connman_element *element;
        gchar *temp;
        int i;
@@ -181,42 +316,67 @@ static void scan_result(struct connman_element *parent,
        temp = g_strdup(network->identifier);
 
        for (i = 0; i < strlen(temp); i++) {
-               if (temp[i] == ' ' || temp[i] == '.' || temp[i] == '-')
-                       temp[i] = '_';
-               if (temp[i] == '(' || temp[i] == ')')
-                       temp[i] = '_';
-               if (g_ascii_isprint(temp[i]) == FALSE)
+               gchar tmp = g_ascii_tolower(temp[i]);
+
+               if (tmp < 'a' || tmp > 'z')
                        temp[i] = '_';
-               temp[i] = g_ascii_tolower(temp[i]);
        }
 
-       g_static_mutex_lock(&data->mutex);
-
-       element = find_element(data, network->identifier);
+       element = find_pending_element(data, network->identifier);
        if (element == NULL) {
+               guint8 strength;
+
                element = connman_element_create(temp);
 
                element->type = CONNMAN_ELEMENT_TYPE_NETWORK;
-               element->index = parent->index;
-
-               data->list = g_slist_append(data->list, element);
+               element->index = device->index;
 
-               connman_element_add_static_property(element, "WiFi.Name",
+               connman_element_add_static_property(element, "Name",
                                DBUS_TYPE_STRING, &network->identifier);
 
                connman_element_add_static_array_property(element, "WiFi.SSID",
                        DBUS_TYPE_BYTE, &network->ssid, network->ssid_len);
 
-               connman_element_register(element, parent);
-       }
+               if (element->wifi.security == NULL) {
+                       const char *security;
+
+                       if (network->has_rsn == TRUE)
+                               security = "wpa2";
+                       else if (network->has_wpa == TRUE)
+                               security = "wpa";
+                       else if (network->has_wep == TRUE)
+                               security = "wep";
+                       else
+                               security = "none";
+
+                       element->wifi.security = g_strdup(security);
+               }
+
+               strength = network->quality;
+
+               connman_element_add_static_property(element, "WiFi.Strength",
+                                               DBUS_TYPE_BYTE, &strength);
+
+               //connman_element_add_static_property(element, "WiFi.Noise",
+               //                      DBUS_TYPE_INT32, &network->noise);
+
+               DBG("%s (%s) strength %d", network->identifier,
+                                       element->wifi.security, strength);
+
+               connman_element_register(element, device);
+       } else
+               data->pending = g_slist_remove(data->pending, element);
+
+       data->current = g_slist_append(data->current, element);
 
-       g_static_mutex_unlock(&data->mutex);
+       element->available = TRUE;
 
        g_free(temp);
 }
 
 static struct supplicant_callback wifi_callback = {
        .state_change   = state_change,
+       .clear_results  = clear_results,
        .scan_result    = scan_result,
 };
 
@@ -230,7 +390,7 @@ static int wifi_probe(struct connman_element *element)
        if (data == NULL)
                return -ENOMEM;
 
-       g_static_mutex_init(&data->mutex);
+       data->connected = FALSE;
 
        connman_element_set_data(element, data);
 
@@ -280,23 +440,40 @@ static int wifi_disable(struct connman_element *element)
 
        DBG("element %p name %s", element, element->name);
 
-       __supplicant_disconnect(element);
+       if (data->cleanup_timer > 0) {
+               g_source_remove(data->cleanup_timer);
+               data->cleanup_timer = 0;
+       }
+
+       if (data->inactive_timer > 0) {
+               g_source_remove(data->inactive_timer);
+               data->inactive_timer = 0;
+       }
 
-       g_static_mutex_lock(&data->mutex);
+       __supplicant_disconnect(element);
 
-       for (list = data->list; list; list = list->next) {
+       for (list = data->pending; list; list = list->next) {
                struct connman_element *network = list->data;
 
                connman_element_unref(network);
        }
 
-       g_slist_free(data->list);
-       data->list = NULL;
+       g_slist_free(data->pending);
+       data->pending = NULL;
+
+       for (list = data->current; list; list = list->next) {
+               struct connman_element *network = list->data;
+
+               connman_element_unref(network);
+       }
 
-       g_static_mutex_unlock(&data->mutex);
+       g_slist_free(data->current);
+       data->current = NULL;
 
        connman_element_unregister_children(element);
 
+       __supplicant_stop(element);
+
        return 0;
 }
 
@@ -311,7 +488,6 @@ static struct connman_driver wifi_driver = {
        .disable        = wifi_disable,
 };
 
-static GStaticMutex device_mutex = G_STATIC_MUTEX_INIT;
 static GSList *device_list = NULL;
 
 static void wifi_newlink(unsigned short type, int index,
@@ -320,7 +496,7 @@ static void wifi_newlink(unsigned short type, int index,
        struct connman_element *device;
        GSList *list;
        gboolean exists = FALSE;
-       gchar *name;
+       gchar *name, *devname;
        struct iwreq iwr;
        int sk;
 
@@ -329,10 +505,11 @@ static void wifi_newlink(unsigned short type, int index,
        if (type != ARPHRD_ETHER)
                return;
 
-       name = inet_index2name(index);
+       name = inet_index2ident(index, "dev_");
+       devname = inet_index2name(index);
 
        memset(&iwr, 0, sizeof(iwr));
-       strncpy(iwr.ifr_ifrn.ifrn_name, name, IFNAMSIZ);
+       strncpy(iwr.ifr_ifrn.ifrn_name, devname, IFNAMSIZ);
 
        sk = socket(PF_INET, SOCK_DGRAM, 0);
 
@@ -344,8 +521,6 @@ static void wifi_newlink(unsigned short type, int index,
 
        close(sk);
 
-       g_static_mutex_lock(&device_mutex);
-
        for (list = device_list; list; list = list->next) {
                struct connman_element *device = list->data;
 
@@ -355,8 +530,6 @@ static void wifi_newlink(unsigned short type, int index,
                }
        }
 
-       g_static_mutex_unlock(&device_mutex);
-
        if (exists == TRUE) {
                g_free(name);
                return;
@@ -368,13 +541,10 @@ static void wifi_newlink(unsigned short type, int index,
 
        device->index = index;
        device->name = name;
-
-       g_static_mutex_lock(&device_mutex);
+       device->devname = devname;
 
        connman_element_register(device, NULL);
        device_list = g_slist_append(device_list, device);
-
-       g_static_mutex_unlock(&device_mutex);
 }
 
 static void wifi_dellink(unsigned short type, int index,
@@ -384,8 +554,6 @@ static void wifi_dellink(unsigned short type, int index,
 
        DBG("index %d", index);
 
-       g_static_mutex_lock(&device_mutex);
-
        for (list = device_list; list; list = list->next) {
                struct connman_element *device = list->data;
 
@@ -396,8 +564,6 @@ static void wifi_dellink(unsigned short type, int index,
                        break;
                }
        }
-
-       g_static_mutex_unlock(&device_mutex);
 }
 
 static struct connman_rtnl wifi_rtnl = {
@@ -426,8 +592,6 @@ static void supplicant_disconnect(DBusConnection *connection, void *user_data)
 
        connman_rtnl_unregister(&wifi_rtnl);
 
-       g_static_mutex_lock(&device_mutex);
-
        for (list = device_list; list; list = list->next) {
                struct connman_element *device = list->data;
 
@@ -438,8 +602,6 @@ static void supplicant_disconnect(DBusConnection *connection, void *user_data)
        g_slist_free(device_list);
        device_list = NULL;
 
-       g_static_mutex_unlock(&device_mutex);
-
        __supplicant_exit();
 }
 
@@ -489,5 +651,5 @@ static void wifi_exit(void)
        dbus_connection_unref(connection);
 }
 
-CONNMAN_PLUGIN_DEFINE("wifi", "WiFi interface plugin", VERSION,
+CONNMAN_PLUGIN_DEFINE(wifi, "WiFi interface plugin", VERSION,
                                                        wifi_init, wifi_exit)