In __connman_dhcp_stop, dhcp_release is called twice on the same
pointer. First, directly in __connman_dhcp_stop itself and again in
the remove_network callback invoked from g_hash_table_remove. Only the
latter is necessary.

Credit to Thierry Boureille <[email protected]> for first
surfacing the issue.

---
 src/dhcp.c |    2 --
 1 files changed, 0 insertions(+), 2 deletions(-)

diff --git a/src/dhcp.c b/src/dhcp.c
index 03a2b90..2656c1c 100644
--- a/src/dhcp.c
+++ b/src/dhcp.c
@@ -446,8 +446,6 @@ void __connman_dhcp_stop(struct connman_network *network)
        if (dhcp == NULL)
                return;
 
-       dhcp_release(dhcp);
-
        g_hash_table_remove(network_table, network);
 }
 
-- 
1.7.5

_______________________________________________
connman mailing list
[email protected]
http://lists.connman.net/listinfo/connman

Reply via email to