Imported Upstream version 1.35
[platform/upstream/connman.git] / src / device.c
index fbfb2b5..a563f46 100644 (file)
@@ -2,7 +2,7 @@
  *
  *  Connection Manager
  *
- *  Copyright (C) 2007-2012  Intel Corporation. All rights reserved.
+ *  Copyright (C) 2007-2014  Intel Corporation. All rights reserved.
  *
  *  This program is free software; you can redistribute it and/or modify
  *  it under the terms of the GNU General Public License version 2 as
@@ -48,18 +48,17 @@ struct connman_device {
        int refcount;
        enum connman_device_type type;
        enum connman_pending_type powered_pending;      /* Indicates a pending
-                                                       enable/disable request */
-       connman_bool_t powered;
-       connman_bool_t scanning;
-       connman_bool_t disconnected;
-       connman_bool_t reconnect;
+                                                        * enable/disable
+                                                        * request
+                                                        */
+       bool powered;
+       bool scanning;
        char *name;
        char *node;
        char *address;
        char *interface;
        char *ident;
        char *path;
-       char *devname;
        int index;
        guint pending_timeout;
 
@@ -127,7 +126,8 @@ static const char *type2string(enum connman_device_type type)
        return NULL;
 }
 
-enum connman_service_type __connman_device_get_service_type(struct connman_device *device)
+enum connman_service_type __connman_device_get_service_type(
+                               struct connman_device *device)
 {
        enum connman_device_type type = connman_device_get_type(device);
 
@@ -181,9 +181,15 @@ int __connman_device_enable(struct connman_device *device)
        if (device->powered_pending == PENDING_ENABLE)
                return -EALREADY;
 
-       if (device->powered_pending == PENDING_NONE && device->powered == TRUE)
+       if (device->powered_pending == PENDING_NONE && device->powered)
                return -EALREADY;
 
+       if (device->index > 0) {
+               err = connman_inet_ifup(device->index);
+               if (err < 0 && err != -EALREADY)
+                       return err;
+       }
+
        device->powered_pending = PENDING_ENABLE;
 
        err = device->driver->enable(device);
@@ -192,13 +198,13 @@ int __connman_device_enable(struct connman_device *device)
         * Invoke the callback
         */
        if (err == 0) {
-               connman_device_set_powered(device, TRUE);
+               connman_device_set_powered(device, true);
                goto done;
        }
 
        if (err == -EALREADY) {
                /* If device is already powered, but connman is not updated */
-               connman_device_set_powered(device, TRUE);
+               connman_device_set_powered(device, true);
                goto done;
        }
        /*
@@ -219,9 +225,6 @@ int __connman_device_disable(struct connman_device *device)
 
        DBG("device %p", device);
 
-       if (!device->driver || !device->driver->disable)
-               return -EOPNOTSUPP;
-
        /* Ongoing power enable request */
        if (device->powered_pending == PENDING_ENABLE)
                return -EBUSY;
@@ -229,25 +232,27 @@ int __connman_device_disable(struct connman_device *device)
        if (device->powered_pending == PENDING_DISABLE)
                return -EALREADY;
 
-       if (device->powered_pending == PENDING_NONE && device->powered == FALSE)
+       if (device->powered_pending == PENDING_NONE && !device->powered)
                return -EALREADY;
 
        device->powered_pending = PENDING_DISABLE;
-       device->reconnect = FALSE;
 
        if (device->network) {
                struct connman_service *service =
                        connman_service_lookup_from_network(device->network);
 
-               if (service != NULL)
+               if (service)
                        __connman_service_disconnect(service);
                else
-                       connman_network_set_connected(device->network, FALSE);
+                       connman_network_set_connected(device->network, false);
        }
 
+       if (!device->driver || !device->driver->disable)
+               return -EOPNOTSUPP;
+
        err = device->driver->disable(device);
        if (err == 0 || err == -EALREADY) {
-               connman_device_set_powered(device, FALSE);
+               connman_device_set_powered(device, false);
                goto done;
        }
 
@@ -264,10 +269,10 @@ static void probe_driver(struct connman_device_driver *driver)
 
        DBG("driver %p name %s", driver, driver->name);
 
-       for (list = device_list; list != NULL; list = list->next) {
+       for (list = device_list; list; list = list->next) {
                struct connman_device *device = list->data;
 
-               if (device->driver != NULL)
+               if (device->driver)
                        continue;
 
                if (driver->type != device->type)
@@ -302,7 +307,7 @@ static void remove_driver(struct connman_device_driver *driver)
 
        DBG("driver %p name %s", driver, driver->name);
 
-       for (list = device_list; list != NULL; list = list->next) {
+       for (list = device_list; list; list = list->next) {
                struct connman_device *device = list->data;
 
                if (device->driver == driver)
@@ -310,12 +315,12 @@ static void remove_driver(struct connman_device_driver *driver)
        }
 }
 
-connman_bool_t __connman_device_has_driver(struct connman_device *device)
+bool __connman_device_has_driver(struct connman_device *device)
 {
-       if (device == NULL || device->driver == NULL)
-               return FALSE;
+       if (!device || !device->driver)
+               return false;
 
-       return TRUE;
+       return true;
 }
 
 static GSList *driver_list = NULL;
@@ -379,19 +384,18 @@ static void device_destruct(struct connman_device *device)
 
        clear_pending_trigger(device);
 
+       g_hash_table_destroy(device->networks);
+       device->networks = NULL;
+
        g_free(device->ident);
        g_free(device->node);
        g_free(device->name);
        g_free(device->address);
        g_free(device->interface);
        g_free(device->path);
-       g_free(device->devname);
 
        g_free(device->last_network);
 
-       g_hash_table_destroy(device->networks);
-       device->networks = NULL;
-
        g_free(device);
 }
 
@@ -412,7 +416,7 @@ struct connman_device *connman_device_create(const char *node,
        DBG("node %s type %d", node, type);
 
        device = g_try_new0(struct connman_device, 1);
-       if (device == NULL)
+       if (!device)
                return NULL;
 
        DBG("device %p", device);
@@ -521,15 +525,12 @@ int connman_device_get_index(struct connman_device *device)
 void connman_device_set_interface(struct connman_device *device,
                                                const char *interface)
 {
-       g_free(device->devname);
-       device->devname = g_strdup(interface);
-
        g_free(device->interface);
        device->interface = g_strdup(interface);
 
-       if (device->name == NULL) {
+       if (!device->name) {
                const char *str = type2description(device->type);
-               if (str != NULL && device->interface != NULL)
+               if (str && device->interface)
                        device->name = g_strdup_printf("%s (%s)", str,
                                                        device->interface);
        }
@@ -562,11 +563,11 @@ const char *connman_device_get_ident(struct connman_device *device)
  * Change power state of device
  */
 int connman_device_set_powered(struct connman_device *device,
-                                               connman_bool_t powered)
+                                               bool powered)
 {
        enum connman_service_type type;
 
-       DBG("driver %p powered %d", device, powered);
+       DBG("device %p powered %d", device, powered);
 
        if (device->powered == powered)
                return -EALREADY;
@@ -579,36 +580,38 @@ int connman_device_set_powered(struct connman_device *device,
 
        type = __connman_device_get_service_type(device);
 
-       if (device->powered == FALSE) {
+       if (!device->powered) {
                __connman_technology_disabled(type);
                return 0;
        }
 
        __connman_technology_enabled(type);
 
-       connman_device_set_disconnected(device, FALSE);
-       device->scanning = FALSE;
+       device->scanning = false;
 
        if (device->driver && device->driver->scan)
-               device->driver->scan(device, NULL, 0, NULL, NULL, NULL);
+               device->driver->scan(CONNMAN_SERVICE_TYPE_UNKNOWN, device,
+                                       NULL, 0, NULL, NULL, NULL, NULL);
 
        return 0;
 }
 
-connman_bool_t connman_device_get_powered(struct connman_device *device)
+bool connman_device_get_powered(struct connman_device *device)
 {
        return device->powered;
 }
 
-static int device_scan(struct connman_device *device)
+static int device_scan(enum connman_service_type type,
+                               struct connman_device *device)
 {
        if (!device->driver || !device->driver->scan)
                return -EOPNOTSUPP;
 
-       if (device->powered == FALSE)
+       if (!device->powered)
                return -ENOLINK;
 
-       return device->driver->scan(device, NULL, 0, NULL, NULL, NULL);
+       return device->driver->scan(type, device, NULL, 0,
+                                       NULL, NULL, NULL, NULL);
 }
 
 int __connman_device_disconnect(struct connman_device *device)
@@ -618,14 +621,12 @@ int __connman_device_disconnect(struct connman_device *device)
 
        DBG("device %p", device);
 
-       connman_device_set_disconnected(device, TRUE);
-
        g_hash_table_iter_init(&iter, device->networks);
 
-       while (g_hash_table_iter_next(&iter, &key, &value) == TRUE) {
+       while (g_hash_table_iter_next(&iter, &key, &value)) {
                struct connman_network *network = value;
 
-               if (connman_network_get_connecting(network) == TRUE) {
+               if (connman_network_get_connecting(network)) {
                        /*
                         * Skip network in the process of connecting.
                         * This is a workaround for WiFi networks serviced
@@ -647,12 +648,21 @@ int __connman_device_disconnect(struct connman_device *device)
        return 0;
 }
 
+int connman_device_reconnect_service(struct connman_device *device)
+{
+       DBG("device %p", device);
+
+       __connman_service_auto_connect(CONNMAN_SERVICE_CONNECT_REASON_AUTO);
+
+       return 0;
+}
+
 static void mark_network_available(gpointer key, gpointer value,
                                                        gpointer user_data)
 {
        struct connman_network *network = value;
 
-       connman_network_set_available(network, TRUE);
+       connman_network_set_available(network, true);
 }
 
 static void mark_network_unavailable(gpointer key, gpointer value,
@@ -660,11 +670,11 @@ static void mark_network_unavailable(gpointer key, gpointer value,
 {
        struct connman_network *network = value;
 
-       if (connman_network_get_connected(network) == TRUE ||
-                       connman_network_get_connecting(network) == TRUE)
+       if (connman_network_get_connected(network) ||
+                       connman_network_get_connecting(network))
                return;
 
-       connman_network_set_available(network, FALSE);
+       connman_network_set_available(network, false);
 }
 
 static gboolean remove_unavailable_network(gpointer key, gpointer value,
@@ -672,10 +682,10 @@ static gboolean remove_unavailable_network(gpointer key, gpointer value,
 {
        struct connman_network *network = value;
 
-       if (connman_network_get_connected(network) == TRUE)
+       if (connman_network_get_connected(network))
                return FALSE;
 
-       if (connman_network_get_available(network) == TRUE)
+       if (connman_network_get_available(network))
                return FALSE;
 
        return TRUE;
@@ -687,7 +697,7 @@ void __connman_device_cleanup_networks(struct connman_device *device)
                                        remove_unavailable_network, NULL);
 }
 
-connman_bool_t connman_device_get_scanning(struct connman_device *device)
+bool connman_device_get_scanning(struct connman_device *device)
 {
        return device->scanning;
 }
@@ -706,7 +716,7 @@ void connman_device_reset_scanning(struct connman_device *device)
  * Change scanning state of device
  */
 int connman_device_set_scanning(struct connman_device *device,
-                                               connman_bool_t scanning)
+                               enum connman_service_type type, bool scanning)
 {
        DBG("device %p scanning %d", device, scanning);
 
@@ -718,7 +728,7 @@ int connman_device_set_scanning(struct connman_device *device,
 
        device->scanning = scanning;
 
-       if (scanning == TRUE) {
+       if (scanning) {
                __connman_technology_scan_started(device);
 
                g_hash_table_foreach(device->networks,
@@ -729,45 +739,14 @@ int connman_device_set_scanning(struct connman_device *device,
 
        __connman_device_cleanup_networks(device);
 
-       __connman_technology_scan_stopped(device);
-
-       __connman_service_auto_connect();
-
-       return 0;
-}
-
-/**
- * connman_device_set_disconnected:
- * @device: device structure
- * @disconnected: disconnected state
- *
- * Change disconnected state of device (only for device with networks)
- */
-int connman_device_set_disconnected(struct connman_device *device,
-                                               connman_bool_t disconnected)
-{
-       DBG("device %p disconnected %d", device, disconnected);
-
-       if (device->disconnected == disconnected)
-               return -EALREADY;
+       __connman_technology_scan_stopped(device, type);
 
-       device->disconnected = disconnected;
+       __connman_service_auto_connect(CONNMAN_SERVICE_CONNECT_REASON_AUTO);
 
        return 0;
 }
 
 /**
- * connman_device_get_disconnected:
- * @device: device structure
- *
- * Get device disconnected state
- */
-connman_bool_t connman_device_get_disconnected(struct connman_device *device)
-{
-       return device->disconnected;
-}
-
-/**
  * connman_device_set_string:
  * @device: device structure
  * @key: unique identifier
@@ -780,16 +759,16 @@ int connman_device_set_string(struct connman_device *device,
 {
        DBG("device %p key %s value %s", device, key, value);
 
-       if (g_str_equal(key, "Address") == TRUE) {
+       if (g_str_equal(key, "Address")) {
                g_free(device->address);
                device->address = g_strdup(value);
-       } else if (g_str_equal(key, "Name") == TRUE) {
+       } else if (g_str_equal(key, "Name")) {
                g_free(device->name);
                device->name = g_strdup(value);
-       } else if (g_str_equal(key, "Node") == TRUE) {
+       } else if (g_str_equal(key, "Node")) {
                g_free(device->node);
                device->node = g_strdup(value);
-       } else if (g_str_equal(key, "Path") == TRUE) {
+       } else if (g_str_equal(key, "Path")) {
                g_free(device->path);
                device->path = g_strdup(value);
        } else {
@@ -811,15 +790,15 @@ const char *connman_device_get_string(struct connman_device *device,
 {
        DBG("device %p key %s", device, key);
 
-       if (g_str_equal(key, "Address") == TRUE)
+       if (g_str_equal(key, "Address"))
                return device->address;
-       else if (g_str_equal(key, "Name") == TRUE)
+       else if (g_str_equal(key, "Name"))
                return device->name;
-       else if (g_str_equal(key, "Node") == TRUE)
+       else if (g_str_equal(key, "Node"))
                return device->node;
-       else if (g_str_equal(key, "Interface") == TRUE)
+       else if (g_str_equal(key, "Interface"))
                return device->interface;
-       else if (g_str_equal(key, "Path") == TRUE)
+       else if (g_str_equal(key, "Path"))
                return device->path;
 
        return NULL;
@@ -839,7 +818,7 @@ int connman_device_add_network(struct connman_device *device,
 
        DBG("device %p network %p", device, network);
 
-       if (identifier == NULL)
+       if (!identifier)
                return -EINVAL;
 
        connman_network_ref(network);
@@ -881,7 +860,7 @@ int connman_device_remove_network(struct connman_device *device,
 
        DBG("device %p network %p", device, network);
 
-       if (network == NULL)
+       if (!network)
                return 0;
 
        identifier = connman_network_get_identifier(network);
@@ -890,23 +869,18 @@ int connman_device_remove_network(struct connman_device *device,
        return 0;
 }
 
-void connman_device_remove_all_networks(struct connman_device *device)
-{
-       g_hash_table_remove_all(device->networks);
-}
-
 void __connman_device_set_network(struct connman_device *device,
                                        struct connman_network *network)
 {
        const char *name;
 
-       if (device == NULL)
+       if (!device)
                return;
 
        if (device->network == network)
                return;
 
-       if (network != NULL) {
+       if (network) {
                name = connman_network_get_string(network, "Name");
                g_free(device->last_network);
                device->last_network = g_strdup(name);
@@ -920,26 +894,14 @@ void __connman_device_set_network(struct connman_device *device,
        }
 }
 
-void __connman_device_set_reconnect(struct connman_device *device,
-                                               connman_bool_t reconnect)
-{
-       device->reconnect = reconnect;
-}
-
-connman_bool_t  __connman_device_get_reconnect(
-                               struct connman_device *device)
-{
-       return device->reconnect;
-}
-
-static gboolean match_driver(struct connman_device *device,
+static bool match_driver(struct connman_device *device,
                                        struct connman_device_driver *driver)
 {
        if (device->type == driver->type ||
                        driver->type == CONNMAN_DEVICE_TYPE_UNKNOWN)
-               return TRUE;
+               return true;
 
-       return FALSE;
+       return false;
 }
 
 /**
@@ -954,13 +916,13 @@ int connman_device_register(struct connman_device *device)
 
        DBG("device %p name %s", device, device->name);
 
-       if (device->driver != NULL)
+       if (device->driver)
                return -EALREADY;
 
        for (list = driver_list; list; list = list->next) {
                struct connman_device_driver *driver = list->data;
 
-               if (match_driver(device, driver) == FALSE)
+               if (!match_driver(device, driver))
                        continue;
 
                DBG("driver %p name %s", driver, driver->name);
@@ -971,7 +933,7 @@ int connman_device_register(struct connman_device *device)
                }
        }
 
-       if (device->driver == NULL)
+       if (!device->driver)
                return 0;
 
        return __connman_technology_add_device(device);
@@ -987,7 +949,7 @@ void connman_device_unregister(struct connman_device *device)
 {
        DBG("device %p name %s", device, device->name);
 
-       if (device->driver == NULL)
+       if (!device->driver)
                return;
 
        remove_device(device);
@@ -1021,7 +983,7 @@ struct connman_device *__connman_device_find_device(
 {
        GSList *list;
 
-       for (list = device_list; list != NULL; list = list->next) {
+       for (list = device_list; list; list = list->next) {
                struct connman_device *device = list->data;
                enum connman_service_type service_type =
                        __connman_device_get_service_type(device);
@@ -1035,6 +997,19 @@ struct connman_device *__connman_device_find_device(
        return NULL;
 }
 
+struct connman_device *connman_device_find_by_index(int index)
+{
+       GSList *list;
+
+       for (list = device_list; list; list = list->next) {
+               struct connman_device *device = list->data;
+               if (device->index == index)
+                       return device;
+       }
+
+       return NULL;
+}
+
 /**
  * connman_device_set_regdom
  * @device: device structure
@@ -1045,10 +1020,10 @@ struct connman_device *__connman_device_find_device(
 int connman_device_set_regdom(struct connman_device *device,
                                                const char *alpha2)
 {
-       if (device->driver == NULL || device->driver->set_regdom == NULL)
+       if (!device->driver || !device->driver->set_regdom)
                return -ENOTSUP;
 
-       if (device->powered == FALSE)
+       if (!device->powered)
                return -EINVAL;
 
        return device->driver->set_regdom(device, alpha2);
@@ -1069,7 +1044,7 @@ void connman_device_regdom_notify(struct connman_device *device,
 
 int __connman_device_request_scan(enum connman_service_type type)
 {
-       connman_bool_t success = FALSE;
+       bool success = false;
        int last_err = -ENOSYS;
        GSList *list;
        int err;
@@ -1085,29 +1060,33 @@ int __connman_device_request_scan(enum connman_service_type type)
        case CONNMAN_SERVICE_TYPE_GADGET:
                return -EOPNOTSUPP;
        case CONNMAN_SERVICE_TYPE_WIFI:
+       case CONNMAN_SERVICE_TYPE_P2P:
                break;
        }
 
-       for (list = device_list; list != NULL; list = list->next) {
+       for (list = device_list; list; list = list->next) {
                struct connman_device *device = list->data;
                enum connman_service_type service_type =
                        __connman_device_get_service_type(device);
 
-               if (service_type != CONNMAN_SERVICE_TYPE_UNKNOWN &&
-                               service_type != type) {
-                       continue;
+               if (service_type != CONNMAN_SERVICE_TYPE_UNKNOWN) {
+                       if (type == CONNMAN_SERVICE_TYPE_P2P) {
+                               if (service_type != CONNMAN_SERVICE_TYPE_WIFI)
+                                       continue;
+                       } else if (service_type != type)
+                               continue;
                }
 
-               err = device_scan(device);
+               err = device_scan(type, device);
                if (err == 0 || err == -EALREADY || err == -EINPROGRESS) {
-                       success = TRUE;
+                       success = true;
                } else {
                        last_err = err;
                        DBG("device %p err %d", device, err);
                }
        }
 
-       if (success == TRUE)
+       if (success)
                return 0;
 
        return last_err;
@@ -1116,19 +1095,17 @@ int __connman_device_request_scan(enum connman_service_type type)
 int __connman_device_request_hidden_scan(struct connman_device *device,
                                const char *ssid, unsigned int ssid_len,
                                const char *identity, const char *passphrase,
-                               void *user_data)
+                               const char *security, void *user_data)
 {
        DBG("device %p", device);
 
-       if (device == NULL || device->driver == NULL ||
-                       device->driver->scan == NULL)
+       if (!device || !device->driver ||
+                       !device->driver->scan)
                return -EINVAL;
 
-       if (device->scanning == TRUE)
-               return -EALREADY;
-
-       return device->driver->scan(device, ssid, ssid_len,
-                                       identity, passphrase, user_data);
+       return device->driver->scan(CONNMAN_SERVICE_TYPE_UNKNOWN,
+                                       device, ssid, ssid_len, identity,
+                                       passphrase, security, user_data);
 }
 
 static char *index2ident(int index, const char *prefix)
@@ -1231,10 +1208,10 @@ struct connman_device *connman_device_create_from_index(int index)
                return NULL;
 
        devname = connman_inet_ifname(index);
-       if (devname == NULL)
+       if (!devname)
                return NULL;
 
-       if (__connman_device_isfiltered(devname) == TRUE) {
+       if (__connman_device_isfiltered(devname)) {
                connman_info("Ignoring interface %s (filtered)", devname);
                g_free(devname);
                return NULL;
@@ -1262,7 +1239,7 @@ struct connman_device *connman_device_create_from_index(int index)
        }
 
        device = connman_device_create(name, type);
-       if (device == NULL)
+       if (!device)
                goto done;
 
        switch (type) {
@@ -1287,7 +1264,7 @@ struct connman_device *connman_device_create_from_index(int index)
        connman_device_set_index(device, index);
        connman_device_set_interface(device, devname);
 
-       if (ident != NULL) {
+       if (ident) {
                connman_device_set_ident(device, ident);
                g_free(ident);
        }
@@ -1302,51 +1279,62 @@ done:
        return device;
 }
 
-connman_bool_t __connman_device_isfiltered(const char *devname)
+bool __connman_device_isfiltered(const char *devname)
 {
        char **pattern;
        char **blacklisted_interfaces;
+       bool match;
 
-       if (device_filter == NULL)
+       if (!device_filter)
                goto nodevice;
 
-       for (pattern = device_filter; *pattern; pattern++) {
-               if (g_pattern_match_simple(*pattern, devname) == FALSE) {
-                       DBG("ignoring device %s (match)", devname);
-                       return TRUE;
+       for (pattern = device_filter, match = false; *pattern; pattern++) {
+               if (g_pattern_match_simple(*pattern, devname)) {
+                       match = true;
+                       break;
                }
        }
 
+       if (!match) {
+               DBG("ignoring device %s (match)", devname);
+               return true;
+       }
+
 nodevice:
-       if (g_pattern_match_simple("dummy*", devname) == TRUE) {
+       if (g_pattern_match_simple("dummy*", devname)) {
                DBG("ignoring dummy networking devices");
-               return TRUE;
+               return true;
        }
 
-       if (nodevice_filter == NULL)
+       if (!nodevice_filter)
                goto list;
 
        for (pattern = nodevice_filter; *pattern; pattern++) {
-               if (g_pattern_match_simple(*pattern, devname) == TRUE) {
+               if (g_pattern_match_simple(*pattern, devname)) {
                        DBG("ignoring device %s (no match)", devname);
-                       return TRUE;
+                       return true;
                }
        }
 
 list:
+       if (__connman_inet_isrootnfs_device(devname)) {
+               DBG("ignoring device %s (rootnfs)", devname);
+               return true;
+       }
+
        blacklisted_interfaces =
                connman_setting_get_string_list("NetworkInterfaceBlacklist");
-       if (blacklisted_interfaces == NULL)
-               return FALSE;
+       if (!blacklisted_interfaces)
+               return false;
 
        for (pattern = blacklisted_interfaces; *pattern; pattern++) {
-               if (g_str_has_prefix(devname, *pattern) == TRUE) {
+               if (g_str_has_prefix(devname, *pattern)) {
                        DBG("ignoring device %s (blacklist)", devname);
-                       return TRUE;
+                       return true;
                }
        }
 
-       return FALSE;
+       return false;
 }
 
 static void cleanup_devices(void)
@@ -1369,21 +1357,41 @@ static void cleanup_devices(void)
 
        interfaces = __connman_inet_get_running_interfaces();
 
-       if (interfaces == NULL)
+       if (!interfaces)
                return;
 
-       for (i = 0; interfaces[i] != NULL; i++) {
-               connman_bool_t filtered;
+       for (i = 0; interfaces[i]; i++) {
+               bool filtered;
                int index;
+               struct sockaddr_in sin_addr, sin_mask;
 
                filtered = __connman_device_isfiltered(interfaces[i]);
-               if (filtered == TRUE)
+               if (filtered)
                        continue;
 
                index = connman_inet_ifindex(interfaces[i]);
                if (index < 0)
                        continue;
 
+               if (!__connman_inet_get_address_netmask(index, &sin_addr,
+                                                       &sin_mask)) {
+                       char *address = g_strdup(inet_ntoa(sin_addr.sin_addr));
+                       char *netmask = g_strdup(inet_ntoa(sin_mask.sin_addr));
+
+                       if (__connman_config_address_provisioned(address,
+                                                               netmask)) {
+                               DBG("Skip %s which is already provisioned "
+                                       "with %s/%s", interfaces[i], address,
+                                       netmask);
+                               g_free(address);
+                               g_free(netmask);
+                               continue;
+                       }
+
+                       g_free(address);
+                       g_free(netmask);
+               }
+
                DBG("cleaning up %s index %d", interfaces[i], index);
 
                connman_inet_ifdown(index);
@@ -1401,10 +1409,10 @@ int __connman_device_init(const char *device, const char *nodevice)
 {
        DBG("");
 
-       if (device != NULL)
+       if (device)
                device_filter = g_strsplit(device, ",", -1);
 
-       if (nodevice != NULL)
+       if (nodevice)
                nodevice_filter = g_strsplit(nodevice, ",", -1);
 
        cleanup_devices();