Changeset 24270 in webkit
- Timestamp:
- Jul 13, 2007 1:24:51 PM (17 years ago)
- Location:
- S60/trunk/JavaScriptCore
- Files:
-
- 3 edited
Legend:
- Unmodified
- Added
- Removed
-
S60/trunk/JavaScriptCore/ChangeLog
r24111 r24270 1 ligman, reviewed by Yongjun 2 DESC: stop the recursion in ArgumentListNode's destructor. 3 http://bugs.webkit.org/show_bug.cgi?id=14578 4 5 * kjs/nodes.cpp: 6 (ArgumentListNode::~ArgumentListNode): 7 * kjs/nodes.h: 8 1 9 ligman, reviewed by <reviewer> 2 10 DESC: Character class array is defined to 32. If we don't limit characters to 255 an overflow occurs. -
S60/trunk/JavaScriptCore/kjs/nodes.cpp
r21373 r24270 482 482 483 483 // ------------------------------ ArgumentListNode ----------------------------- 484 484 #ifdef NOKIA_CHANGES 485 ArgumentListNode::~ArgumentListNode() 486 { 487 // avoid recusive link destruction. 488 if (list) { 489 // get the next in the list 490 ArgumentListNode* next = list.get(); 491 // break the link between this and the next 492 list.setNull(); 493 while (next) { 494 ArgumentListNode* curr = next; 495 // get the next 496 next = curr->list.get(); 497 // break the link between curr and next 498 curr->list.setNull(); 499 // deref current...this should destroy it. 500 curr->deref(); 501 } 502 } 503 } 504 #endif 505 485 506 Value ArgumentListNode::evaluate(ExecState */*exec*/) 486 507 { -
S60/trunk/JavaScriptCore/kjs/nodes.h
r21373 r24270 308 308 ArgumentListNode(ArgumentListNode *l, Node *e) 309 309 : list(l->list), expr(e) { l->list = this; } 310 #ifdef NOKIA_CHANGES 311 virtual ~ArgumentListNode(); 312 #endif 310 313 Value evaluate(ExecState *exec); 311 314 List evaluateList(ExecState *exec);
Note: See TracChangeset
for help on using the changeset viewer.