Closed Bug 638963 Opened 13 years ago Closed 13 years ago

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


(Core :: JavaScript Engine, defect)

Not set



Tracking Status
fennec 2.0+ ---


(Reporter: Logan, Assigned: mbrubeck)



(Keywords: crash, mobile, Whiteboard: [has patch][has review])

Crash Data


(1 file)

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

Relevant crashes:

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.
tracking-fennec: --- → ?
Keywords: crash, mobile
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)
Summary: Fennec crashes due to JS errors on Samsung Continuum → Fennec crash [@ js::RegExp::executeInternal ] on Samsung Continuum
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).
Depends on: 603624, 601222
It crashes on, for example.

Build: Mozilla/5.0 (android; Linux armv7l; rv:2.0b13pre) Gecko/20110304
Firefox/4.0b13pre Fennec/4.0b6pre
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.)
Assignee: general → mbrubeck
Ever confirmed: true
Attachment #517037 - Flags: review?(dvander)
Attachment #517037 - Flags: review?(dvander) → review+
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.
Attachment #517037 - Flags: approval2.0?
Whiteboard: [has patch][has review][needs approval]
Attachment #517037 - Flags: approval2.0? → approval2.0+
Whiteboard: [has patch][has review][needs approval] → [has patch][has review]
Closed: 13 years ago
Resolution: --- → FIXED
Crash Signature: [@ js::RegExp::executeInternal ]
tracking-fennec: ? → 2.0+
You need to log in before you can comment on or make changes to this bug.