885770 - Assertion failure: script && script->filename(), at shell/js.cpp

Status: RESOLVED FIXED

(Core :: JavaScript Engine, defect)

Description

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

Attachment: stack

evaluate("\
    for each(a in this) {\
        try {\
            a(__defineGetter__([], decodeURI))\
        } catch (e) {}\
    }\
", {
    fileName: null
})

asserts js debug shell on m-c changeset 61c3c8b85563 without any CLI arguments at Assertion failure: script && script->filename(), at shell/js.cpp

Comment 1

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

autoBisect shows this is probably related to the following changeset:

The first bad revision is:
changeset:   http://hg.mozilla.org/mozilla-central/rev/37474b43d064
user:        Steve Fink
date:        Mon Mar 11 15:36:16 2013 -0700
summary:     Bug 835552 - Make load() be script-relative and read()/snarf() be cwd-relative.. r=njn

Steve, is bug 835552 a likely regressor?

Comment 2

[Steve Fink [:sfink] [:s:]]

Attachment: script filename can be NULL in shell load

Yes it is. (Sorry for the random review request, terrence.)

Comment 3

[Nicholas Nethercote [:njn] (On Long Service Leave)]

Comment on attachment 766078 [details] [diff] [review]
script filename can be NULL in shell load

Review of attachment 766078 [details] [diff] [review]:
-----------------------------------------------------------------

Drive-by!

::: js/src/shell/js.cpp
@@ +797,3 @@
>              return false;
> +        }
> +        if (!str || !(str = ResolvePath(cx, str, scriptRelative))) {

I think you can remove the "!str ||" part of that condition, given that you test for it in the previous if statement.

Comment 4

[Steve Fink [:sfink] [:s:]]

http://hg.mozilla.org/integration/mozilla-inbound/rev/d3ca13f229af

Comment 5

[Steve Fink [:sfink] [:s:]]

(In reply to Nicholas Nethercote [:njn] from comment #3)
> Comment on attachment 766078 [details] [diff] [review]
> script filename can be NULL in shell load
> 
> Review of attachment 766078 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> Drive-by!

Awesome, thanks!

> ::: js/src/shell/js.cpp
> @@ +797,3 @@
> >              return false;
> > +        }
> > +        if (!str || !(str = ResolvePath(cx, str, scriptRelative))) {
> 
> I think you can remove the "!str ||" part of that condition, given that you
> test for it in the previous if statement.

Er, yeah.

Comment 6

[Phil Ringnalda (:philor)]

https://hg.mozilla.org/mozilla-central/rev/d3ca13f229af