Clear exception set by secMan before proceeding with the alternative security mode.

NEW
Unassigned

Status

()

16 years ago
9 years ago

People

(Reporter: harishd, Unassigned)

Tracking

Trunk
x86
Windows XP
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

2.53 KB, patch
hjtoi-bugzilla
: review+
Details | Diff | Splinter Review
(Reporter)

Description

16 years ago
The new secutiy model code is executed only if the security manager has denied
access. On denying access, the security manager, pushes in an exception into the
JS exception stack that makes the new security code's decision worthless. To
avoid this problem we have to clear the JS exception.
(Reporter)

Comment 1

16 years ago
Created attachment 118466 [details] [diff] [review]
patch v1.0

Thsi patch does the following:
1) Checks if "UnversalBrowserRead" is set.
2) Clears JS exception if CheckSameOrigin fails.
(Reporter)

Updated

16 years ago
Status: NEW → ASSIGNED
(Reporter)

Updated

16 years ago
Attachment #118466 - Flags: superreview?(jst)
Attachment #118466 - Flags: review?(heikki)
(Reporter)

Comment 2

16 years ago
Created attachment 118483 [details] [diff] [review]
patch v1.1
Attachment #118466 - Attachment is obsolete: true
(Reporter)

Updated

16 years ago
Attachment #118466 - Flags: superreview?(jst)
Attachment #118466 - Flags: review?(heikki)
(Reporter)

Updated

16 years ago
Attachment #118483 - Flags: superreview?(jst)
Attachment #118483 - Flags: review?(heikki)
Comment on attachment 118483 [details] [diff] [review]
patch v1.1

sr=jst
Attachment #118483 - Flags: superreview?(jst) → superreview+
Comment on attachment 118483 [details] [diff] [review]
patch v1.1

r=heikki, but please get in touch with mstoltz later to see if there is a
better way to do this.
Attachment #118483 - Flags: review?(heikki) → review+
Assignee: harishd → nobody
Status: ASSIGNED → NEW
QA Contact: ashshbhatt → xml
You need to log in before you can comment on or make changes to this bug.