<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>15760</bug_id>
          
          <creation_ts>2007-10-30 01:00:37 -0700</creation_ts>
          <short_desc>Crash with ridiculous number of classes on an element</short_desc>
          <delta_ts>2007-12-14 14:36:16 -0800</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>WebCore Misc.</component>
          <version>523.x (Safari 3)</version>
          <rep_platform>Mac</rep_platform>
          <op_sys>OS X 10.5</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc>http://dscoder.com/crazyclasstest.html</bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="David Smith">catfish.man</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>59835</commentid>
    <comment_count>0</comment_count>
    <who name="David Smith">catfish.man</who>
    <bug_when>2007-10-30 01:00:37 -0700</bug_when>
    <thetext>The destructor for AtomicStringList is recursive, and can run out of stack space when the number of classes is too high. Load the test url, then close the window or reload to trigger the crash. The testcase is currently 14MB; I haven&apos;t yet determined what the cutoff point where it stops crashing is.

A reasonable solution (which I hope to find time to try) would be to make AtomicStringList be Vector based instead of a linked list.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>64207</commentid>
    <comment_count>1</comment_count>
    <who name="Sam Weinig">sam</who>
    <bug_when>2007-12-14 13:59:31 -0800</bug_when>
    <thetext>This was probably fixed in r28722 which switched the data structure to use a Vector.  David, can you verify this.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>64210</commentid>
    <comment_count>2</comment_count>
    <who name="David Smith">catfish.man</who>
    <bug_when>2007-12-14 14:36:16 -0800</bug_when>
    <thetext>This is fixed as of r28722 :)</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>