[PATCH v2 1/4] config: Get configurations that are provisioned

2012-07-09 Thread Jukka Rissanen
We need the list of provisioned services so that all the hidden ones can be scanned. --- Makefile.am |2 +- include/provision.h | 51 ++ src/config.c| 68 +++ 3 files changed, 120

[PATCH v2 3/4] wifi: Scan all hidden and provisioned AP

2012-07-09 Thread Jukka Rissanen
--- plugins/wifi.c | 78 ++-- 1 file changed, 65 insertions(+), 13 deletions(-) diff --git a/plugins/wifi.c b/plugins/wifi.c index c0f7ddc..c807ab9 100644 --- a/plugins/wifi.c +++ b/plugins/wifi.c @@ -51,6 +51,7 @@ #include connman/option.h

[PATCH v2 2/4] config: Read hidden AP information from config file

2012-07-09 Thread Jukka Rissanen
--- src/config.c |8 1 file changed, 8 insertions(+) diff --git a/src/config.c b/src/config.c index 3b66a73..5a2bd11 100644 --- a/src/config.c +++ b/src/config.c @@ -91,6 +91,7 @@ static connman_bool_t cleanup = FALSE; #define SERVICE_KEY_IDENTITY Identity #define

[PATCH v2 4/4] doc: Hidden AP provisioning information added

2012-07-09 Thread Jukka Rissanen
--- doc/config-format.txt |2 ++ 1 file changed, 2 insertions(+) diff --git a/doc/config-format.txt b/doc/config-format.txt index 4f76832..d8146be 100644 --- a/doc/config-format.txt +++ b/doc/config-format.txt @@ -53,6 +53,8 @@ Allowed fields: Prefix the value with EAP- to indicate the

[PATCH] service: Allow connection if there are interfaces available

2012-07-09 Thread Jukka Rissanen
Allow user to initiate connection if there are multiple devices of the same technology type available (like having multiple wifi cards). --- Hi, This is a resend of the patch I sent two months ago. I have been testing connman with two wifi cards and it is very annoying that if the connection

[PATCH v3 1/5] doc: Hidden AP provisioning information added

2012-07-10 Thread Jukka Rissanen
--- doc/config-format.txt |2 ++ 1 file changed, 2 insertions(+) diff --git a/doc/config-format.txt b/doc/config-format.txt index 4f76832..d8146be 100644 --- a/doc/config-format.txt +++ b/doc/config-format.txt @@ -53,6 +53,8 @@ Allowed fields: Prefix the value with EAP- to indicate the

[PATCH v3 0/5] Enable hidden AP provisioning

2012-07-10 Thread Jukka Rissanen
entries in one go. Also there is now a function to free the allocated entries in patch #3. The patch #4 refactors the for loop so there is no more goto's there. Patch #5 changes a warn message to debug one. Cheers, Jukka Jukka Rissanen (5): doc: Hidden AP provisioning information added

[PATCH v3 2/5] config: Read hidden AP information from config file

2012-07-10 Thread Jukka Rissanen
--- src/config.c |9 + 1 file changed, 9 insertions(+) diff --git a/src/config.c b/src/config.c index 5363dc3..de43933 100644 --- a/src/config.c +++ b/src/config.c @@ -51,6 +51,7 @@ struct connman_config_service { GSList *service_identifiers; char *config_ident; /*

[PATCH v3 3/5] config: Get configurations that are provisioned

2012-07-10 Thread Jukka Rissanen
We need the list of provisioned services so that all the hidden ones can be scanned. --- Makefile.am |2 +- include/provision.h | 52 + src/config.c| 81 +++ 3 files changed, 134 insertions(+),

[PATCH v3 4/5] wifi: Scan all hidden and provisioned AP

2012-07-10 Thread Jukka Rissanen
--- plugins/wifi.c | 72 ++-- 1 file changed, 59 insertions(+), 13 deletions(-) diff --git a/plugins/wifi.c b/plugins/wifi.c index c0f7ddc..cf7cd95 100644 --- a/plugins/wifi.c +++ b/plugins/wifi.c @@ -51,6 +51,7 @@ #include connman/option.h

[PATCH v3 5/5] wifi: Debug print fixed

2012-07-10 Thread Jukka Rissanen
Convert a warn message to debug one because it is printed too often to be usefull as a warning message. --- plugins/wifi.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/plugins/wifi.c b/plugins/wifi.c index cf7cd95..a6da298 100644 --- a/plugins/wifi.c +++ b/plugins/wifi.c

Re: [PATCH 2/3] log: Rely on systemd sending stderr to journal

2012-07-11 Thread Jukka Rissanen
Hi Lucas, On 11 July 2012 18:23, Lucas De Marchi lucas.demar...@profusion.mobi wrote: Systemd can use the daemon output and send it to the journal (or a compatible syslog implementation). Currently in the unit file we are starting connmand with the recommended way by systemd (without forking)

Re: [PATCH v2 1/4] config: Get configurations that are provisioned

2012-07-12 Thread Jukka Rissanen
On 07/10/2012 06:07 PM, Marcel Holtmann wrote: Hi Jukka, +struct connman_config_item **connman_config_get_configs(void) +{ + GHashTableIter iter_file, iter_config; + gpointer value, key; + struct connman_config_item **configs = NULL; + int i, count = 0; + +

[PATCH] service: Gateway IP address property is not sent

2012-07-12 Thread Jukka Rissanen
The address of the IP gateway is not sent when the IP property changes. This was because of timing issues i.e., the gateway was not yet set when the other IP properties were sent. Fixes BMC#25471 --- src/service.c | 11 +++ 1 file changed, 11 insertions(+) diff --git a/src/service.c

[PATCH 0/3] NTP problems

2012-07-13 Thread Jukka Rissanen
the other address if the first timeserver cannot be contacted. This is fixed in patch #2. We should accept NTP packet containing leap second flag set. This is fixed by patch #3. Cheers, Jukka Jukka Rissanen (3): ntp: If the server cannot be contacted, try next one timeserver: Try all resolved

[PATCH 3/3] ntp: Accept ntp packet containing leap second info

2012-07-13 Thread Jukka Rissanen
Fixes BMC#25486 --- src/ntp.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/ntp.c b/src/ntp.c index 8c31fdb..f6b0f97 100644 --- a/src/ntp.c +++ b/src/ntp.c @@ -168,7 +168,7 @@ static void decode_msg(void *base, size_t len, struct timeval *tv) transmit_delay

[PATCH 2/3] timeserver: Try all resolved timeserver IP addresses

2012-07-13 Thread Jukka Rissanen
If we get more than one IP address, then add those server addresses to the time server list so that they can be tried if the first entry fails. --- src/timeserver.c |6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/src/timeserver.c b/src/timeserver.c index

Re: [PATCH] ntp: use mnemonics rather than magic numbers

2012-07-16 Thread Jukka Rissanen
Hi Grant, On 07/13/2012 07:35 PM, Grant Erickson wrote: Convert a magic number into equivalent mnemonics for the flags (leap, version, mode) packet field. --- src/ntp.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/ntp.c b/src/ntp.c index 87017ed..f257cda 100644

Re: [PATCH 2/2] ntp: specify leap-not-in-sync in flags

2012-07-16 Thread Jukka Rissanen
Hi Grant, On 07/13/2012 07:52 PM, Grant Erickson wrote: This patch sets the non-in-sync flag of the leap subfield of the packet flags field. This reflects behavior implemented by ntpdate or ntpd in one-shot mode. --- src/ntp.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff

[PATCH v2 0/2] NTP problems

2012-07-16 Thread Jukka Rissanen
Hi, this is v2 which prepares for the Grant's NTP patches. The patch #3 (ntp flags) patch from v1 is removed as it is handled by Grant's patchset that is coming soon. The patch #1 in this set fixes the timeserver list setting. Cheers, Jukka Jukka Rissanen (2): ntp: If the server cannot

[PATCH v2 1/2] ntp: If the server cannot be contacted, try next one

2012-07-16 Thread Jukka Rissanen
This could happen if the network routes are not yet setup so the sendto() could fail. In this case try next address and hope for the best. This is a rare issue but I have seen it couple of times. --- src/ntp.c |7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/src/ntp.c

[PATCH v2 2/2] timeserver: Try all resolved timeserver IP addresses

2012-07-16 Thread Jukka Rissanen
If we get more than one IP address, then add those server addresses to the time server list so that they can be tried if the first entry fails. --- src/timeserver.c | 10 -- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/src/timeserver.c b/src/timeserver.c index

Re: [PATCH] ntp: use mnemonics rather than magic numbers

2012-07-16 Thread Jukka Rissanen
On 07/16/2012 10:13 AM, Jukka Rissanen wrote: Hi Grant, On 07/13/2012 07:35 PM, Grant Erickson wrote: Convert a magic number into equivalent mnemonics for the flags (leap, version, mode) packet field. --- src/ntp.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src

[PATCH 00/34] Compile fixes

2012-07-16 Thread Jukka Rissanen
Hi, because of commit 9ae3391b2b06a9b49f94886322e3b3adb690948f the latest upstream is not compilable everywhere. This patchset fixes that. Cheers, Jukka Jukka Rissanen (34): vpn: Avoid shadowing index variable ppp: Avoid multiple shadowing variables gweb: Avoid shadowing index variable

[PATCH 01/34] vpn: Avoid shadowing index variable

2012-07-16 Thread Jukka Rissanen
--- plugins/vpn.c | 24 1 file changed, 12 insertions(+), 12 deletions(-) diff --git a/plugins/vpn.c b/plugins/vpn.c index 165c325..6c97645 100644 --- a/plugins/vpn.c +++ b/plugins/vpn.c @@ -175,20 +175,20 @@ vpn_exit: int vpn_set_ifname(struct connman_provider

[PATCH 02/34] ppp: Avoid multiple shadowing variables

2012-07-16 Thread Jukka Rissanen
--- scripts/libppp-plugin.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/scripts/libppp-plugin.c b/scripts/libppp-plugin.c index e77b8cc..cae7d8c 100644 --- a/scripts/libppp-plugin.c +++ b/scripts/libppp-plugin.c @@ -84,7 +84,7 @@ static int

[PATCH 03/34] gweb: Avoid shadowing index variable

2012-07-16 Thread Jukka Rissanen
--- gweb/gweb.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/gweb/gweb.c b/gweb/gweb.c index 27ed634..96e67ef 100644 --- a/gweb/gweb.c +++ b/gweb/gweb.c @@ -201,11 +201,11 @@ static void flush_sessions(GWeb *web) web-session_list = NULL; } -GWeb

[PATCH 04/34] gresolv: Avoid shadowing index variable

2012-07-16 Thread Jukka Rissanen
--- gweb/gresolv.c |6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/gweb/gresolv.c b/gweb/gresolv.c index e93bf4f..c79c06c 100644 --- a/gweb/gresolv.c +++ b/gweb/gresolv.c @@ -806,11 +806,11 @@ static int connect_udp_channel(struct resolv_nameserver *nameserver)

[PATCH 05/34] tools: Avoid shadowing multiple variables

2012-07-16 Thread Jukka Rissanen
--- tools/dhcp-server-test.c |8 tools/dhcp-test.c|8 tools/resolv-test.c |4 ++-- tools/supplicant.c |4 ++-- tools/web-test.c |4 ++-- tools/wispr.c|4 ++-- tools/wpad-test.c|4 ++-- 7 files changed, 18

[PATCH 27/34] rtnl: Avoid shadowing index variable

2012-07-16 Thread Jukka Rissanen
--- src/rtnl.c | 101 +++- 1 file changed, 52 insertions(+), 49 deletions(-) diff --git a/src/rtnl.c b/src/rtnl.c index 4e7ef4d..8e103b6 100644 --- a/src/rtnl.c +++ b/src/rtnl.c @@ -193,12 +193,12 @@ static void read_uevent(struct

[PATCH 07/34] gdhcp: Avoid shadowing index variable

2012-07-16 Thread Jukka Rissanen
--- gdhcp/client.c | 26 +- gdhcp/common.c | 14 +++--- gdhcp/server.c |4 ++-- 3 files changed, 22 insertions(+), 22 deletions(-) diff --git a/gdhcp/client.c b/gdhcp/client.c index 7e8e494..5327dce 100644 --- a/gdhcp/client.c +++ b/gdhcp/client.c @@

[PATCH 09/34] unit: Avoid shadowing index variable

2012-07-16 Thread Jukka Rissanen
--- unit/test-session.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/unit/test-session.c b/unit/test-session.c index 38ed2bf..80d8e4f 100644 --- a/unit/test-session.c +++ b/unit/test-session.c @@ -48,9 +48,9 @@ static void set_session_state(struct test_session

[PATCH 10/34] ippool: Avoid shadowing index variable

2012-07-16 Thread Jukka Rissanen
--- src/ippool.c | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/src/ippool.c b/src/ippool.c index 52446ce..98b926f 100644 --- a/src/ippool.c +++ b/src/ippool.c @@ -207,14 +207,14 @@ static uint32_t get_free_block(unsigned int size) return 0; }

[PATCH 08/34] dbus: Avoid shadowing signal variable

2012-07-16 Thread Jukka Rissanen
--- src/dbus.c | 33 ++--- 1 file changed, 18 insertions(+), 15 deletions(-) diff --git a/src/dbus.c b/src/dbus.c index 38a6b92..5b46afa 100644 --- a/src/dbus.c +++ b/src/dbus.c @@ -213,20 +213,21 @@ dbus_bool_t connman_dbus_property_changed_basic(const char *path,

[PATCH 11/34] gsupplicant: Avoid shadowing signal variable

2012-07-16 Thread Jukka Rissanen
--- gsupplicant/supplicant.c |6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/gsupplicant/supplicant.c b/gsupplicant/supplicant.c index be42958..13ac617 100644 --- a/gsupplicant/supplicant.c +++ b/gsupplicant/supplicant.c @@ -1493,11 +1493,11 @@ static void

[PATCH 28/34] timeserver: Avoid shadowing index

2012-07-16 Thread Jukka Rissanen
--- src/timeserver.c |6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/timeserver.c b/src/timeserver.c index 467da52..3880024 100644 --- a/src/timeserver.c +++ b/src/timeserver.c @@ -172,7 +172,7 @@ GSList *__connman_timeserver_get_all(struct connman_service

[PATCH 13/34] ofono: Avoid shadowing index variable

2012-07-16 Thread Jukka Rissanen
--- plugins/ofono.c | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/plugins/ofono.c b/plugins/ofono.c index 1fb075c..edd6186 100644 --- a/plugins/ofono.c +++ b/plugins/ofono.c @@ -745,7 +745,7 @@ static void extract_ipv4_settings(DBusMessageIter

[PATCH 14/34] dundee: Avoid shadowing index variable

2012-07-16 Thread Jukka Rissanen
--- plugins/dundee.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/plugins/dundee.c b/plugins/dundee.c index 0dae0c9..e5813a9 100644 --- a/plugins/dundee.c +++ b/plugins/dundee.c @@ -421,7 +421,7 @@ static void extract_settings(DBusMessageIter *array,

[PATCH 18/34] service: Avoid shadowing multiple variables

2012-07-16 Thread Jukka Rissanen
--- src/service.c | 111 + 1 file changed, 56 insertions(+), 55 deletions(-) diff --git a/src/service.c b/src/service.c index dfe1752..bc6e86f 100644 --- a/src/service.c +++ b/src/service.c @@ -1123,29 +1123,29 @@ void

[PATCH 19/34] connection: Avoid shadowing index variable

2012-07-16 Thread Jukka Rissanen
--- src/connection.c | 84 +++--- 1 file changed, 42 insertions(+), 42 deletions(-) diff --git a/src/connection.c b/src/connection.c index d40f428..3fd4db1 100644 --- a/src/connection.c +++ b/src/connection.c @@ -54,7 +54,7 @@ struct gateway_data

[PATCH 20/34] provider: Avoid shadowing index variable

2012-07-16 Thread Jukka Rissanen
--- src/provider.c | 22 +++--- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git a/src/provider.c b/src/provider.c index 3f4bb1c..5380480 100644 --- a/src/provider.c +++ b/src/provider.c @@ -464,16 +464,16 @@ static void provider_append_routes(gpointer key, gpointer

[PATCH 21/34] resolver: Avoid shadowing index variable

2012-07-16 Thread Jukka Rissanen
--- src/resolver.c | 26 +- 1 file changed, 13 insertions(+), 13 deletions(-) diff --git a/src/resolver.c b/src/resolver.c index a9d1341..f772721 100644 --- a/src/resolver.c +++ b/src/resolver.c @@ -239,17 +239,17 @@ static gboolean resolver_expire_cb(gpointer

[PATCH 22/34] ipconfig: Avoid shadowing index variable

2012-07-16 Thread Jukka Rissanen
--- src/ipconfig.c | 100 1 file changed, 50 insertions(+), 50 deletions(-) diff --git a/src/ipconfig.c b/src/ipconfig.c index 5e34ac0..84d943c 100644 --- a/src/ipconfig.c +++ b/src/ipconfig.c @@ -618,7 +618,7 @@ static void

[PATCH 23/34] detect: Avoid shadowing index variable

2012-07-16 Thread Jukka Rissanen
--- src/detect.c | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/src/detect.c b/src/detect.c index f4b5d29..c1deb78 100644 --- a/src/detect.c +++ b/src/detect.c @@ -29,29 +29,29 @@ static GSList *device_list = NULL; -static struct connman_device

[PATCH 24/34] inet: Avoid shadowing index variable

2012-07-16 Thread Jukka Rissanen
--- src/inet.c | 216 ++-- 1 file changed, 108 insertions(+), 108 deletions(-) diff --git a/src/inet.c b/src/inet.c index c28c3e4..1874979 100644 --- a/src/inet.c +++ b/src/inet.c @@ -73,7 +73,7 @@ int __connman_inet_rtnl_addattr_l(struct

[PATCH 25/34] dhcp: Avoid shadowing index variable

2012-07-16 Thread Jukka Rissanen
--- src/dhcp.c |6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/dhcp.c b/src/dhcp.c index 3425b6d..c0455c8 100644 --- a/src/dhcp.c +++ b/src/dhcp.c @@ -396,13 +396,13 @@ static int dhcp_request(struct connman_dhcp *dhcp) GDHCPClient *dhcp_client;

[PATCH 26/34] dhcpv6: Avoid shadowing index variable

2012-07-16 Thread Jukka Rissanen
--- src/dhcpv6.c | 24 1 file changed, 12 insertions(+), 12 deletions(-) diff --git a/src/dhcpv6.c b/src/dhcpv6.c index 574b19e..9df2698 100644 --- a/src/dhcpv6.c +++ b/src/dhcpv6.c @@ -160,7 +160,7 @@ static gchar *convert_to_hex(unsigned char *buf, int len) */

[PATCH 29/34] technology: Avoid shadowing multiple variables

2012-07-16 Thread Jukka Rissanen
--- src/technology.c | 42 +- 1 file changed, 21 insertions(+), 21 deletions(-) diff --git a/src/technology.c b/src/technology.c index 6d020cb..e3313e1 100644 --- a/src/technology.c +++ b/src/technology.c @@ -478,21 +478,21 @@ static void

[PATCH 30/34] tethering: Avoid shadowing index variable

2012-07-16 Thread Jukka Rissanen
--- src/tethering.c | 26 +- 1 file changed, 13 insertions(+), 13 deletions(-) diff --git a/src/tethering.c b/src/tethering.c index decb112..0905b3b 100644 --- a/src/tethering.c +++ b/src/tethering.c @@ -131,15 +131,15 @@ static GDHCPServer *dhcp_server_start(const char

[PATCH 31/34] wpad: Avoid shadowing index variable

2012-07-16 Thread Jukka Rissanen
--- src/wpad.c | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/src/wpad.c b/src/wpad.c index 4e5834e..771a4b1 100644 --- a/src/wpad.c +++ b/src/wpad.c @@ -126,7 +126,7 @@ int __connman_wpad_start(struct connman_service *service) struct connman_wpad

[PATCH 32/34] wispr: Avoid shadowing index variable

2012-07-16 Thread Jukka Rissanen
--- src/wispr.c | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/src/wispr.c b/src/wispr.c index 9412f1e..9d4de6d 100644 --- a/src/wispr.c +++ b/src/wispr.c @@ -859,26 +859,26 @@ int __connman_wispr_start(struct connman_service *service, { struct

[PATCH 33/34] 6to4: Avoid shadowing index variable

2012-07-16 Thread Jukka Rissanen
--- src/6to4.c | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/src/6to4.c b/src/6to4.c index 340b69e..5990265 100644 --- a/src/6to4.c +++ b/src/6to4.c @@ -137,13 +137,13 @@ static int tunnel_add_route() { struct __connman_inet_rtnl_handle rth;

[PATCH 34/34] bridge: Avoid shadowing index variable

2012-07-16 Thread Jukka Rissanen
--- src/bridge.c | 22 +++--- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git a/src/bridge.c b/src/bridge.c index 2849ef7..f251e2e 100644 --- a/src/bridge.c +++ b/src/bridge.c @@ -112,28 +112,28 @@ int __connman_bridge_remove(const char *name) int

Re: [PATCH] rtnl: support PPP connections

2012-07-17 Thread Jukka Rissanen
Hi Forest, On 06/21/2012 12:06 AM, Forest Bond wrote: From: Forest Bond forest.b...@rapidrollout.com The ARP header type for PPP connections is ARPHRD_PPP (512). --- src/rtnl.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/src/rtnl.c b/src/rtnl.c index

Re: [PATCH 00/13] NTP: Address Failures to Synchronize

2012-07-17 Thread Jukka Rissanen
Hi Grant, On 07/16/2012 07:47 PM, Grant Erickson wrote: This patch set is an omnibus roll-up of several patches submitted Friday and over the weekend to address BMC#25486 https://bugs.meego.com/show_bug.cgi?id=25486. Two key problems are addressed in this set: 1) The NTP client will send

Re: [PATCH] gdhcp: Set client state to INIT_SELECTING on restart

2012-07-17 Thread Jukka Rissanen
Hi Forest, On 07/16/2012 04:09 PM, Forest Bond wrote: From: Forest Bond forest.b...@rapidrollout.com With the previous behavior, we would end up sending a discover message and then ignoring the reply due to the client state mismatch, so a restart would never result in successful acquisition of

Re: [PATCH] Emit PropertyChanged signal for the Clock.Time property if ntp sets a new time

2012-07-17 Thread Jukka Rissanen
Hi Fernando, On 07/16/2012 01:16 PM, Fernando Herrera wrote: --- src/clock.c | 14 ++ src/connman.h |1 + src/ntp.c |1 + 3 files changed, 16 insertions(+) diff --git a/src/clock.c b/src/clock.c index e2cc19f..f3f118a 100644 --- a/src/clock.c +++ b/src/clock.c

Re: making ConnMan do DNS instead of writing /etc/resolv.conf

2012-07-18 Thread Jukka Rissanen
Hi Erik, On 07/18/2012 03:06 PM, Erik Bernoth wrote: Hi Marcel, thanks for your reply! 2012/7/18 Marcel Holtmann mar...@holtmann.org: Hi Erik, actually ConnMan creates a local DNS proxy and cache at localhost. And it should put 127.0.0.1 nameserver in /etc/resolv.conf. The -r option

Re: connman fails with Aborting (signal 4)

2012-07-20 Thread Jukka Rissanen
Hi Jan, On 07/20/2012 11:45 AM, Jan Pohanka wrote: Hello, I'd like to try connman on our embedded system on arm (dm365) with linux. There are two interfaces eth0 and wlan0 (both as usb dongles). After running connman it unfortunately fails immediately with following log. Could you please give

Re: [PATCH] technology: save tethering identifier and passphrase

2012-07-24 Thread Jukka Rissanen
Hi Julien, On 07/24/2012 01:31 PM, Julien Massot wrote: --- src/technology.c | 25 +++-- 1 files changed, 19 insertions(+), 6 deletions(-) diff --git a/src/technology.c b/src/technology.c index 6d020cb..74871a6 100644 --- a/src/technology.c +++ b/src/technology.c @@

Re: BUG# 25559 - Enhance Wi-Fi connection

2012-07-24 Thread Jukka Rissanen
Hi Danny, On 07/24/2012 03:00 PM, Danny Jeongseok Seo wrote: BUG# 25559 - Enhance Wi-Fi connection Current ConnMan, the DHCP process shows quite worse performance than Android and iPhone. I have tested with TIZEN-ConnMan (0.78.4) and only IPv4 case. (with the same H/W) ConnMan takes 3.12

Re: [PATCH 0/5] dhcp: Enhance Wi-Fi connection

2012-07-25 Thread Jukka Rissanen
Hi Danny, On 07/25/2012 09:36 AM, Danny Jeongseok Seo wrote: This patch set is to skip DHCP discovery if server address already known. Bug# 2559 - Enhance Wi-Fi connection https://bugs.meego.com/show_bug.cgi?id=25559 Most of the patches do not apply because they are malformed (long lines are

Re: [PATCH 0/5] dhcp: Enhance Wi-Fi connection

2012-07-25 Thread Jukka Rissanen
On 07/25/2012 12:09 PM, Jukka Rissanen wrote: Hi Danny, On 07/25/2012 09:36 AM, Danny Jeongseok Seo wrote: This patch set is to skip DHCP discovery if server address already known. Bug# 2559 - Enhance Wi-Fi connection https://bugs.meego.com/show_bug.cgi?id=25559 Most of the patches do

Re: [PATCH 0/5] dhcp: Enhance Wi-Fi connection

2012-07-25 Thread Jukka Rissanen
On 07/25/2012 12:39 PM, Jukka Rissanen wrote: On 07/25/2012 12:09 PM, Jukka Rissanen wrote: Hi Danny, On 07/25/2012 09:36 AM, Danny Jeongseok Seo wrote: This patch set is to skip DHCP discovery if server address already known. Bug# 2559 - Enhance Wi-Fi connection https://bugs.meego.com

Re: BeagleBone - must unplug Ethernet before wireless can connect

2012-07-26 Thread Jukka Rissanen
Hi Jerrill, On 07/26/2012 06:26 AM, Jerrill Johnson wrote: On the BeagleBone, one must disconnect the Ethernet cable before a wireless connection can be made. Out of curiosity, what's the reason for this limitation? Thanks, Jerrill There is no such limitation in connman side. Perhaps you

Re: Ethernet Preference over Cellular

2012-07-26 Thread Jukka Rissanen
Hi Brian, On 26 July 2012 20:35, Brian Ruptash brupt...@gmail.com wrote: I'm using connman 1.1 and ofono 1.7 on an embedded device with a 3G/GPRS modem, Ethernet, and eventually I'll add USB cdc_ethernet. But that's not what I want ;) I need to use the Ethernet/cdc_ethernet interfaces for

Re: BeagleBone - must unplug Ethernet before wireless can connect

2012-07-27 Thread Jukka Rissanen
Hi Jerrill, On 07/27/2012 05:02 AM, Jerrill Johnson wrote: On Thu, Jul 26, 2012 at 4:12 AM, Jukka Rissanen jukka.rissa...@linux.intel.com wrote: Hi Jerrill, On 07/26/2012 06:26 AM, Jerrill Johnson wrote: On the BeagleBone, one must disconnect the Ethernet cable before a wireless

[PATCH 1/3] connection: Add more debugging when setting VPN routes

2012-07-27 Thread Jukka Rissanen
--- src/connection.c |7 +++ 1 file changed, 7 insertions(+) diff --git a/src/connection.c b/src/connection.c index d40f428..2e5cea6 100644 --- a/src/connection.c +++ b/src/connection.c @@ -149,6 +149,9 @@ static void set_vpn_routes(struct gateway_data *new_gateway, char *dest;

[PATCH 0/3] Misc VPN fixes

2012-07-27 Thread Jukka Rissanen
to connect to OpenConnect VPN once. Cheers, Jukka Jukka Rissanen (3): connection: Add more debugging when setting VPN routes connection: Set VPN default route to the VPN interface provider: Allow VPN connection if not in connected state src/connection.c | 27 +++ src

[PATCH 2/3] connection: Set VPN default route to the VPN interface

2012-07-27 Thread Jukka Rissanen
Do not bother setting individual routes for default gateway because they do not seem to work properly (there is extra and incorrect route created). For VPN we set the whole interface as a default gateway route. Example of non working routes: Kernel IP routing table Destination Gateway

[PATCH 3/3] provider: Allow VPN connection if not in connected state

2012-07-27 Thread Jukka Rissanen
Allow user to connect to VPN via dbus and supplying all VPN parameters if the provider already exists but the VPN is not in connected state. This means that the user can call the Manager.ConnectProvider interface multiple times to connect to VPN service. This patch is needed so that OpenConnect

Re: BeagleBone tethering support (in lieu of adhoc support)

2012-07-30 Thread Jukka Rissanen
Hi Jerrill, On 07/27/2012 09:35 PM, Jerrill Johnson wrote: So your adapter supports ad-hoc mode. To support tethering, your adapater needs support AP(or master) mode. Another thing is that your kernel needs CONFIG_BRIDGE enabled. Bests Jeff Jeff, Thanks for the info. I wasn't

Re: Reconnect after switching IPv4 method from off to dhcp or manual

2012-07-30 Thread Jukka Rissanen
Hi Jonathan, On 07/28/2012 07:42 AM, Jonathan Liu wrote: Hi, I have IPv4 method set to dhcp. If I set it to off, wait a few moments and set it back to dhcp the connection is not restored. Same thing happens if I switch from off to manual. Is this a bug or do I need to do something else to

Re: [PATCH v2 2/2] wifi: try only once when connecting to a non-favorite service

2012-07-30 Thread Jukka Rissanen
Hi Julien, On 07/30/2012 12:04 PM, Julien Massot wrote: --- plugins/wifi.c | 10 +- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/plugins/wifi.c b/plugins/wifi.c index 8f3fc6c..6856ee2 100644 --- a/plugins/wifi.c +++ b/plugins/wifi.c @@ -1423,12 +1423,20 @@ static

Re: BeagleBone tethering support (in lieu of adhoc support)

2012-07-31 Thread Jukka Rissanen
Hi Jerrill, On 07/31/2012 05:35 AM, Jerrill Johnson wrote: On Mon, Jul 30, 2012 at 3:50 AM, Jukka Rissanen jukka.rissa...@linux.intel.com wrote: Hi Jerrill, See http://linuxwireless.org/en/**users/Drivershttp://linuxwireless.org/en/users/Drivers I have D-link DWA-160 rev A2 http

Re: BeagleBone tethering support (in lieu of adhoc support)

2012-08-02 Thread Jukka Rissanen
Hi Jerrill, On 08/02/2012 01:21 AM, Jerrill Johnson wrote: On Tue, Jul 31, 2012 at 5:58 AM, Jukka Rissanen jukka.rissa...@linux.intel.com wrote: Hi Jerrill, The culprit is probably the wpa_supplicant, it needs CONFIG_AP set in the config file when supplicant is compiled. You can see

[PATCH 0/2] Fix IPv46 method setting from OFF to DHCP/AUTO

2012-08-02 Thread Jukka Rissanen
and IPv4 methods are OFF, then skip the service. This patch fixes this bug https://bugs.meego.com/show_bug.cgi?id=25587 Cheers, Jukka Jukka Rissanen (2): ipconfig: Add function to check if the ipconfig is OFF service: Do not try to connect to service that cannot create connection src

[PATCH 2/2] service: Do not try to connect to service that cannot create connection

2012-08-02 Thread Jukka Rissanen
Check if the service IPv4/6 ipconfig is turned OFF, in this case do not try to connect to that service because it will not succeed. Fixes BMC#25587 --- src/service.c | 18 ++ 1 file changed, 18 insertions(+) diff --git a/src/service.c b/src/service.c index dfe1752..90e280d

[RFC 0/5] Separate routing table for each interface

2012-08-06 Thread Jukka Rissanen
Hi, This patchset provides functionality for setting up a separate routing table for each interface. The patchset needs cgroup patches from Daniel to fully support application based routing so this is just preliminary work. Cheers, Jukka Jukka Rissanen (5): inet: Add functions to setup fwmark

[RFC 1/5] inet: Add functions to setup fwmark to routing table

2012-08-06 Thread Jukka Rissanen
--- src/connman.h |2 ++ src/inet.c| 77 + 2 files changed, 79 insertions(+) diff --git a/src/connman.h b/src/connman.h index 6726078..71ea1ba 100644 --- a/src/connman.h +++ b/src/connman.h @@ -193,6 +193,8 @@ int

[RFC 2/5] inet: Add function to setup default route to a routing table

2012-08-06 Thread Jukka Rissanen
--- src/connman.h |1 + src/inet.c| 63 + 2 files changed, 64 insertions(+) diff --git a/src/connman.h b/src/connman.h index 71ea1ba..c70e5cc 100644 --- a/src/connman.h +++ b/src/connman.h @@ -195,6 +195,7 @@ int

[RFC 5/5] ipconfig: Set a default route to a routing table

2012-08-06 Thread Jukka Rissanen
--- src/ipconfig.c |5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/src/ipconfig.c b/src/ipconfig.c index 5e34ac0..5d597de 100644 --- a/src/ipconfig.c +++ b/src/ipconfig.c @@ -1206,11 +1206,14 @@ int __connman_ipconfig_gateway_add(struct connman_ipconfig *ipconfig)

[PATCH v2 2/4] service: Do not try to connect to service that cannot create connection

2012-08-07 Thread Jukka Rissanen
Check if the service IPv4/6 ipconfig is turned OFF, in this case do not try to connect to that service because it will not succeed. Fixes BMC#25587 --- src/service.c | 16 1 file changed, 16 insertions(+) diff --git a/src/service.c b/src/service.c index dfe1752..abd429c

[PATCH v2 3/4] service: Do not set the service state when changing IPv4 method

2012-08-07 Thread Jukka Rissanen
The service state will be set to proper state if we are actually doing the connection so do not set the state when we are just setting the IPv4 method. --- src/service.c |3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/service.c b/src/service.c index abd429c..e4c13dc

[PATCH v2 4/4] service: Initiate autoconnect after changing ipconfig method

2012-08-07 Thread Jukka Rissanen
--- src/service.c |2 ++ 1 file changed, 2 insertions(+) diff --git a/src/service.c b/src/service.c index e4c13dc..9275e54 100644 --- a/src/service.c +++ b/src/service.c @@ -2945,6 +2945,7 @@ static int set_ipconfig(struct connman_service *service, method ==

[PATCH v2 0/4] Misc VPN fixes

2012-08-07 Thread Jukka Rissanen
Hi, this v2 only adds patch #4 where we check if the VPN server or user is trying to set invalid host route to VPN server via the VPN tunnel. That will not work so we must discard that route. Cheers, Jukka Jukka Rissanen (4): connection: Add more debugging when setting VPN routes

[PATCH v2 1/4] connection: Add more debugging when setting VPN routes

2012-08-07 Thread Jukka Rissanen
--- src/connection.c |7 +++ 1 file changed, 7 insertions(+) diff --git a/src/connection.c b/src/connection.c index d40f428..2e5cea6 100644 --- a/src/connection.c +++ b/src/connection.c @@ -149,6 +149,9 @@ static void set_vpn_routes(struct gateway_data *new_gateway, char *dest;

[PATCH v2 2/4] connection: Set VPN default route to the VPN interface

2012-08-07 Thread Jukka Rissanen
Do not bother setting individual routes for default gateway because they do not seem to work properly (there is extra and incorrect route created). For VPN we set the whole interface as a default gateway route. Example of non working routes: Kernel IP routing table Destination Gateway

[PATCH v2 3/4] provider: Allow VPN connection if not in connected state

2012-08-07 Thread Jukka Rissanen
Allow user to connect to VPN via dbus and supplying all VPN parameters if the provider already exists but the VPN is not in connected state. This means that the user can call the Manager.ConnectProvider interface multiple times to connect to VPN service. This patch is needed so that OpenConnect

[PATCH v2 4/4] provider: Discard routes to VPN server via VPN tunnel

2012-08-07 Thread Jukka Rissanen
These obsolete routes must be discarded as we cannot contact VPN server via VPN tunnel. --- src/provider.c | 11 +++ 1 file changed, 11 insertions(+) diff --git a/src/provider.c b/src/provider.c index ebc7055..7ba1670 100644 --- a/src/provider.c +++ b/src/provider.c @@ -466,6 +466,17

[PATCH v3 0/4] Misc VPN fixes

2012-08-08 Thread Jukka Rissanen
Hi, this v3 resolves the VPN server address if necessary. This is needed so that we can discard any routes to VPN server via VPN tunnel. Cheers, Jukka Jukka Rissanen (4): connection: Add more debugging when setting VPN routes connection: Set VPN default route to the VPN interface

[PATCH v3 1/4] connection: Add more debugging when setting VPN routes

2012-08-08 Thread Jukka Rissanen
--- src/connection.c |7 +++ 1 file changed, 7 insertions(+) diff --git a/src/connection.c b/src/connection.c index d40f428..2e5cea6 100644 --- a/src/connection.c +++ b/src/connection.c @@ -149,6 +149,9 @@ static void set_vpn_routes(struct gateway_data *new_gateway, char *dest;

[PATCH v3 3/4] provider: Allow VPN connection if not in connected state

2012-08-08 Thread Jukka Rissanen
Allow user to connect to VPN via dbus and supplying all VPN parameters if the provider already exists but the VPN is not in connected state. This means that the user can call the Manager.ConnectProvider interface multiple times to connect to VPN service. This patch is needed so that OpenConnect

[PATCH v3 4/4] provider: Discard routes to VPN server via VPN tunnel

2012-08-08 Thread Jukka Rissanen
These obsolete routes must be discarded as we cannot contact VPN server via VPN tunnel. If the VPN server address is not yet resolved, we do that while starting the VPN client process. Example of non working routes: Kernel IP routing table Destination Gateway Genmask Flags

[PATCH 0/2] Disconnect VPN when underlaying service disconnects

2012-08-09 Thread Jukka Rissanen
Hi, The VPN connection was not disconnected in all cases when the service that was serving the VPN was disconnected. Cheers, Jukka Jukka Rissanen (2): inet: Add function to return route to a given host connection: Disconnect VPN when underlaying service disconnects src/connection.c

[PATCH 1/2] inet: Add function to return route to a given host

2012-08-09 Thread Jukka Rissanen
--- src/connman.h |6 +++ src/inet.c| 160 + 2 files changed, 166 insertions(+) diff --git a/src/connman.h b/src/connman.h index e72bc2a..6593360 100644 --- a/src/connman.h +++ b/src/connman.h @@ -153,6 +153,12 @@ int

[PATCH 2/2] connection: Disconnect VPN when underlaying service disconnects

2012-08-09 Thread Jukka Rissanen
Fixes BMC#25128 --- src/connection.c | 109 ++ 1 file changed, 68 insertions(+), 41 deletions(-) diff --git a/src/connection.c b/src/connection.c index 7212af3..a26a13a 100644 --- a/src/connection.c +++ b/src/connection.c @@ -106,10 +106,7 @@

[PATCH 0/2] Ignore received DHCP hostname option

2012-08-09 Thread Jukka Rissanen
Hi, This patchset disables the setting of system hostname according to received DHCP hostname option if system config file option is set. This one fixes https://bugs.meego.com/show_bug.cgi?id=25597 Cheers, Jukka Jukka Rissanen (2): main: Introduce main.conf option to disable DHCP hostname

[PATCH 1/2] main: Introduce conf file option to ignore DHCP hostname option

2012-08-09 Thread Jukka Rissanen
The DisableDHCPHostnameOption in system config file can be used to ignore the received DHCP hostname option (12). --- src/main.c| 13 + src/main.conf |6 ++ 2 files changed, 19 insertions(+) diff --git a/src/main.c b/src/main.c index 1e4809f..70c73ef 100644 ---

[PATCH 2/2] dhcp: Apply received hostname option according to conf file setting

2012-08-09 Thread Jukka Rissanen
We do not use the received DHCP hostname option (12) if system config file setting (DisableDHCPHostnameOption) says so. Fixes BMC#25597 --- src/dhcp.c | 10 +++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/src/dhcp.c b/src/dhcp.c index 3425b6d..ab3b0cf 100644 ---

<    4   5   6   7   8   9   10   11   12   13   >