Connman upstream test result_20130426

2013-04-26 Thread Li, XiaX
Hi all, This is test report for commit 88943c8550d7f79d503c65b6eb68558e742b1a42. In this testing, we ran 261 cases. 251 cases passed and 10 cases failed. The pass rate is 96%. We found 1 new bug, reopen 1 bug and verify 0 bug . New bug === 26001 - Con

Connman upstream test result_20130426

2013-04-26 Thread Li, XiaX
Hi all, This is test report for commit 88943c8550d7f79d503c65b6eb68558e742b1a42. In this testing, we ran 261 cases. 251 cases passed and 10 cases failed. The pass rate is 96%. We found 1 new bug, reopen 1 bug and verify 0 bug . New bug === 26001 - Con

Re: [PATCH v3 0/5] Add support for ethernet tethering

2013-04-26 Thread Patrik Flykt
On Thu, 2013-04-25 at 14:47 +0300, Jukka Rissanen wrote: > Hi, > > v3: > - Changed the name and semantics of the main.conf variable. > Now it is a list of technologies that are allowed to > tether. Patches 1, 2 and 5 > - Reworked the rtnl patch 4, get rid of extra variable > - Combined patches

Re: [PATCH 0/2] Fix tethering if dnsproxy is disabled

2013-04-26 Thread Patrik Flykt
On Tue, 2013-04-23 at 18:38 +0300, Jukka Rissanen wrote: > Hi, > > If tethering is enabled with -r option, then clients are unable > to resolve DNS addresses. This happens because tethering code > reports the gateway host (tethering device) address to clients. > If dnsproxy is disabled, there is n

Re: [PATCH v4 0/3] Start tethering automatically

2013-04-26 Thread Patrik Flykt
On Thu, 2013-04-25 at 16:24 +0300, Jukka Rissanen wrote: > Hi, > > v4: > - rebase because of ethernet tethering changes > - fix the ethernet tethering autostart Applied, thanks! Patrik ___ connman mailing list connman@connman.net https://lists.

[PATCH v2 0/2] Manual pages into dist tar ball

2013-04-26 Thread Jukka Rissanen
Hi, v2: - updated connman.conf.5 file because of changes in ethernet tethering description v1: this patchset install connmand and connman.conf manual pages into dist tar ball. The connmanctl man pages is still left out as its syntax is still fluctuating a bit. Also updated the connman.conf man

[PATCH v2 1/2] man: Update connman.conf.5 manual page

2013-04-26 Thread Jukka Rissanen
Updated PreferredTechnologies description. Added SingleConnectedTechnology, AllowedTetheringTechnologies and PersistentTetheringMode descriptions. --- doc/connman.conf.5 | 42 -- 1 file changed, 36 insertions(+), 6 deletions(-) diff --git a/doc/connman.conf

[PATCH v2 2/2] build: Install connmand and connman.conf manual pages

2013-04-26 Thread Jukka Rissanen
--- Makefile.am | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/Makefile.am b/Makefile.am index cb532a3..4680b8e 100644 --- a/Makefile.am +++ b/Makefile.am @@ -226,7 +226,7 @@ include Makefile.plugins if CLIENT noinst_PROGRAMS += client/connmanctl -MANUAL_PAGES += doc/c

Re: [PATCH 0/6] Clean up connman_uint8_t and connman_uint16_t

2013-04-26 Thread Patrik Flykt
On Tue, 2013-04-23 at 13:05 +0300, Patrik Flykt wrote: > Hi, > > The low-hanging fruits of the type definitions clean-up tasks have been > picked. With this patch set connman_uint8_t is substituted with uint8_t > and connman_uint16_t with uint16_t. > > The remaining connman_bool_t will take

Re: [PATCH 0/6] DHCPv6 transaction id fixed

2013-04-26 Thread Patrik Flykt
On Fri, 2013-04-26 at 09:51 +0300, Jukka Rissanen wrote: > Hi, > > these transaction id problems in DHCPv6 client were > reported by Harshal Patel in the ConnMan mailing list. > > First three patches are preparation for transaction > id fixes. The DHCPv6 transaction id must not be changed during

[PATCH] dnsproxy: Conditional jump depends on uninitialized value

2013-04-26 Thread Jukka Rissanen
This came from Daniel when running ConnMan using valgrind ==20635== Conditional jump or move depends on uninitialised value(s) ==20635==at 0x49ECF4: get_name (dnsproxy.c:867) ==20635==by 0x49F3F7: parse_response (dnsproxy.c:1096) ==20635==by 0x49FD40: cache_update (dnsproxy.c:1410) ==2

Re: [PATCH] make: Add main.conf example file into dist tar ball

2013-04-26 Thread Patrik Flykt
On Tue, 2013-04-23 at 13:06 +0300, Jukka Rissanen wrote: > --- > Makefile.am | 2 ++ > 1 file changed, 2 insertions(+) Applied, thanks! Patrik ___ connman mailing list connman@connman.net https://lists.connman.net/mailman/listinfo/connman

Re: [PATCH v2 0/2] Manual pages into dist tar ball

2013-04-26 Thread Patrik Flykt
On Fri, 2013-04-26 at 12:54 +0300, Jukka Rissanen wrote: > Hi, > > v2: > - updated connman.conf.5 file because of changes in ethernet > tethering description Applied, thanks! Patrik ___ connman mailing list connman@connman.net https://lists.c

Re: [PATCH] dnsproxy: Conditional jump depends on uninitialized value

2013-04-26 Thread Patrik Flykt
On Fri, 2013-04-26 at 13:12 +0300, Jukka Rissanen wrote: > This came from Daniel when running ConnMan using valgrind > > ==20635== Conditional jump or move depends on uninitialised value(s) > ==20635==at 0x49ECF4: get_name (dnsproxy.c:867) > ==20635==by 0x49F3F7: parse_response (dnsproxy.c

[PATCH 2/3] client: Remove unneeded members from structs

2013-04-26 Thread Patrik Flykt
Remove now obsolete memebers from the option struct and instead add a description field. Also remove now unnecessary include. --- client/commands.c | 160 +++-- 1 file changed, 70 insertions(+), 90 deletions(-) diff --git a/client/commands.c b/cli

[PATCH 3/3] client: Fix up wrongly included typedef

2013-04-26 Thread Patrik Flykt
--- client/dbus_helpers.c | 11 +++ client/dbus_helpers.h | 12 2 files changed, 15 insertions(+), 8 deletions(-) diff --git a/client/dbus_helpers.c b/client/dbus_helpers.c index 2cc6e61..101fa4c 100644 --- a/client/dbus_helpers.c +++ b/client/dbus_helpers.c @@ -277,7 +27

[PATCH 0/3] A few connmanctl related cleanups

2013-04-26 Thread Patrik Flykt
Hi, Two unnecessary includes detected in connmanctl. Properly define the needed structs and add a field for help text. Also nitpick about command ordering in the help section. Cheers, Patrik Patrik Flykt (3): client: Move commands around in the table into logical places cli

[PATCH 1/3] client: Move commands around in the table into logical places

2013-04-26 Thread Patrik Flykt
--- client/commands.c |8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/client/commands.c b/client/commands.c index 32a8220..14f552a 100644 --- a/client/commands.c +++ b/client/commands.c @@ -1184,12 +1184,14 @@ static const struct { int (*func) (char *args[],

[PATCH] dnsproxy: Do not unref g_io_channel if we know it is null

2013-04-26 Thread Jukka Rissanen
Currently tethering does not support IPv6 so its listener is not created and is null. This will cause following output connmand[18363]: src/dnsproxy.c:destroy_tcp_listener() index 31 (connmand:18363): GLib-CRITICAL **: g_io_channel_unref: assertion `channel != NULL' failed connmand[18363]: src/dn

[PATCH] technology: Disable tethering when device disappears

2013-04-26 Thread Jukka Rissanen
If tethering is active when device is removed, then shutdown tethering cleanly so that tether interface and bridge are properly removed. --- src/technology.c | 4 1 file changed, 4 insertions(+) diff --git a/src/technology.c b/src/technology.c index f214367..10e6132 100644 --- a/src/technolo

Re: [PATCH v3 0/5] Add support for ethernet tethering

2013-04-26 Thread Marcel Holtmann
Hi Patrik, >> v3: >> - Changed the name and semantics of the main.conf variable. >> Now it is a list of technologies that are allowed to >> tether. Patches 1, 2 and 5 >> - Reworked the rtnl patch 4, get rid of extra variable >> - Combined patches 5 and 6 together > > Applied, thanks! actually