Closed
Bug 827801
Opened 8 years ago
Closed 8 years ago
[OS.File] Test that OS.Constants is not defined in content
Categories
(Toolkit :: OS.File, defect)
Tracking
()
RESOLVED
WONTFIX
mozilla21
People
(Reporter: Yoric, Assigned: Yoric)
References
Details
(Whiteboard: [mentor=Yoric][lang=js])
Attachments
(1 file)
2.37 KB,
patch
|
khuey
:
review+
|
Details | Diff | Splinter Review |
To avoid regressions, we should add a test to ensure that non-privileged web workers do not have access to OS.Constants.
Assignee | ||
Comment 1•8 years ago
|
||
Assignee: nobody → dteller
Attachment #699842 -
Flags: review?(khuey)
Attachment #699842 -
Flags: review?(khuey) → review+
Assignee | ||
Updated•8 years ago
|
Depends on: CVE-2013-0774
Assignee | ||
Updated•8 years ago
|
Keywords: checkin-needed
Comment 2•8 years ago
|
||
I landed a test that covers this as part of bug 827193.
Comment 3•8 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/8d16c92e1bf4
Keywords: checkin-needed
Comment 4•8 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/8d16c92e1bf4
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla21
Comment 5•8 years ago
|
||
Fun fact: we don't run tests starting with "test-" instead of "test_".
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Assignee | ||
Comment 6•8 years ago
|
||
Ms2ger: Gasp. How comes there are a few dozens of these on m-c? Well, anyway, since gavin has landed something already, I guess we should revert that patch and mark the bug as WONTFIX.
Comment 7•8 years ago
|
||
(In reply to David Rajchenbach Teller [:Yoric] from comment #6) > Ms2ger: Gasp. How comes there are a few dozens of these on m-c? These are either reftests or resources for other tests to use.
Assignee | ||
Comment 8•8 years ago
|
||
Reverted, as this test is 1/ not executed 2/ not useful. https://hg.mozilla.org/integration/mozilla-inbound/rev/bf0d25a344fb
Comment 9•8 years ago
|
||
Backout of the makefile changes too: https://hg.mozilla.org/integration/mozilla-inbound/rev/a200bde12057 This bug now wontfix?
Assignee | ||
Comment 10•8 years ago
|
||
Indeed
Status: REOPENED → RESOLVED
Closed: 8 years ago → 8 years ago
Resolution: --- → WONTFIX
Comment 11•8 years ago
|
||
Merge of backouts: https://hg.mozilla.org/mozilla-central/rev/bf0d25a344fb https://hg.mozilla.org/mozilla-central/rev/a200bde12057
You need to log in
before you can comment on or make changes to this bug.
Description
•