Re: [PATCH] ipconfig: Do not enable/disable ipv6 for all ifs

[Jussi Laakkonen <jussi.laakkonen@jolla.com>]

Hi Christian,

On 11/22/21 3:03 PM, Christian Taedcke wrote:
> If the interface name could not be determined (e.g. because the usb
> device was unplugged and removed from the system), do not disable ipv6
> for all devices.
> ---
>   src/ipconfig.c | 6 ++++++
>   1 file changed, 6 insertions(+)
> 
> diff --git a/src/ipconfig.c b/src/ipconfig.c
> index 1551826b..34b1724a 100644
> --- a/src/ipconfig.c
> +++ b/src/ipconfig.c
> @@ -1593,6 +1593,9 @@ static void disable_ipv6(struct connman_ipconfig *ipconfig)
>   
>   	ifname = connman_inet_ifname(ipconfig->index);
>   
> +	if (!ifname)
> +	        return;
> +
>   	set_ipv6_state(ifname, false);
>   
>   	g_free(ifname);
> @@ -1612,6 +1615,9 @@ static void enable_ipv6(struct connman_ipconfig *ipconfig)
>   
>   	ifname = connman_inet_ifname(ipconfig->index);
>   
> +	if (!ifname)
> +	        return;
> +
>   	if (ipconfig->method == CONNMAN_IPCONFIG_METHOD_AUTO)
>   		set_ipv6_privacy(ifname, ipconfig->ipv6_privacy_config);
>   
> 

Good find. This indeed fixes this particular issue, IPv6 does seem to 
get disabled after removing a USB ethernet device. Surprising side effect.

With a quick glance I cannot see any problems related to this, I'm not 
100% sure if there is a case where this disable_ipv6() is used via 
__connman_ipconfig_disable_ipv6() to completely disable IPv6, Daniel?

Cheers,
  Jussi