closed window holds its last security context




13 years ago
12 years ago


(Reporter: sync2d, Assigned: jst)


({fixed1.7.13, fixed1.8, qawanted})

Windows 98
fixed1.7.13, fixed1.8, qawanted
Dependency tree / graph
Bug Flags:
blocking1.7.13 +
blocking-aviary1.0.8 +
blocking1.8rc1 -
blocking1.9a1 +
in-testsuite ?

Firefox Tracking Flags

(Not tracked)


(Whiteboard: [sg:high] XSS fixed by 313236)


(2 attachments)



13 years ago
Closed window holds its last security context,
but does not perform any security checks.

Comment 1

13 years ago
Created attachment 200430 [details]

Works on:
Mozilla/5.0 (Windows; U; Win98; en-US; rv:1.9a1) Gecko/20051021 Firefox/1.6a1
Mozilla/5.0 (Windows; U; Win98; en-US; rv:1.8b5) Gecko/20051021 Firefox/1.5
Mozilla/5.0 (Windows; U; Win98; en-US; rv:1.7.12) Gecko/20051021 Firefox/1.0.7
I swear I've recently seen a similar sounding bug that involved playing with closed windows, but I can't find it now.

Confirming. This could lead to an automatic arbitrary code exploit if combined with another bug that allows a CheckLoadURI bypass. Without such a second bug this could still be used as an arbitrary code execution if you can convince a user to open a privileged page in the new window (by typing it into the url bar). That seems somewhat remote, but you might be able to convince a few people (until the word got out) that you were offering useful pref-twiddling steps. Or maybe you could convince someone to drag a disguised chrome URL to the location bar.
Assignee: general → jst
Ever confirmed: true
Flags: blocking1.9a1+
Flags: blocking1.8rc1?
Flags: blocking1.7.13?
Flags: blocking-aviary1.0.8?
Whiteboard: [sg:moderate] stepping stone
So.. a trunk seamonkey gives me a security exception.  A trunk firefox doesn't even open a window on, so clearly something is badly busted there.
My trunk firefox (pulled at 11:30 today) throws a security exception as well. I'm pulling a branch firefox even as I type this.
My branch build does the same thing.

Comment 6

13 years ago
I tested Firefox 1.0.6 (win32) and I see the same thing as well (exception thrown).

Comment 7

13 years ago
Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.9a1) Gecko/20051021 Firefox/1.6a1 - attack succeeds

Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.9a1) Gecko/20051022 Firefox/1.6a1 - attack fails with the following error message:

Error: function Function must be called directly, and not by way of a function of another name
Source File:
Line: 17

That check was added by mrbkap in bug 313236 (the "shinier belt" patch).

Comment 8

13 years ago
In case it matters, I was testing with "Force links that open new windows to open in new tabs instead" checked.

Comment 9

13 years ago
I was able to reproduce in Firefox 1.0.7 RC2, but only after checking "Force links that open new windows to open in new tabs instead", which is hidden in 1.0.x.  (I made browser.tabs.showSingleWindowModePrefs true to make the pref appear.)
Hmm... so what security checks do we do differently here when the thing lives in a tab?

Comment 11

13 years ago
Created attachment 200696 [details]
xss testcase

XSS testcase for reference. works on 20051021 or older builds.
Of course, this does not work on latest trunk/mozilla1.8 builds.


13 years ago
Whiteboard: [sg:moderate] stepping stone → [sg:high] XSS

Comment 12

13 years ago
Dan, can you look into this further? What would it take to get a fix for this and what kind of risk would that add to our release?
Flags: blocking1.8rc1? → blocking1.8rc1-
We can mark this fixed for trunk/1.8 due to bug 313236
Last Resolved: 13 years ago
Depends on: 313236
Flags: blocking1.7.13?
Flags: blocking1.7.13+
Flags: blocking-aviary1.0.8?
Flags: blocking-aviary1.0.8+
Resolution: --- → FIXED


13 years ago
Flags: testcase+
Whiteboard: [sg:high] XSS → [sg:high] XSS fixed by 313236
Keywords: fixed-aviary1.0.8, fixed1.7.13, fixed1.8
qawanted to make sure the bug 313236 really fixed this in older branches.
Keywords: qawanted

Comment 15

13 years ago
I tested these in ff 1.0.8 on all three platforms and passed each.
Keywords: fixed-aviary1.0.8 → verified-aviary1.0.8
Group: security


12 years ago
Flags: in-testsuite+ → in-testsuite?
You need to log in before you can comment on or make changes to this bug.