Hi Martin,

Bug1118(https://bugzilla.moblin.org/show_bug.cgi?id=1118) is a show stop issue and need to fix right away, so I spent some time to work out a patch to fix it.
The patch now is maintained in RPM package.

I found the issue comes from:
connman inserts unmodified “identifier” to harsh table to identify the network (see src/device.c connman_device_add_network())
However,
next time connman will used modified(using ‘_’ substitute “none number and letter” char) “identifier” to search the hash table, so the registered network will be registered second time. (at src/ device.c connman_device_get_network())

I find that the network->element->name is modified. Why we need to modify the name?

I think there are two methods to fix the issue, one is to add network->element->name to harsh table to identify the network, the other one is use unmodified “identifier” to search harsh table. I prefer the last one. Below is the patch.

Can you let me know you comments? J

thanks for the analysis. Pushed your patch.

Further question, what if the two AP has the same essid? I think the same essid leads to the same identify and may cause the same issue with bug 1118. What do you think of it?

We need to be a little bit smarter with SSID grouping. It has room for improvements.

Regards

Marcel

_______________________________________________
connman mailing list
[email protected]
https://lists.moblin.org/mailman/listinfo/connman

Reply via email to