From: Daniel Wagner <[email protected]>
dhcpv6_set_addresses() should return an error code and not a boolean,
because the caller tests for values less then 0. Hence we haven
taken always the false branch instead stopping the DHCPv6 in the
error case.
Reported by coverity.
---
src/network.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/network.c b/src/network.c
index bd5796e..cb84361 100644
--- a/src/network.c
+++ b/src/network.c
@@ -346,7 +346,7 @@ static void dhcpv6_info_callback(struct connman_network
*network,
stop_dhcpv6(network);
}
-static bool dhcpv6_set_addresses(struct connman_network *network)
+static int dhcpv6_set_addresses(struct connman_network *network)
{
struct connman_service *service;
struct connman_ipconfig *ipconfig_ipv6;
--
1.8.4.474.g128a96c
_______________________________________________
connman mailing list
[email protected]
https://lists.connman.net/mailman/listinfo/connman
