Attachment 109456 Details for Bug 85012 – Patch

[patch] Patch

0001-icera-IPv6-is-a-placeholder-for-no-IPv6-so-don-t-err.patch (text/plain), 4.66 KB, created by Aleksander Morgado on 2014-11-14 11:35:41 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Aleksander Morgado

Created: 2014-11-14 11:35:41 UTC

Size: 4.66 KB

patch

obsolete

>From e272d57db6c56110238072fc23603e9af84ca82e Mon Sep 17 00:00:00 2001
>From: Aleksander Morgado <aleksander@aleksander.es>
>Date: Fri, 14 Nov 2014 12:32:10 +0100
>Subject: [PATCH] icera: IPv6 '::' is a placeholder for 'no IPv6', so don't
> error out if found
>
>---
> plugins/icera/mm-modem-helpers-icera.c         | 27 +++++++++++++++-----------
> plugins/icera/tests/test-modem-helpers-icera.c | 10 +++++++++-
> 2 files changed, 25 insertions(+), 12 deletions(-)
>
>diff --git a/plugins/icera/mm-modem-helpers-icera.c b/plugins/icera/mm-modem-helpers-icera.c
>index f9aa612..8a0d7e6 100644
>--- a/plugins/icera/mm-modem-helpers-icera.c
>+++ b/plugins/icera/mm-modem-helpers-icera.c
>@@ -145,6 +145,10 @@ parse_ipdpaddr_v6 (const gchar **items, guint num_items, GError **error)
>     if (num_items < 12)
>         return NULL;
> 
>+    /* No IPv6 config, return NULL without error */
>+    if (g_strcmp0 (items[9], "::") == 0)
>+        return NULL;
>+
>     /* It appears that for IPv6 %IPDPADDR returns only the expected
>      * link-local address and a DNS address, and that to retrieve the
>      * default router, extra DNS, and search domains, the host must listen
>@@ -172,16 +176,18 @@ parse_ipdpaddr_v6 (const gchar **items, guint num_items, GError **error)
>         mm_bearer_ip_config_set_method (config, MM_BEARER_IP_METHOD_DHCP);
> 
>     /* DNS server */
>-    memset (&tmp6, 0, sizeof (tmp6));
>-    if (inet_pton (AF_INET6, items[11], &tmp6) == 1 &&
>+    if (g_strcmp0 (items[11], "::") != 0) {
>+        memset (&tmp6, 0, sizeof (tmp6));
>+        if (inet_pton (AF_INET6, items[11], &tmp6) == 1 &&
>             !IN6_IS_ADDR_UNSPECIFIED (&tmp6)) {
>-        dns[0] = items[11];
>-        dns[1] = NULL;
>-        mm_bearer_ip_config_set_dns (config, (const gchar **) dns);
>-    } else {
>-        g_set_error (error, MM_CORE_ERROR, MM_CORE_ERROR_FAILED,
>-                     "Couldn't parse DNS address '%s'", items[11]);
>-        goto error;
>+            dns[0] = items[11];
>+            dns[1] = NULL;
>+            mm_bearer_ip_config_set_dns (config, (const gchar **) dns);
>+        } else {
>+            g_set_error (error, MM_CORE_ERROR, MM_CORE_ERROR_FAILED,
>+                         "Couldn't parse DNS address '%s'", items[11]);
>+            goto error;
>+        }
>     }
> 
>     return config;
>@@ -223,7 +229,7 @@ mm_icera_parse_ipdpaddr_response (const gchar *response,
>      * Sierra USB305: %IPDPADDR: 2, 21.93.217.11, 21.93.217.10, 10.177.0.34, 10.161.171.220, 0.0.0.0, 0.0.0.0
>      * K3805-Z: %IPDPADDR: 2, 21.93.217.11, 21.93.217.10, 10.177.0.34, 10.161.171.220, 0.0.0.0, 0.0.0.0, 255.0.0.0, 255.255.255.0, 21.93.217.10,
>      * Nokia 21M: %IPDPADDR: 2, 33.196.7.127, 33.196.7.128, 10.177.0.34, 10.161.171.220, 0.0.0.0, 0.0.0.0, 255.0.0.0, 33.196.7.128, fe80::f:9135:5901, ::, fd00:976a::9, ::, ::, ::, ::, ::
>-     * Nokia 21M: %IPDPADDR: 3, 0.0.0.0, 0.0.0.0, 0.0.0.0, 0.0.0.0, 0.0.0.0, 0.0.0.0, 0.0.0.0, 0.0.0.0, fe80::2e:437b:7901, ::, fd00:976a::9, ::, ::, ::, ::, ::  
>+     * Nokia 21M: %IPDPADDR: 3, 0.0.0.0, 0.0.0.0, 0.0.0.0, 0.0.0.0, 0.0.0.0, 0.0.0.0, 0.0.0.0, 0.0.0.0, fe80::2e:437b:7901, ::, fd00:976a::9, ::, ::, ::, ::, ::
>      */
>     response = mm_strip_tag (response, IPDPADDR_TAG);
>     items = g_strsplit_set (response, ",", 0);
>@@ -274,4 +280,3 @@ out:
>     *out_ip6_config = ip6_config;
>     return success;
> }
>-
>diff --git a/plugins/icera/tests/test-modem-helpers-icera.c b/plugins/icera/tests/test-modem-helpers-icera.c
>index 8eaff4c..3342ed4 100644
>--- a/plugins/icera/tests/test-modem-helpers-icera.c
>+++ b/plugins/icera/tests/test-modem-helpers-icera.c
>@@ -49,7 +49,7 @@ typedef struct {
> 
> static const IpdpaddrTest ipdpaddr_tests[] = {
>     /* Sierra USB305 */
>-    { "%IPDPADDR: 2, 21.93.217.11, 21.93.217.10, 10.177.0.34, 10.161.171.220, 0.0.0.0, 0.0.0.0\r\n", 
>+    { "%IPDPADDR: 2, 21.93.217.11, 21.93.217.10, 10.177.0.34, 10.161.171.220, 0.0.0.0, 0.0.0.0\r\n",
>         2, "21.93.217.11", 32, "21.93.217.10", "10.177.0.34", "10.161.171.220",
>         NULL, NULL },
> 
>@@ -87,6 +87,14 @@ static const IpdpaddrTest ipdpaddr_tests[] = {
>         2, "188.150.116.13", 16, "188.150.116.14", "188.149.250.16", NULL,
>         "fe80::1:e414:eb01", "2a00:e18:0:3::6" },
> 
>+    { "%IPDPADDR: 1, 0.0.0.0, 0.0.0.0, 0.0.0.0, 0.0.0.0, 0.0.0.0, 0.0.0.0, 0.0.0.0, 0.0.0.0, fe80::1f:fad1:4c01, ::, 2001:4600:4:fff::54, 2001:4600:4:1fff::54, ::, ::, ::, ::\r\n",
>+      1, NULL, 0, NULL, NULL, NULL,
>+      "fe80::1f:fad1:4c01", "2001:4600:4:fff::54" },
>+
>+    { "%IPDPADDR: 1, 46.157.76.179, 46.157.76.180, 193.213.112.4, 130.67.15.198, 0.0.0.0, 0.0.0.0, 255.0.0.0, 46.157.76.180, ::, ::, ::, ::, ::, ::, ::, ::\r\n",
>+      1, "46.157.76.179", 32, "46.157.76.180", "193.213.112.4", "130.67.15.198",
>+      NULL, NULL },
>+
>     { NULL }
> };
> 
>-- 
>2.1.3
>

Actions: View | Splinter Review

Attachments on bug 85012: 109456 | 109466 | 109513 | 109514

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.