From: Saurav Babu Date: Mon, 18 Jul 2016 11:13:53 +0000 (+0530) Subject: Upstream: service: Enable ipconfig only for the changed IP type X-Git-Tag: accepted/tizen/common/20160907.154208^0 X-Git-Url: http://review.tizen.org/git/?p=platform%2Fupstream%2Fconnman.git;a=commitdiff_plain;h=5298369f5625fa7cab9ce742f9abbf2cdfbfea2d Upstream: service: Enable ipconfig only for the changed IP type When IPv6.Configuration is changed to "off" then connman starts fresh DHCP Requests for service after changing its state to Configuration. In an ideal scenario IPv4 Configurations should not be affected on changing IPv6.Configuration property. This patch only enables ipconfig for the changed IP type and leaves other IP type unchanged Change-Id: I65eacb6711314d2674e709711f432ef706167fc4 Signed-off-by: Saurav Babu --- diff --git a/src/service.c b/src/service.c index f77f033..1eae380 100755 --- a/src/service.c +++ b/src/service.c @@ -4040,9 +4040,11 @@ static DBusMessage *set_property(DBusConnection *conn, if (err < 0) { if (is_connected_state(service, state) || is_connecting_state(service, state)) { - __connman_network_enable_ipconfig(service->network, + if (type == CONNMAN_IPCONFIG_TYPE_IPV4) + __connman_network_enable_ipconfig(service->network, service->ipconfig_ipv4); - __connman_network_enable_ipconfig(service->network, + else + __connman_network_enable_ipconfig(service->network, service->ipconfig_ipv6); } @@ -4055,10 +4057,12 @@ static DBusMessage *set_property(DBusConnection *conn, ipv6_configuration_changed(service); if (is_connecting(service) || is_connected(service)) { - __connman_network_enable_ipconfig(service->network, - service->ipconfig_ipv4); - __connman_network_enable_ipconfig(service->network, - service->ipconfig_ipv6); + if (type == CONNMAN_IPCONFIG_TYPE_IPV4) + __connman_network_enable_ipconfig(service->network, + service->ipconfig_ipv4); + else + __connman_network_enable_ipconfig(service->network, + service->ipconfig_ipv6); } service_save(service);