1227573 - Unify Wifi setup among emulators

Status: RESOLVED WONTFIX

(Firefox OS Graveyard :: Wifi, defect)

Description

[Thomas Zimmermann [:tzimmermann] [:tdz]]

This is something like a follow-up bug to bug 1160913. Emulators for KK and later fetch random data from /dev/urandom to guarantee progress. Older emulators should probably do the same.

Comment 1

[Thomas Zimmermann [:tzimmermann] [:tdz]]

Attachment: Github pull request for b2g-manifest

Comment 2

[Thomas Zimmermann [:tzimmermann] [:tdz]]

Attachment: Add entropy file for ICS wpa_supplicant

Hi Henry,

originally I wanted to port the existing patches from bug 1160913, but looking closer at the code, it seems possible to feed the necessary entropy from and external file into wpa_supplicant.

This patch loads the entropy into wpa_supplicant, which should then still fail to read from /dev/random [1], but pick up the mixed-in entropy afterwards. [2]

[1] https://android.googlesource.com/platform/external/wpa_supplicant_8/+/android-4.0.4_r2.1/src/crypto/random.c#217
[2] https://android.googlesource.com/platform/external/wpa_supplicant_8/+/android-4.0.4_r2.1/src/crypto/random.c#252

Comment 3

[Thomas Zimmermann [:tzimmermann] [:tdz]]

I'd like to reconsider the current patches of bug 1160913 to ease long-term maintenance.

Comment 4

[Henry Chang [:hchang]]

Comment on attachment 8691443 [details]
Add entropy file for ICS wpa_supplicant

This is awesome! It seems what vendor does to wpa_supplicant! Thank for investigating it :)

Comment 5

[Thomas Zimmermann [:tzimmermann] [:tdz]]

The other bug says that there's an automated test for Wifi on KK. How can I run it?

Comment 6

[Henry Chang [:hchang]]

Do you mean the on on emulator? It's

./mach marionette-webapi [GECKO-PATH]/dom/wifi/test/marionette/test_wifi_associate.js

Comment 7

[Thomas Zimmermann [:tzimmermann] [:tdz]]

(In reply to Henry Chang [:henry] from comment #6)
> Do you mean the on on emulator? It's
> 
> ./mach marionette-webapi
> [GECKO-PATH]/dom/wifi/test/marionette/test_wifi_associate.js

Yes, thanks. If I find some time, I'll prepare a patch and try this out.