Closed
Bug 487320
Opened 17 years ago
Closed 17 years ago
named function objects can escape without detection by the upvar analysis
Categories
(Core :: JavaScript Engine, defect, P1)
Core
JavaScript Engine
Tracking
()
VERIFIED
FIXED
mozilla1.9.2a1
People
(Reporter: brendan, Assigned: brendan)
Details
(4 keywords)
Attachments
(2 files, 1 obsolete file)
|
178 bytes,
text/plain
|
Details | |
|
1.21 KB,
patch
|
mrbkap
:
review+
|
Details | Diff | Splinter Review |
Second unaddressed todo item from bug 487215:
* named function expr name not flagged via PND_FUNARG
We can handle this too.
/be
Flags: blocking1.9.1?
Updated•17 years ago
|
Flags: blocking1.9.1? → blocking1.9.1+
| Assignee | ||
Updated•17 years ago
|
Assignee: mrbkap → brendan
Status: NEW → ASSIGNED
Priority: -- → P1
| Assignee | ||
Updated•17 years ago
|
Summary: Crash [@ js_Interpret ] → named function objects can escape without detection by the upvar analysis
| Assignee | ||
Comment 1•17 years ago
|
||
Attachment #371550 -
Attachment is obsolete: true
| Assignee | ||
Comment 2•17 years ago
|
||
See the comment for the whole story.
/be
Attachment #371571 -
Flags: review?(mrbkap)
Updated•17 years ago
|
Attachment #371571 -
Flags: review?(mrbkap) → review+
| Assignee | ||
Comment 3•17 years ago
|
||
Fixed on tm and m-c:
http://hg.mozilla.org/tracemonkey/rev/7758faede5b2
http://hg.mozilla.org/mozilla-central/rev/202671f30992
/be
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
| Assignee | ||
Comment 4•17 years ago
|
||
Tweaked the comment after attaching the patch that got r+ -- see the hgweb links.
/be
Comment 5•17 years ago
|
||
is there a concise test case for this?
| Assignee | ||
Comment 6•17 years ago
|
||
(In reply to comment #5)
> is there a concise test case for this?
The attachment 371570 [details] is pretty short -- is that not good enough?
/be
Comment 8•17 years ago
|
||
Keywords: fixed1.9.1
Updated•16 years ago
|
Status: RESOLVED → VERIFIED
Target Milestone: mozilla1.9.1b4 → mozilla1.9.2a1
Comment 9•13 years ago
|
||
Automatically extracted testcase for this bug was committed:
https://hg.mozilla.org/mozilla-central/rev/efaf8960a929
Flags: in-testsuite? → in-testsuite+
You need to log in
before you can comment on or make changes to this bug.
Description
•