[PATCH resend] Add assigning for element name in connman element create

2011-01-11 Thread Li Jian
connman_element_create function forgot to assign for name field of element, need to add --- src/element.c |2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/src/element.c b/src/element.c index 1dcc695..78120ce 100644 --- a/src/element.c +++ b/src/element.c @@ -597,6 +597,8

[PATCH] adjust the order of some statements for better check

2011-01-11 Thread Li Jian
___ connman mailing list connman@connman.net http://lists.connman.net/listinfo/connman

ConnMan (0.65.34) Weekly Test Report - WW502

2011-01-11 Thread Zheng, Jeff
Hi all, This is test report for ConnMan upstream connman-0.65.34. In this testing, we ran 217 cases. 205 cases passed and 12 case failed because of known/new bug. The pass rate is 95%. We found 1 new bugs Verified Bugs = NA New bugs ===

RE: [PATCH autoconf v3 06/12] ipv6: Clear the IPv6 address if that address is removed from system

2011-01-11 Thread Jukka.Rissanen
Hi Samuel, ext Samuel Ortiz wrote: > Hi Jukka, > > On Tue, Jan 11, 2011 at 12:45:33PM +, jukka.rissa...@nokia.com > wrote: >>> g_slist_remove(ipdevice->address_list, ipaddress); >>> >>> - connman_ipaddress_free(ipaddress); >>> + connman_ipaddress_clear(ipaddress); >>> + g_

Re: [PATCH] add assigning for element name in connman element create

2011-01-11 Thread Samuel Ortiz
Hi Li, On Tue, Jan 11, 2011 at 11:36:28AM +0800, Li Jian wrote: > Add assigning for element name in connman element create > > --- > src/element.c |2 ++ > 1 files changed, 2 insertions(+), 0 deletions(-) > > diff --git a/src/element.c b/src/element.c > index 1dcc695..c1c34b1 100644 > --- a

Re: [PATCH] adjust the order of some statements for better check

2011-01-11 Thread Samuel Ortiz
Hi Li, On Tue, Jan 11, 2011 at 10:26:48AM +0800, Li Jian wrote: > device: adjust the orders of some statements for better check > > In order to fastly find the crash reason, DBG() should be in the front of > functions to prevent NULL points. > > --- > src/device.c |9 ++--- > 1 files

Re: [PATCH autoconf v3 06/12] ipv6: Clear the IPv6 address if that address is removed from system

2011-01-11 Thread Samuel Ortiz
Hi Jukka, On Tue, Jan 11, 2011 at 12:45:33PM +, jukka.rissa...@nokia.com wrote: > > g_slist_remove(ipdevice->address_list, ipaddress); > > > > - connman_ipaddress_free(ipaddress); > > + connman_ipaddress_clear(ipaddress); > > + g_free(ipaddress); > > But isn't connman_ipad

[PATCH 09/15] WPS: service's wps property support in test scripts.

2011-01-11 Thread Tomasz Bursztyka
--- test/get-services |2 +- test/list-services |2 +- test/test-manager |2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/test/get-services b/test/get-services index 29d4ea0..605e278 100755 --- a/test/get-services +++ b/test/get-services @@ -43,7 +43,7 @@ for ent

[PATCH 13/15] WPS: connection logic in service.c.

2011-01-11 Thread Tomasz Bursztyka
--- src/service.c | 22 +- 1 files changed, 17 insertions(+), 5 deletions(-) diff --git a/src/service.c b/src/service.c index 38ef855..937b7e5 100644 --- a/src/service.c +++ b/src/service.c @@ -2299,9 +2299,12 @@ static void request_input_cb (struct connman_service *service

[PATCH 08/15] WPS: service's wps attribute documented in service-api.txt.

2011-01-11 Thread Tomasz Bursztyka
--- doc/service-api.txt | 10 ++ 1 files changed, 10 insertions(+), 0 deletions(-) diff --git a/doc/service-api.txt b/doc/service-api.txt index 4a8a490..40d5deb 100644 --- a/doc/service-api.txt +++ b/doc/service-api.txt @@ -214,6 +214,16 @@ Properties string State [readonly]

[PATCH 15/15] gsupplicant: Disconnect request might not need to go through RemoveNetwork call.

2011-01-11 Thread Tomasz Bursztyka
--- gsupplicant/supplicant.c |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/gsupplicant/supplicant.c b/gsupplicant/supplicant.c index 162cbc7..13eff19 100644 --- a/gsupplicant/supplicant.c +++ b/gsupplicant/supplicant.c @@ -2809,6 +2809,9 @@ static int network_remove(st

[PATCH 14/15] WPS: input added in agent api test script.

2011-01-11 Thread Tomasz Bursztyka
--- test/simple-agent | 44 1 files changed, 36 insertions(+), 8 deletions(-) diff --git a/test/simple-agent b/test/simple-agent index 859d403..5c73599 100755 --- a/test/simple-agent +++ b/test/simple-agent @@ -11,7 +11,8 @@ class Canceled(dbus.DBusE

[PATCH 12/15] WPS: input request logic in agent.c.

2011-01-11 Thread Tomasz Bursztyka
--- src/agent.c | 77 ++- 1 files changed, 76 insertions(+), 1 deletions(-) diff --git a/src/agent.c b/src/agent.c index d000a56..4d75f39 100644 --- a/src/agent.c +++ b/src/agent.c @@ -94,7 +94,9 @@ struct request_input_reply { static voi

[PATCH 11/15] WPS: service internal accessor to know wether or not wps is enabled.

2011-01-11 Thread Tomasz Bursztyka
--- src/connman.h |1 + src/service.c |8 2 files changed, 9 insertions(+), 0 deletions(-) diff --git a/src/connman.h b/src/connman.h index b58f903..a084748 100644 --- a/src/connman.h +++ b/src/connman.h @@ -474,6 +474,7 @@ const char *__connman_service_get_path(struct connman_s

[PATCH 07/15] WPS: support in service core part.

2011-01-11 Thread Tomasz Bursztyka
--- src/service.c | 23 ++- 1 files changed, 22 insertions(+), 1 deletions(-) diff --git a/src/service.c b/src/service.c index 4572e59..966588b 100644 --- a/src/service.c +++ b/src/service.c @@ -105,6 +105,7 @@ struct connman_service { char **proxies; char **

[PATCH 10/15] WPS: method input for wps added in agent api documentation.

2011-01-11 Thread Tomasz Bursztyka
--- doc/agent-api.txt | 25 - 1 files changed, 24 insertions(+), 1 deletions(-) diff --git a/doc/agent-api.txt b/doc/agent-api.txt index b4b8a39..98499a3 100644 --- a/doc/agent-api.txt +++ b/doc/agent-api.txt @@ -66,10 +66,19 @@ Fields string Name

[PATCH 05/15] WPS: support in network core part.

2011-01-11 Thread Tomasz Bursztyka
--- src/network.c | 17 + 1 files changed, 17 insertions(+), 0 deletions(-) diff --git a/src/network.c b/src/network.c index 612e923..a83fc40 100644 --- a/src/network.c +++ b/src/network.c @@ -67,6 +67,9 @@ struct connman_network { char *private_key_path;

[PATCH 06/15] WPS: support in wifi plugin (non legacy one).

2011-01-11 Thread Tomasz Bursztyka
--- plugins/wifi.c | 70 1 files changed, 70 insertions(+), 0 deletions(-) diff --git a/plugins/wifi.c b/plugins/wifi.c index 040d2f9..7d87235 100644 --- a/plugins/wifi.c +++ b/plugins/wifi.c @@ -70,6 +70,10 @@ struct wifi_data {

[PATCH 04/15] WPS: Start() method enabler added in gsupplicant library.

2011-01-11 Thread Tomasz Bursztyka
--- gsupplicant/gsupplicant.h |2 + gsupplicant/supplicant.c | 82 - 2 files changed, 83 insertions(+), 1 deletions(-) diff --git a/gsupplicant/gsupplicant.h b/gsupplicant/gsupplicant.h index 05f5c35..0df8640 100644 --- a/gsupplicant/gsupplicant.

[PATCH 03/15] WPS: event signal support added in gsupplicant library.

2011-01-11 Thread Tomasz Bursztyka
--- gsupplicant/gsupplicant.h |7 ++ gsupplicant/supplicant.c | 52 + 2 files changed, 59 insertions(+), 0 deletions(-) diff --git a/gsupplicant/gsupplicant.h b/gsupplicant/gsupplicant.h index 4f507e3..05f5c35 100644 --- a/gsupplicant/gsuppli

[PATCH 00/15] WPS support

2011-01-11 Thread Tomasz Bursztyka
Hi, Got WPS working in ConnMan. At least Push-Button works fine. There are still issues with PIN method but I am strongly suspecting wpa_supplicant. Since it adds anyway no regression whatsoever I think it is the right time to integrate that feature, if there is a bug about pin method we will

[PATCH 02/15] WPS: credentials signal support added in gsupplicant library.

2011-01-11 Thread Tomasz Bursztyka
--- gsupplicant/gsupplicant.h |3 + gsupplicant/supplicant.c | 91 + 2 files changed, 94 insertions(+), 0 deletions(-) diff --git a/gsupplicant/gsupplicant.h b/gsupplicant/gsupplicant.h index 9404402..4f507e3 100644 --- a/gsupplicant/gsupplicant.

[PATCH 01/15] WPS: bss/network wps enabled support in gsupplicant library.

2011-01-11 Thread Tomasz Bursztyka
--- gsupplicant/gsupplicant.h |1 + gsupplicant/supplicant.c | 87 - 2 files changed, 87 insertions(+), 1 deletions(-) diff --git a/gsupplicant/gsupplicant.h b/gsupplicant/gsupplicant.h index 1da9661..9404402 100644 --- a/gsupplicant/gsupplicant.

RE: [PATCH autoconf v3 06/12] ipv6: Clear the IPv6 address if that address is removed from system

2011-01-11 Thread Jukka.Rissanen
Hi Samuel, ext Samuel Ortiz wrote: > I think it would make sense to clear the address every time deladdr > is called, not only for the IPv6 system address. Something like that: > > diff --git a/src/ipconfig.c b/src/ipconfig.c index 6ef39c5..bd664f3 > 100644 --- a/src/ipconfig.c > +++ b/src/ipcon

Re: [PACRunner PATCH] Always require thread support

2011-01-11 Thread Samuel Ortiz
Hi David, On Mon, Jan 03, 2011 at 02:09:15AM +, David Woodhouse wrote: > We use threads unconditionally. Even if we don't have a JS plugin. That makes sense, I applied this patch. Cheers, Samuel. -- Intel Open Source Technology Centre http://oss.intel.com/ __

[PATCH v0 1/5] gdbus: invaldate_parent_data: walk the whole path down

2011-01-11 Thread Daniel Wagner
From: Daniel Wagner Assume there is only one object registerd at "/". If we add a new object at "/foo/bar" the introspection of "/" has to be updated. A new node has to be added at "/". invalidate_parent_data stops invaldating the whole path because the boolean return value of dbus_connection_ge

[PATCH v0 2/5] gdbus: Remove root node 'name' attribute in introspection

2011-01-11 Thread Daniel Wagner
From: Daniel Wagner generate_introspection_xml generates the root tags with a 'name' attribute. This seems to be a valid attribute but it is not consistent with the way the D-Bus daemon generates empty nodes. For example if we register "/foo/bar", D-Bus daemon will generate for "/foo" a introsp