Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#6868 closed Bug (fixed)

Crash with definition lists (Firefox, Chrome)

Reported by: devtek Owned by: Garry Yao
Priority: Normal Milestone: CKEditor 3.5.1
Component: Core : Parser Version: 3.4.2
Keywords: Cc:

Description

I am facing the same problem as this user: http://cksource.com/forums/viewtopic.php?f=11&t=20955&start=0

The following source code makes CKEditor 3.4.2 crash when switching to wysiwyg mode:

<dd>crash</dd><dd>crash</dd>

Note that this one does not:

<dd>crash</dd>

And apparently there is no problem with well-formed definition lists.

Attachments (1)

6868.patch (886 bytes) - added by Garry Yao 7 years ago.

Download all attachments as: .zip

Change History (6)

comment:1 Changed 7 years ago by Sa'ar Zac Elias

Milestone: CKEditor 3.5.1
Status: newconfirmed

Note that dd and dt must be enclosed within dl. Similar to #5626.

Changed 7 years ago by Garry Yao

Attachment: 6868.patch added

comment:2 Changed 7 years ago by Garry Yao

Component: GeneralCore : Parser
Owner: set to Garry Yao
Status: confirmedreview

comment:3 Changed 7 years ago by Tobiasz Cudnik

Status: reviewreview_passed

comment:4 Changed 7 years ago by Garry Yao

Resolution: fixed
Status: review_passedclosed

Fixed with [6265].

comment:5 Changed 7 years ago by Sa'ar Zac Elias

#6932 is a dup.

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