diff options
author | Kent Hansen <kent.hansen@nokia.com> | 2009-11-23 14:26:36 (GMT) |
---|---|---|
committer | Jason McDonald <jason.mcdonald@nokia.com> | 2009-11-24 02:56:06 (GMT) |
commit | 762a7d37f961235cf48364760762c7457897a8f8 (patch) | |
tree | ed2fcec623b34eb0ed64909b1f802f4544603f8a /src/corelib/kernel | |
parent | 551d72554e50a61015b2b95b0cb2517eb7ac633b (diff) | |
download | Qt-762a7d37f961235cf48364760762c7457897a8f8.zip Qt-762a7d37f961235cf48364760762c7457897a8f8.tar.gz Qt-762a7d37f961235cf48364760762c7457897a8f8.tar.bz2 |
Don't crash in eval() function when QtScript debugger is attached
The built-in eval() function bypasses the script registration
performed by QScriptEngine::evaluate(), so if we get an
atStatement() callback from JSC from that script, the
scriptID-to-sourceProvider lookup will fail.
In this case, just return from atStatement() without delivering
the positionChange() callback to the QScriptEngineAgent, since
the agent will not have received the scriptLoad() callback for
that script anyway.
This is a change in behavior from 4.5, but we consider it
the minimum-impact fix at this point to keep 4.6.0 from
crashing. The only downside is that debugging will
effectively be "disabled" for the script passed to eval(),
but that's a lot better than crashing.
Task-number: QTBUG-6108
Reviewed-by: Jedrzej Nowacki
(cherry picked from commit 23002374d11598b26b6585e78dc073071a13f0ec)
Diffstat (limited to 'src/corelib/kernel')
0 files changed, 0 insertions, 0 deletions