Opened 8 years ago

Closed 7 years ago

#4642 closed Bug (fixed)

Invalid HTML causes infinite loop in IE

Reported by: wwalc Owned by: alfonsoml
Priority: Normal Milestone: FCKeditor 2.6.6
Component: General Version: FCKeditor 2.6.5
Keywords: Confirmed Review+ Cc:


This bug was introduced here:

In IE, paste the following content in the source mode:

<p id="1"><strong></span></p>
<p id="2"><strong></span></p>
<p id="3"><strong></span></p>
<p id="4"><strong></span></p>
<p id="5"><strong></span></p>
<p id="6"><strong></span></p>
<p id="7"><strong></span></p>
<p id="8"><strong></span></p>
<p id="9"><strong></span></p>
<p id="10"><strong></span></p>
<p id="11"><strong></span></p>
<p id="12"><strong></span></p>
<p id="13"><strong></span></p>
<p id="14"><strong></span></p>
<p id="15"><strong></span></p>
<p id="16"><strong></span></p>
<p id="17"><strong></span></p>
<p id="18"><strong></span></p>
<p id="19"><strong></span></p>
<p id="20"><strong></span></p>
<p id="21"><strong></span></p>
<p id="22"><strong></span></p>

then switch back to wysiwyg, press source button again - IE hangs and enters an infinite loop, after a while it asks whether script execution should be stopped.

The problem is caused by FCKXHtml._RemoveXHtmlJobProperties() function in fckxhtml_ie.js.

Attachments (1)

4642.patch (1015 bytes) - added by alfonsoml 7 years ago.
Proposed patch

Download all attachments as: .zip

Change History (4)

Changed 7 years ago by alfonsoml

Proposed patch

comment:1 Changed 7 years ago by alfonsoml

  • Keywords Review? added
  • Milestone set to FCKeditor 2.6.6
  • Owner set to alfonsoml
  • Status changed from new to assigned

comment:2 Changed 7 years ago by wwalc

  • Keywords Review+ added; Review? removed

Looks reasonable and fixed the issue for me.

comment:3 Changed 7 years ago by alfonsoml

  • Resolution set to fixed
  • Status changed from assigned to closed

Fixed with [5113]

Note: See TracTickets for help on using tickets.
© 2003 – 2016 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy