Hi Samuel,
> I smell some long debugging session here :) Thanks a lot for that.
Yes indeed :)
> The patch makes sense to me, but I wonder if something like:
>
> diff --git a/gsupplicant/supplicant.c b/gsupplicant/supplicant.c
> index 42f4920..b30dce5 100644
> --- a/gsupplicant/supplicant.c
> +++ b/gsupplicant/supplicant.c
> @@ -1170,7 +1170,8 @@ static void interface_bss_added(DBusMessageIter *iter,
> void *user_data)
> bss->path = g_strdup(path);
>
> dbus_message_iter_next(iter);
> - if (dbus_message_iter_get_arg_type(iter) != DBUS_TYPE_INVALID) {
> + if (dbus_message_iter_get_arg_type(iter) != DBUS_TYPE_INVALID &&
> + dbus_message_iter_get_arg_type(iter) != DBUS_TYPE_OBJECT_PATH) {
> supplicant_dbus_property_foreach(iter, bss_property, bss);
> bss_property(NULL, NULL, bss);
> return;
>
>
> I may be missing something here though, feel free to point me at it.
I did try this at the first go but the problem with this is that we still miss
alternate BSSs due to dbus_message_iter_next being done twice. Hence I felt
separating the cases is a better approach.
Thanks,
Leena.
_______________________________________________
connman mailing list
[email protected]
http://lists.connman.net/listinfo/connman
