Changeset 61588 in webkit for trunk/JavaScriptCore/runtime/Executable.h

Timestamp:

Jun 21, 2010, 4:17:48 PM (15 years ago)

Author:

[email protected]

Message:

2010-06-21 Oliver Hunt <​[email protected]>

Reviewed by Geoffrey Garen.

Make JSC more resilient in the face of parse failures
​https://bugs.webkit.org/show_bug.cgi?id=40951

A number of recent bugs have occurred due to issues like miscounting
BOMs, etc which lead to interesting crashes later on. Adding this
logic hardens JSC in the face of these errors, and has no impact on
performance (32bit jit actually gets 0.7% faster but I put that down
to cache effects).

• bytecode/CodeBlock.cpp: (JSC::CodeBlock::reparseForExceptionInfoIfNecessary): (JSC::CodeBlock::lineNumberForBytecodeOffset): (JSC::CodeBlock::expressionRangeForBytecodeOffset): (JSC::CodeBlock::getByIdExceptionInfoForBytecodeOffset):
• bytecode/CodeBlock.h: (JSC::CodeBlock::bytecodeOffset):
• interpreter/Interpreter.cpp: (JSC::Interpreter::execute): (JSC::Interpreter::executeCall): (JSC::Interpreter::executeConstruct): (JSC::Interpreter::prepareForRepeatCall): (JSC::Interpreter::privateExecute):
• jit/JITOpcodes.cpp: (JSC::JIT::privateCompileCTIMachineTrampolines):
• jit/JITOpcodes32_64.cpp: (JSC::JIT::privateCompileCTIMachineTrampolines):
• jit/JITStubs.cpp: (JSC::DEFINE_STUB_FUNCTION):
• runtime/ArrayPrototype.cpp: (JSC::isNumericCompareFunction):
• runtime/Executable.cpp: (JSC::FunctionExecutable::compileForCall): (JSC::FunctionExecutable::compileForConstruct): (JSC::FunctionExecutable::generateJITCodeForCall): (JSC::FunctionExecutable::generateJITCodeForConstruct): (JSC::FunctionExecutable::reparseExceptionInfo): (JSC::EvalExecutable::reparseExceptionInfo):
• runtime/Executable.h: (JSC::FunctionExecutable::bytecodeForCall): (JSC::FunctionExecutable::bytecodeForConstruct):
• runtime/JSGlobalData.cpp: (JSC::JSGlobalData::numericCompareFunction):

File:

• trunk/JavaScriptCore/runtime/Executable.h (modified) (3 diffs)

trunk/JavaScriptCore/runtime/Executable.h

@@ -306 +306 @@
         }
 
-        FunctionCodeBlock& bytecodeForCall(ExecState* exec, ScopeChainNode* scopeChainNode) 
+        FunctionCodeBlock* bytecodeForCall(ExecState* exec, ScopeChainNode* scopeChainNode) 
         {
             ASSERT(scopeChainNode);
             if (!m_codeBlockForCall)
                 compileForCall(exec, scopeChainNode);
-            return *m_codeBlockForCall;
+            return m_codeBlockForCall;
         }
 
@@ -325 +325 @@
         }
 
-        FunctionCodeBlock& bytecodeForConstruct(ExecState* exec, ScopeChainNode* scopeChainNode) 
+        FunctionCodeBlock* bytecodeForConstruct(ExecState* exec, ScopeChainNode* scopeChainNode) 
         {
             ASSERT(scopeChainNode);
             if (!m_codeBlockForConstruct)
                 compileForConstruct(exec, scopeChainNode);
-            return *m_codeBlockForConstruct;
+            return m_codeBlockForConstruct;
         }
 
@@ -384 +384 @@
         }
 
-        void compileForCall(ExecState*, ScopeChainNode*);
-        void compileForConstruct(ExecState*, ScopeChainNode*);
+        bool compileForCall(ExecState*, ScopeChainNode*);
+        bool compileForConstruct(ExecState*, ScopeChainNode*);
 
         unsigned m_numVariables : 31;