Opened 14 years ago

Closed 14 years ago

#5434 closed Bug (fixed)

Access Denied error on IE7 and IPv6

Reported by: Damian Owned by: Frederico Caldeira Knabben
Priority: Normal Milestone: CKEditor 3.3
Component: General Version:
Keywords: IE7 IBM Review+ Cc: satya_minnekanti@…

Description

We are testing the editor in an IPv6 environment on IE7.

Script debugger reports the error at:

return i&&new g(i.contentWindow.document);

in ckditor.js. This looks to be in issue in element.js.

I suspect it is the call to contentWindow.document that is causing the "Access Denied" error.

The editor loads, but is not usable.

Attachments (2)

5434.patch (454 bytes) - added by Damian 14 years ago.
Proposed fix
5434_2.patch (533 bytes) - added by Frederico Caldeira Knabben 14 years ago.

Download all attachments as: .zip

Change History (11)

comment:1 Changed 14 years ago by Frederico Caldeira Knabben

Keywords: IE7 added

Works well with IE8, in all modes.

comment:2 Changed 14 years ago by Satya Minnekanti

Cc: satya_minnekanti@… added

comment:3 Changed 14 years ago by Garry Yao

Keywords: Pending added

WFM in IE7, with both IP and normal domain name.

Changed 14 years ago by Damian

Attachment: 5434.patch added

Proposed fix

comment:4 Changed 14 years ago by Damian

The added patch indicates the problem.

In IE7 document.domain = "[fe80::fe80:fe80:fe80:fe80]"
and window.location.hostname = "fe80::fe80:fe80:fe80:fe80" (i.e. no brackets [] )

This difference is causing isCustomDomain() to return true when in fact it should return false.

Changed 14 years ago by Frederico Caldeira Knabben

Attachment: 5434_2.patch added

comment:5 Changed 14 years ago by Frederico Caldeira Knabben

Keywords: Review? added; Pending removed

@damo, wonderful job. thanks for the patch.

I've provided a new patch, based on your code, which should produce a more compact output. Please give us the last ok for it, so it'll get approved and committed. Thanks!

comment:6 Changed 14 years ago by Frederico Caldeira Knabben

Owner: set to Frederico Caldeira Knabben
Status: newassigned

comment:7 Changed 14 years ago by Damian

Thanks Fred! The proposed fix works.

One side note here is that the user must add the host to their list of trusted sites in IE otherwise the "Access Denied" error will still occur. There doesn't seem to be any way around this.

comment:8 Changed 14 years ago by Garry Yao

Keywords: Review+ added; Review? removed

comment:9 Changed 14 years ago by Frederico Caldeira Knabben

Resolution: fixed
Status: assignedclosed

Fixed with [5407].

Note: See TracTickets for help on using tickets.
© 2003 – 2022, CKSource sp. z o.o. sp.k. All rights reserved. | Terms of use | Privacy policy