Re: Connman network refcount

2015-01-15 Thread Patrik Flykt
Hi, On Thu, 2015-01-15 at 12:16 -0800, Naveen Singh wrote: > Sorry for the confusion. I am on 1.27 with some of our local changes. Since we have no idea what your local changes are, we will not help you any further with this bug. Please look into your own modifications first. > @@ -220

Re: Connman network refcount

2015-01-15 Thread Naveen Singh
Thanks Guys. Sorry for the confusion. I am on 1.27 with some of our local changes. Here is the snippet from log stating that: messages#1523:2015-01-15 04:54:09.00 02AA01AC4013043X daemon.info connmand[299]: Connection Manager version 1.27 One of the local changes that I had was to ensure that

Re: Connman network refcount

2015-01-15 Thread Patrik Flykt
On Wed, 2015-01-14 at 19:40 -0800, Naveen Singh wrote: > Do you guys think if the following make sense: > > diff --git a/plugins/wifi.c b/plugins/wifi.c > index 97f7144..863109d 100644 > --- a/plugins/wifi.c > +++ b/plugins/wifi.c > @@ -2207,7 +2207,7 @@ static int network_connect(struct connm

Re: Connman network refcount

2015-01-14 Thread Tomasz Bursztyka
Hi Naveen, Again you don't seem to have the latest upstream version of ConnMan (1.27) network_connect() has no ret variable. It either returns g_supplicant_interface_connect() status directly or returns -EINPROGRESS. Which version of ConnMan are you using? With what patches on top, if any? We

Re: Connman network refcount

2015-01-14 Thread Tomasz Bursztyka
Hi Naveen, *Could it be because I am not using the version of wpa_supplicant that is required for working with latest connman (1.27). * 2.0 is fine, but latest and greatest it always the best (so 2.3 here). However, I already pin pointed out that you don't seem to have ConnMan 1.27, at least

Re: Connman network refcount

2015-01-14 Thread Naveen Singh
_destruct() network 0xbbba8 name > GoogleGuest ==> Network destruct tries to free the memory again and later > on there would be crash.* > > > *The call to this function **g_supplicant_interface_connect **will always > return -EINPROGRESS so there would be an increment and dec

Re: Connman network refcount

2015-01-14 Thread Naveen Singh
00 > From: Tomasz Bursztyka > To: connman@connman.net > Subject: Re: Connman network refcount > Message-ID: <54b617d8.4090...@linux.intel.com> > Content-Type: text/plain; charset=windows-1252; format=flowed > > Hi, > > > So Looks like decrementing the ref count f

Re: Connman network refcount

2015-01-13 Thread Patrik Flykt
On Tue, 2015-01-13 at 19:30 -0800, Naveen Singh wrote: > I did a simple test of connecting to a WiFi AP and What version of ConnMan and wpa_supplicant are you talking about? Cheers, Patrik ___ connman mailing list connman@connman.net https:/

Re: Connman network refcount

2015-01-13 Thread Tomasz Bursztyka
Hi, So Looks like decrementing the ref count from network removed was not correct. So either we add the network ref count from network_added function in supplicant or remove it from the network_removed function. Any thoughts? I am sorry, but what your are experiencing is really unclear. Con

Connman network refcount

2015-01-13 Thread Naveen Singh
I did a simple test of connecting to a WiFi AP and then doing a disconnect using connmanctl. I found that the connman crashed while trying to decrement the network ref count from network_removed() function in plugins/wifi.c static void network_removed(GSupplicantNetwork *network){ GSupplic