GlobalWindowImpl::SetTimeoutOrInterval() silently fails

VERIFIED FIXED

Status

()

Core
Security: CAPS
P3
normal
VERIFIED FIXED
19 years ago
11 months ago

People

(Reporter: Chris Waterson, Assigned: Norris Boyd)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

19 years ago
If you have a setTimeout() call that runs inline during the _first_ document
load (in viewer only), it silently fails here:

http://lxr.mozilla.org/seamonkey/source/dom/src/base/nsGlobalWindow.cpp#1819

because it can't get the document's principal.

This scares me, because it makes me wonder if we have an off-by-one case where
the timeout is always using the last principal.
(Reporter)

Comment 1

19 years ago
Created attachment 1397 [details]
test case
(Reporter)

Updated

19 years ago
Assignee: mwelch → norris
Component: Security → CAPS
(Assignee)

Updated

19 years ago
Status: NEW → ASSIGNED
(Assignee)

Comment 2

19 years ago
I'd already reworked GlobalWindowImpl::GetPrincipal as part of some other work
I've been doing. With my existing changes, the problem doesn't appear.

My changes affect over thirty files, so getting them into the tree will take a
little effort. I'll try to do it by Monday.
(Assignee)

Updated

19 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 19 years ago
Resolution: --- → FIXED
(Assignee)

Comment 3

19 years ago
Fixed with 8/29 checkin

Comment 4

18 years ago
Moving all CAPS bugs to Security: CAPS component.  CAPS component will be 
deleted.
Component: CAPS → Security: CAPS

Comment 5

18 years ago
Verified.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.