network: Fix failed to reconnect to 3G network when connect failed
[platform/upstream/connman.git] / src / network.c
index 02ee3c0..f9dfada 100644 (file)
 
 #include "connman.h"
 
-static unsigned int hidden_counter = 0;
+static GSList *network_list = NULL;
+static GSList *driver_list = NULL;
 
 struct connman_network {
-       struct connman_element element;
+       int refcount;
        enum connman_network_type type;
        connman_bool_t available;
        connman_bool_t connected;
        connman_bool_t roaming;
-       connman_bool_t hidden;
        connman_uint8_t strength;
        connman_uint16_t frequency;
        char *identifier;
        char *name;
        char *node;
        char *group;
+       char *path;
+       int index;
 
        struct connman_network_driver *driver;
        void *driver_data;
@@ -59,8 +61,10 @@ struct connman_network {
                unsigned short channel;
                char *security;
                char *passphrase;
+               char *agent_passphrase;
                char *eap;
                char *identity;
+               char *agent_identity;
                char *ca_cert_path;
                char *client_cert_path;
                char *private_key_path;
@@ -70,6 +74,11 @@ struct connman_network {
                connman_bool_t use_wps;
                char *pin_wps;
        } wifi;
+
+       struct {
+               char *nsp_name;
+               int nsp_name_len;
+       } wimax;
 };
 
 static const char *type2string(enum connman_network_type type)
@@ -102,7 +111,152 @@ connman_bool_t __connman_network_has_driver(struct connman_network *network)
        return TRUE;
 }
 
-static GSList *driver_list = NULL;
+static gboolean match_driver(struct connman_network *network,
+                                       struct connman_network_driver *driver)
+{
+       if (network->type == driver->type ||
+                       driver->type == CONNMAN_NETWORK_TYPE_UNKNOWN)
+               return TRUE;
+
+       return FALSE;
+}
+
+static int network_probe(struct connman_network *network)
+{
+       GSList *list;
+       struct connman_network_driver *driver = NULL;
+
+       DBG("network %p name %s", network, network->name);
+
+       if (network->driver != NULL)
+               return -EALREADY;
+
+       for (list = driver_list; list; list = list->next) {
+               driver = list->data;
+
+               if (match_driver(network, driver) == FALSE)
+                       continue;
+
+               DBG("driver %p name %s", driver, driver->name);
+
+               if (driver->probe(network) == 0)
+                       break;
+
+               driver = NULL;
+       }
+
+       if (driver == NULL)
+               return -ENODEV;
+
+       if (network->group == NULL)
+               return -EINVAL;
+
+       switch (network->type) {
+       case CONNMAN_NETWORK_TYPE_UNKNOWN:
+       case CONNMAN_NETWORK_TYPE_VENDOR:
+               return 0;
+       case CONNMAN_NETWORK_TYPE_ETHERNET:
+       case CONNMAN_NETWORK_TYPE_BLUETOOTH_PAN:
+       case CONNMAN_NETWORK_TYPE_BLUETOOTH_DUN:
+       case CONNMAN_NETWORK_TYPE_CELLULAR:
+       case CONNMAN_NETWORK_TYPE_WIFI:
+       case CONNMAN_NETWORK_TYPE_WIMAX:
+               network->driver = driver;
+               if (__connman_service_create_from_network(network) == NULL) {
+                       network->driver = NULL;
+                       return -EINVAL;
+               }
+       }
+
+       return 0;
+}
+
+static void network_remove(struct connman_network *network)
+{
+       DBG("network %p name %s", network, network->name);
+
+       if (network->driver == NULL)
+               return;
+
+       connman_network_set_connected(network, FALSE);
+
+       switch (network->type) {
+       case CONNMAN_NETWORK_TYPE_UNKNOWN:
+       case CONNMAN_NETWORK_TYPE_VENDOR:
+               break;
+       case CONNMAN_NETWORK_TYPE_ETHERNET:
+       case CONNMAN_NETWORK_TYPE_BLUETOOTH_PAN:
+       case CONNMAN_NETWORK_TYPE_BLUETOOTH_DUN:
+       case CONNMAN_NETWORK_TYPE_CELLULAR:
+       case CONNMAN_NETWORK_TYPE_WIFI:
+       case CONNMAN_NETWORK_TYPE_WIMAX:
+               if (network->group != NULL) {
+                       __connman_service_remove_from_network(network);
+
+                       g_free(network->group);
+                       network->group = NULL;
+               }
+               break;
+       }
+
+       if (network->driver->remove)
+               network->driver->remove(network);
+
+       network->driver = NULL;
+}
+
+static void network_change(struct connman_network *network)
+{
+       DBG("network %p name %s", network, network->name);
+
+       if (network->connected == FALSE)
+               return;
+
+       connman_device_set_disconnected(network->device, TRUE);
+
+       if (network->driver && network->driver->disconnect) {
+               network->driver->disconnect(network);
+               return;
+       }
+
+       network->connected = FALSE;
+}
+
+static void probe_driver(struct connman_network_driver *driver)
+{
+       GSList *list;
+
+       DBG("driver %p name %s", driver, driver->name);
+
+       for (list = network_list; list != NULL; list = list->next) {
+               struct connman_network *network = list->data;
+
+               if (network->driver != NULL)
+                       continue;
+
+               if (driver->type != network->type)
+                       continue;
+
+               if (driver->probe(network) < 0)
+                       continue;
+
+               network->driver = driver;
+       }
+}
+
+static void remove_driver(struct connman_network_driver *driver)
+{
+       GSList *list;
+
+       DBG("driver %p name %s", driver, driver->name);
+
+       for (list = network_list; list != NULL; list = list->next) {
+               struct connman_network *network = list->data;
+
+               if (network->driver == driver)
+                       network_remove(network);
+       }
+}
 
 static gint compare_priority(gconstpointer a, gconstpointer b)
 {
@@ -137,6 +291,8 @@ int connman_network_driver_register(struct connman_network_driver *driver)
        driver_list = g_slist_insert_sorted(driver_list, driver,
                                                        compare_priority);
 
+       probe_driver(driver);
+
        return 0;
 }
 
@@ -151,20 +307,22 @@ void connman_network_driver_unregister(struct connman_network_driver *driver)
        DBG("driver %p name %s", driver, driver->name);
 
        driver_list = g_slist_remove(driver_list, driver);
+
+       remove_driver(driver);
 }
 
-static void network_destruct(struct connman_element *element)
+static void network_destruct(struct connman_network *network)
 {
-       struct connman_network *network = element->network;
-
-       DBG("element %p name %s", element, element->name);
+       DBG("network %p name %s", network, network->name);
 
        g_free(network->wifi.ssid);
        g_free(network->wifi.mode);
        g_free(network->wifi.security);
        g_free(network->wifi.passphrase);
+       g_free(network->wifi.agent_passphrase);
        g_free(network->wifi.eap);
        g_free(network->wifi.identity);
+       g_free(network->wifi.agent_identity);
        g_free(network->wifi.ca_cert_path);
        g_free(network->wifi.client_cert_path);
        g_free(network->wifi.private_key_path);
@@ -172,12 +330,15 @@ static void network_destruct(struct connman_element *element)
        g_free(network->wifi.phase2_auth);
        g_free(network->wifi.pin_wps);
 
+       g_free(network->path);
        g_free(network->group);
        g_free(network->node);
        g_free(network->name);
        g_free(network->identifier);
 
        network->device = NULL;
+
+       g_free(network);
 }
 
 /**
@@ -192,9 +353,7 @@ struct connman_network *connman_network_create(const char *identifier,
                                                enum connman_network_type type)
 {
        struct connman_network *network;
-       connman_uint8_t strength = 0;
-       const char *str;
-       char *temp;
+       char *ident;
 
        DBG("identifier %s type %d", identifier, type);
 
@@ -204,33 +363,19 @@ struct connman_network *connman_network_create(const char *identifier,
 
        DBG("network %p", network);
 
-       __connman_element_initialize(&network->element);
+       network->refcount = 1;
 
-       if (identifier == NULL) {
-               temp = g_strdup_printf("hidden_%d", hidden_counter++);
-               network->hidden = TRUE;
-       } else
-               temp = g_strdup(identifier);
+       ident = g_strdup(identifier);
 
-       if (temp == NULL) {
+       if (ident == NULL) {
                g_free(network);
                return NULL;
        }
 
-       network->element.name = temp;
-       network->element.type = CONNMAN_ELEMENT_TYPE_NETWORK;
-
-       network->element.network = network;
-       network->element.destruct = network_destruct;
-
-       str = type2string(type);
-       if (str != NULL)
-               connman_element_set_string(&network->element, "Type", str);
-
-       connman_element_set_uint8(&network->element, "Strength", strength);
-
        network->type       = type;
-       network->identifier = g_strdup(temp);
+       network->identifier = ident;
+
+       network_list = g_slist_append(network_list, network);
 
        return network;
 }
@@ -243,8 +388,10 @@ struct connman_network *connman_network_create(const char *identifier,
  */
 struct connman_network *connman_network_ref(struct connman_network *network)
 {
-       if (connman_element_ref(&network->element) == NULL)
-               return NULL;
+       DBG("network %p name %s refcount %d", network, network->name,
+               network->refcount + 1);
+
+       __sync_fetch_and_add(&network->refcount, 1);
 
        return network;
 }
@@ -257,7 +404,15 @@ struct connman_network *connman_network_ref(struct connman_network *network)
  */
 void connman_network_unref(struct connman_network *network)
 {
-       connman_element_unref(&network->element);
+       DBG("network %p name %s refcount %d", network, network->name,
+               network->refcount - 1);
+
+       if (__sync_fetch_and_sub(&network->refcount, 1) != 1)
+               return;
+
+       network_list = g_slist_remove(network_list, network);
+
+       network_destruct(network);
 }
 
 const char *__connman_network_get_type(struct connman_network *network)
@@ -305,10 +460,10 @@ void connman_network_set_index(struct connman_network *network, int index)
 
        ipconfig = __connman_service_get_ip4config(service);
 
-       DBG("index %d service %p ip4config %p", network->element.index,
+       DBG("index %d service %p ip4config %p", network->index,
                service, ipconfig);
 
-       if (network->element.index < 0 && ipconfig == NULL) {
+       if (network->index < 0 && ipconfig == NULL) {
 
                ipconfig = __connman_service_get_ip4config(service);
                if (ipconfig == NULL)
@@ -330,7 +485,7 @@ void connman_network_set_index(struct connman_network *network, int index)
        }
 
 done:
-       network->element.index = index;
+       network->index = index;
 }
 
 /**
@@ -341,19 +496,7 @@ done:
  */
 int connman_network_get_index(struct connman_network *network)
 {
-       return network->element.index;
-}
-
-/**
- * connman_network_get_element:
- * @network: network structure
- *
- * Get connman_element of network
- */
-struct connman_element *connman_network_get_element(
-                               struct connman_network *network)
-{
-       return &network->element;
+       return network->index;
 }
 
 /**
@@ -394,7 +537,7 @@ void connman_network_set_group(struct connman_network *network,
        network->group = g_strdup(group);
 
        if (network->group != NULL)
-               __connman_service_create_from_network(network);
+               network_probe(network);
 }
 
 /**
@@ -471,9 +614,6 @@ int connman_network_set_available(struct connman_network *network,
  */
 connman_bool_t connman_network_get_available(struct connman_network *network)
 {
-       if (network->hidden == TRUE)
-               return TRUE;
-
        return network->available;
 }
 
@@ -498,9 +638,12 @@ int connman_network_set_associating(struct connman_network *network,
                struct connman_service *service;
 
                service = __connman_service_lookup_from_network(network);
-               __connman_service_indicate_state(service,
+               __connman_service_ipconfig_indicate_state(service,
                                        CONNMAN_SERVICE_STATE_ASSOCIATION,
                                        CONNMAN_IPCONFIG_TYPE_IPV4);
+               __connman_service_ipconfig_indicate_state(service,
+                                       CONNMAN_SERVICE_STATE_ASSOCIATION,
+                                       CONNMAN_IPCONFIG_TYPE_IPV6);
        }
 
        return 0;
@@ -510,14 +653,9 @@ static void set_associate_error(struct connman_network *network)
 {
        struct connman_service *service;
 
-       if (network->associating == FALSE)
-               return ;
-
-       network->associating = FALSE;
-
        service = __connman_service_lookup_from_network(network);
 
-       __connman_service_indicate_state(service,
+       __connman_service_ipconfig_indicate_state(service,
                                        CONNMAN_SERVICE_STATE_FAILURE,
                                        CONNMAN_IPCONFIG_TYPE_IPV4);
 }
@@ -526,11 +664,9 @@ static void set_configure_error(struct connman_network *network)
 {
        struct connman_service *service;
 
-       network->connecting = FALSE;
-
        service = __connman_service_lookup_from_network(network);
 
-       __connman_service_indicate_state(service,
+       __connman_service_ipconfig_indicate_state(service,
                                        CONNMAN_SERVICE_STATE_FAILURE,
                                        CONNMAN_IPCONFIG_TYPE_IPV4);
 }
@@ -595,6 +731,7 @@ void connman_network_set_error(struct connman_network *network,
        DBG("nework %p, error %d", network, error);
 
        network->connecting = FALSE;
+       network->associating = FALSE;
 
        switch (error) {
        case CONNMAN_NETWORK_ERROR_UNKNOWN:
@@ -612,6 +749,8 @@ void connman_network_set_error(struct connman_network *network,
                set_connect_error(network);
                break;
        }
+
+       network_change(network);
 }
 
 void connman_network_clear_error(struct connman_network *network)
@@ -636,14 +775,15 @@ static void set_configuration(struct connman_network *network)
 
        DBG("network %p", network);
 
-       __connman_device_increase_connections(network->device);
+       if (network->device == NULL)
+               return;
 
        __connman_device_set_network(network->device, network);
 
        connman_device_set_disconnected(network->device, FALSE);
 
        service = __connman_service_lookup_from_network(network);
-       __connman_service_indicate_state(service,
+       __connman_service_ipconfig_indicate_state(service,
                                        CONNMAN_SERVICE_STATE_CONFIGURATION,
                                        CONNMAN_IPCONFIG_TYPE_IPV4);
 }
@@ -671,9 +811,6 @@ static void dhcp_success(struct connman_network *network)
        if (err < 0)
                goto err;
 
-       __connman_service_indicate_state(service, CONNMAN_SERVICE_STATE_READY,
-                                               CONNMAN_IPCONFIG_TYPE_IPV4);
-
        return;
 
 err:
@@ -684,17 +821,14 @@ err:
 static void dhcp_failure(struct connman_network *network)
 {
        struct connman_service *service;
-       struct connman_ipconfig *ipconfig_ipv4;
 
        service = __connman_service_lookup_from_network(network);
        if (service == NULL)
                return;
 
-       ipconfig_ipv4 = __connman_service_get_ip4config(service);
-       __connman_ipconfig_address_remove(ipconfig_ipv4);
-
-       __connman_service_indicate_state(service, CONNMAN_SERVICE_STATE_IDLE,
-                                               CONNMAN_IPCONFIG_TYPE_IPV4);
+       __connman_service_ipconfig_indicate_state(service,
+                                       CONNMAN_SERVICE_STATE_IDLE,
+                                       CONNMAN_IPCONFIG_TYPE_IPV4);
 }
 
 static void dhcp_callback(struct connman_network *network,
@@ -734,9 +868,6 @@ static int set_connected_fixed(struct connman_network *network)
        if (err < 0)
                goto err;
 
-       __connman_service_indicate_state(service,
-                                       CONNMAN_SERVICE_STATE_READY,
-                                       CONNMAN_IPCONFIG_TYPE_IPV4);
        return 0;
 
 err:
@@ -758,6 +889,9 @@ static void set_connected_manual(struct connman_network *network)
 
        ipconfig = __connman_service_get_ip4config(service);
 
+       if (__connman_ipconfig_get_local(ipconfig) == NULL)
+               __connman_service_read_ip4config(service);
+
        set_configuration(network);
 
        err = __connman_ipconfig_address_add(ipconfig);
@@ -772,9 +906,6 @@ static void set_connected_manual(struct connman_network *network)
 
        connman_network_set_associating(network, FALSE);
 
-       __connman_service_indicate_state(service, CONNMAN_SERVICE_STATE_READY,
-                                       CONNMAN_IPCONFIG_TYPE_IPV4);
-
        return;
 
 err:
@@ -806,10 +937,15 @@ static int manual_ipv6_set(struct connman_network *network,
        struct connman_service *service;
        int err;
 
+       DBG("network %p ipv6 %p", network, ipconfig_ipv6);
+
        service = __connman_service_lookup_from_network(network);
        if (service == NULL)
                return -EINVAL;
 
+       if (__connman_ipconfig_get_local(ipconfig_ipv6) == NULL)
+               __connman_service_read_ip6config(service);
+
        err = __connman_ipconfig_address_add(ipconfig_ipv6);
        if (err < 0) {
                connman_network_set_error(network,
@@ -817,24 +953,26 @@ static int manual_ipv6_set(struct connman_network *network,
                return err;
        }
 
-       /*
-        * READY state will be indicated by IPV4 setting
-        * gateway will be set by IPV4 setting
-        */
+       err = __connman_ipconfig_gateway_add(ipconfig_ipv6);
+       if (err < 0)
+               return err;
+
+       __connman_connection_gateway_activate(service,
+                                               CONNMAN_IPCONFIG_TYPE_IPV6);
+
+       __connman_device_set_network(network->device, network);
+
+       connman_device_set_disconnected(network->device, FALSE);
+
+       network->connecting = FALSE;
 
        return 0;
 }
 
 static void autoconf_ipv6_set(struct connman_network *network)
 {
-       struct connman_service *service;
-
        DBG("network %p", network);
 
-       service = __connman_service_lookup_from_network(network);
-
-       __connman_device_increase_connections(network->device);
-
        __connman_device_set_network(network->device, network);
 
        connman_device_set_disconnected(network->device, FALSE);
@@ -842,10 +980,6 @@ static void autoconf_ipv6_set(struct connman_network *network)
        /* XXX: Append IPv6 nameservers here */
 
        network->connecting = FALSE;
-
-       __connman_service_indicate_state(service,
-                                       CONNMAN_SERVICE_STATE_READY,
-                                       CONNMAN_IPCONFIG_TYPE_IPV6);
 }
 
 static gboolean set_connected(gpointer user_data)
@@ -916,30 +1050,65 @@ static gboolean set_connected(gpointer user_data)
                }
 
        } else {
-               struct connman_service *service;
-
-               connman_element_unregister_children(&network->element);
+               enum connman_service_state state;
 
                __connman_device_set_network(network->device, NULL);
-               network->hidden = FALSE;
 
-               service = __connman_service_lookup_from_network(network);
+               switch (ipv4_method) {
+               case CONNMAN_IPCONFIG_METHOD_UNKNOWN:
+               case CONNMAN_IPCONFIG_METHOD_OFF:
+               case CONNMAN_IPCONFIG_METHOD_AUTO:
+               case CONNMAN_IPCONFIG_METHOD_FIXED:
+               case CONNMAN_IPCONFIG_METHOD_MANUAL:
+                       break;
+               case CONNMAN_IPCONFIG_METHOD_DHCP:
+                       __connman_dhcp_stop(network);
+                       break;
+               }
 
-               __connman_service_indicate_state(service,
+               /*
+                * We only set the disconnect state if we were not in idle
+                * or in failure. It does not make sense to go to disconnect
+                * state if we were not connected.
+                */
+               state = __connman_service_ipconfig_get_state(service,
+                                               CONNMAN_IPCONFIG_TYPE_IPV4);
+               if (state != CONNMAN_SERVICE_STATE_IDLE &&
+                                       state != CONNMAN_SERVICE_STATE_FAILURE)
+                       __connman_service_ipconfig_indicate_state(service,
                                        CONNMAN_SERVICE_STATE_DISCONNECT,
                                        CONNMAN_IPCONFIG_TYPE_IPV4);
 
-               __connman_service_indicate_state(service,
+               state = __connman_service_ipconfig_get_state(service,
+                                               CONNMAN_IPCONFIG_TYPE_IPV6);
+               if (state != CONNMAN_SERVICE_STATE_IDLE &&
+                                       state != CONNMAN_SERVICE_STATE_FAILURE)
+                       __connman_service_ipconfig_indicate_state(service,
                                        CONNMAN_SERVICE_STATE_DISCONNECT,
                                        CONNMAN_IPCONFIG_TYPE_IPV6);
 
-               __connman_connection_gateway_remove(service);
+               __connman_connection_gateway_remove(service,
+                                               CONNMAN_IPCONFIG_TYPE_ALL);
+
+               __connman_ipconfig_address_unset(ipconfig_ipv4);
+               __connman_ipconfig_address_unset(ipconfig_ipv6);
+
+               /*
+                * Special handling for IPv6 autoconfigured address.
+                * The simplest way to remove autoconfigured routes is to
+                * disable IPv6 temporarily so that kernel will do the cleanup
+                * automagically.
+                */
+               if (ipv6_method == CONNMAN_IPCONFIG_METHOD_AUTO) {
+                       __connman_ipconfig_disable_ipv6(ipconfig_ipv6);
+                       __connman_ipconfig_enable_ipv6(ipconfig_ipv6);
+               }
 
-               __connman_service_indicate_state(service,
+               __connman_service_ipconfig_indicate_state(service,
                                        CONNMAN_SERVICE_STATE_IDLE,
                                        CONNMAN_IPCONFIG_TYPE_IPV4);
 
-               __connman_service_indicate_state(service,
+               __connman_service_ipconfig_indicate_state(service,
                                        CONNMAN_SERVICE_STATE_IDLE,
                                        CONNMAN_IPCONFIG_TYPE_IPV6);
        }
@@ -973,9 +1142,6 @@ int connman_network_set_connected(struct connman_network *network,
        if (network->connected == connected)
                return -EALREADY;
 
-       if (connected == FALSE)
-               __connman_device_decrease_connections(network->device);
-
        network->connected = connected;
 
        set_connected(network);
@@ -1042,7 +1208,6 @@ int __connman_network_connect(struct connman_network *network)
                        connman_network_set_associating(network, TRUE);
                else {
                        network->connecting = FALSE;
-                       network->hidden = FALSE;
                }
 
                return err;
@@ -1104,12 +1269,7 @@ static int manual_ipv4_set(struct connman_network *network,
                return err;
        }
 
-       __connman_ipconfig_gateway_add(ipconfig);
-
-       __connman_service_indicate_state(service, CONNMAN_SERVICE_STATE_READY,
-                                       CONNMAN_IPCONFIG_TYPE_IPV4);
-
-       return 0;
+       return __connman_ipconfig_gateway_add(ipconfig);
 }
 
 int __connman_network_clear_ipconfig(struct connman_network *network,
@@ -1141,11 +1301,11 @@ int __connman_network_clear_ipconfig(struct connman_network *network,
        }
 
        if (type == CONNMAN_IPCONFIG_TYPE_IPV6)
-               __connman_service_indicate_state(service,
+               __connman_service_ipconfig_indicate_state(service,
                                        CONNMAN_SERVICE_STATE_CONFIGURATION,
                                        CONNMAN_IPCONFIG_TYPE_IPV6);
        else if (type == CONNMAN_IPCONFIG_TYPE_IPV4)
-               __connman_service_indicate_state(service,
+               __connman_service_ipconfig_indicate_state(service,
                                        CONNMAN_SERVICE_STATE_CONFIGURATION,
                                        CONNMAN_IPCONFIG_TYPE_IPV4);
 
@@ -1159,6 +1319,9 @@ int __connman_network_set_ipconfig(struct connman_network *network,
        enum connman_ipconfig_method method;
        int ret;
 
+       if (network == NULL)
+               return -EINVAL;
+
        if (ipconfig_ipv6) {
                method = __connman_ipconfig_get_method(ipconfig_ipv6);
 
@@ -1227,22 +1390,6 @@ int connman_network_set_ipaddress(struct connman_network *network,
        return 0;
 }
 
-int connman_network_set_pac(struct connman_network *network,
-                               const char *pac)
-{
-       struct connman_service *service;
-
-       DBG("network %p pac %s", network, pac);
-
-       service = __connman_service_lookup_from_network(network);
-       if (service == NULL)
-               return -EINVAL;
-
-       __connman_service_set_pac(service, pac);
-
-       return 0;
-}
-
 int connman_network_set_nameservers(struct connman_network *network,
                                const char *nameservers)
 {
@@ -1302,7 +1449,7 @@ int connman_network_set_name(struct connman_network *network,
        g_free(network->name);
        network->name = g_strdup(name);
 
-       return connman_element_set_string(&network->element, "Name", name);
+       return 0;
 }
 
 /**
@@ -1312,6 +1459,7 @@ int connman_network_set_name(struct connman_network *network,
  *
  * Set signal strength value for network
  */
+
 int connman_network_set_strength(struct connman_network *network,
                                                connman_uint8_t strength)
 {
@@ -1319,8 +1467,42 @@ int connman_network_set_strength(struct connman_network *network,
 
        network->strength = strength;
 
-       return connman_element_set_uint8(&network->element,
-                                               "Strength", strength);
+       return 0;
+}
+
+connman_uint8_t connman_network_get_strength(struct connman_network *network)
+{
+       return network->strength;
+}
+
+int connman_network_set_frequency(struct connman_network *network,
+                                               connman_uint16_t frequency)
+{
+       DBG("network %p frequency %d", network, frequency);
+
+       network->frequency = frequency;
+
+       return 0;
+}
+
+connman_uint16_t connman_network_get_frequency(struct connman_network *network)
+{
+       return network->frequency;
+}
+
+int connman_network_set_wifi_channel(struct connman_network *network,
+                                               connman_uint16_t channel)
+{
+       DBG("network %p wifi channel %d", network, channel);
+
+       network->wifi.channel = channel;
+
+       return 0;
+}
+
+connman_uint16_t connman_network_get_wifi_channel(struct connman_network *network)
+{
+       return network->wifi.channel;
 }
 
 /**
@@ -1337,8 +1519,7 @@ int connman_network_set_roaming(struct connman_network *network,
 
        network->roaming = roaming;
 
-       return connman_element_set_bool(&network->element,
-                                               "Roaming", roaming);
+       return 0;
 }
 
 /**
@@ -1352,14 +1533,15 @@ int connman_network_set_roaming(struct connman_network *network,
 int connman_network_set_string(struct connman_network *network,
                                        const char *key, const char *value)
 {
-       int err;
-
        DBG("network %p key %s value %s", network, key, value);
 
        if (g_strcmp0(key, "Name") == 0)
                return connman_network_set_name(network, value);
 
-       if (g_str_equal(key, "Node") == TRUE) {
+       if (g_str_equal(key, "Path") == TRUE) {
+               g_free(network->path);
+               network->path = g_strdup(value);
+       } else if (g_str_equal(key, "Node") == TRUE) {
                g_free(network->node);
                network->node = g_strdup(value);
        } else if (g_str_equal(key, "WiFi.Mode") == TRUE) {
@@ -1371,12 +1553,18 @@ int connman_network_set_string(struct connman_network *network,
        } else if (g_str_equal(key, "WiFi.Passphrase") == TRUE) {
                g_free(network->wifi.passphrase);
                network->wifi.passphrase = g_strdup(value);
+       } else if (g_str_equal(key, "WiFi.AgentPassphrase") == TRUE) {
+               g_free(network->wifi.agent_passphrase);
+               network->wifi.agent_passphrase = g_strdup(value);
        } else if (g_str_equal(key, "WiFi.EAP") == TRUE) {
                g_free(network->wifi.eap);
                network->wifi.eap = g_strdup(value);
        } else if (g_str_equal(key, "WiFi.Identity") == TRUE) {
                g_free(network->wifi.identity);
                network->wifi.identity = g_strdup(value);
+       } else if (g_str_equal(key, "WiFi.AgentIdentity") == TRUE) {
+               g_free(network->wifi.agent_identity);
+               network->wifi.agent_identity = g_strdup(value);
        } else if (g_str_equal(key, "WiFi.CACertFile") == TRUE) {
                g_free(network->wifi.ca_cert_path);
                network->wifi.ca_cert_path = g_strdup(value);
@@ -1395,18 +1583,10 @@ int connman_network_set_string(struct connman_network *network,
        } else if (g_str_equal(key, "WiFi.PinWPS") == TRUE) {
                g_free(network->wifi.pin_wps);
                network->wifi.pin_wps = g_strdup(value);
+       } else {
+               return -EINVAL;
        }
 
-       err = connman_element_set_string(&network->element, key, value);
-       if (err < 0)
-               return err;
-
-       if (network->driver == NULL)
-               return 0;
-
-       if (network->driver->setup)
-               return network->driver->setup(network, key);
-
        return 0;
 }
 
@@ -1422,7 +1602,9 @@ const char *connman_network_get_string(struct connman_network *network,
 {
        DBG("network %p key %s", network, key);
 
-       if (g_str_equal(key, "Name") == TRUE)
+       if (g_str_equal(key, "Path") == TRUE)
+               return network->path;
+       else if (g_str_equal(key, "Name") == TRUE)
                return network->name;
        else if (g_str_equal(key, "Node") == TRUE)
                return network->node;
@@ -1432,10 +1614,14 @@ const char *connman_network_get_string(struct connman_network *network,
                return network->wifi.security;
        else if (g_str_equal(key, "WiFi.Passphrase") == TRUE)
                return network->wifi.passphrase;
+       else if (g_str_equal(key, "WiFi.AgentPassphrase") == TRUE)
+               return network->wifi.agent_passphrase;
        else if (g_str_equal(key, "WiFi.EAP") == TRUE)
                return network->wifi.eap;
        else if (g_str_equal(key, "WiFi.Identity") == TRUE)
                return network->wifi.identity;
+       else if (g_str_equal(key, "WiFi.AgentIdentity") == TRUE)
+               return network->wifi.agent_identity;
        else if (g_str_equal(key, "WiFi.CACertFile") == TRUE)
                return network->wifi.ca_cert_path;
        else if (g_str_equal(key, "WiFi.ClientCertFile") == TRUE)
@@ -1449,7 +1635,7 @@ const char *connman_network_get_string(struct connman_network *network,
        else if (g_str_equal(key, "WiFi.PinWPS") == TRUE)
                return network->wifi.pin_wps;
 
-       return connman_element_get_string(&network->element, key);
+       return NULL;
 }
 
 /**
@@ -1472,7 +1658,7 @@ int connman_network_set_bool(struct connman_network *network,
        else if (g_strcmp0(key, "WiFi.UseWPS") == 0)
                network->wifi.use_wps = value;
 
-       return connman_element_set_bool(&network->element, key, value);
+       return -EINVAL;
 }
 
 /**
@@ -1494,85 +1680,7 @@ connman_bool_t connman_network_get_bool(struct connman_network *network,
        else if (g_str_equal(key, "WiFi.UseWPS") == TRUE)
                return network->wifi.use_wps;
 
-       return connman_element_get_bool(&network->element, key);
-}
-
-/**
- * connman_network_set_uint8:
- * @network: network structure
- * @key: unique identifier
- * @value: integer value
- *
- * Set integer value for specific key
- */
-int connman_network_set_uint8(struct connman_network *network,
-                                       const char *key, connman_uint8_t value)
-{
-       DBG("network %p key %s value %d", network, key, value);
-
-       if (g_strcmp0(key, "Strength") == 0)
-               return connman_network_set_strength(network, value);
-
-       return connman_element_set_uint8(&network->element, key, value);
-}
-
-/**
- * connman_network_get_uint8:
- * @network: network structure
- * @key: unique identifier
- *
- * Get integer value for specific key
- */
-connman_uint8_t connman_network_get_uint8(struct connman_network *network,
-                                                       const char *key)
-{
-       DBG("network %p key %s", network, key);
-
-       if (g_str_equal(key, "Strength") == TRUE)
-               return network->strength;
-
-       return connman_element_get_uint8(&network->element, key);
-}
-
-/**
- * connman_network_set_uint16:
- * @network: network structure
- * @key: unique identifier
- * @value: integer value
- *
- * Set integer value for specific key
- */
-int connman_network_set_uint16(struct connman_network *network,
-                               const char *key, connman_uint16_t value)
-{
-       DBG("network %p key %s value %d", network, key, value);
-
-       if (g_str_equal(key, "Frequency") == TRUE)
-               network->frequency = value;
-       else if (g_str_equal(key, "WiFi.Channel") == TRUE)
-               network->wifi.channel = value;
-
-       return -EINVAL;
-}
-
-/**
- * connman_network_get_uint16:
- * @network: network structure
- * @key: unique identifier
- *
- * Get integer value for specific key
- */
-connman_uint16_t connman_network_get_uint16(struct connman_network *network,
-                                                       const char *key)
-{
-       DBG("network %p key %s", network, key);
-
-       if (g_str_equal(key, "Frequency") == TRUE)
-               return network->frequency;
-       else if (g_str_equal(key, "WiFi.Channel") == TRUE)
-               return network->wifi.channel;
-
-       return 0;
+       return FALSE;
 }
 
 /**
@@ -1597,9 +1705,11 @@ int connman_network_set_blob(struct connman_network *network,
                        network->wifi.ssid_len = size;
                } else
                        network->wifi.ssid_len = 0;
+       } else {
+               return -EINVAL;
        }
 
-       return connman_element_set_blob(&network->element, key, data, size);
+       return 0;
 }
 
 /**
@@ -1621,13 +1731,22 @@ const void *connman_network_get_blob(struct connman_network *network,
                return network->wifi.ssid;
        }
 
-       return connman_element_get_blob(&network->element, key, size);
+       return NULL;
 }
 
 void __connman_network_set_device(struct connman_network *network,
                                        struct connman_device *device)
 {
+       if (network->device == device)
+               return;
+
+       if (network->device != NULL)
+               network_remove(network);
+
        network->device = device;
+
+       if (network->device != NULL)
+               network_probe(network);
 }
 
 /**
@@ -1681,143 +1800,16 @@ void connman_network_update(struct connman_network *network)
 
        if (network->group != NULL)
                __connman_service_update_from_network(network);
-
-       return;
-}
-
-static gboolean match_driver(struct connman_network *network,
-                                       struct connman_network_driver *driver)
-{
-       if (network->type == driver->type ||
-                       driver->type == CONNMAN_NETWORK_TYPE_UNKNOWN)
-               return TRUE;
-
-       return FALSE;
-}
-
-static int network_probe(struct connman_element *element)
-{
-       struct connman_network *network = element->network;
-       GSList *list;
-
-       DBG("element %p name %s", element, element->name);
-
-       if (network == NULL)
-               return -ENODEV;
-
-       for (list = driver_list; list; list = list->next) {
-               struct connman_network_driver *driver = list->data;
-
-               if (match_driver(network, driver) == FALSE)
-                       continue;
-
-               DBG("driver %p name %s", driver, driver->name);
-
-               if (driver->probe(network) == 0) {
-                       network->driver = driver;
-                       break;
-               }
-       }
-
-       if (network->driver == NULL)
-               return -ENODEV;
-
-       switch (network->type) {
-       case CONNMAN_NETWORK_TYPE_UNKNOWN:
-       case CONNMAN_NETWORK_TYPE_VENDOR:
-               break;
-       case CONNMAN_NETWORK_TYPE_ETHERNET:
-       case CONNMAN_NETWORK_TYPE_BLUETOOTH_PAN:
-       case CONNMAN_NETWORK_TYPE_BLUETOOTH_DUN:
-       case CONNMAN_NETWORK_TYPE_CELLULAR:
-       case CONNMAN_NETWORK_TYPE_WIFI:
-       case CONNMAN_NETWORK_TYPE_WIMAX:
-               if (network->group != NULL &&
-                        __connman_service_create_from_network(network) == NULL)
-                               return -EINVAL;
-       }
-
-       return 0;
-}
-
-static void network_remove(struct connman_element *element)
-{
-       struct connman_network *network = element->network;
-
-       DBG("element %p name %s", element, element->name);
-
-       if (network == NULL)
-               return;
-
-       if (network->driver == NULL)
-               return;
-
-       switch (network->type) {
-       case CONNMAN_NETWORK_TYPE_UNKNOWN:
-       case CONNMAN_NETWORK_TYPE_VENDOR:
-               break;
-       case CONNMAN_NETWORK_TYPE_ETHERNET:
-       case CONNMAN_NETWORK_TYPE_BLUETOOTH_PAN:
-       case CONNMAN_NETWORK_TYPE_BLUETOOTH_DUN:
-       case CONNMAN_NETWORK_TYPE_CELLULAR:
-       case CONNMAN_NETWORK_TYPE_WIFI:
-       case CONNMAN_NETWORK_TYPE_WIMAX:
-               if (network->group != NULL) {
-                       __connman_service_remove_from_network(network);
-
-                       g_free(network->group);
-                       network->group = NULL;
-               }
-               break;
-       }
-
-       if (network->driver->remove)
-               network->driver->remove(network);
-}
-
-static void network_change(struct connman_element *element)
-{
-       struct connman_network *network = element->network;
-
-       DBG("element %p name %s", element, element->name);
-
-       if (element->state != CONNMAN_ELEMENT_STATE_ERROR)
-               return;
-
-       if (network->connected == FALSE)
-               return;
-
-       connman_element_unregister_children(element);
-
-       connman_device_set_disconnected(network->device, TRUE);
-
-       if (network->driver && network->driver->disconnect) {
-               network->driver->disconnect(network);
-               return;
-       }
-
-       network->connected = FALSE;
 }
 
-static struct connman_driver network_driver = {
-       .name           = "network",
-       .type           = CONNMAN_ELEMENT_TYPE_NETWORK,
-       .priority       = CONNMAN_DRIVER_PRIORITY_LOW,
-       .probe          = network_probe,
-       .remove         = network_remove,
-       .change         = network_change,
-};
-
 int __connman_network_init(void)
 {
        DBG("");
 
-       return connman_driver_register(&network_driver);
+       return 0;
 }
 
 void __connman_network_cleanup(void)
 {
        DBG("");
-
-       connman_driver_unregister(&network_driver);
 }