Closed Bug 417851 Opened 17 years ago Closed 13 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: 13 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.