On Mon, 2014-12-22 at 13:01 +0200, pasi.sjoh...@jolla.com wrote:
> From: Pasi Sjöholm <pasi.sjoh...@jollamobile.com>
>
> ---
> plugins/ofono.c | 25 ++++++++++++++-----------
> 1 file changed, 14 insertions(+), 11 deletions(-)
>
> diff --git a/plugins/ofono.c b/plugins/ofono.c
> index 1617764..782c93e 100644
> --- a/plugins/ofono.c
> +++ b/plugins/ofono.c
> @@ -824,20 +824,23 @@ static void extract_ipv4_settings(DBusMessageIter
> *array,
>
> context->index = index;
>
> - if (context->ipv4_method != CONNMAN_IPCONFIG_METHOD_FIXED)
> - goto out;
> + if (address) {
>
> - context->ipv4_address =
> connman_ipaddress_alloc(CONNMAN_IPCONFIG_TYPE_IPV4);
> - if (!context->ipv4_address) {
> - context->index = -1;
> - goto out;
> - }
> + if (context->ipv4_method != CONNMAN_IPCONFIG_METHOD_FIXED)
> + goto out;
>
> - connman_ipaddress_set_ipv4(context->ipv4_address, address,
> - netmask, gateway);
> + context->ipv4_address =
> connman_ipaddress_alloc(CONNMAN_IPCONFIG_TYPE_IPV4);
> + if (!context->ipv4_address) {
> + context->index = -1;
> + goto out;
> + }
>
> - g_free(context->ipv4_nameservers);
> - context->ipv4_nameservers = nameservers;
> + connman_ipaddress_set_ipv4(context->ipv4_address, address,
> + netmask, gateway);
> +
> + g_free(context->ipv4_nameservers);
> + context->ipv4_nameservers = nameservers;
> + }
>
> out:
> if (context->ipv4_nameservers != nameservers)
NACK.
From your other mail:
{ConnectionContext} [/ril_0/context1] Settings = { Gateway =
10.233.105.246, Address = 10.233.105.245, Method = static, Netmask =
255.255.255.0, Interface = rmnet0, DomainNameServers = 62.241.198.245
62.241.198.246 }
...
{ConnectionContext} [/ril_0/context1] IPv6.Settings = { Gateway =
2001::1223, Address = 2001::8a0f, Interface = rmnet0, DomainNameServers
= 2001::0001 2001::0002 }
This clearly shows oFono is sending DomainNameServers which ConnMan does
not have any reason not to use. Unless some RFC clearly says
DomainNameServers are to be ignored if updated via DHCP or stateless
DHCPv6.
I don't see anything *invalid* being sent/stale information being used
by the code.
Cheers,
Patrik
As your ofono trace shows,
_______________________________________________
connman mailing list
connman@connman.net
https://lists.connman.net/mailman/listinfo/connman
