The network_change() function called when a network error is set will
disconnect a network driver identical to __connman_network_disconnect()
but without any error handling. Therefore the specific network_change()
function is better replaced with __connman_network_disconnect(). With
connman_network_set_error() now calling __connman_network_disconnect(),
the additional function call can now be removed from
connman_network_set_connected().
Also let the associating and connecting variables be handled by the
normal code paths and don't forcefully unset them when an error happens.
too early, leave
---
src/network.c | 23 ++---------------------
1 file changed, 2 insertions(+), 21 deletions(-)
diff --git a/src/network.c b/src/network.c
index 7945cb4..d70cfdb 100644
--- a/src/network.c
+++ b/src/network.c
@@ -798,21 +798,6 @@ static void network_remove(struct connman_network *network)
network->driver = NULL;
}
-static void network_change(struct connman_network *network)
-{
- DBG("network %p name %s", network, network->name);
-
- if (!network->connected)
- return;
-
- 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;
@@ -1301,9 +1286,6 @@ void connman_network_set_error(struct connman_network
*network,
{
DBG("network %p error %d", network, error);
- network->connecting = false;
- network->associating = false;
-
switch (error) {
case CONNMAN_NETWORK_ERROR_UNKNOWN:
return;
@@ -1321,7 +1303,7 @@ void connman_network_set_error(struct connman_network
*network,
break;
}
- network_change(network);
+ __connman_network_disconnect(network);
}
/**
@@ -1342,8 +1324,7 @@ int connman_network_set_connected(struct connman_network
*network,
!connected) {
connman_network_set_error(network,
CONNMAN_NETWORK_ERROR_CONNECT_FAIL);
- if (__connman_network_disconnect(network) == 0)
- return 0;
+ return 0;
}
if (network->connected == connected)
--
2.1.4
_______________________________________________
connman mailing list
[email protected]
https://lists.connman.net/mailman/listinfo/connman
