Imported Upstream version 1.29
[platform/upstream/connman.git] / src / technology.c
index 3f41d7f..55303a0 100644 (file)
@@ -2,7 +2,7 @@
  *
  *  Connection Manager
  *
- *  Copyright (C) 2007-2012  Intel Corporation. All rights reserved.
+ *  Copyright (C) 2007-2013  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
@@ -41,13 +41,13 @@ static GSList *technology_list = NULL;
 static GSList *techless_device_list = NULL;
 static GHashTable *rfkill_list;
 
-static connman_bool_t global_offlinemode;
+static bool global_offlinemode;
 
 struct connman_rfkill {
        unsigned int index;
        enum connman_service_type type;
-       connman_bool_t softblock;
-       connman_bool_t hardblock;
+       bool softblock;
+       bool hardblock;
 };
 
 struct connman_technology {
@@ -55,15 +55,19 @@ struct connman_technology {
        enum connman_service_type type;
        char *path;
        GSList *device_list;
-       connman_bool_t enabled;
+       bool enabled;
        char *regdom;
-       connman_bool_t connected;
+       bool connected;
 
-       connman_bool_t tethering;
+       bool tethering;
+       bool tethering_persistent; /* Tells the save status, needed
+                                             * as offline mode might set
+                                             * tethering OFF.
+                                             */
        char *tethering_ident;
        char *tethering_passphrase;
 
-       connman_bool_t enable_persistent; /* Save the tech state */
+       bool enable_persistent; /* Save the tech state */
 
        GSList *driver_list;
 
@@ -72,14 +76,17 @@ struct connman_technology {
 
        GSList *scan_pending;
 
-       connman_bool_t rfkill_driven;
-       connman_bool_t softblocked;
-       connman_bool_t hardblocked;
-       connman_bool_t dbus_registered;
+       bool rfkill_driven;
+       bool softblocked;
+       bool hardblocked;
+       bool dbus_registered;
 };
 
 static GSList *driver_list = NULL;
 
+static int technology_enabled(struct connman_technology *technology);
+static int technology_disabled(struct connman_technology *technology);
+
 static gint compare_priority(gconstpointer a, gconstpointer b)
 {
        const struct connman_technology_driver *driver1 = a;
@@ -99,101 +106,114 @@ static void rfkill_check(gpointer key, gpointer value, gpointer user_data)
                                rfkill->softblock, rfkill->hardblock);
 }
 
-/**
- * connman_technology_driver_register:
- * @driver: Technology driver definition
- *
- * Register a new technology driver
- *
- * Returns: %0 on success
- */
-int connman_technology_driver_register(struct connman_technology_driver *driver)
+bool
+connman_technology_is_tethering_allowed(enum connman_service_type type)
 {
-       GSList *list;
-       struct connman_device *device;
-       enum connman_service_type type;
-
-       DBG("Registering %s driver", driver->name);
-
-       driver_list = g_slist_insert_sorted(driver_list, driver,
-                                                       compare_priority);
-
-       /*
-        * Check for technology less devices if this driver
-        * can service any of them.
-       */
-       for (list = techless_device_list; list != NULL; list = list->next) {
-               device = list->data;
-
-               type = __connman_device_get_service_type(device);
-               if (type != driver->type)
-                       continue;
+       static char *allowed_default[] = { "wifi", "bluetooth", "gadget",
+                                          NULL };
+       const char *type_str = __connman_service_type2string(type);
+       char **allowed;
+       int i;
+
+       if (!type_str)
+               return false;
+
+       allowed = connman_setting_get_string_list("TetheringTechnologies");
+       if (!allowed)
+               allowed = allowed_default;
+
+       for (i = 0; allowed[i]; i++) {
+               if (g_strcmp0(allowed[i], type_str) == 0)
+                       return true;
+       }
 
-               techless_device_list = g_slist_remove(techless_device_list,
-                                                               device);
+       return false;
+}
 
-               __connman_technology_add_device(device);
+static const char *get_name(enum connman_service_type type)
+{
+       switch (type) {
+       case CONNMAN_SERVICE_TYPE_UNKNOWN:
+       case CONNMAN_SERVICE_TYPE_SYSTEM:
+       case CONNMAN_SERVICE_TYPE_GPS:
+       case CONNMAN_SERVICE_TYPE_VPN:
+               break;
+       case CONNMAN_SERVICE_TYPE_GADGET:
+               return "Gadget";
+       case CONNMAN_SERVICE_TYPE_ETHERNET:
+               return "Wired";
+       case CONNMAN_SERVICE_TYPE_WIFI:
+               return "WiFi";
+       case CONNMAN_SERVICE_TYPE_BLUETOOTH:
+               return "Bluetooth";
+       case CONNMAN_SERVICE_TYPE_CELLULAR:
+               return "Cellular";
+       case CONNMAN_SERVICE_TYPE_P2P:
+               return "P2P";
        }
 
-       /* Check for orphaned rfkill switches. */
-       g_hash_table_foreach(rfkill_list, rfkill_check,
-                                       GINT_TO_POINTER(driver->type));
-
-       return 0;
+       return NULL;
 }
 
-/**
- * connman_technology_driver_unregister:
- * @driver: Technology driver definition
- *
- * Remove a previously registered technology driver
- */
-void connman_technology_driver_unregister(struct connman_technology_driver *driver)
+static void technology_save(struct connman_technology *technology)
 {
-       GSList *list, *tech_drivers;
-       struct connman_technology *technology;
-       struct connman_technology_driver *current;
+       GKeyFile *keyfile;
+       gchar *identifier;
+       const char *name = get_name(technology->type);
 
-       DBG("Unregistering driver %p name %s", driver, driver->name);
+       DBG("technology %p type %d name %s", technology, technology->type,
+                                                                       name);
+       if (!name)
+               return;
 
-       for (list = technology_list; list; list = list->next) {
-               technology = list->data;
+       keyfile = __connman_storage_load_global();
+       if (!keyfile)
+               keyfile = g_key_file_new();
 
-               for (tech_drivers = technology->driver_list;
-                    tech_drivers != NULL;
-                    tech_drivers = g_slist_next(tech_drivers)) {
+       identifier = g_strdup_printf("%s", name);
+       if (!identifier)
+               goto done;
 
-                       current = tech_drivers->data;
-                       if (driver != current)
-                               continue;
+       g_key_file_set_boolean(keyfile, identifier, "Enable",
+                               technology->enable_persistent);
 
-                       if (driver->remove != NULL)
-                               driver->remove(technology);
+       g_key_file_set_boolean(keyfile, identifier, "Tethering",
+                               technology->tethering_persistent);
 
-                       technology->driver_list =
-                               g_slist_remove(technology->driver_list, driver);
+       if (technology->tethering_ident)
+               g_key_file_set_string(keyfile, identifier,
+                                       "Tethering.Identifier",
+                                       technology->tethering_ident);
 
-                       break;
-               }
-       }
+       if (technology->tethering_passphrase)
+               g_key_file_set_string(keyfile, identifier,
+                                       "Tethering.Passphrase",
+                                       technology->tethering_passphrase);
 
-       driver_list = g_slist_remove(driver_list, driver);
+done:
+       g_free(identifier);
+
+       __connman_storage_save_global(keyfile);
+
+       g_key_file_free(keyfile);
+
+       return;
 }
 
 static void tethering_changed(struct connman_technology *technology)
 {
-       connman_bool_t tethering = technology->tethering;
+       dbus_bool_t tethering = technology->tethering;
 
        connman_dbus_property_changed_basic(technology->path,
                                CONNMAN_TECHNOLOGY_INTERFACE, "Tethering",
                                                DBUS_TYPE_BOOLEAN, &tethering);
+
+       technology_save(technology);
 }
 
 void connman_technology_tethering_notify(struct connman_technology *technology,
-                                                       connman_bool_t enabled)
+                                                       bool enabled)
 {
-       GSList *list;
-
        DBG("technology %p enabled %u", technology, enabled);
 
        if (technology->tethering == enabled)
@@ -203,21 +223,14 @@ void connman_technology_tethering_notify(struct connman_technology *technology,
 
        tethering_changed(technology);
 
-       if (enabled == TRUE)
+       if (enabled)
                __connman_tethering_set_enabled();
-       else {
-               for (list = technology_list; list; list = list->next) {
-                       struct connman_technology *other_tech = list->data;
-                       if (other_tech->tethering == TRUE)
-                               break;
-               }
-               if (list == NULL)
-                       __connman_tethering_set_disabled();
-       }
+       else
+               __connman_tethering_set_disabled();
 }
 
 static int set_tethering(struct connman_technology *technology,
-                               connman_bool_t enabled)
+                               bool enabled)
 {
        int result = -EOPNOTSUPP;
        int err;
@@ -228,22 +241,22 @@ static int set_tethering(struct connman_technology *technology,
        passphrase = technology->tethering_passphrase;
 
        __sync_synchronize();
-       if (technology->enabled == FALSE)
+       if (!technology->enabled)
                return -EACCES;
 
        bridge = __connman_tethering_get_bridge();
-       if (bridge == NULL)
+       if (!bridge)
                return -EOPNOTSUPP;
 
        if (technology->type == CONNMAN_SERVICE_TYPE_WIFI &&
-           (ident == NULL || passphrase == NULL))
+           (!ident || !passphrase))
                return -EINVAL;
 
-       for (tech_drivers = technology->driver_list; tech_drivers != NULL;
+       for (tech_drivers = technology->driver_list; tech_drivers;
             tech_drivers = g_slist_next(tech_drivers)) {
                struct connman_technology_driver *driver = tech_drivers->data;
 
-               if (driver == NULL || driver->set_tethering == NULL)
+               if (!driver || !driver->set_tethering)
                        continue;
 
                err = driver->set_tethering(technology, ident, passphrase,
@@ -266,7 +279,7 @@ void connman_technology_regdom_notify(struct connman_technology *technology,
 {
        DBG("");
 
-       if (alpha2 == NULL)
+       if (!alpha2)
                connman_error("Failed to set regulatory domain");
        else
                DBG("Regulatory domain set to %s", alpha2);
@@ -300,13 +313,13 @@ int connman_technology_set_regdom(const char *alpha2)
                if (set_regdom_by_device(technology, alpha2) != 0) {
 
                        for (tech_drivers = technology->driver_list;
-                            tech_drivers != NULL;
+                            tech_drivers;
                             tech_drivers = g_slist_next(tech_drivers)) {
 
                                struct connman_technology_driver *driver =
                                        tech_drivers->data;
 
-                               if (driver->set_regdom != NULL)
+                               if (driver->set_regdom)
                                        driver->set_regdom(technology, alpha2);
                        }
                }
@@ -315,71 +328,50 @@ int connman_technology_set_regdom(const char *alpha2)
        return 0;
 }
 
-static void free_rfkill(gpointer data)
+static struct connman_technology *technology_find(enum connman_service_type type)
 {
-       struct connman_rfkill *rfkill = data;
+       GSList *list;
 
-       g_free(rfkill);
-}
+       DBG("type %d", type);
 
-static const char *get_name(enum connman_service_type type)
-{
-       switch (type) {
-       case CONNMAN_SERVICE_TYPE_UNKNOWN:
-       case CONNMAN_SERVICE_TYPE_SYSTEM:
-       case CONNMAN_SERVICE_TYPE_GPS:
-       case CONNMAN_SERVICE_TYPE_VPN:
-       case CONNMAN_SERVICE_TYPE_GADGET:
-               break;
-       case CONNMAN_SERVICE_TYPE_ETHERNET:
-               return "Wired";
-       case CONNMAN_SERVICE_TYPE_WIFI:
-               return "WiFi";
-       case CONNMAN_SERVICE_TYPE_BLUETOOTH:
-               return "Bluetooth";
-       case CONNMAN_SERVICE_TYPE_CELLULAR:
-               return "Cellular";
+       for (list = technology_list; list; list = list->next) {
+               struct connman_technology *technology = list->data;
+
+               if (technology->type == type)
+                       return technology;
        }
 
        return NULL;
 }
 
-static void technology_save(struct connman_technology *technology)
+bool connman_technology_get_wifi_tethering(const char **ssid,
+                                                       const char **psk)
 {
-       GKeyFile *keyfile;
-       gchar *identifier;
-
-       DBG("technology %p", technology);
-
-       keyfile = __connman_storage_load_global();
-       if (keyfile == NULL)
-               keyfile = g_key_file_new();
+       struct connman_technology *technology;
 
-       identifier = g_strdup_printf("%s", get_name(technology->type));
-       if (identifier == NULL)
-               goto done;
+       if (!ssid || !psk)
+               return false;
 
-       g_key_file_set_boolean(keyfile, identifier, "Enable",
-                               technology->enable_persistent);
+       *ssid = *psk = NULL;
 
-       if (technology->tethering_ident != NULL)
-               g_key_file_set_string(keyfile, identifier,
-                                       "Tethering.Identifier",
-                                       technology->tethering_ident);
+       technology = technology_find(CONNMAN_SERVICE_TYPE_WIFI);
+       if (!technology)
+               return false;
 
-       if (technology->tethering_passphrase != NULL)
-               g_key_file_set_string(keyfile, identifier,
-                                       "Tethering.Passphrase",
-                                       technology->tethering_passphrase);
+       if (!technology->tethering)
+               return false;
 
-done:
-       g_free(identifier);
+       *ssid = technology->tethering_ident;
+       *psk = technology->tethering_passphrase;
 
-       __connman_storage_save_global(keyfile);
+       return true;
+}
 
-       g_key_file_free(keyfile);
+static void free_rfkill(gpointer data)
+{
+       struct connman_rfkill *rfkill = data;
 
-       return;
+       g_free(rfkill);
 }
 
 static void technology_load(struct connman_technology *technology)
@@ -387,38 +379,50 @@ static void technology_load(struct connman_technology *technology)
        GKeyFile *keyfile;
        gchar *identifier;
        GError *error = NULL;
-       connman_bool_t enable;
+       bool enable, need_saving = false;
 
        DBG("technology %p", technology);
 
        keyfile = __connman_storage_load_global();
        /* Fallback on disabling technology if file not found. */
-       if (keyfile == NULL) {
+       if (!keyfile) {
                if (technology->type == CONNMAN_SERVICE_TYPE_ETHERNET)
                        /* We enable ethernet by default */
-                       technology->enable_persistent = TRUE;
+                       technology->enable_persistent = true;
                else
-                       technology->enable_persistent = FALSE;
+                       technology->enable_persistent = false;
                return;
        }
 
        identifier = g_strdup_printf("%s", get_name(technology->type));
-       if (identifier == NULL)
+       if (!identifier)
                goto done;
 
        enable = g_key_file_get_boolean(keyfile, identifier, "Enable", &error);
-       if (error == NULL)
+       if (!error)
                technology->enable_persistent = enable;
        else {
                if (technology->type == CONNMAN_SERVICE_TYPE_ETHERNET)
-                       technology->enable_persistent = TRUE;
+                       technology->enable_persistent = true;
                else
-                       technology->enable_persistent = FALSE;
+                       technology->enable_persistent = false;
 
-               technology_save(technology);
+               need_saving = true;
                g_clear_error(&error);
        }
 
+       enable = g_key_file_get_boolean(keyfile, identifier,
+                                       "Tethering", &error);
+       if (!error)
+               technology->tethering_persistent = enable;
+       else {
+               need_saving = true;
+               g_clear_error(&error);
+       }
+
+       if (need_saving)
+               technology_save(technology);
+
        technology->tethering_ident = g_key_file_get_string(keyfile,
                                identifier, "Tethering.Identifier", NULL);
 
@@ -432,17 +436,17 @@ done:
        return;
 }
 
-connman_bool_t __connman_technology_get_offlinemode(void)
+bool __connman_technology_get_offlinemode(void)
 {
        return global_offlinemode;
 }
 
-static void connman_technology_save_offlinemode()
+static void connman_technology_save_offlinemode(void)
 {
        GKeyFile *keyfile;
 
        keyfile = __connman_storage_load_global();
-       if (keyfile == NULL)
+       if (!keyfile)
                keyfile = g_key_file_new();
 
        g_key_file_set_boolean(keyfile, "global",
@@ -455,21 +459,21 @@ static void connman_technology_save_offlinemode()
        return;
 }
 
-static connman_bool_t connman_technology_load_offlinemode()
+static bool connman_technology_load_offlinemode(void)
 {
        GKeyFile *keyfile;
        GError *error = NULL;
-       connman_bool_t offlinemode;
+       bool offlinemode;
 
        /* If there is a error, we enable offlinemode */
        keyfile = __connman_storage_load_global();
-       if (keyfile == NULL)
-               return FALSE;
+       if (!keyfile)
+               return false;
 
        offlinemode = g_key_file_get_boolean(keyfile, "global",
                                                "OfflineMode", &error);
-       if (error != NULL) {
-               offlinemode = FALSE;
+       if (error) {
+               offlinemode = false;
                g_clear_error(&error);
        }
 
@@ -482,39 +486,43 @@ static void append_properties(DBusMessageIter *iter,
                struct connman_technology *technology)
 {
        DBusMessageIter dict;
+       dbus_bool_t val;
        const char *str;
 
        connman_dbus_dict_open(iter, &dict);
 
        str = get_name(technology->type);
-       if (str != NULL)
+       if (str)
                connman_dbus_dict_append_basic(&dict, "Name",
                                                DBUS_TYPE_STRING, &str);
 
        str = __connman_service_type2string(technology->type);
-       if (str != NULL)
+       if (str)
                connman_dbus_dict_append_basic(&dict, "Type",
                                                DBUS_TYPE_STRING, &str);
 
        __sync_synchronize();
+       val = technology->enabled;
        connman_dbus_dict_append_basic(&dict, "Powered",
                                        DBUS_TYPE_BOOLEAN,
-                                       &technology->enabled);
+                                       &val);
 
+       val = technology->connected;
        connman_dbus_dict_append_basic(&dict, "Connected",
                                        DBUS_TYPE_BOOLEAN,
-                                       &technology->connected);
+                                       &val);
 
+       val = technology->tethering;
        connman_dbus_dict_append_basic(&dict, "Tethering",
                                        DBUS_TYPE_BOOLEAN,
-                                       &technology->tethering);
+                                       &val);
 
-       if (technology->tethering_ident != NULL)
+       if (technology->tethering_ident)
                connman_dbus_dict_append_basic(&dict, "TetheringIdentifier",
                                        DBUS_TYPE_STRING,
                                        &technology->tethering_ident);
 
-       if (technology->tethering_passphrase != NULL)
+       if (technology->tethering_passphrase)
                connman_dbus_dict_append_basic(&dict, "TetheringPassphrase",
                                        DBUS_TYPE_STRING,
                                        &technology->tethering_passphrase);
@@ -529,7 +537,7 @@ static void technology_added_signal(struct connman_technology *technology)
 
        signal = dbus_message_new_signal(CONNMAN_MANAGER_PATH,
                        CONNMAN_MANAGER_INTERFACE, "TechnologyAdded");
-       if (signal == NULL)
+       if (!signal)
                return;
 
        dbus_message_iter_init_append(signal, &iter);
@@ -557,7 +565,7 @@ static DBusMessage *get_properties(DBusConnection *conn,
        DBusMessageIter iter;
 
        reply = dbus_message_new_method_return(message);
-       if (reply == NULL)
+       if (!reply)
                return NULL;
 
        dbus_message_iter_init_append(reply, &iter);
@@ -574,9 +582,9 @@ void __connman_technology_list_struct(DBusMessageIter *array)
        for (list = technology_list; list; list = list->next) {
                struct connman_technology *technology = list->data;
 
-               if (technology->path == NULL ||
-                               (technology->rfkill_driven == TRUE &&
-                                technology->hardblocked == TRUE))
+               if (!technology->path ||
+                               (technology->rfkill_driven &&
+                                technology->hardblocked))
                        continue;
 
                dbus_message_iter_open_container(array, DBUS_TYPE_STRUCT,
@@ -594,9 +602,9 @@ static gboolean technology_pending_reply(gpointer user_data)
        DBusMessage *reply;
 
        /* Power request timedout, send ETIMEDOUT. */
-       if (technology->pending_reply != NULL) {
+       if (technology->pending_reply) {
                reply = __connman_error_failed(technology->pending_reply, ETIMEDOUT);
-               if (reply != NULL)
+               if (reply)
                        g_dbus_send_message(connection, reply);
 
                dbus_message_unref(technology->pending_reply);
@@ -608,23 +616,96 @@ static gboolean technology_pending_reply(gpointer user_data)
 }
 
 static int technology_affect_devices(struct connman_technology *technology,
-                                               connman_bool_t enable_device)
+                                               bool enable_device)
 {
+       int err = 0, err_dev;
        GSList *list;
-       int err = 0;
+
+       if (technology->type == CONNMAN_SERVICE_TYPE_P2P) {
+               if (enable_device)
+                       __connman_technology_enabled(technology->type);
+               else
+                       __connman_technology_disabled(technology->type);
+               return 0;
+       }
 
        for (list = technology->device_list; list; list = list->next) {
                struct connman_device *device = list->data;
 
-               if (enable_device == TRUE)
-                       err = __connman_device_enable(device);
+               if (enable_device)
+                       err_dev = __connman_device_enable(device);
                else
-                       err = __connman_device_disable(device);
+                       err_dev = __connman_device_disable(device);
+
+               if (err_dev < 0 && err_dev != -EALREADY)
+                       err = err_dev;
        }
 
        return err;
 }
 
+static void powered_changed(struct connman_technology *technology)
+{
+       dbus_bool_t enabled;
+
+       if (!technology->dbus_registered)
+               return;
+
+       if (technology->pending_reply) {
+               g_dbus_send_reply(connection,
+                               technology->pending_reply, DBUS_TYPE_INVALID);
+               dbus_message_unref(technology->pending_reply);
+               technology->pending_reply = NULL;
+
+               g_source_remove(technology->pending_timeout);
+               technology->pending_timeout = 0;
+       }
+
+       __sync_synchronize();
+       enabled = technology->enabled;
+       connman_dbus_property_changed_basic(technology->path,
+                       CONNMAN_TECHNOLOGY_INTERFACE, "Powered",
+                       DBUS_TYPE_BOOLEAN, &enabled);
+}
+
+static void enable_tethering(struct connman_technology *technology)
+{
+       int ret;
+
+       if (!connman_setting_get_bool("PersistentTetheringMode"))
+               return;
+
+       ret = set_tethering(technology, true);
+       if (ret < 0 && ret != -EALREADY)
+               DBG("Cannot enable tethering yet for %s (%d/%s)",
+                       get_name(technology->type),
+                       -ret, strerror(-ret));
+}
+
+static int technology_enabled(struct connman_technology *technology)
+{
+       __sync_synchronize();
+       if (technology->enabled)
+               return -EALREADY;
+
+       technology->enabled = true;
+
+       if (technology->type == CONNMAN_SERVICE_TYPE_WIFI) {
+               struct connman_technology *p2p;
+
+               p2p = technology_find(CONNMAN_SERVICE_TYPE_P2P);
+               if (p2p && !p2p->enabled && p2p->enable_persistent)
+                       technology_enabled(p2p);
+       }
+
+       if (technology->tethering_persistent)
+               enable_tethering(technology);
+
+       powered_changed(technology);
+
+       return 0;
+}
+
 static int technology_enable(struct connman_technology *technology)
 {
        int err = 0;
@@ -633,23 +714,50 @@ static int technology_enable(struct connman_technology *technology)
        DBG("technology %p enable", technology);
 
        __sync_synchronize();
-       if (technology->enabled == TRUE)
+
+       if (technology->type == CONNMAN_SERVICE_TYPE_P2P) {
+               struct connman_technology *wifi;
+
+               wifi = technology_find(CONNMAN_SERVICE_TYPE_WIFI);
+               if (wifi && wifi->enabled)
+                       return technology_enabled(technology);
+               return 0;
+       }
+
+       if (technology->enabled)
                return -EALREADY;
 
-       if (technology->pending_reply != NULL)
+       if (technology->pending_reply)
                return -EBUSY;
 
-       if (technology->rfkill_driven == TRUE)
-               err = __connman_rfkill_block(technology->type, FALSE);
+       if (connman_setting_get_bool("PersistentTetheringMode") &&
+                                       technology->tethering)
+               set_tethering(technology, true);
+
+       if (technology->rfkill_driven)
+               err = __connman_rfkill_block(technology->type, false);
 
-       err_dev = technology_affect_devices(technology, TRUE);
+       err_dev = technology_affect_devices(technology, true);
 
-       if (technology->rfkill_driven == FALSE)
+       if (!technology->rfkill_driven)
                err = err_dev;
 
        return err;
 }
 
+static int technology_disabled(struct connman_technology *technology)
+{
+       __sync_synchronize();
+       if (!technology->enabled)
+               return -EALREADY;
+
+       technology->enabled = false;
+
+       powered_changed(technology);
+
+       return 0;
+}
+
 static int technology_disable(struct connman_technology *technology)
 {
        int err;
@@ -657,35 +765,49 @@ static int technology_disable(struct connman_technology *technology)
        DBG("technology %p disable", technology);
 
        __sync_synchronize();
-       if (technology->enabled == FALSE)
+
+       if (technology->type == CONNMAN_SERVICE_TYPE_P2P) {
+               technology->enable_persistent = false;
+               return technology_disabled(technology);
+       } else if (technology->type == CONNMAN_SERVICE_TYPE_WIFI) {
+               struct connman_technology *p2p;
+
+               p2p = technology_find(CONNMAN_SERVICE_TYPE_P2P);
+               if (p2p && p2p->enabled) {
+                       p2p->enable_persistent = true;
+                       technology_disabled(p2p);
+               }
+       }
+
+       if (!technology->enabled)
                return -EALREADY;
 
-       if (technology->pending_reply != NULL)
+       if (technology->pending_reply)
                return -EBUSY;
 
-       if (technology->tethering == TRUE)
-               set_tethering(technology, FALSE);
+       if (technology->tethering)
+               set_tethering(technology, false);
 
-       err = technology_affect_devices(technology, FALSE);
+       err = technology_affect_devices(technology, false);
 
-       if (technology->rfkill_driven == TRUE)
-               err = __connman_rfkill_block(technology->type, TRUE);
+       if (technology->rfkill_driven)
+               err = __connman_rfkill_block(technology->type, true);
 
        return err;
 }
 
 static DBusMessage *set_powered(struct connman_technology *technology,
-                               DBusMessage *msg, connman_bool_t powered)
+                               DBusMessage *msg, bool powered)
 {
        DBusMessage *reply = NULL;
        int err = 0;
 
-       if (technology->rfkill_driven && technology->hardblocked == TRUE) {
+       if (technology->rfkill_driven && technology->hardblocked) {
                err = -EACCES;
                goto make_reply;
        }
 
-       if (powered == TRUE)
+       if (powered)
                err = technology_enable(technology);
        else
                err = technology_disable(technology);
@@ -701,7 +823,7 @@ make_reply:
                technology->pending_timeout = g_timeout_add_seconds(10,
                                        technology_pending_reply, technology);
        } else if (err == -EALREADY) {
-               if (powered == TRUE)
+               if (powered)
                        reply = __connman_error_already_enabled(msg);
                else
                        reply = __connman_error_already_disabled(msg);
@@ -723,7 +845,7 @@ static DBusMessage *set_property(DBusConnection *conn,
 
        DBG("conn %p", conn);
 
-       if (dbus_message_iter_init(msg, &iter) == FALSE)
+       if (!dbus_message_iter_init(msg, &iter))
                return __connman_error_invalid_arguments(msg);
 
        if (dbus_message_iter_get_arg_type(&iter) != DBUS_TYPE_STRING)
@@ -741,17 +863,23 @@ static DBusMessage *set_property(DBusConnection *conn,
 
        DBG("property %s", name);
 
-       if (g_str_equal(name, "Tethering") == TRUE) {
+       if (g_str_equal(name, "Tethering")) {
+               dbus_bool_t tethering;
                int err;
-               connman_bool_t tethering;
 
                if (type != DBUS_TYPE_BOOLEAN)
                        return __connman_error_invalid_arguments(msg);
 
+               if (!connman_technology_is_tethering_allowed(technology->type)) {
+                       DBG("%s tethering not allowed by config file",
+                               __connman_service_type2string(technology->type));
+                       return __connman_error_not_supported(msg);
+               }
+
                dbus_message_iter_get_basic(&value, &tethering);
 
                if (technology->tethering == tethering) {
-                       if (tethering == FALSE)
+                       if (!tethering)
                                return __connman_error_already_disabled(msg);
                        else
                                return __connman_error_already_enabled(msg);
@@ -761,7 +889,11 @@ static DBusMessage *set_property(DBusConnection *conn,
                if (err < 0)
                        return __connman_error_failed(msg, -err);
 
-       } else if (g_str_equal(name, "TetheringIdentifier") == TRUE) {
+               technology->tethering_persistent = tethering;
+
+               technology_save(technology);
+
+       } else if (g_str_equal(name, "TetheringIdentifier")) {
                const char *str;
 
                dbus_message_iter_get_basic(&value, &str);
@@ -783,7 +915,7 @@ static DBusMessage *set_property(DBusConnection *conn,
                                                DBUS_TYPE_STRING,
                                                &technology->tethering_ident);
                }
-       } else if (g_str_equal(name, "TetheringPassphrase") == TRUE) {
+       } else if (g_str_equal(name, "TetheringPassphrase")) {
                const char *str;
 
                dbus_message_iter_get_basic(&value, &str);
@@ -805,8 +937,8 @@ static DBusMessage *set_property(DBusConnection *conn,
                                        DBUS_TYPE_STRING,
                                        &technology->tethering_passphrase);
                }
-       } else if (g_str_equal(name, "Powered") == TRUE) {
-               connman_bool_t enable;
+       } else if (g_str_equal(name, "Powered")) {
+               dbus_bool_t enable;
 
                if (type != DBUS_TYPE_BOOLEAN)
                        return __connman_error_invalid_arguments(msg);
@@ -820,29 +952,13 @@ static DBusMessage *set_property(DBusConnection *conn,
        return g_dbus_create_reply(msg, DBUS_TYPE_INVALID);
 }
 
-static struct connman_technology *technology_find(enum connman_service_type type)
-{
-       GSList *list;
-
-       DBG("type %d", type);
-
-       for (list = technology_list; list; list = list->next) {
-               struct connman_technology *technology = list->data;
-
-               if (technology->type == type)
-                       return technology;
-       }
-
-       return NULL;
-}
-
 static void reply_scan_pending(struct connman_technology *technology, int err)
 {
        DBusMessage *reply;
 
        DBG("technology %p err %d", technology, err);
 
-       while (technology->scan_pending != NULL) {
+       while (technology->scan_pending) {
                DBusMessage *msg = technology->scan_pending->data;
 
                DBG("reply to %s", dbus_message_get_sender(msg));
@@ -865,22 +981,21 @@ void __connman_technology_scan_started(struct connman_device *device)
        DBG("device %p", device);
 }
 
-void __connman_technology_scan_stopped(struct connman_device *device)
+void __connman_technology_scan_stopped(struct connman_device *device,
+                                       enum connman_service_type type)
 {
        int count = 0;
        struct connman_technology *technology;
-       enum connman_service_type type;
        GSList *list;
 
-       type = __connman_device_get_service_type(device);
        technology = technology_find(type);
 
        DBG("technology %p device %p", technology, device);
 
-       if (technology == NULL)
+       if (!technology)
                return;
 
-       for (list = technology->device_list; list != NULL; list = list->next) {
+       for (list = technology->device_list; list; list = list->next) {
                struct connman_device *other_device = list->data;
 
                if (device == other_device)
@@ -889,7 +1004,7 @@ void __connman_technology_scan_stopped(struct connman_device *device)
                if (__connman_device_get_service_type(other_device) != type)
                        continue;
 
-               if (connman_device_get_scanning(other_device) == TRUE)
+               if (connman_device_get_scanning(other_device))
                        count += 1;
        }
 
@@ -900,7 +1015,7 @@ void __connman_technology_scan_stopped(struct connman_device *device)
 void __connman_technology_notify_regdom_by_device(struct connman_device *device,
                                                int result, const char *alpha2)
 {
-       connman_bool_t regdom_set = FALSE;
+       bool regdom_set = false;
        struct connman_technology *technology;
        enum connman_service_type type;
        GSList *tech_drivers;
@@ -908,25 +1023,25 @@ void __connman_technology_notify_regdom_by_device(struct connman_device *device,
        type = __connman_device_get_service_type(device);
        technology = technology_find(type);
 
-       if (technology == NULL)
+       if (!technology)
                return;
 
        if (result < 0) {
 
                for (tech_drivers = technology->driver_list;
-                    tech_drivers != NULL;
+                    tech_drivers;
                     tech_drivers = g_slist_next(tech_drivers)) {
                        struct connman_technology_driver *driver =
                                tech_drivers->data;
 
-                       if (driver->set_regdom != NULL) {
+                       if (driver->set_regdom) {
                                driver->set_regdom(technology, alpha2);
-                               regdom_set = TRUE;
+                               regdom_set = true;
                        }
 
                }
 
-               if (regdom_set == FALSE)
+               if (!regdom_set)
                        alpha2 = NULL;
        }
 
@@ -938,7 +1053,7 @@ static DBusMessage *scan(DBusConnection *conn, DBusMessage *msg, void *data)
        struct connman_technology *technology = data;
        int err;
 
-       DBG ("technology %p request from %s", technology,
+       DBG("technology %p request from %s", technology,
                        dbus_message_get_sender(msg));
 
        dbus_message_ref(msg);
@@ -969,25 +1084,72 @@ static const GDBusSignalTable technology_signals[] = {
        { },
 };
 
-static gboolean technology_dbus_register(struct connman_technology *technology)
+static bool technology_dbus_register(struct connman_technology *technology)
 {
-       if (technology->dbus_registered == TRUE ||
-                               (technology->rfkill_driven == TRUE &&
-                                technology->hardblocked == TRUE))
-               return TRUE;
-
-       if (g_dbus_register_interface(connection, technology->path,
-                               CONNMAN_TECHNOLOGY_INTERFACE,
-                               technology_methods, technology_signals,
-                               NULL, technology, NULL) == FALSE) {
+       if (technology->dbus_registered ||
+                               (technology->rfkill_driven &&
+                                technology->hardblocked))
+               return true;
+
+       if (!g_dbus_register_interface(connection, technology->path,
+                                       CONNMAN_TECHNOLOGY_INTERFACE,
+                                       technology_methods, technology_signals,
+                                       NULL, technology, NULL)) {
                connman_error("Failed to register %s", technology->path);
-               return FALSE;
+               return false;
        }
 
        technology_added_signal(technology);
-       technology->dbus_registered = TRUE;
+       technology->dbus_registered = true;
+
+       return true;
+}
+
+static void technology_dbus_unregister(struct connman_technology *technology)
+{
+       if (!technology->dbus_registered)
+               return;
+
+       technology_removed_signal(technology);
+       g_dbus_unregister_interface(connection, technology->path,
+               CONNMAN_TECHNOLOGY_INTERFACE);
+
+       technology->dbus_registered = false;
+}
+
+static void technology_put(struct connman_technology *technology)
+{
+       DBG("technology %p", technology);
+
+       if (__sync_sub_and_fetch(&technology->refcount, 1) > 0)
+               return;
+
+       reply_scan_pending(technology, -EINTR);
+
+       while (technology->driver_list) {
+               struct connman_technology_driver *driver;
+
+               driver = technology->driver_list->data;
+
+               if (driver->remove)
+                       driver->remove(technology);
+
+               technology->driver_list =
+                       g_slist_delete_link(technology->driver_list,
+                                       technology->driver_list);
+       }
+
+       technology_list = g_slist_remove(technology_list, technology);
+
+       technology_dbus_unregister(technology);
 
-       return TRUE;
+       g_slist_free(technology->device_list);
+
+       g_free(technology->path);
+       g_free(technology->regdom);
+       g_free(technology->tethering_ident);
+       g_free(technology->tethering_passphrase);
+       g_free(technology);
 }
 
 static struct connman_technology *technology_get(enum connman_service_type type)
@@ -1001,12 +1163,13 @@ static struct connman_technology *technology_get(enum connman_service_type type)
        DBG("type %d", type);
 
        str = __connman_service_type2string(type);
-       if (str == NULL)
+       if (!str)
                return NULL;
 
        technology = technology_find(type);
-       if (technology != NULL) {
-               __sync_fetch_and_add(&technology->refcount, 1);
+       if (technology) {
+               if (type != CONNMAN_SERVICE_TYPE_P2P)
+                       __sync_fetch_and_add(&technology->refcount, 1);
                return technology;
        }
 
@@ -1020,107 +1183,145 @@ static struct connman_technology *technology_get(enum connman_service_type type)
                }
        }
 
-       if (tech_drivers == NULL) {
+       if (!tech_drivers) {
                DBG("No matching drivers found for %s.",
                                __connman_service_type2string(type));
                return NULL;
        }
 
        technology = g_try_new0(struct connman_technology, 1);
-       if (technology == NULL)
+       if (!technology)
                return NULL;
 
        technology->refcount = 1;
-
-       technology->rfkill_driven = FALSE;
-       technology->softblocked = FALSE;
-       technology->hardblocked = FALSE;
-
        technology->type = type;
        technology->path = g_strdup_printf("%s/technology/%s",
                                                        CONNMAN_PATH, str);
 
-       technology->device_list = NULL;
+       technology_load(technology);
+       technology_list = g_slist_prepend(technology_list, technology);
+       technology->driver_list = tech_drivers;
 
-       technology->pending_reply = NULL;
+       for (list = tech_drivers; list; list = list->next) {
+               driver = list->data;
 
-       technology_load(technology);
+               if (driver->probe && driver->probe(technology) < 0)
+                       DBG("Driver probe failed for technology %p",
+                                       technology);
+       }
 
-       if (technology_dbus_register(technology) == FALSE) {
-               g_free(technology);
+       if (!technology_dbus_register(technology)) {
+               technology_put(technology);
                return NULL;
        }
 
-       technology_list = g_slist_prepend(technology_list, technology);
+       if (type == CONNMAN_SERVICE_TYPE_P2P) {
+               struct connman_technology *wifi;
+               bool enable;
 
-       technology->driver_list = tech_drivers;
+               enable = technology->enable_persistent;
 
-       for (list = tech_drivers; list != NULL; list = g_slist_next(list)) {
-               driver = list->data;
+               wifi = technology_find(CONNMAN_SERVICE_TYPE_WIFI);
+               if (enable && wifi)
+                       enable = wifi->enabled;
 
-               if (driver->probe != NULL && driver->probe(technology) < 0)
-                       DBG("Driver probe failed for technology %p",
-                                       technology);
+               technology_affect_devices(technology, enable);
        }
 
-       DBG("technology %p", technology);
+       DBG("technology %p %s", technology, get_name(technology->type));
 
        return technology;
 }
 
-static void technology_dbus_unregister(struct connman_technology *technology)
+int connman_technology_driver_register(struct connman_technology_driver *driver)
 {
-       if (technology->dbus_registered == FALSE)
-               return;
+       GSList *list;
+       struct connman_device *device;
+       enum connman_service_type type;
 
-       technology_removed_signal(technology);
-       g_dbus_unregister_interface(connection, technology->path,
-               CONNMAN_TECHNOLOGY_INTERFACE);
+       for (list = driver_list; list; list = list->next) {
+               if (list->data == driver)
+                       goto exist;
+       }
 
-       technology->dbus_registered = FALSE;
-}
+       DBG("Registering %s driver", driver->name);
 
-static void technology_put(struct connman_technology *technology)
-{
-       DBG("technology %p", technology);
+       driver_list = g_slist_insert_sorted(driver_list, driver,
+                                                       compare_priority);
 
-       if (__sync_sub_and_fetch(&technology->refcount, 1) > 0)
-               return;
+       /*
+        * Check for technology less devices if this driver
+        * can service any of them.
+       */
+       for (list = techless_device_list; list; list = list->next) {
+               device = list->data;
 
-       reply_scan_pending(technology, -EINTR);
+               type = __connman_device_get_service_type(device);
+               if (type != driver->type)
+                       continue;
 
-       while (technology->driver_list != NULL) {
-               struct connman_technology_driver *driver;
+               techless_device_list = g_slist_remove(techless_device_list,
+                                                               device);
 
-               driver = technology->driver_list->data;
+               __connman_technology_add_device(device);
+       }
 
-               if (driver->remove != NULL)
-                       driver->remove(technology);
+       /* Check for orphaned rfkill switches. */
+       g_hash_table_foreach(rfkill_list, rfkill_check,
+                                       GINT_TO_POINTER(driver->type));
 
-               technology->driver_list =
-                       g_slist_delete_link(technology->driver_list,
-                                       technology->driver_list);
+exist:
+       if (driver->type == CONNMAN_SERVICE_TYPE_P2P) {
+               if (!technology_get(CONNMAN_SERVICE_TYPE_P2P))
+                       return -ENOMEM;
        }
 
-       technology_list = g_slist_remove(technology_list, technology);
+       return 0;
+}
 
-       technology_dbus_unregister(technology);
+void connman_technology_driver_unregister(struct connman_technology_driver *driver)
+{
+       GSList *list, *tech_drivers;
+       struct connman_technology *technology;
+       struct connman_technology_driver *current;
 
-       g_slist_free(technology->device_list);
+       DBG("Unregistering driver %p name %s", driver, driver->name);
 
-       g_free(technology->path);
-       g_free(technology->regdom);
-       g_free(technology->tethering_ident);
-       g_free(technology->tethering_passphrase);
-       g_free(technology);
+       for (list = technology_list; list; list = list->next) {
+               technology = list->data;
+
+               for (tech_drivers = technology->driver_list; tech_drivers;
+                               tech_drivers = g_slist_next(tech_drivers)) {
+                       current = tech_drivers->data;
+                       if (driver != current)
+                               continue;
+
+                       if (driver->remove)
+                               driver->remove(technology);
+
+                       technology->driver_list =
+                               g_slist_remove(technology->driver_list,
+                                                               driver);
+                       break;
+               }
+       }
+
+       driver_list = g_slist_remove(driver_list, driver);
+
+       if (driver->type == CONNMAN_SERVICE_TYPE_P2P) {
+               technology = technology_find(CONNMAN_SERVICE_TYPE_P2P);
+               if (technology)
+                       technology_put(technology);
+       }
 }
 
 void __connman_technology_add_interface(enum connman_service_type type,
-                               int index, const char *name, const char *ident)
+                               int index, const char *ident)
 {
        struct connman_technology *technology;
        GSList *tech_drivers;
        struct connman_technology_driver *driver;
+       char *name;
 
        switch (type) {
        case CONNMAN_SERVICE_TYPE_UNKNOWN:
@@ -1133,32 +1334,45 @@ void __connman_technology_add_interface(enum connman_service_type type,
        case CONNMAN_SERVICE_TYPE_GPS:
        case CONNMAN_SERVICE_TYPE_VPN:
        case CONNMAN_SERVICE_TYPE_GADGET:
+       case CONNMAN_SERVICE_TYPE_P2P:
                break;
        }
 
+       name = connman_inet_ifname(index);
        connman_info("Adding interface %s [ %s ]", name,
                                __connman_service_type2string(type));
 
        technology = technology_find(type);
 
-       if (technology == NULL)
-               return;
+       if (!technology)
+               goto out;
 
-       for (tech_drivers = technology->driver_list; tech_drivers != NULL;
+       for (tech_drivers = technology->driver_list; tech_drivers;
             tech_drivers = g_slist_next(tech_drivers)) {
                driver = tech_drivers->data;
 
-               if(driver->add_interface != NULL)
+               if (driver->add_interface)
                        driver->add_interface(technology, index, name, ident);
        }
+
+       /*
+        * At this point we can try to enable tethering automatically as
+        * now the interfaces are set properly.
+        */
+       if (technology->tethering_persistent)
+               enable_tethering(technology);
+
+out:
+       g_free(name);
 }
 
 void __connman_technology_remove_interface(enum connman_service_type type,
-                               int index, const char *name, const char *ident)
+                               int index, const char *ident)
 {
        struct connman_technology *technology;
        GSList *tech_drivers;
        struct connman_technology_driver *driver;
+       char *name;
 
        switch (type) {
        case CONNMAN_SERVICE_TYPE_UNKNOWN:
@@ -1171,22 +1385,25 @@ void __connman_technology_remove_interface(enum connman_service_type type,
        case CONNMAN_SERVICE_TYPE_GPS:
        case CONNMAN_SERVICE_TYPE_VPN:
        case CONNMAN_SERVICE_TYPE_GADGET:
+       case CONNMAN_SERVICE_TYPE_P2P:
                break;
        }
 
+       name = connman_inet_ifname(index);
        connman_info("Remove interface %s [ %s ]", name,
                                __connman_service_type2string(type));
+       g_free(name);
 
        technology = technology_find(type);
 
-       if (technology == NULL)
+       if (!technology)
                return;
 
-       for (tech_drivers = technology->driver_list; tech_drivers != NULL;
+       for (tech_drivers = technology->driver_list; tech_drivers;
             tech_drivers = g_slist_next(tech_drivers)) {
                driver = tech_drivers->data;
 
-               if(driver->remove_interface != NULL)
+               if (driver->remove_interface)
                        driver->remove_interface(technology, index);
        }
 }
@@ -1196,12 +1413,12 @@ int __connman_technology_add_device(struct connman_device *device)
        struct connman_technology *technology;
        enum connman_service_type type;
 
-       DBG("device %p", device);
-
        type = __connman_device_get_service_type(device);
 
+       DBG("device %p type %s", device, get_name(type));
+
        technology = technology_get(type);
-       if (technology == NULL) {
+       if (!technology) {
                /*
                 * Since no driver can be found for this device at the moment we
                 * add it to the techless device list.
@@ -1213,8 +1430,8 @@ int __connman_technology_add_device(struct connman_device *device)
        }
 
        __sync_synchronize();
-       if (technology->rfkill_driven == TRUE) {
-               if (technology->enabled == TRUE)
+       if (technology->rfkill_driven) {
+               if (technology->enabled)
                        __connman_device_enable(device);
                else
                        __connman_device_disable(device);
@@ -1222,8 +1439,8 @@ int __connman_technology_add_device(struct connman_device *device)
                goto done;
        }
 
-       if (technology->enable_persistent == TRUE &&
-                                       global_offlinemode == FALSE) {
+       if (technology->enable_persistent &&
+                                       !global_offlinemode) {
                int err = __connman_device_enable(device);
                /*
                 * connman_technology_add_device() calls __connman_device_enable()
@@ -1235,7 +1452,7 @@ int __connman_technology_add_device(struct connman_device *device)
                        __connman_technology_enabled(type);
        }
        /* if technology persistent state is offline */
-       if (technology->enable_persistent == FALSE)
+       if (!technology->enable_persistent)
                __connman_device_disable(device);
 
 done:
@@ -1255,7 +1472,7 @@ int __connman_technology_remove_device(struct connman_device *device)
        type = __connman_device_get_service_type(device);
 
        technology = technology_find(type);
-       if (technology == NULL) {
+       if (!technology) {
                techless_device_list = g_slist_remove(techless_device_list,
                                                                device);
                return -ENXIO;
@@ -1263,41 +1480,11 @@ int __connman_technology_remove_device(struct connman_device *device)
 
        technology->device_list = g_slist_remove(technology->device_list,
                                                                device);
-       technology_put(technology);
-
-       return 0;
-}
 
-static void powered_changed(struct connman_technology *technology)
-{
-       if (technology->dbus_registered == FALSE)
-               return;
+       if (technology->tethering)
+               set_tethering(technology, false);
 
-       if (technology->pending_reply != NULL) {
-               g_dbus_send_reply(connection,
-                               technology->pending_reply, DBUS_TYPE_INVALID);
-               dbus_message_unref(technology->pending_reply);
-               technology->pending_reply = NULL;
-
-               g_source_remove(technology->pending_timeout);
-               technology->pending_timeout = 0;
-       }
-
-       __sync_synchronize();
-       connman_dbus_property_changed_basic(technology->path,
-                       CONNMAN_TECHNOLOGY_INTERFACE, "Powered",
-                       DBUS_TYPE_BOOLEAN, &technology->enabled);
-}
-
-static int technology_enabled(struct connman_technology *technology)
-{
-       __sync_synchronize();
-       if (technology->enabled == TRUE)
-               return -EALREADY;
-
-       technology->enabled = TRUE;
-
-       powered_changed(technology);
+       technology_put(technology);
 
        return 0;
 }
@@ -1307,54 +1494,48 @@ int __connman_technology_enabled(enum connman_service_type type)
        struct connman_technology *technology;
 
        technology = technology_find(type);
-       if (technology == NULL)
+       if (!technology)
                return -ENXIO;
 
-       if (technology->rfkill_driven == TRUE)
+       DBG("technology %p type %s rfkill %d enabled %d", technology,
+               get_name(type), technology->rfkill_driven,
+               technology->enabled);
+
+       if (technology->rfkill_driven) {
+               if (technology->tethering_persistent)
+                       enable_tethering(technology);
                return 0;
+       }
 
        return technology_enabled(technology);
 }
 
-static int technology_disabled(struct connman_technology *technology)
-{
-       __sync_synchronize();
-       if (technology->enabled == FALSE)
-               return -EALREADY;
-
-       technology->enabled = FALSE;
-
-       powered_changed(technology);
-
-       return 0;
-}
-
 int __connman_technology_disabled(enum connman_service_type type)
 {
        struct connman_technology *technology;
        GSList *list;
 
        technology = technology_find(type);
-       if (technology == NULL)
+       if (!technology)
                return -ENXIO;
 
-       if (technology->rfkill_driven == TRUE)
+       if (technology->rfkill_driven)
                return 0;
 
-       for (list = technology->device_list; list != NULL; list = list->next) {
+       for (list = technology->device_list; list; list = list->next) {
                struct connman_device *device = list->data;
 
-               if (connman_device_get_powered(device) == TRUE)
+               if (connman_device_get_powered(device))
                        return 0;
        }
 
        return technology_disabled(technology);
 }
 
-int __connman_technology_set_offlinemode(connman_bool_t offlinemode)
+int __connman_technology_set_offlinemode(bool offlinemode)
 {
        GSList *list;
-       int err = -EINVAL;
+       int err = -EINVAL, enabled_tech_count = 0;
 
        if (global_offlinemode == offlinemode)
                return 0;
@@ -1378,12 +1559,19 @@ int __connman_technology_set_offlinemode(connman_bool_t offlinemode)
 
                if (offlinemode)
                        err = technology_disable(technology);
+               else {
+                       if (technology->hardblocked)
+                               continue;
 
-               if (!offlinemode && technology->enable_persistent)
-                       err = technology_enable(technology);
+                       if (technology->enable_persistent) {
+                               err = technology_enable(technology);
+                               enabled_tech_count++;
+                       }
+               }
        }
 
-       if (err == 0 || err == -EINPROGRESS || err == -EALREADY) {
+       if (err == 0 || err == -EINPROGRESS || err == -EALREADY ||
+                       (err == -EINVAL && enabled_tech_count == 0)) {
                connman_technology_save_offlinemode();
                __connman_notifier_offlinemode(offlinemode);
        } else
@@ -1393,30 +1581,32 @@ int __connman_technology_set_offlinemode(connman_bool_t offlinemode)
 }
 
 void __connman_technology_set_connected(enum connman_service_type type,
-               connman_bool_t connected)
+               bool connected)
 {
        struct connman_technology *technology;
+       dbus_bool_t val;
 
        technology = technology_find(type);
-       if (technology == NULL)
+       if (!technology)
                return;
 
        DBG("technology %p connected %d", technology, connected);
 
        technology->connected = connected;
 
+       val = connected;
        connman_dbus_property_changed_basic(technology->path,
                        CONNMAN_TECHNOLOGY_INTERFACE, "Connected",
-                       DBUS_TYPE_BOOLEAN, &connected);
+                       DBUS_TYPE_BOOLEAN, &val);
 }
 
-static connman_bool_t technology_apply_rfkill_change(struct connman_technology *technology,
-                                               connman_bool_t softblock,
-                                               connman_bool_t hardblock,
-                                               connman_bool_t new_rfkill)
+static bool technology_apply_rfkill_change(struct connman_technology *technology,
+                                               bool softblock,
+                                               bool hardblock,
+                                               bool new_rfkill)
 {
-       gboolean hardblock_changed = FALSE;
-       gboolean apply = TRUE;
+       bool hardblock_changed = false;
+       bool apply = true;
        GList *start, *list;
 
        DBG("technology %p --> %d/%d vs %d/%d",
@@ -1426,53 +1616,58 @@ static connman_bool_t technology_apply_rfkill_change(struct connman_technology *
        if (technology->hardblocked == hardblock)
                goto softblock_change;
 
-       if (!(new_rfkill == TRUE && hardblock == FALSE)) {
+       if (!(new_rfkill && !hardblock)) {
                start = g_hash_table_get_values(rfkill_list);
 
-               for (list = start; list != NULL; list = list->next) {
+               for (list = start; list; list = list->next) {
                        struct connman_rfkill *rfkill = list->data;
 
                        if (rfkill->type != technology->type)
                                continue;
 
                        if (rfkill->hardblock != hardblock)
-                               apply = FALSE;
+                               apply = false;
                }
 
                g_list_free(start);
        }
 
-       if (apply == FALSE)
+       if (!apply)
                goto softblock_change;
 
        technology->hardblocked = hardblock;
-       hardblock_changed = TRUE;
+       hardblock_changed = true;
 
 softblock_change:
-       if (apply == FALSE && technology->softblocked != softblock)
-               apply = TRUE;
+       if (!apply && technology->softblocked != softblock)
+               apply = true;
 
-       if (apply == FALSE)
+       if (!apply)
                return technology->hardblocked;
 
        technology->softblocked = softblock;
 
-       if (technology->hardblocked == TRUE ||
-                                       technology->softblocked == TRUE) {
+       if (technology->hardblocked ||
+                                       technology->softblocked) {
                if (technology_disabled(technology) != -EALREADY)
-                       technology_affect_devices(technology, FALSE);
-       } else if (technology->hardblocked == FALSE &&
-                                       technology->softblocked == FALSE) {
+                       technology_affect_devices(technology, false);
+       } else if (!technology->hardblocked &&
+                                       !technology->softblocked) {
                if (technology_enabled(technology) != -EALREADY)
-                       technology_affect_devices(technology, TRUE);
+                       technology_affect_devices(technology, true);
        }
 
-       if (hardblock_changed == TRUE) {
-               if (technology->hardblocked == TRUE) {
+       if (hardblock_changed) {
+               if (technology->hardblocked) {
                        DBG("%s is switched off.", get_name(technology->type));
                        technology_dbus_unregister(technology);
-               } else
+               } else {
+                       DBG("%s is switched on.", get_name(technology->type));
                        technology_dbus_register(technology);
+
+                       if (global_offlinemode)
+                               __connman_rfkill_block(technology->type, true);
+               }
        }
 
        return technology->hardblocked;
@@ -1480,8 +1675,8 @@ softblock_change:
 
 int __connman_technology_add_rfkill(unsigned int index,
                                        enum connman_service_type type,
-                                               connman_bool_t softblock,
-                                               connman_bool_t hardblock)
+                                               bool softblock,
+                                               bool hardblock)
 {
        struct connman_technology *technology;
        struct connman_rfkill *rfkill;
@@ -1490,11 +1685,11 @@ int __connman_technology_add_rfkill(unsigned int index,
                                                        softblock, hardblock);
 
        rfkill = g_hash_table_lookup(rfkill_list, GINT_TO_POINTER(index));
-       if (rfkill != NULL)
+       if (rfkill)
                goto done;
 
        rfkill = g_try_new0(struct connman_rfkill, 1);
-       if (rfkill == NULL)
+       if (!rfkill)
                return -ENOMEM;
 
        rfkill->index = index;
@@ -1507,36 +1702,37 @@ int __connman_technology_add_rfkill(unsigned int index,
 done:
        technology = technology_get(type);
        /* If there is no driver for this type, ignore it. */
-       if (technology == NULL)
+       if (!technology)
                return -ENXIO;
 
-       technology->rfkill_driven = TRUE;
+       technology->rfkill_driven = true;
 
        /* If hardblocked, there is no need to handle softblocked state */
        if (technology_apply_rfkill_change(technology,
-                               softblock, hardblock, TRUE) == TRUE)
+                               softblock, hardblock, true))
+               return 0;
+
+       if (global_offlinemode)
                return 0;
 
        /*
         * Depending on softblocked state we unblock/block according to
         * offlinemode and persistente state.
         */
-       if (technology->softblocked == TRUE &&
-                               global_offlinemode == FALSE &&
-                               technology->enable_persistent == TRUE)
-               return __connman_rfkill_block(type, FALSE);
-       else if (technology->softblocked == FALSE &&
-                       (global_offlinemode == TRUE ||
-                               technology->enable_persistent == FALSE))
-               return __connman_rfkill_block(type, TRUE);
+       if (technology->softblocked &&
+                               technology->enable_persistent)
+               return __connman_rfkill_block(type, false);
+       else if (!technology->softblocked &&
+                               !technology->enable_persistent)
+               return __connman_rfkill_block(type, true);
 
        return 0;
 }
 
 int __connman_technology_update_rfkill(unsigned int index,
                                        enum connman_service_type type,
-                                               connman_bool_t softblock,
-                                               connman_bool_t hardblock)
+                                               bool softblock,
+                                               bool hardblock)
 {
        struct connman_technology *technology;
        struct connman_rfkill *rfkill;
@@ -1544,7 +1740,7 @@ int __connman_technology_update_rfkill(unsigned int index,
        DBG("index %u soft %u hard %u", index, softblock, hardblock);
 
        rfkill = g_hash_table_lookup(rfkill_list, GINT_TO_POINTER(index));
-       if (rfkill == NULL)
+       if (!rfkill)
                return -ENXIO;
 
        if (rfkill->softblock == softblock &&
@@ -1556,27 +1752,17 @@ int __connman_technology_update_rfkill(unsigned int index,
 
        technology = technology_find(type);
        /* If there is no driver for this type, ignore it. */
-       if (technology == NULL)
+       if (!technology)
                return -ENXIO;
 
-       /* If hardblocked, there is no need to handle softblocked state */
-       if (technology_apply_rfkill_change(technology,
-                               softblock, hardblock, FALSE) == TRUE)
-               return 0;
-
-       if (global_offlinemode == TRUE)
-               return 0;
+       technology_apply_rfkill_change(technology, softblock, hardblock,
+                                                               false);
 
-       /*
-        * Depending on softblocked state we unblock/block according to
-        * persistent state.
-        */
-       if (technology->softblocked == TRUE &&
-                               technology->enable_persistent == TRUE)
-               return __connman_rfkill_block(type, FALSE);
-       else if (technology->softblocked == FALSE &&
-                               technology->enable_persistent == FALSE)
-               return __connman_rfkill_block(type, TRUE);
+       if (technology->hardblocked)
+               DBG("%s hardblocked", get_name(technology->type));
+       else
+               DBG("%s is%s softblocked", get_name(technology->type),
+                       technology->softblocked ? "" : " not");
 
        return 0;
 }
@@ -1590,17 +1776,17 @@ int __connman_technology_remove_rfkill(unsigned int index,
        DBG("index %u", index);
 
        rfkill = g_hash_table_lookup(rfkill_list, GINT_TO_POINTER(index));
-       if (rfkill == NULL)
+       if (!rfkill)
                return -ENXIO;
 
        g_hash_table_remove(rfkill_list, GINT_TO_POINTER(index));
 
        technology = technology_find(type);
-       if (technology == NULL)
+       if (!technology)
                return -ENXIO;
 
        technology_apply_rfkill_change(technology,
-               technology->softblocked, !technology->hardblocked, FALSE);
+               technology->softblocked, !technology->hardblocked, false);
 
        technology_put(technology);
 
@@ -1628,6 +1814,12 @@ void __connman_technology_cleanup(void)
 {
        DBG("");
 
+       while (technology_list) {
+               struct connman_technology *technology = technology_list->data;
+               technology_list = g_slist_remove(technology_list, technology);
+               technology_put(technology);
+       }
+
        g_hash_table_destroy(rfkill_list);
 
        dbus_connection_unref(connection);