Ticket #6868 (closed Bug: fixed)

Opened 2 years ago

Last modified 2 years ago

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

6868.patch​ (886 bytes) - added by garry.yao 2 years ago.

Change History

comment:1 Changed 2 years ago by Saare

  • Status changed from new to confirmed
  • Milestone set to CKEditor 3.5.1

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

Changed 2 years ago by garry.yao

comment:2 Changed 2 years ago by garry.yao

  • Status changed from confirmed to review
  • Owner set to garry.yao
  • Component changed from General to Core : Parser

comment:3 Changed 2 years ago by tobiasz.cudnik

  • Status changed from review to review_passed

comment:4 Changed 2 years ago by garry.yao

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

Fixed with [6265].

comment:5 Changed 2 years ago by Saare

#6932 is a dup.

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