[Upstream] ipconfig: Ensure ifname is not NULL when setting IPv6 state

This is a quick fix for the issue where removal of an interface may
cause IPv6 to be disabled/enabled on other interfaces.

When removing an ipdevice in free_ipdevice(), it is ensured that the
ifname (which we get from connman_inet_ifname()) is not NULL,
before invoking set_ipv6_state(). If a NULL ifname is passed into
set_ipv6_state(), it will apply the change to all interfaces instead
of the intended one.

Change-Id: Iefab1e1c87d2f08d1b73c80ec51dbdfcb3238fad
Signed-off-by: Jaehyun Kim <jeik01.kim@samsung.com>

diff --git a/src/ipconfig.c b/src/ipconfig.c
index 700384f..ded1c11 100755 (executable)
--- a/src/ipconfig.c
+++ b/src/ipconfig.c
@@ -404,8 +404,10 @@ static void free_ipdevice(gpointer data)
 
        g_free(ipdevice->address);
 
-       set_ipv6_state(ifname, ipdevice->ipv6_enabled);
-       set_ipv6_privacy(ifname, ipdevice->ipv6_privacy);
+       if (ifname) {
+               set_ipv6_state(ifname, ipdevice->ipv6_enabled);
+               set_ipv6_privacy(ifname, ipdevice->ipv6_privacy);
+       }
 
        g_free(ifname);
        g_free(ipdevice);