Changeset 2847 in webkit for trunk/JavaScriptCore/kjs/nodes.cpp

Timestamp:

Nov 24, 2002, 2:12:02 AM (23 years ago)

Author:

mjs

Message:

• fixed 3095446 - Crash on AppleScript page due to very long argument list

• kjs/grammar.y: Don't try to construct the argument list in the right order, since that blows out the parser stack.
• kjs/nodes.cpp: (ArgumentsNode::ArgumentsNode): Instead reverse the argument list here.
• kjs/nodes.h: Make ArgumentsNode a friend of ArgumentListNode.
• kjs/grammar.cpp: Updated from grammar.y.

File:

• trunk/JavaScriptCore/kjs/nodes.cpp (modified) (1 diff)

trunk/JavaScriptCore/kjs/nodes.cpp

@@ -616 +616 @@
 ArgumentsNode::ArgumentsNode(ArgumentListNode *l) : list(l)
 {
+  if (list == NULL) {
+    return;
+  }
+
+  ArgumentListNode *prev = list;
+  ArgumentListNode *cur = prev->list;
+  
+  while (cur != NULL) {
+    prev->list = cur->list;
+    cur->list = list;
+    list = cur;
+    cur = prev->list;
+  }
 }