Title: [92804] trunk/Source/_javascript_Core
Revision
92804
Author
[email protected]
Date
2011-08-10 17:17:05 -0700 (Wed, 10 Aug 2011)

Log Message

REGRESSION(r92670-r92744): WebKit crashes when opening Gmail
https://bugs.webkit.org/show_bug.cgi?id=66010

Reviewed by Oliver Hunt.

Made sure that Construct calls use() on the this argument.

* dfg/DFGJITCodeGenerator.cpp:
(JSC::DFG::JITCodeGenerator::emitCall):

Modified Paths

Diff

Modified: trunk/Source/_javascript_Core/ChangeLog (92803 => 92804)


--- trunk/Source/_javascript_Core/ChangeLog	2011-08-11 00:00:39 UTC (rev 92803)
+++ trunk/Source/_javascript_Core/ChangeLog	2011-08-11 00:17:05 UTC (rev 92804)
@@ -1,3 +1,15 @@
+2011-08-10  Filip Pizlo  <[email protected]>
+
+        REGRESSION(r92670-r92744): WebKit crashes when opening Gmail
+        https://bugs.webkit.org/show_bug.cgi?id=66010
+
+        Reviewed by Oliver Hunt.
+        
+        Made sure that Construct calls use() on the this argument.
+
+        * dfg/DFGJITCodeGenerator.cpp:
+        (JSC::DFG::JITCodeGenerator::emitCall):
+
 2011-08-10  Mark Hahnenberg  <[email protected]>
 
         JSC should always throw when function arg list is too long

Modified: trunk/Source/_javascript_Core/dfg/DFGJITCodeGenerator.cpp (92803 => 92804)


--- trunk/Source/_javascript_Core/dfg/DFGJITCodeGenerator.cpp	2011-08-11 00:00:39 UTC (rev 92803)
+++ trunk/Source/_javascript_Core/dfg/DFGJITCodeGenerator.cpp	2011-08-11 00:17:05 UTC (rev 92804)
@@ -999,6 +999,9 @@
     m_jit.storePtr(MacroAssembler::TrustedImmPtr(JSValue::encode(jsNumber(numArgs))), addressOfCallData(RegisterFile::ArgumentCount));
     m_jit.storePtr(GPRInfo::callFrameRegister, addressOfCallData(RegisterFile::CallerFrame));
     
+    if (node.op == Construct)
+        use(m_jit.graph().m_varArgChildren[node.firstChild() + 1]);
+    
     for (int argIdx = (node.op == Call ? 0 : 1); argIdx < numArgs; argIdx++) {
         NodeIndex argNodeIndex = m_jit.graph().m_varArgChildren[node.firstChild() + 1 + argIdx];
         JSValueOperand arg(this, argNodeIndex);
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to