Closed Bug 417851 Opened 14 years ago Closed 9 years ago

get rid of access check in ComputeGlobalThis

Categories

(Core :: JavaScript Engine, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: brendan, Assigned: mrbkap)

References

Details

Blake is the man.

/be
Brendan, I can reproduce a crash on exact this lines. Running Venkman and setting a breakpoint at applicationManager.js/appManager_onOk()/"if (!this...)

(gdb) frame 0
#0  JS_Assert (s=0x110e554 "!fp->thisp && fp->argv == argv", file=0x110e124 "/Users/henrik/Projects/mozilla/source/mozilla/js/src/jsinterp.c", ln=928) at /Users/henrik/Projects/mozilla/source/mozilla/js/src/jsutil.c:63
63	    abort();
(gdb) frame 1
#1  0x010626e1 in ComputeGlobalThis (cx=0x3e1162f0, lazy=1, argv=0x3d98778c) at /Users/henrik/Projects/mozilla/source/mozilla/js/src/jsinterp.c:928
928	            JS_ASSERT(!fp->thisp && fp->argv == argv);
(gdb) p fp->thisp
$1 = (JSObject *) 0x3a2be620
(gdb) p fp->argv
$2 = (jsval *) 0x3d98781c
(gdb) p argv
$3 = (jsval *) 0x3d98778c

Should I file that as a new bug or can it be covered here?
Henrik, please file a new bug, CC brendan and me. Thanks.
This actually got fixed quite a while ago.
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.