Opened 8 years ago

Closed 8 years ago

Last modified 7 years ago

#4614 closed Bug (fixed)

Attribute protection fails because of line-break

Reported by: Garry Yao Owned by: Garry Yao
Priority: Normal Milestone: CKEditor 3.1
Component: General Version: SVN (CKEditor) - OLD
Keywords: Confirmed Review+ Cc:

Description (last modified by Garry Yao)

It's quite a serious bug that our "href" protection logic leaks when we have the following form of input:

<!-- line break after tag name -->
<a 
href="http://ckeditor.com" ></a>

Attachments (1)

4614.patch (1.1 KB) - added by Garry Yao 8 years ago.

Download all attachments as: .zip

Change History (6)

Changed 8 years ago by Garry Yao

Attachment: 4614.patch added

comment:1 Changed 8 years ago by Garry Yao

Description: modified (diff)
Keywords: Review? added
Owner: set to Garry Yao
Status: newassigned

Ticket Test added at :
http://ckeditor.t/tt/4614/1.html.

comment:2 Changed 8 years ago by Garry Yao

Priority: NormalHigh

comment:3 Changed 8 years ago by Frederico Caldeira Knabben

Keywords: Review+ added; Review? removed
Priority: HighNormal

Well... not "that" serious. Good fix though.

comment:4 Changed 8 years ago by Garry Yao

Resolution: fixed
Status: assignedclosed

Fixed with [4448].

comment:5 in reply to:  1 Changed 7 years ago by Krzysztof Studnik

Replying to garry.yao:

Ticket Test added at :
http://ckeditor.t/tt/4614/1.html.

Test updated

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