Re: Understanding connection flow in connman

2012-10-23 Thread Tomasz Bursztyka
Hi, Which version of connman? indicator-network is pretty old, and probably not compatible with version 1.x of connman. ___ connman mailing list connman@connman.net http://lists.connman.net/listinfo/connman

Re: Understanding connection flow in connman

2012-10-23 Thread Patrik Flykt
On Mon, 2012-10-22 at 19:52 +, Raseel Bhagat wrote: I have uninstalled Network Manager and installed indicator-network so that connman is my Connection Manager I'm trying to get my basic flows right with cmcc. I don't know how compatible cmcc or indicator-network is with recent ConnMan 1.x

[PATCH] wispr: Free wispr portal context in the relevant place

2012-10-23 Thread Tomasz Bursztyka
This fixes 2 issues: - calling __connman_service_ipconfig_indicate_state() might lead to restart wispr_portal check, so context should not be freed afterward but before hand. - freeing the context in wispr_manage_message() should not happen since wispr_manage_message() will return back to

Re: Understanding connection flow in connman

2012-10-23 Thread Tomasz Bursztyka
Hi I was using connman v1.3. Maybe I should take a look at the more recent version. This will not work with cmcc/indicator-network. Will take a look at them and see if I can figure things out. Are there, by any chance, other application front-ends that use connman for connection management

RE: Understanding connection flow in connman

2012-10-23 Thread Raseel Bhagat
-Original Message- From: connman-boun...@connman.net [mailto:connman-boun...@connman.net] On Behalf Of Tomasz Bursztyka Sent: Tuesday, October 23, 2012 1:05 PM To: connman@connman.net Subject: Re: Understanding connection flow in connman Hi I was using connman v1.3. Maybe I should take

Re: [PATCH] wispr: Free wispr portal context in the relevant place

2012-10-23 Thread Patrik Flykt
On Tue, 2012-10-23 at 10:26 +0300, Tomasz Bursztyka wrote: This fixes 2 issues: - calling __connman_service_ipconfig_indicate_state() might lead to restart wispr_portal check, so context should not be freed afterward but before hand. - freeing the context in wispr_manage_message() should not

Re: [PATCH v1 10/24] gdbus: Add GetConnectionSELinuxSecurityContext

2012-10-23 Thread Daniel Wagner
Hi Patrik, On 19.10.2012 21:35, Patrik Flykt wrote: Hi, gdbus/ should be kept in sync between Bluez, ConnMan, oFono and neard. Any other code should be added elsewhere. Maybe we should move this part into a plugin for LSM, so that we can support more than SELinux. On Wed,

[PATCH v2 1/2] openvpn: Add more options

2012-10-23 Thread patrik . flykt
From: Emil Renner Berthing es...@mailme.dk Connman OpenVPN OpenVPN.NSCertType --ns-cert-type client|server OpenVPN.AskPass --askpass file OpenVPN.AuthNoCache --auth-nocache --- plugins/openvpn.c |3 +++ 1 file changed, 3 insertions(+) diff --git a/plugins/openvpn.c

[PATCH v2 0/2] Openvpn -tls-auth file with optional argument

2012-10-23 Thread patrik . flykt
From: Patrik Flykt patrik.fl...@linux.intel.com Hi, Here's a fixed version 2 of the patch set by Emil Renner Berthing. Please review, Patrik Emil Renner Berthing (2): openvpn: Add more options openvpn: Support '--tls-auth file' with optional 'direction' argument

[PATCH v2 2/2] openvpn: Support '--tls-auth file' with optional 'direction' argument

2012-10-23 Thread patrik . flykt
From: Emil Renner Berthing es...@mailme.dk Connman OpenVPN OpenVPN.TLSAuth --tls-auth file [direction] OpenVPN.TLSAuthDir optional direction argument to '--tls-auth' Since 'OpenVPN.TLSAuthDir' contains the optional direction argument for 'OpenVPN.TLSAuth', handle them

Re: [PATCH v1 10/24] gdbus: Add GetConnectionSELinuxSecurityContext

2012-10-23 Thread Patrik Flykt
Hi, On Tue, 2012-10-23 at 10:42 +0200, Daniel Wagner wrote: +static int parse_context(DBusMessage *msg, unsigned char **context, + unsigned int *size) +{ + DBusMessageIter iter, array; + unsigned char *ctx; + int i = 0; + +

Re: [PATCH v1 10/24] gdbus: Add GetConnectionSELinuxSecurityContext

2012-10-23 Thread Daniel Wagner
On 23.10.2012 12:36, Patrik Flykt wrote: Hi, On Tue, 2012-10-23 at 10:42 +0200, Daniel Wagner wrote: +static int parse_context(DBusMessage *msg, unsigned char **context, + unsigned int *size) +{ + DBusMessageIter iter, array; + unsigned char

Re: [PATCH v1 10/24] gdbus: Add GetConnectionSELinuxSecurityContext

2012-10-23 Thread Lucas De Marchi
On Tue, Oct 23, 2012 at 6:42 AM, Daniel Wagner w...@monom.org wrote: Hi Patrik, On 19.10.2012 21:35, Patrik Flykt wrote: Hi, gdbus/ should be kept in sync between Bluez, ConnMan, oFono and neard. Any other code should be added elsewhere. Maybe we should move this part into a

Re: [PATCH v1 10/24] gdbus: Add GetConnectionSELinuxSecurityContext

2012-10-23 Thread Daniel Wagner
On 23.10.2012 15:21, Lucas De Marchi wrote: On Tue, Oct 23, 2012 at 6:42 AM, Daniel Wagner w...@monom.org wrote: Hi Patrik, On 19.10.2012 21:35, Patrik Flykt wrote: Hi, gdbus/ should be kept in sync between Bluez, ConnMan, oFono and neard. Any other code should be added

Re: [PATCH v1 21/24] session: Export session parsing functions

2012-10-23 Thread Daniel Wagner
On 22.10.2012 09:51, Patrik Flykt wrote: On Mon, 2012-10-22 at 09:43 +0200, Daniel Wagner wrote: I don't see it in my tree. Which line (assuming I am blind again)? parse_service_types() ? You were talking about: Functionality for connman_session_parse_allowed_bearers already exists in

Re: [PATCH v1 10/24] gdbus: Add GetConnectionSELinuxSecurityContext

2012-10-23 Thread Patrik Flykt
Hi, On Tue, 2012-10-23 at 11:21 -0200, Lucas De Marchi wrote: Btw, if you are going to touch gdbus, please consider checking the BlueZ version. We are in the middle of the transition to support DBus.Properties and DBus.ObjectManager. A lot of patches were integrated there. Since