Ticket #4614 (closed Bug: fixed)

Opened 4 years ago

Last modified 2 years ago

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) (diff)

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

4614.patch​ (1.1 KB) - added by garry.yao 4 years ago.

Change History

Changed 4 years ago by garry.yao

comment:1 follow-up: ↓ 5 Changed 4 years ago by garry.yao

  • Owner set to garry.yao
  • Keywords Review? added
  • Status changed from new to assigned
  • Description modified (diff)

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

comment:2 Changed 4 years ago by garry.yao

  • Priority changed from Normal to High

comment:3 Changed 4 years ago by fredck

  • Keywords Review+ added; Review? removed
  • Priority changed from High to Normal

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

comment:4 Changed 4 years ago by garry.yao

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

Fixed with [4448].

comment:5 in reply to: ↑ 1 Changed 2 years ago by krst

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 – 2012 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy