Send connman mailing list submissions to
        connman@lists.01.org

To subscribe or unsubscribe via the World Wide Web, visit
        https://lists.01.org/mailman/listinfo/connman
or, via email, send a message with subject or body 'help' to
        connman-requ...@lists.01.org

You can reach the person managing the list at
        connman-ow...@lists.01.org

When replying, please edit your Subject line so it is more specific
than "Re: Contents of connman digest..."


Today's Topics:

   1. Re: [PATCH 06/11] resolve: musl does not implement res_ninit
      (Burton, Ross)
   2. [PATCH] Fix some warnings due to GCC8's cast-function-type
      (Peter Meerwald-Stadler)
   3. RE: Wrong state of a service if only IPv6 is enabled
      (Marek Szanyi)


----------------------------------------------------------------------

Message: 1
Date: Wed, 3 Oct 2018 11:52:47 +0100
From: "Burton, Ross" <ross.bur...@intel.com>
To: connman@lists.01.org
Subject: Re: [PATCH 06/11] resolve: musl does not implement res_ninit
Message-ID:
        <CAJTo0LaOoXbCNP=88SeFpu=gdturo4aq-6tbas98-e8sk9v...@mail.gmail.com>
Content-Type: text/plain; charset="UTF-8"

Whoops, meant to strip this before sending.  Ignore this patch for now.

Ross
On Wed, 3 Oct 2018 at 11:51, Ross Burton <ross.bur...@intel.com> wrote:
>
> From: Khem Raj <raj.k...@gmail.com>
>
> ported from
> http://git.alpinelinux.org/cgit/aports/plain/testing/connman/libresolv.patch
>
> Upstream-Status: Pending
>
> Signed-off-by: Khem Raj <raj.k...@gmail.com>
> Signed-off-by: Ross Burton <ross.bur...@intel.com>
> ---
>  gweb/gresolv.c | 34 +++++++++++++---------------------
>  1 file changed, 13 insertions(+), 21 deletions(-)
>
> diff --git a/gweb/gresolv.c b/gweb/gresolv.c
> index 38a554e0..a9e87402 100644
> --- a/gweb/gresolv.c
> +++ b/gweb/gresolv.c
> @@ -36,6 +36,7 @@
>  #include <arpa/inet.h>
>  #include <arpa/nameser.h>
>  #include <net/if.h>
> +#include <ctype.h>
>
>  #include "gresolv.h"
>
> @@ -877,8 +878,6 @@ GResolv *g_resolv_new(int index)
>         resolv->index = index;
>         resolv->nameserver_list = NULL;
>
> -       res_ninit(&resolv->res);
> -
>         return resolv;
>  }
>
> @@ -918,8 +917,6 @@ void g_resolv_unref(GResolv *resolv)
>
>         flush_nameservers(resolv);
>
> -       res_nclose(&resolv->res);
> -
>         g_free(resolv);
>  }
>
> @@ -1022,24 +1019,19 @@ guint g_resolv_lookup_hostname(GResolv *resolv, const 
> char *hostname,
>         debug(resolv, "hostname %s", hostname);
>
>         if (!resolv->nameserver_list) {
> -               int i;
> -
> -               for (i = 0; i < resolv->res.nscount; i++) {
> -                       char buf[100];
> -                       int family = resolv->res.nsaddr_list[i].sin_family;
> -                       void *sa_addr = &resolv->res.nsaddr_list[i].sin_addr;
> -
> -                       if (family != AF_INET &&
> -                                       resolv->res._u._ext.nsaddrs[i]) {
> -                               family = AF_INET6;
> -                               sa_addr = 
> &resolv->res._u._ext.nsaddrs[i]->sin6_addr;
> +               FILE *f = fopen("/etc/resolv.conf", "r");
> +               if (f) {
> +                       char line[256], *s;
> +                       int i;
> +                       while (fgets(line, sizeof(line), f)) {
> +                               if (strncmp(line, "nameserver", 10) || 
> !isspace(line[10]))
> +                                       continue;
> +                               for (s = &line[11]; isspace(s[0]); s++);
> +                               for (i = 0; s[i] && !isspace(s[i]); i++);
> +                               s[i] = 0;
> +                               g_resolv_add_nameserver(resolv, s, 53, 0);
>                         }
> -
> -                       if (family != AF_INET && family != AF_INET6)
> -                               continue;
> -
> -                       if (inet_ntop(family, sa_addr, buf, sizeof(buf)))
> -                               g_resolv_add_nameserver(resolv, buf, 53, 0);
> +                       fclose(f);
>                 }
>
>                 if (!resolv->nameserver_list)
> --
> 2.11.0
>


------------------------------

Message: 2
Date: Wed,  3 Oct 2018 16:10:05 +0200
From: Peter Meerwald-Stadler <pme...@pmeerw.net>
To: connman@lists.01.org
Cc: ross.bur...@intel.com
Subject: [PATCH] Fix some warnings due to GCC8's cast-function-type
Message-ID: <20181003141005.28441-1-pme...@pmeerw.net>
Content-Type: text/plain; charset=UTF-8

Reported by Ross Burton

gweb/giognutls.c: In function ?g_io_gnutls_dispatch?:
gweb/giognutls.c:307:17: error: cast between incompatible function types from 
?GSourceFunc? {aka ?int (*)$

tools/session-utils.c: In function ?util_test_add?:
tools/session-utils.c:234:4: error: cast between incompatible function types 
from ?void (*)(const void *)$
    (GTestFixtureFunc) run_test_cb,
tools/session-utils.c:235:4: error: cast between incompatible function types 
from ?void (*)(void *)? to ?$
    (GTestFixtureFunc) g_free);
---
 gweb/giognutls.c      | 2 +-
 tools/session-utils.c | 1 +
 2 files changed, 2 insertions(+), 1 deletion(-)

diff --git a/gweb/giognutls.c b/gweb/giognutls.c
index c029a8b0..b5c476cb 100644
--- a/gweb/giognutls.c
+++ b/gweb/giognutls.c
@@ -304,7 +304,7 @@ static gboolean g_io_gnutls_dispatch(GSource *source, 
GSourceFunc callback,
                                                        gpointer user_data)
 {
        GIOGnuTLSWatch *watch = (GIOGnuTLSWatch *) source;
-       GIOFunc func = (GIOFunc) callback;
+       GIOFunc func = (GIOFunc) (void (*) (void)) callback;
        GIOCondition condition = watch->pollfd.revents;
 
        DBG("source %p condition %u", source, condition);
diff --git a/tools/session-utils.c b/tools/session-utils.c
index 51cec5c3..77485f7c 100644
--- a/tools/session-utils.c
+++ b/tools/session-utils.c
@@ -26,6 +26,7 @@
 #include <stdlib.h>
 
 #pragma GCC diagnostic ignored "-Wdeprecated-declarations"
+#pragma GCC diagnostic ignored "-Wcast-function-type"
 
 #include <gdbus.h>
 
-- 
2.19.0



------------------------------

Message: 3
Date: Wed, 3 Oct 2018 14:26:17 +0000
From: Marek Szanyi <marek.sza...@kistler.com>
To: Daniel Wagner <w...@monom.org>
Cc: "connman@lists.01.org" <connman@lists.01.org>
Subject: RE: Wrong state of a service if only IPv6 is enabled
Message-ID:
        <47f3088e9b0a4140b7c47e41dc12f9f719dea...@kihagwinex01.int.kistler.com>
        
Content-Type: text/plain; charset="us-ascii"

Hi Daniel,

Yes, I was using the D-Bus API. To be more specific, I have used the 
SetProperty function from net.connman.Service object and simply set "Method" to 
"off" for IPv4 configuration.

Kind Regards,
Marek Szanyi

> -----Original Message-----
> From: Daniel Wagner [mailto:w...@monom.org]
> Sent: Tuesday, October 02, 2018 8:58 AM
> To: Szanyi Marek
> Cc: connman@lists.01.org
> Subject: Re: Wrong state of a service if only IPv6 is enabled
> 
> Hi Marek,
> 
> On Wed, Sep 26, 2018 at 11:31:31AM +0000, Marek Szanyi wrote:
> > I have a device running connman v 1.35. For the Ethernet service if
> >  IPv4 is enabled or both IPv4 and IPv6 is enabled the state of the
> >  Ethernet service is reported correctly. Because the device have
> >  internet access the state is "online". However if I disable IPv4 and
> >  leave only IPv6 enabled the state of the service reported by connman
> >  is "idle", which is wrong, because I still have internet access
> >  (verified by using curl). Is this an expected behavior or is this a
> >  bug in connman?
> 
> I would expect that the state should remain in online if IPv6 is still
> up. How do you disable IPv4? Is it via ConnMan's D-Bus API?
> 
> Thanks,
> Daniel


------------------------------

Subject: Digest Footer

_______________________________________________
connman mailing list
connman@lists.01.org
https://lists.01.org/mailman/listinfo/connman


------------------------------

End of connman Digest, Vol 36, Issue 5
**************************************

Reply via email to