From: Daniel Wagner <wagi@cesium.(none)>
---
include/element.h | 2 -
include/network.h | 3 +
plugins/ethernet.c | 15 ++-
plugins/wifi.c | 13 ++-
src/device.c | 13 +--
src/element.c | 6 -
src/network.c | 419 ++++++++++++++++++++++++++++------------------------
src/service.c | 14 ++-
8 files changed, 269 insertions(+), 216 deletions(-)
diff --git a/include/element.h b/include/element.h
index 57a7632..3499cdf 100644
--- a/include/element.h
+++ b/include/element.h
@@ -42,7 +42,6 @@ extern "C" {
enum connman_element_type {
CONNMAN_ELEMENT_TYPE_UNKNOWN = 0,
CONNMAN_ELEMENT_TYPE_ROOT = 1,
- CONNMAN_ELEMENT_TYPE_NETWORK = 4,
};
enum connman_element_state {
@@ -80,7 +79,6 @@ struct connman_element {
union {
void *private;
- struct connman_network *network;
};
GHashTable *properties;
diff --git a/include/network.h b/include/network.h
index 0649a4d..d2aa347 100644
--- a/include/network.h
+++ b/include/network.h
@@ -140,6 +140,9 @@ void connman_network_set_data(struct connman_network
*network, void *data);
void connman_network_update(struct connman_network *network);
+int connman_network_register(struct connman_network *network);
+void connman_network_unregister(struct connman_network *network);
+
struct connman_network_driver {
const char *name;
enum connman_network_type type;
diff --git a/plugins/ethernet.c b/plugins/ethernet.c
index 2ccd498..0cf0755 100644
--- a/plugins/ethernet.c
+++ b/plugins/ethernet.c
@@ -92,19 +92,24 @@ static void add_network(struct connman_device *device)
if (network == NULL)
return;
- index = connman_device_get_index(device);
- connman_network_set_index(network, index);
-
connman_network_set_name(network, "Wired");
+ connman_network_set_group(network, "cable");
+
if (connman_device_add_network(device, network) < 0) {
connman_network_unref(network);
return;
}
- connman_network_set_available(network, TRUE);
+ index = connman_device_get_index(device);
+ connman_network_set_index(network, index);
- connman_network_set_group(network, "cable");
+ if (connman_network_register(network) < 0) {
+ connman_network_unref(network);
+ return;
+ }
+
+ connman_network_set_available(network, TRUE);
connman_network_set_connected(network, TRUE);
}
diff --git a/plugins/wifi.c b/plugins/wifi.c
index 119f7c7..efe2e89 100644
--- a/plugins/wifi.c
+++ b/plugins/wifi.c
@@ -790,6 +790,11 @@ static void network_added(GSupplicantNetwork
*supplicant_network)
connman_network_unref(network);
return;
}
+
+ if (connman_network_register(network) < 0) {
+ connman_network_unref(network);
+ return;
+ }
}
if (name != NULL && name[0] != '\0')
@@ -814,6 +819,7 @@ static void network_removed(GSupplicantNetwork *network)
GSupplicantInterface *interface;
struct wifi_data *wifi;
const char *name, *identifier;
+ struct connman_network *connman_network;
interface = g_supplicant_network_get_interface(network);
wifi = g_supplicant_interface_get_data(interface);
@@ -822,8 +828,13 @@ static void network_removed(GSupplicantNetwork *network)
DBG("name %s", name);
- if (wifi != NULL)
+ if (wifi != NULL) {
+ connman_network = connman_device_get_network(wifi->device,
identifier);
+ if (connman_network != NULL)
+ connman_network_unregister(connman_network);
+
connman_device_remove_network(wifi->device, identifier);
+ }
}
static void debug(const char *str)
diff --git a/src/device.c b/src/device.c
index 48be10f..628f1aa 100644
--- a/src/device.c
+++ b/src/device.c
@@ -424,7 +424,7 @@ static void unregister_network(gpointer data)
DBG("network %p", network);
- connman_element_unregister((struct connman_element *) network);
+ connman_network_unregister(network);
__connman_network_set_device(network, NULL);
@@ -1067,7 +1067,6 @@ int connman_device_add_network(struct connman_device
*device,
struct connman_network *network)
{
const char *identifier = connman_network_get_identifier(network);
- int err;
DBG("device %p network %p", device, network);
@@ -1076,13 +1075,6 @@ int connman_device_add_network(struct connman_device
*device,
__connman_network_set_device(network, device);
- err = connman_element_register((struct connman_element *) network,
- NULL);
- if (err < 0) {
- __connman_network_set_device(network, NULL);
- return err;
- }
-
g_hash_table_insert(device->networks, g_strdup(identifier),
network);
@@ -1122,7 +1114,8 @@ int connman_device_remove_network(struct connman_device
*device,
if (network == NULL)
return 0;
- connman_element_unregister((struct connman_element *) network);
+ __connman_network_set_device(network, NULL);
+
g_hash_table_remove(device->networks, identifier);
return 0;
diff --git a/src/element.c b/src/element.c
index a9a643b..75d5126 100644
--- a/src/element.c
+++ b/src/element.c
@@ -46,8 +46,6 @@ static const char *type2string(enum connman_element_type type)
return "unknown";
case CONNMAN_ELEMENT_TYPE_ROOT:
return "root";
- case CONNMAN_ELEMENT_TYPE_NETWORK:
- return "network";
}
return NULL;
@@ -119,10 +117,6 @@ static gboolean append_path(GNode *node, gpointer
user_data)
filter->type != element->type)
return FALSE;
- if (filter->type == CONNMAN_ELEMENT_TYPE_NETWORK &&
- __connman_network_has_driver(element->network) == FALSE)
- return FALSE;
-
dbus_message_iter_append_basic(filter->iter,
DBUS_TYPE_OBJECT_PATH, &element->path);
diff --git a/src/network.c b/src/network.c
index 95b4d4a..62847a9 100644
--- a/src/network.c
+++ b/src/network.c
@@ -28,10 +28,12 @@
#include "connman.h"
+static GSList *network_list = NULL;
+static GSList *driver_list = NULL;
static unsigned int hidden_counter = 0;
struct connman_network {
- struct connman_element element;
+ gint refcount;
enum connman_network_type type;
connman_bool_t available;
connman_bool_t connected;
@@ -43,6 +45,8 @@ struct connman_network {
char *name;
char *node;
char *group;
+ char *path;
+ int index;
struct connman_network_driver *driver;
void *driver_data;
@@ -70,6 +74,12 @@ struct connman_network {
connman_bool_t use_wps;
char *pin_wps;
} wifi;
+
+ struct {
+ char *network_type;
+ char *nsp_name;
+ int nsp_name_len;
+ } wimax;
};
static const char *type2string(enum connman_network_type type)
@@ -102,7 +112,155 @@ connman_bool_t __connman_network_has_driver(struct
connman_network *network)
return TRUE;
}
-static GSList *driver_list = NULL;
+static gboolean match_driver(struct connman_network *network,
+ struct connman_network_driver *driver)
+{
+ if (network->type == driver->type ||
+ driver->type == CONNMAN_NETWORK_TYPE_UNKNOWN)
+ return TRUE;
+
+ return FALSE;
+}
+
+static int network_probe(struct connman_network *network)
+{
+ GSList *list;
+
+ DBG("network %p name %s", network, network->name);
+
+ for (list = driver_list; list; list = list->next) {
+ struct connman_network_driver *driver = list->data;
+
+ if (match_driver(network, driver) == FALSE)
+ continue;
+
+ DBG("driver %p name %s", driver, driver->name);
+
+ if (driver->probe(network) == 0) {
+ network->driver = driver;
+ break;
+ }
+ }
+
+ if (network->driver == NULL)
+ return -ENODEV;
+
+ switch (network->type) {
+ case CONNMAN_NETWORK_TYPE_UNKNOWN:
+ case CONNMAN_NETWORK_TYPE_VENDOR:
+ break;
+ case CONNMAN_NETWORK_TYPE_ETHERNET:
+ case CONNMAN_NETWORK_TYPE_BLUETOOTH_PAN:
+ case CONNMAN_NETWORK_TYPE_BLUETOOTH_DUN:
+ case CONNMAN_NETWORK_TYPE_CELLULAR:
+ case CONNMAN_NETWORK_TYPE_WIFI:
+ case CONNMAN_NETWORK_TYPE_WIMAX:
+ if (network->group != NULL &&
+ __connman_service_create_from_network(network) == NULL)
+ return -EINVAL;
+ }
+
+ return 0;
+}
+
+static void network_remove(struct connman_network *network)
+{
+ DBG("network %p name %s", network, network->name);
+
+ switch (network->type) {
+ case CONNMAN_NETWORK_TYPE_UNKNOWN:
+ case CONNMAN_NETWORK_TYPE_VENDOR:
+ break;
+ case CONNMAN_NETWORK_TYPE_ETHERNET:
+ case CONNMAN_NETWORK_TYPE_BLUETOOTH_PAN:
+ case CONNMAN_NETWORK_TYPE_BLUETOOTH_DUN:
+ case CONNMAN_NETWORK_TYPE_CELLULAR:
+ case CONNMAN_NETWORK_TYPE_WIFI:
+ case CONNMAN_NETWORK_TYPE_WIMAX:
+ if (network->group != NULL) {
+ __connman_service_remove_from_network(network);
+
+ g_free(network->group);
+ network->group = NULL;
+ }
+ break;
+ }
+
+ if (network->driver->remove)
+ network->driver->remove(network);
+
+ network->driver = NULL;
+}
+
+static void network_change(struct connman_network *network)
+{
+ DBG("network %p name %s", network, network->name);
+
+ if (network->connected == FALSE)
+ return;
+
+ connman_network_unref(network);
+
+ connman_device_set_disconnected(network->device, TRUE);
+
+ if (network->driver && network->driver->disconnect) {
+ network->driver->disconnect(network);
+ return;
+ }
+
+ network->connected = FALSE;
+}
+
+static void probe_driver(struct connman_network_driver *driver)
+{
+ GSList *list;
+
+ DBG("driver %p name %s", driver, driver->name);
+
+ for (list = network_list; list != NULL; list = list->next) {
+ struct connman_network *network = list->data;
+
+ if (network->driver != NULL)
+ continue;
+
+ if (driver->type != network->type)
+ continue;
+
+ if (driver->probe(network) < 0)
+ continue;
+
+ network->driver = driver;
+ }
+}
+
+int connman_network_register(struct connman_network *network)
+{
+ network_list = g_slist_append(network_list, network);
+
+ return network_probe(network);
+}
+
+
+void connman_network_unregister(struct connman_network *network)
+{
+ network_list = g_slist_remove(network_list, network);
+
+ network_remove(network);
+}
+
+static void remove_driver(struct connman_network_driver *driver)
+{
+ GSList *list;
+
+ DBG("driver %p name %s", driver, driver->name);
+
+ for (list = network_list; list != NULL; list = list->next) {
+ struct connman_network *network = list->data;
+
+ if (network->driver == driver)
+ network_remove(network);
+ }
+}
static gint compare_priority(gconstpointer a, gconstpointer b)
{
@@ -137,6 +295,8 @@ int connman_network_driver_register(struct
connman_network_driver *driver)
driver_list = g_slist_insert_sorted(driver_list, driver,
compare_priority);
+ probe_driver(driver);
+
return 0;
}
@@ -151,13 +311,13 @@ void connman_network_driver_unregister(struct
connman_network_driver *driver)
DBG("driver %p name %s", driver, driver->name);
driver_list = g_slist_remove(driver_list, driver);
+
+ remove_driver(driver);
}
-static void network_destruct(struct connman_element *element)
+static void network_destruct(struct connman_network *network)
{
- struct connman_network *network = element->network;
-
- DBG("element %p name %s", element, element->name);
+ DBG("network %p name %s", network, network->name);
g_free(network->wifi.ssid);
g_free(network->wifi.mode);
@@ -172,12 +332,18 @@ static void network_destruct(struct connman_element
*element)
g_free(network->wifi.phase2_auth);
g_free(network->wifi.pin_wps);
+ g_free(network->wimax.network_type);
+ g_free(network->wimax.nsp_name);
+
g_free(network->group);
g_free(network->node);
g_free(network->name);
g_free(network->identifier);
+ g_free(network->path);
network->device = NULL;
+
+ g_free(network);
}
/**
@@ -192,8 +358,6 @@ struct connman_network *connman_network_create(const char
*identifier,
enum connman_network_type type)
{
struct connman_network *network;
- connman_uint8_t strength = 0;
- const char *str;
char *temp;
DBG("identifier %s type %d", identifier, type);
@@ -204,7 +368,7 @@ struct connman_network *connman_network_create(const char
*identifier,
DBG("network %p", network);
- __connman_element_initialize(&network->element);
+ network->refcount = 1;
if (identifier == NULL) {
temp = g_strdup_printf("hidden_%d", hidden_counter++);
@@ -217,18 +381,6 @@ struct connman_network *connman_network_create(const char
*identifier,
return NULL;
}
- network->element.name = temp;
- network->element.type = CONNMAN_ELEMENT_TYPE_NETWORK;
-
- network->element.network = network;
- network->element.destruct = network_destruct;
-
- str = type2string(type);
- if (str != NULL)
- connman_element_set_string(&network->element, "Type", str);
-
- connman_element_set_uint8(&network->element, "Strength", strength);
-
network->type = type;
network->identifier = g_strdup(temp);
@@ -243,8 +395,10 @@ struct connman_network *connman_network_create(const char
*identifier,
*/
struct connman_network *connman_network_ref(struct connman_network *network)
{
- if (connman_element_ref(&network->element) == NULL)
- return NULL;
+ DBG("network %p name %s refcount %d", network, network->name,
+ g_atomic_int_get(&network->refcount) + 1);
+
+ g_atomic_int_inc(&network->refcount);
return network;
}
@@ -257,7 +411,13 @@ struct connman_network *connman_network_ref(struct
connman_network *network)
*/
void connman_network_unref(struct connman_network *network)
{
- connman_element_unref(&network->element);
+ DBG("network %p name %s refcount %d", network, network->name,
+ g_atomic_int_get(&network->refcount) - 1);
+
+ if (g_atomic_int_dec_and_test(&network->refcount) == FALSE)
+ return;
+
+ network_destruct(network);
}
const char *__connman_network_get_type(struct connman_network *network)
@@ -305,10 +465,10 @@ void connman_network_set_index(struct connman_network
*network, int index)
ipconfig = __connman_service_get_ip4config(service);
- DBG("index %d service %p ip4config %p", network->element.index,
+ DBG("index %d service %p ip4config %p", network->index,
service, ipconfig);
- if (network->element.index < 0 && ipconfig == NULL) {
+ if (network->index < 0 && ipconfig == NULL) {
ipconfig = __connman_service_get_ip4config(service);
if (ipconfig == NULL)
@@ -330,7 +490,7 @@ void connman_network_set_index(struct connman_network
*network, int index)
}
done:
- network->element.index = index;
+ network->index = index;
}
/**
@@ -341,19 +501,7 @@ done:
*/
int connman_network_get_index(struct connman_network *network)
{
- return network->element.index;
-}
-
-/**
- * connman_network_get_element:
- * @network: network structure
- *
- * Get connman_element of network
- */
-struct connman_element *connman_network_get_element(
- struct connman_network *network)
-{
- return &network->element;
+ return network->index;
}
/**
@@ -392,9 +540,6 @@ void connman_network_set_group(struct connman_network
*network,
}
network->group = g_strdup(group);
-
- if (network->group != NULL)
- __connman_service_create_from_network(network);
}
/**
@@ -612,6 +757,8 @@ void connman_network_set_error(struct connman_network
*network,
set_connect_error(network);
break;
}
+
+ network_change(network);
}
void connman_network_clear_error(struct connman_network *network)
@@ -909,8 +1056,6 @@ static gboolean set_connected(gpointer user_data)
} else {
struct connman_service *service;
- connman_element_unregister_children(&network->element);
-
__connman_device_set_network(network->device, NULL);
network->hidden = FALSE;
@@ -1292,7 +1437,7 @@ int connman_network_set_name(struct connman_network
*network,
g_free(network->name);
network->name = g_strdup(name);
- return connman_element_set_string(&network->element, "Name", name);
+ return 0;
}
/**
@@ -1309,8 +1454,7 @@ int connman_network_set_strength(struct connman_network
*network,
network->strength = strength;
- return connman_element_set_uint8(&network->element,
- "Strength", strength);
+ return 0;
}
/**
@@ -1327,8 +1471,7 @@ int connman_network_set_roaming(struct connman_network
*network,
network->roaming = roaming;
- return connman_element_set_bool(&network->element,
- "Roaming", roaming);
+ return 0;
}
/**
@@ -1342,8 +1485,6 @@ int connman_network_set_roaming(struct connman_network
*network,
int connman_network_set_string(struct connman_network *network,
const char *key, const char *value)
{
- int err;
-
DBG("network %p key %s value %s", network, key, value);
if (g_strcmp0(key, "Name") == 0)
@@ -1385,12 +1526,14 @@ int connman_network_set_string(struct connman_network
*network,
} else if (g_str_equal(key, "WiFi.PinWPS") == TRUE) {
g_free(network->wifi.pin_wps);
network->wifi.pin_wps = g_strdup(value);
+ } else if (g_str_equal(key, "Path") == TRUE) {
+ g_free(network->path);
+ network->path = g_strdup(value);
+ } else if (g_str_equal(key, "WiMAX Network Type") == TRUE) {
+ g_free(network->wimax.network_type);
+ network->wimax.network_type = g_strdup(value);
}
- err = connman_element_set_string(&network->element, key, value);
- if (err < 0)
- return err;
-
if (network->driver == NULL)
return 0;
@@ -1438,8 +1581,12 @@ const char *connman_network_get_string(struct
connman_network *network,
return network->wifi.phase2_auth;
else if (g_str_equal(key, "WiFi.PinWPS") == TRUE)
return network->wifi.pin_wps;
+ else if (g_str_equal(key, "Path") == TRUE)
+ return network->path;
+ else if (g_str_equal(key, "WiMAX Network Type") == TRUE)
+ return network->wimax.network_type;
- return connman_element_get_string(&network->element, key);
+ return NULL;
}
/**
@@ -1462,7 +1609,7 @@ int connman_network_set_bool(struct connman_network
*network,
else if (g_strcmp0(key, "WiFi.UseWPS") == 0)
network->wifi.use_wps = value;
- return connman_element_set_bool(&network->element, key, value);
+ return 0;
}
/**
@@ -1484,7 +1631,7 @@ connman_bool_t connman_network_get_bool(struct
connman_network *network,
else if (g_str_equal(key, "WiFi.UseWPS") == TRUE)
return network->wifi.use_wps;
- return connman_element_get_bool(&network->element, key);
+ return FALSE;
}
/**
@@ -1503,7 +1650,7 @@ int connman_network_set_uint8(struct connman_network
*network,
if (g_strcmp0(key, "Strength") == 0)
return connman_network_set_strength(network, value);
- return connman_element_set_uint8(&network->element, key, value);
+ return -EINVAL;
}
/**
@@ -1521,7 +1668,12 @@ connman_uint8_t connman_network_get_uint8(struct
connman_network *network,
if (g_str_equal(key, "Strength") == TRUE)
return network->strength;
- return connman_element_get_uint8(&network->element, key);
+ /*
+ * XXX Cellular.Mode is not handled, though it's not set
+ * anywhere...
+ */
+
+ return 0;
}
/**
@@ -1587,9 +1739,17 @@ int connman_network_set_blob(struct connman_network
*network,
network->wifi.ssid_len = size;
} else
network->wifi.ssid_len = 0;
+ } else if (g_str_equal(key, "WiMAX.NSP.name") == TRUE) {
+ g_free(network->wimax.nsp_name);
+ network->wimax.nsp_name = g_try_malloc(size);
+ if (network->wimax.nsp_name != NULL) {
+ memcpy(network->wimax.nsp_name, data, size);
+ network->wimax.nsp_name_len = size;
+ } else
+ network->wimax.nsp_name_len = 0;
}
- return connman_element_set_blob(&network->element, key, data, size);
+ return 0;
}
/**
@@ -1609,9 +1769,13 @@ const void *connman_network_get_blob(struct
connman_network *network,
if (size != NULL)
*size = network->wifi.ssid_len;
return network->wifi.ssid;
+ } else if (g_str_equal(key, "WiMAX.NSP.name") == TRUE) {
+ if (size != NULL)
+ *size = network->wimax.nsp_name_len;
+ return network->wimax.nsp_name;
}
- return connman_element_get_blob(&network->element, key, size);
+ return NULL;
}
void __connman_network_set_device(struct connman_network *network,
@@ -1671,143 +1835,16 @@ void connman_network_update(struct connman_network
*network)
if (network->group != NULL)
__connman_service_update_from_network(network);
-
- return;
-}
-
-static gboolean match_driver(struct connman_network *network,
- struct connman_network_driver *driver)
-{
- if (network->type == driver->type ||
- driver->type == CONNMAN_NETWORK_TYPE_UNKNOWN)
- return TRUE;
-
- return FALSE;
-}
-
-static int network_probe(struct connman_element *element)
-{
- struct connman_network *network = element->network;
- GSList *list;
-
- DBG("element %p name %s", element, element->name);
-
- if (network == NULL)
- return -ENODEV;
-
- for (list = driver_list; list; list = list->next) {
- struct connman_network_driver *driver = list->data;
-
- if (match_driver(network, driver) == FALSE)
- continue;
-
- DBG("driver %p name %s", driver, driver->name);
-
- if (driver->probe(network) == 0) {
- network->driver = driver;
- break;
- }
- }
-
- if (network->driver == NULL)
- return -ENODEV;
-
- switch (network->type) {
- case CONNMAN_NETWORK_TYPE_UNKNOWN:
- case CONNMAN_NETWORK_TYPE_VENDOR:
- break;
- case CONNMAN_NETWORK_TYPE_ETHERNET:
- case CONNMAN_NETWORK_TYPE_BLUETOOTH_PAN:
- case CONNMAN_NETWORK_TYPE_BLUETOOTH_DUN:
- case CONNMAN_NETWORK_TYPE_CELLULAR:
- case CONNMAN_NETWORK_TYPE_WIFI:
- case CONNMAN_NETWORK_TYPE_WIMAX:
- if (network->group != NULL &&
- __connman_service_create_from_network(network) == NULL)
- return -EINVAL;
- }
-
- return 0;
-}
-
-static void network_remove(struct connman_element *element)
-{
- struct connman_network *network = element->network;
-
- DBG("element %p name %s", element, element->name);
-
- if (network == NULL)
- return;
-
- if (network->driver == NULL)
- return;
-
- switch (network->type) {
- case CONNMAN_NETWORK_TYPE_UNKNOWN:
- case CONNMAN_NETWORK_TYPE_VENDOR:
- break;
- case CONNMAN_NETWORK_TYPE_ETHERNET:
- case CONNMAN_NETWORK_TYPE_BLUETOOTH_PAN:
- case CONNMAN_NETWORK_TYPE_BLUETOOTH_DUN:
- case CONNMAN_NETWORK_TYPE_CELLULAR:
- case CONNMAN_NETWORK_TYPE_WIFI:
- case CONNMAN_NETWORK_TYPE_WIMAX:
- if (network->group != NULL) {
- __connman_service_remove_from_network(network);
-
- g_free(network->group);
- network->group = NULL;
- }
- break;
- }
-
- if (network->driver->remove)
- network->driver->remove(network);
}
-static void network_change(struct connman_element *element)
-{
- struct connman_network *network = element->network;
-
- DBG("element %p name %s", element, element->name);
-
- if (element->state != CONNMAN_ELEMENT_STATE_ERROR)
- return;
-
- if (network->connected == FALSE)
- return;
-
- connman_element_unregister_children(element);
-
- connman_device_set_disconnected(network->device, TRUE);
-
- if (network->driver && network->driver->disconnect) {
- network->driver->disconnect(network);
- return;
- }
-
- network->connected = FALSE;
-}
-
-static struct connman_driver network_driver = {
- .name = "network",
- .type = CONNMAN_ELEMENT_TYPE_NETWORK,
- .priority = CONNMAN_DRIVER_PRIORITY_LOW,
- .probe = network_probe,
- .remove = network_remove,
- .change = network_change,
-};
-
int __connman_network_init(void)
{
DBG("");
- return connman_driver_register(&network_driver);
+ return 0;
}
void __connman_network_cleanup(void)
{
DBG("");
-
- connman_driver_unregister(&network_driver);
}
diff --git a/src/service.c b/src/service.c
index 1b45f74..232fbc6 100644
--- a/src/service.c
+++ b/src/service.c
@@ -2961,8 +2961,13 @@ static void service_free(gpointer user_data)
g_hash_table_destroy(service->counter_table);
- if (service->network != NULL)
+ if (service->network != NULL) {
+ /*
+ * XXX If it'ss hidden wifi network then we should
+ * call connman_network_unregister too, right?
+ */
connman_network_unref(service->network);
+ }
if (service->provider != NULL)
connman_provider_unref(service->provider);
@@ -4080,6 +4085,11 @@ static struct connman_network *create_hidden_wifi(struct
connman_device *device,
if (network == NULL)
return NULL;
+ if (connman_network_register(network) < 0) {
+ connman_network_unref(network);
+ return NULL;
+ }
+
connman_network_set_blob(network, "WiFi.SSID",
(unsigned char *) ssid, ssid_len);
@@ -4088,6 +4098,7 @@ static struct connman_network *create_hidden_wifi(struct
connman_device *device,
name = g_try_malloc0(ssid_len + 1);
if (name == NULL) {
+ connman_network_unregister(network);
connman_network_unref(network);
return NULL;
}
@@ -4107,6 +4118,7 @@ static struct connman_network *create_hidden_wifi(struct
connman_device *device,
connman_network_set_index(network, index);
if (connman_device_add_network(device, network) < 0) {
+ connman_network_unregister(network);
connman_network_unref(network);
return NULL;
}
--
1.7.4.4
_______________________________________________
connman mailing list
[email protected]
http://lists.connman.net/listinfo/connman
