Closed
Bug 1272644
Opened 9 years ago
Closed 6 years ago
Script hangs Firefox
Categories
(Core :: General, defect)
Core
General
Tracking
()
RESOLVED
WONTFIX
People
(Reporter: karan.srivas, Unassigned)
References
(Depends on 1 open bug)
Details
(4 keywords)
Attachments
(1 file, 5 obsolete files)
|
(deleted),
text/html
|
Details |
Screenshot of browser crash
User Agent: Mozilla/5.0 (Windows NT 6.3; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/50.0.2661.94 Safari/537.36
Firefox for Android
Steps to reproduce:
1) Just created a script to make cache full.
2) Opened the script.
3) Browser crashed !
Actual results:
When the script is executed the browser hanged totally within few seconds and ultimately it forcefully closed the browser.
This crash worked perfectly on 45 version,46 version and the latest on 46.0.1 on both desktop and mobile platforms.
I request to make this bug as Public Disclosure.
Expected results:
It can affect users as all the current sessions are closed once the script is triggered.
| Comment hidden (obsolete) |
| Comment hidden (obsolete) |
|
Reporter | |
Updated•9 years ago
|
Severity: normal → critical
Component: Untriaged → General
| Comment hidden (obsolete) |
| Comment hidden (obsolete) |
|
|
Reporter | |
Comment 5•9 years ago
|
||
I have tried the script on the latest Mozilla Firefox version (46.0.1) for Android platform.
|
|
Reporter | |
Comment 7•9 years ago
|
||
(In reply to :Gijs Kruitbosch from comment #6)
> Please provide the script/page in question.
<html><script>txt = "a";while(1){txt = txt += "a"; }</script></html>
|
|
Reporter | |
Updated•9 years ago
|
OS: Unspecified → Windows 8.1
Hardware: Unspecified → x86_64
That sounds like another 'dummy' script leading too OOM crash by consming all the RAM, like many others in the past.
And I doubt it worked fine in 45 because I'm able to crash old versions of Firefox.
|
|
Reporter | |
Comment 9•9 years ago
|
||
(In reply to Loic from comment #8)
> That sounds like another 'dummy' script leading too OOM crash by consming
> all the RAM, like many others in the past.
> And I doubt it worked fine in 45 because I'm able to crash old versions of
> Firefox.
You can check in my attachments. It works fine even in the latest version also.
|
||
Comment 11•9 years ago
|
||
There is no crash.
Without e10s Firefox hangs completely and it's not possible to close browser without forcing it with task manager. Stop script warning didn't show.
With e10s Firefox hangs, but it's possible to close browser without using task manager and force close. Stop slow site warning shows, but it didn't do anything.
Blocks: eviltraps
Severity: critical → major
Status: UNCONFIRMED → NEW
Ever confirmed: true
OS: Windows 8.1 → All
Hardware: x86_64 → All
Summary: Firefox Browser Crash → Script hangs Firefox
Version: 46 Branch → unspecified
|
|
||
Comment 12•9 years ago
|
||
Attachment #8752176 -
Attachment is obsolete: true
Attachment #8752177 -
Attachment is obsolete: true
Attachment #8752178 -
Attachment is obsolete: true
Attachment #8752184 -
Attachment is obsolete: true
Attachment #8752185 -
Attachment is obsolete: true
|
|
Reporter | |
Updated•9 years ago
|
Group: firefox-core-security
|
|
Reporter | |
Updated•9 years ago
|
CC list accessible: false
|
||
Updated•9 years ago
|
Group: firefox-core-security
|
|
||
Comment 14•9 years ago
|
||
Morgan, any chance you would be interested in this testcase? There's this, document.write() and window.open() that all present similar problems (cf. bug 685828, bug 380223). From your recent blogpost, you seem like you'd be in an excellent position to kill this stuff off. :-)
Flags: needinfo?(winter2718)
|
||
Comment 15•9 years ago
|
||
(In reply to :Gijs Kruitbosch from comment #14)
> Morgan, any chance you would be interested in this testcase? There's this,
> document.write() and window.open() that all present similar problems (cf.
> bug 685828, bug 380223). From your recent blogpost, you seem like you'd be
> in an excellent position to kill this stuff off. :-)
Yes! This is exactly the sort of situation we're trying to solve with bug 1269917. I'll add it as a blocker here. Thanks for flagging me! :)
Flags: needinfo?(winter2718)
|
|
||
Updated•9 years ago
|
| Comment hidden (obsolete) |
|
||
Comment 18•7 years ago
|
||
(In reply to Virtual_ManPL [:Virtual] - (please needinfo? me - so I will see your comment/reply/question/etc.) from comment #11)
> There is no crash.
There is on Android: brings down the whole browser.
On desktop this essentially just hangs the tab (assuming the default e10s is on), and the slow script dialog shows up and successfully stops the page. I'm not getting a slow script dialog on Android. Do we not have that there? If we did it would be a solution.
Flags: needinfo?(kbrosnan)
Keywords: crash
|
||
Comment 19•7 years ago
|
||
Yes, the slow script dialog exists on Android. Though the browser may hang, crash or the OS will kill us before we get to show the dialog.
Flags: needinfo?(kbrosnan)
|
|
Reporter | |
Comment 20•7 years ago
|
||
So, this we can consider as a bug ???
if yes , what more is needed ?
|
|
||
Comment 21•7 years ago
|
||
It is marked as new which means that others can reproduce it. Hanging the browser is generally not a top priority. This may get resolved as Firefox for Android picks up more parts of mulitprocess Firefox (e10s).
|
||
Comment 22•6 years ago
|
||
This is not an evil trap if it's fixed by e10s. Let's close it.
You need to log in
before you can comment on or make changes to this bug.
Description
•