core: Remove WiMAX definitions from code

[platform/upstream/connman.git] / src / technology.c

diff --git a/src/technology.c b/src/technology.c
index 44eeb13..8166d17 100644 (file)
--- a/src/technology.c
+++ b/src/technology.c
@@ -223,6 +223,10 @@ static int set_tethering(struct connman_technology *technology,
                        technology->driver->set_tethering == NULL)
                return -EOPNOTSUPP;
 
+       __sync_synchronize();
+       if (technology->enabled == FALSE)
+               return -EACCES;
+
        bridge = __connman_tethering_get_bridge();
        if (bridge == NULL)
                return -EOPNOTSUPP;
@@ -304,8 +308,6 @@ static const char *get_name(enum connman_service_type type)
                return "Wired";
        case CONNMAN_SERVICE_TYPE_WIFI:
                return "WiFi";
-       case CONNMAN_SERVICE_TYPE_WIMAX:
-               return "WiMAX";
        case CONNMAN_SERVICE_TYPE_BLUETOOTH:
                return "Bluetooth";
        case CONNMAN_SERVICE_TYPE_CELLULAR:
@@ -920,7 +922,7 @@ static const GDBusSignalTable technology_signals[] = {
 static gboolean technology_dbus_register(struct connman_technology *technology)
 {
        if (technology->dbus_registered == TRUE ||
-                               (technology->rfkill_driven &&
+                               (technology->rfkill_driven == TRUE &&
                                 technology->hardblocked == TRUE))
                return TRUE;
 
@@ -982,11 +984,7 @@ static struct connman_technology *technology_get(enum connman_service_type type)
 
        technology->rfkill_driven = FALSE;
        technology->softblocked = FALSE;
-
-       if (type == CONNMAN_SERVICE_TYPE_ETHERNET)
-               technology->hardblocked = FALSE;
-       else
-               technology->hardblocked = TRUE;
+       technology->hardblocked = FALSE;
 
        technology->type = type;
        technology->path = g_strdup_printf("%s/technology/%s",
@@ -1065,7 +1063,6 @@ void __connman_technology_add_interface(enum connman_service_type type,
                return;
        case CONNMAN_SERVICE_TYPE_ETHERNET:
        case CONNMAN_SERVICE_TYPE_WIFI:
-       case CONNMAN_SERVICE_TYPE_WIMAX:
        case CONNMAN_SERVICE_TYPE_BLUETOOTH:
        case CONNMAN_SERVICE_TYPE_CELLULAR:
        case CONNMAN_SERVICE_TYPE_GPS:
@@ -1098,7 +1095,6 @@ void __connman_technology_remove_interface(enum connman_service_type type,
                return;
        case CONNMAN_SERVICE_TYPE_ETHERNET:
        case CONNMAN_SERVICE_TYPE_WIFI:
-       case CONNMAN_SERVICE_TYPE_WIMAX:
        case CONNMAN_SERVICE_TYPE_BLUETOOTH:
        case CONNMAN_SERVICE_TYPE_CELLULAR:
        case CONNMAN_SERVICE_TYPE_GPS:
@@ -1340,27 +1336,35 @@ void __connman_technology_set_connected(enum connman_service_type type,
 
 static connman_bool_t technology_apply_rfkill_change(struct connman_technology *technology,
                                                connman_bool_t softblock,
-                                               connman_bool_t hardblock)
+                                               connman_bool_t hardblock,
+                                               connman_bool_t new_rfkill)
 {
        gboolean hardblock_changed = FALSE;
        gboolean apply = TRUE;
        GList *start, *list;
 
+       DBG("technology %p --> %d/%d vs %d/%d",
+                       technology, softblock, hardblock,
+                       technology->softblocked, technology->hardblocked);
+
        if (technology->hardblocked == hardblock)
                goto softblock_change;
 
-       start = g_hash_table_get_values(rfkill_list);
-       for (list = start; list != NULL; list = list->next) {
-               struct connman_rfkill *rfkill = list->data;
+       if (!(new_rfkill == TRUE && hardblock == FALSE)) {
+               start = g_hash_table_get_values(rfkill_list);
 
-               if (rfkill->type != technology->type)
-                       continue;
+               for (list = start; list != NULL; list = list->next) {
+                       struct connman_rfkill *rfkill = list->data;
 
-               if (rfkill->hardblock != hardblock)
-                       apply = FALSE;
-       }
+                       if (rfkill->type != technology->type)
+                               continue;
+
+                       if (rfkill->hardblock != hardblock)
+                               apply = FALSE;
+               }
 
-       g_list_free(start);
+               g_list_free(start);
+       }
 
        if (apply == FALSE)
                goto softblock_change;
@@ -1434,7 +1438,7 @@ done:
 
        /* If hardblocked, there is no need to handle softblocked state */
        if (technology_apply_rfkill_change(technology,
-                                       softblock, hardblock) == TRUE)
+                               softblock, hardblock, TRUE) == TRUE)
                return 0;
 
        /*
@@ -1446,8 +1450,8 @@ done:
                                technology->enable_persistent == TRUE)
                return __connman_rfkill_block(type, FALSE);
        else if (technology->softblocked == FALSE &&
-                               global_offlinemode == TRUE &&
-                               technology->enable_persistent == FALSE)
+                       (global_offlinemode == TRUE ||
+                               technology->enable_persistent == FALSE))
                return __connman_rfkill_block(type, TRUE);
 
        return 0;
@@ -1481,7 +1485,7 @@ int __connman_technology_update_rfkill(unsigned int index,
 
        /* If hardblocked, there is no need to handle softblocked state */
        if (technology_apply_rfkill_change(technology,
-                                       softblock, hardblock) == TRUE)
+                               softblock, hardblock, FALSE) == TRUE)
                return 0;
 
        if (global_offlinemode == TRUE)
@@ -1519,6 +1523,9 @@ int __connman_technology_remove_rfkill(unsigned int index,
        if (technology == NULL)
                return -ENXIO;
 
+       technology_apply_rfkill_change(technology,
+               technology->softblocked, !technology->hardblocked, FALSE);
+
        technology_put(technology);
 
        return 0;