1160913 - [Emulator][Wifi] Unable to connect to secure Wifi AP on emulator

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Wifi, defect)

Description

[Henry Chang [:hchang]]

With Bug 1001261 fixed, we could enable wifi with test driver and connect to a virtual wifi AP established by hostapd on emulator KK (arm/x86). However, when configurating hostapd with security, we are unable to connect to the AP. This bug is to fix this kind of issues.

Comment 1

[Henry Chang [:hchang]]

Attachment: logcat for connecting to secure AP

Comment 2

[Henry Chang [:hchang]]

(In reply to Henry Chang [:henry] from comment #1)
> Created attachment 8607954 [details]
> logcat for connecting to secure AP

Observed from the log, the new wpa_supplicant has "CTRL-EVENT-SSID-TEMP-DISABLED" 
design. It would temporarily disable the secure network which failed to authenticate 
the client. For some reason, the simulated AP (hostapd) reported

"WPA: Not enough entropy in random pool to proceed - reject first 4-way handshake"

error so that it cannot be connected and then be temporarily disabled.

As a result, it takes longer time to successfully connect to a secure network.
Using a longer timeout resolve this issue like a charm.

Comment 3

[Henry Chang [:hchang]]

Attachment: Bug1160913.patch

Comment 4

[Vincent Chang[:vchang][changyihsin]]

> "WPA: Not enough entropy in random pool to proceed - reject first 4-way
> handshake"
> 
> error so that it cannot be connected and then be temporarily disabled.
> 
> As a result, it takes longer time to successfully connect to a secure
> network.
> Using a longer timeout resolve this issue like a charm.

It's quite similar error found in Bug 1151726, and change /dev/random to /dev/urandom in random.c seems a better solution to fix the problem [1]. 

[1] https://bugzilla.mozilla.org/show_bug.cgi?id=1151726#c8

Comment 5

[Henry Chang [:hchang]]

(In reply to Vincent Chang[:vchang] from comment #4)
> > "WPA: Not enough entropy in random pool to proceed - reject first 4-way
> > handshake"
> > 
> > error so that it cannot be connected and then be temporarily disabled.
> > 
> > As a result, it takes longer time to successfully connect to a secure
> > network.
> > Using a longer timeout resolve this issue like a charm.
> 
> It's quite similar error found in Bug 1151726, and change /dev/random to
> /dev/urandom in random.c seems a better solution to fix the problem [1]. 
> 
> [1] https://bugzilla.mozilla.org/show_bug.cgi?id=1151726#c8

Thanks for your information. I knew this before (replace with urandom). 
However, since it's not something like test driver specific change, 
I would rather find other solution that we can control by test case
or the testbed itself for portability. 

I had it a try with urandom and it did work perfectly. Since this
change can also benefit Bug 1151726, I think it's worthy of adopting
this solution.

I am going to apply the mod and apply to the wpa_supplicant used by
emulator ICS as well.

Thanks!

Comment 6

[Henry Chang [:hchang]]

Attachment: Use /dev/urandom instead of /dev/random

Comment 7

[Henry Chang [:hchang]]

Attachment: Use /dev/urandom instead of /dev/random (for 5.1. r+'d)

Comment 8

[Kai-Zhen Li [:seinlin][:kli]]

b2g-4.4.2_r1:
b2g-5.1.0_r1 https://github.com/mozilla-b2g/platform_external_wpa_supplicant_8/commit/0c6a6547cd1fd302fa2b0f6e375654df36bf0ec4

b2g-5.1.0_r1:
https://github.com/mozilla-b2g/platform_external_wpa_supplicant_8/commit/cbda29a58abc4ea1f7f4611fe354ab67b606219d