638963 - Fennec crash [@ js::RegExp::executeInternal ] on Samsung Continuum

Status: RESOLVED FIXED

(Core :: JavaScript Engine, defect)

Description

[Logan Rosen [:Logan]]

User-Agent:       Mozilla/5.0 (Windows NT 6.1; WOW64; rv:2.0b13pre) Gecko/20110303 Firefox/4.0b13pre
Build Identifier: 20110304042311

Relevant crashes:
https://crash-stats.mozilla.com/report/index/b19c98d5-1e50-46ff-b1a2-ba7572110304
https://crash-stats.mozilla.com/report/index/bp-7e01861a-462c-4814-ac7c-145942110304

They both are RegExp crashes, and it happens intermittently, but usually the first time loading a website when opening Fennec.

Reproducible: Sometimes

Steps to Reproduce:
1. Get a Samsung Continuum.
2. Download Fennec.
3. Install Fennec.
4. Open Fennec.
5. Load a website.
Actual Results:  
Fennec crashes with a JS error.

Expected Results:  
It should load the website.

Comment 1

[Tony Chung [:tchung]]

Can you specify which website?   does it reproduce on any page?
Also, please add the buildID of fennec.  (goto about:fennec, scroll to the bottom)

Comment 2

[Matt Brubeck (:mbrubeck)]

This device is in the Galaxy S class, and is running Android 2.1 with Linux 2.6.29, just like the other Galaxy S phones that experienced JIT crashes.

We might need to extend the JIT blacklist to this device (SCH-I400).

Comment 3

[Logan Rosen [:Logan]]

It crashes on techcrunch.com, for example.

Build: Mozilla/5.0 (android; Linux armv7l; rv:2.0b13pre) Gecko/20110304
Firefox/4.0b13pre Fennec/4.0b6pre

Comment 4

[Chris Jones [:cjones] inactive; ni?/f?/r? if you need me]

Instead of continuing to inflame this ugly pustule, I wonder if it's time to cut our losses and start refusing to install on the 2.1 galaxy s***.  Skype is already doing this (dunno why though).  If not now, when ...  I had hoped to fix bug 604774 around November.

Samsung and the carriers have just totally dropped the ball.

(Yes, I have a galaxy s*** still running 2.1.)

Comment 5

[Matt Brubeck (:mbrubeck)]

Attachment: patch: add Continuum to JIT blacklist

Comment 6

[Matt Brubeck (:mbrubeck)]

There is a test build with this patch at:
http://people.mozilla.com/~mbrubeck/fennec-4.0b6pre-mbrubeck-continuum.apk

Comment 7

[Matt Brubeck (:mbrubeck)]

Comment on attachment 517037 [details] [diff] [review]
patch: add Continuum to JIT blacklist

Requesting approval2.0: Trivial patch that fixes a crash by adding a new device to the JIT blocklist.

Comment 8

[Matt Brubeck (:mbrubeck)]

http://hg.mozilla.org/mozilla-central/rev/4e771e65764a