621123 - Assertion failure: dn->pn_atom == atom, at jsparse.cpp

Status: RESOLVED FIXED

(Core :: JavaScript Engine, defect)

Description

[Christian Holler (:decoder)]

The following two lines

options('strict');
eval("function V(eval){function eval(){}");


cause the following output in my debug shell:

min.js:2: strict warning: redeclaration of argument eval:
min.js:2: strict warning: function V(eval){function eval(){}
min.js:2: strict warning: ..........................^
min.js:2: warning: redefining eval is deprecated:
min.js:2: warning: function V(eval){function eval(){}
min.js:2: warning: ..........................^
Assertion failure: dn->pn_atom == atom, at jsparse.cpp:1346
Aborted


Tested on mozilla-central-trunk.

Comment 1

[Alex Miller]

I managed to reproduce this on Win7_64, so "platform" shouldn't be OS specific.

Comment 2

[Gary Kwong [:gkw] [:nth10sd] (NOT official MoCo now)]

Attachment: stack

Comment 3

[Gary Kwong [:gkw] [:nth10sd] (NOT official MoCo now)]

autoBisect shows this is probably related to the following changeset:

The first bad revision is:
changeset:   49719:e5a8e971443c
user:        Jeff Walden
date:        Mon Aug 16 16:04:08 2010 -0700
summary:     Bug 580932 - Firefox's strict error reporting prevents ES5 test suite from running.  r=jimb over IRC

Comment 4

[David Mandelin [:dmandelin]]

Jeff says the assert doesn't lead to anything particularly bad.

Comment 6

[Gary Kwong [:gkw] [:nth10sd] (NOT official MoCo now)]

Fixed by bug 630770.

autoBisect shows this is probably related to the following changeset:

The first good revision is:
changeset:   61773:e518c56eaffd
user:        Jeff Walden
date:        Wed Feb 02 01:33:23 2011 -0800
summary:     Bug 630770 - Correctly warn about duplicate parameters when the strict option is enabled.  r=jimb