Opened 10 years ago

Closed 10 years ago

Last modified 6 years ago

#3744 closed Bug (fixed)

Paste from Word - Stack overflow exception

Reported by: Damian Owned by: Frederico Caldeira Knabben
Priority: Normal Milestone: CKEditor 3.0
Component: Plugin : Paste from Word Version:
Keywords: IBM Confirmed IE Review+ Cc:

Description

When using Paste from Word with a large document (see attachment). The editor hangs and causes a stack overflow exception in IE. As usual, worst offender is IE6.

The test case is to select the content of the Word doc and paste it into the Paste from Word dialog.

I see that there are two issues here:

  1. The editor cannot handle the size/complexity of this document.
  2. The editor needs to handle situations like this more gracefully, notifying the user that the doc is too large etc.

What the restrictions in case (1.)?

Attachments (2)

BigWikiArticle.doc (540.0 KB) - added by Damian 10 years ago.
3744.patch (1.8 KB) - added by Frederico Caldeira Knabben 10 years ago.

Download all attachments as: .zip

Change History (9)

Changed 10 years ago by Damian

Attachment: BigWikiArticle.doc added

comment:1 Changed 10 years ago by Martin Kou

Owner: set to Martin Kou
Status: newassigned

comment:2 Changed 10 years ago by Martin Kou

Keywords: Pending added

I can't reproduce this issue with Office2003/IE6/WinXP. Installing Office 2003 to another computer to test it with IE7 and 8.

comment:3 Changed 10 years ago by Martin Kou

Keywords: Confirmed IE added; Pending removed

Confirmed, I was able to get the browser to hang with IE8.

Changed 10 years ago by Frederico Caldeira Knabben

Attachment: 3744.patch added

comment:4 Changed 10 years ago by Frederico Caldeira Knabben

Keywords: Review? added
Owner: changed from Martin Kou to Frederico Caldeira Knabben
Status: assignednew

This is not actually related to the document size. It's was a problem in our parser when handling the terribly bad HTML generated by Word and accepted by IE. I've added a new test case that illustrates this case.

comment:5 Changed 10 years ago by Martin Kou

Keywords: Review+ added; Review? removed

comment:6 Changed 10 years ago by Frederico Caldeira Knabben

Resolution: fixed
Status: newclosed

Fixed with [3694].

comment:7 Changed 6 years ago by Frederico Caldeira Knabben

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