Opened 11 years ago

Closed 11 years ago

#2774 closed Bug (fixed)

plugin:htmlwriter Deadly recursive while dealing with nodes with namespace

Reported by: Garry Yao Owned by: Frederico Caldeira Knabben
Priority: Normal Milestone: CKEditor 3.0
Component: General Version: SVN (FCKeditor) - Retired
Keywords: Cc:

Description

This bug cause unable to switch to 'source' mode after pasting MS-WORD(2003) documents which containing several empty paragraphs into 'wysiwyg' mode, reason reduced as xml namespace on nested nodes:

  1. Open an editor instance with 'source' mode and paste the content of attachment 'msword-Paras-xhtml' into it
  2. Switch to 'wysiwyg' mode and switch back
    • Expected: 'source' mode opened with original content
    • Actual: Editor freezing on FF, Out of Memory reported on IE.

Attachments (1)

msword-paras-xhtml.html (816 bytes) - added by Garry Yao 11 years ago.
Reproducing attachment

Download all attachments as: .zip

Change History (5)

Changed 11 years ago by Garry Yao

Attachment: msword-paras-xhtml.html added

Reproducing attachment

comment:1 Changed 11 years ago by Martin Kou

Owner: set to Martin Kou
Status: newassigned

comment:2 Changed 11 years ago by Martin Kou

Owner: Martin Kou deleted
Status: assignednew

The problem seems to come from the htmldataprocessor plugin, in which it isn't processing tags that are not defined in dtd.js correctly. Fred is looking into the issue, so I'm de-assigning myself from this ticket.

comment:3 Changed 11 years ago by Frederico Caldeira Knabben

Owner: set to Frederico Caldeira Knabben
Status: newassigned

Ok... I've found the problem here. A patch is coming soon.

comment:4 Changed 11 years ago by Frederico Caldeira Knabben

Resolution: fixed
Status: assignedclosed

Fixed with [2917].

I'm not waiting for review this time to not delay the release process at this point. I'm mostly sure these fixes are good, but please let me know if the changes look wrong.

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