1192393 - InternalError: too much recursion

Status: RESOLVED WONTFIX

(Core :: JavaScript Engine, defect)

Description

[Spandan Veggalam]

User Agent: Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:39.0) Gecko/20100101 Firefox/39.0
Build ID: 20150629114049

Steps to reproduce:

mozilla-central revision 892594bdad30 (build with: --enable-optimize --enable-posix-nspr-emulation --enable-valgrind --enable-gczeal --enable-debug)

options('strict_mode');
var g = newGlobal();
var dbg = new Debugger(g);
dbg.onEnterFrame = function(f) {
    if (g[0] > 2) return;
    f.eval("1");
};
g.eval("1234");


Actual results:

InternalError: too much recursion 



Expected results:

There is no recursion in this fragment. With strict mode most of times, it results in crash.

Comment 1

[Shu-yu Guo [:shu]]

There is recursion in this example.

onEnterFrame is called Debugger eval frames as well, so the |f.eval("1")| actually recursively calls the onEnterFrame hook that's set.

Comment 2

[Jan de Mooij [:jandem]]

Spandan, why are you changing the resolution of multiple bugs from INVALID to WONTFIX? As I understand it, WONTFIX means it's a bug which will never be fixed for some reason, but this was not a bug in the first place.

Comment 3

[Spandan Veggalam]

I was told by a person, that bugs which are not considered for development must be WONTFIX. So thats the reason, changed the status of the '--fuzzing-safe and recursion bugs' reported by me.

Comment 4

[Emanuel Hoogeveen [:ehoogeveen]]

That is not untrue, however INVALID means we don't consider something a bug *at all*. WONTFIX means we consider it a bug, but have decided not to fix it for whatever (legitimate) reason.