Hi,

>At the start of both functions, nameservers is set to NULL. If the
>'DomainNameServers' property is set in the message, the nameservers take
>the value supplied. At the end of both functions there are these two
>lines which appear to remove the old values from the context like so:
>
>       g_free(context->ipv6_nameservers);
>        context->ipv6_nameservers = nameservers;
>
>Are you sure the extra freeing is needed?

I will modify it a bit so that the freeing is done only "if (address)" fails.. 
as the new addresses will come through dhcp/dhcpv6 in that case.

Br,
Pasi
_______________________________________________
connman mailing list
[email protected]
https://lists.connman.net/mailman/listinfo/connman

Reply via email to