diff options
author | Gabriel de Dietrich <gabriel.dietrich-de@nokia.com> | 2011-02-15 16:10:20 (GMT) |
---|---|---|
committer | Gabriel de Dietrich <gabriel.dietrich-de@nokia.com> | 2011-02-15 16:10:20 (GMT) |
commit | 96abeb8021c48f9651b6a5c62b7cb427b2bc8ccf (patch) | |
tree | 69a2ed4ce05b1839d6e050668273c70245076d68 /src/script/api/qscriptcontext.cpp | |
parent | a38a14b9fb411231443e80cf14d397492f3412dc (diff) | |
parent | 0a24d2d6269d509e775f7da91c8002eddc2dcbd7 (diff) | |
download | Qt-96abeb8021c48f9651b6a5c62b7cb427b2bc8ccf.zip Qt-96abeb8021c48f9651b6a5c62b7cb427b2bc8ccf.tar.gz Qt-96abeb8021c48f9651b6a5c62b7cb427b2bc8ccf.tar.bz2 |
Merge branch 'master-upstream'
Diffstat (limited to 'src/script/api/qscriptcontext.cpp')
-rw-r--r-- | src/script/api/qscriptcontext.cpp | 9 |
1 files changed, 8 insertions, 1 deletions
diff --git a/src/script/api/qscriptcontext.cpp b/src/script/api/qscriptcontext.cpp index 59ea52d..2468a46 100644 --- a/src/script/api/qscriptcontext.cpp +++ b/src/script/api/qscriptcontext.cpp @@ -299,6 +299,12 @@ QScriptValue QScriptContext::argumentsObject() const //for a js function if (frame->codeBlock() && frame->callee()) { + if (!QScriptEnginePrivate::hasValidCodeBlockRegister(frame)) { + // We have a built-in JS host call. + // codeBlock is needed by retrieveArguments(), but since it + // contains junk, we would crash. Return an invalid value for now. + return QScriptValue(); + } JSC::JSValue result = frame->interpreter()->retrieveArguments(frame, JSC::asFunction(frame->callee())); return QScript::scriptEngineFromExec(frame)->scriptValueFromJSCValue(result); } @@ -309,7 +315,8 @@ QScriptValue QScriptContext::argumentsObject() const } //for a native function - if (!frame->optionalCalleeArguments()) { + if (!frame->optionalCalleeArguments() + && QScriptEnginePrivate::hasValidCodeBlockRegister(frame)) { // Make sure we don't go here for host JSFunctions Q_ASSERT(frame->argumentCount() > 0); //we need at least 'this' otherwise we'll crash later JSC::Arguments* arguments = new (&frame->globalData())JSC::Arguments(frame, JSC::Arguments::NoParameters); frame->setCalleeArguments(arguments); |