Kevin Brosnan [Ex-Mozilla] Reporter
	Description • 11 years ago

This bug was filed from the Socorro interface and is 
report bp-e78d512a-4f44-4cc0-839a-713752130920.
=============================================================

We likely need to take a similar fix to the LG Optimus Black crash on the SMS web API.

	Kevin Brosnan [Ex-Mozilla] Reporter
	Comment 1 • 11 years ago

Seems similar to https://bugzilla.mozilla.org/show_bug.cgi?id=856445#c35

	Kevin Brosnan [Ex-Mozilla] Reporter
	Comment 2 • 11 years ago

Crash happens when Firefox is open and connecting to a WiFi network.

STR:
* Open Firefox Beta
* Open the Android WiFi settings
* Connect to a network
* Switch back to Firefox Beta
* Crash dialog appears

	Brad Lassey [:blassey] (use needinfo?) Assignee
	Comment 3 • 11 years ago

from the stack it looks like we're failing on a call to WifiManager.getDhcpInfo(), which requires no permissions and has been around since API level 1 (though deprecated in API level 18)

http://developer.android.com/reference/android/net/wifi/WifiManager.html#getDhcpInfo%28%29

	Alex Keybl [:akeybl]
	Comment 4 • 11 years ago

We'll temporarily block the LG Optimus Black for this bug, although upon further review we may deem these STR non-critical.

	Ioana Chiorean
	Comment 5 • 11 years ago

Reproducing it on LG Optimus 4x (ARMv7 - Paul Feher) and LG Optimus Slider (ARMv6)

	Alex Keybl [:akeybl]
	Comment 6 • 11 years ago

(In reply to Alex Keybl [:akeybl] from comment #4)
> We'll temporarily block the LG Optimus Black for this bug, although upon
> further review we may deem these STR non-critical.

We've been able to reproduce on multiple LG devices at this point - taking a look at the Play Store that would mean blocking 292 devices. Let's only take that drastic action if we start hearing complaints in reviews. We can get a fix into b2, once found.

	Kevin Brosnan [Ex-Mozilla] Reporter
	Comment 7 • 11 years ago

So this is device independent and looks rather bad atm https://crash-stats.mozilla.com/query/?product=FennecAndroid&version=FennecAndroid%3A25.0b1&range_value=1&range_unit=weeks&query_search=signature&query_type=contains&query=&reason=&release_channels=&build_id=&process_type=any&hang_type=any

	Aaron Train [:aaronmt]
	Comment 8 • 11 years ago

Seeing plenty of mixed devices in the reports, not just LG.

	Brad Lassey [:blassey] (use needinfo?) Assignee
	Comment 9 • 11 years ago

so.. the problem here is bug 888268 landed in 25 that requires the WiFi permissions that were already requested for bug 866957.

	Brad Lassey [:blassey] (use needinfo?) Assignee
	Comment 10 • 11 years ago

I haven't built or tested this patch

	Patrick McManus [:mcmanus]
	Comment 11 • 11 years ago

Comment on attachment 807829 [details] [diff] [review]
getDhcpInfo.patch

thanks brad

	Alex Keybl [:akeybl]
	Comment 12 • 11 years ago

Current thinking is to get this landed for Monday, and fast turn around push out Monday or Tuesday. Will of course require verification.

	Aaron Train [:aaronmt]
	Comment 13 • 11 years ago

Reproducible on my LG Nexus 4 (Android 4.3). Safe to say reproducible everywhere.

↳ Launch browser → disable Wi-Fi → enable Wi-Fi  → return to browser → crash

Using the attached patch in Brad's custom build fixes this for me

	Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)
	Comment 14 • 11 years ago

https://hg.mozilla.org/mozilla-central/rev/b0ec597b7453

	Brad Lassey [:blassey] (use needinfo?) Assignee
	Comment 15 • 11 years ago

Comment on attachment 807829 [details] [diff] [review]
getDhcpInfo.patch

[Approval Request Comment]
Bug caused by (feature/regressing bug #): 918378
User impact if declined: 
Testing completed (on m-c, etc.): 
Risk to taking this patch (and alternatives if risky): 
String or IDL/UUID changes made by this patch:

	Robert Kaiser
	Comment 16 • 11 years ago

This is single-handedly pushing the crash rate for 25.0b1 on Android to 13 crashes per 100 ADI

	Alex Keybl [:akeybl]
	Comment 17 • 11 years ago

(In reply to Robert Kaiser (:kairo@mozilla.com) from comment #16)
> This is single-handedly pushing the crash rate for 25.0b1 on Android to 13
> crashes per 100 ADI

It's a beta product, and we're planning on releasing b2 tomorrow morning. We'll make sure to communicate it as fixed in the next update.

	Ryan VanderMeulen [:RyanVM]
	Comment 18 • 11 years ago

https://hg.mozilla.org/releases/mozilla-aurora/rev/e9e0d1bf1425
https://hg.mozilla.org/releases/mozilla-beta/rev/eda4dbed18fe