Re: [PATCH] gsupplicant: Fix the issue about get an invalid group structure

[Tomasz Bursztyka]

Hi,


peer-groups store group's path, not GSupplicantGroup, so get it
from group_mapping


Of course... Don't ask me why I introduced this bug. Nice catch.

@Patrik: Apply that patch in priority please!

Thanks,

Tomasz
___
connman mailing list
connman@connman.net
https://lists.connman.net/mailman/listinfo/connman

Re: [PATCH] gsupplicant: Fix the issue about get an invalid group structure

[Patrik Flykt]

On Thu, 2014-08-07 at 09:35 +0800, Guoqiang Liu wrote:
 From: Guoqiang Liu guoqiang@archermind.com
 
 peer-groups store group's path, not GSupplicantGroup, so get it
 from group_mapping

Applied, thanks!

Patrik

___
connman mailing list
connman@connman.net
https://lists.connman.net/mailman/listinfo/connman

[PATCH] gsupplicant: Fix the issue about get an invalid group structure

[Guoqiang Liu]

From: Guoqiang Liu guoqiang@archermind.com

peer-groups store group's path, not GSupplicantGroup, so get it
from group_mapping
---
 gsupplicant/supplicant.c |6 +-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/gsupplicant/supplicant.c b/gsupplicant/supplicant.c
index 94bf611..a76ee2e 100644
--- a/gsupplicant/supplicant.c
+++ b/gsupplicant/supplicant.c
@@ -1116,7 +1116,11 @@ bool g_supplicant_peer_is_client(GSupplicantPeer *peer)
return false;
 
for (list = peer-groups; list; list = list-next) {
-   group = list-data;
+   const char *path = list-data;
+
+   group = g_hash_table_lookup(group_mapping, path);
+   if (!group)
+   continue;
 
if (group-role != G_SUPPLICANT_GROUP_ROLE_CLIENT ||
group-orig_interface != peer-interface)
-- 
1.7.9.5

___
connman mailing list
connman@connman.net
https://lists.connman.net/mailman/listinfo/connman