Closed
Bug 518942
Opened 15 years ago
Closed 15 years ago
Invalid read [@ NativeToValueBase]
Categories
(Core :: JavaScript Engine, defect)
Tracking
()
RESOLVED
WORKSFORME
People
(Reporter: jruderman, Unassigned)
References
Details
(Keywords: valgrind, Whiteboard: [sg:critical?])
python trace-test.py --valgrind-all ../debug/js
putargsReturn.js and putargsNoReturn.js hit:
Using Valgrind-3.6.0.SVN and LibVEX; rerun with -h for copyright info
Command: ../debug/js -j -e const\ platform='darwin';\ const\ libdir='/Users/jruderman/tracemonkey/js/src/trace-test/lib/'; -f /Users/jruderman/tracemonkey/js/src/trace-test/lib/prolog.js -f /Users/jruderman/tracemonkey/js/src/trace-test/tests/basic/putargsReturn.js
Invalid read of size 4
at 0x132E0A: bool NativeToValueBase<FailDoubleOOMHandler>(JSContext*, long&, JSTraceType_, double*) (jstracer.cpp:2668)
by 0x133294: js_NativeToValue(JSContext*, long&, JSTraceType_, double*) (jstracer.cpp:2766)
by 0x6011C: ArgGetter(JSContext*, JSObject*, long, long*) (jsfun.cpp:510)
by 0xB58F0: JSScopeProperty::get(JSContext*, JSObject*, JSObject*, long*) (jsscope.h:800)
by 0xAF952: js_NativeGet (jsobj.cpp:4279)
by 0xAFF63: js_GetPropertyHelper (jsobj.cpp:4460)
by 0xB00D3: js_GetProperty (jsobj.cpp:4470)
by 0x160F78: JSObject::getProperty(JSContext*, long, long*) (jsobj.h:267)
by 0x85BA3: js_Interpret (jsops.cpp:1933)
by 0x9971A: js_Execute (jsinterp.cpp:1599)
by 0x1EB6B: JS_ExecuteScript (jsapi.cpp:4947)
by 0x8472: Process(JSContext*, JSObject*, char*, int) (shell/js.cpp:436)
Address 0xbfffcf98 is not stack'd, malloc'd or (recently) free'd
Comment 1•15 years ago
|
||
I want to kill the reserve double list really badly. Working on gregor's conservative stack scanning patch which can also scan the native stack, eliminating the need for reserve lists.
Reporter | ||
Comment 2•15 years ago
|
||
To reproduce a little more directly, run this in js/src/trace-tests:
../debug/js -j -e const\ platform='darwin';\ const\ libdir='/Users/jruderman/tracemonkey/js/src/trace-test/lib/'; -f /Users/jruderman/tracemonkey/js/src/trace-test/lib/prolog.js -f /Users/jruderman/tracemonkey/js/src/trace-test/tests/basic/putargsReturn.js
Reporter | ||
Comment 3•15 years ago
|
||
Based on comment 1, marking as dependent on bug 516832.
Depends on: 516832
Reporter | ||
Updated•15 years ago
|
Whiteboard: [sg:critical?]
Comment 4•15 years ago
|
||
is this still reproducible?
Reporter | ||
Comment 5•15 years ago
|
||
WFM on tracemonkey branch.
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → WORKSFORME
Updated•9 years ago
|
Group: core-security → core-security-release
Updated•9 years ago
|
Group: core-security-release
You need to log in
before you can comment on or make changes to this bug.
Description
•