Closed
Bug 1344106
Opened 7 years ago
Closed 7 years ago
Permaorange in browser_content_sandbox_fs.js and browser_content_sandbox_syscalls.js from bug 1337162 when Gecko 54 merges to aurora on 2017-03-06
Categories
(Core :: Security: Process Sandboxing, enhancement)
Tracking
()
RESOLVED
FIXED
mozilla54
Tracking | Status | |
---|---|---|
firefox-esr45 | --- | unaffected |
firefox51 | --- | unaffected |
firefox52 | --- | unaffected |
firefox-esr52 | --- | unaffected |
firefox53 | --- | unaffected |
firefox54 | + | fixed |
People
(Reporter: philor, Unassigned)
References
Details
Attachments
(2 files)
3.68 KB,
patch
|
Details | Diff | Splinter Review | |
59 bytes,
text/x-review-board-request
|
haik
:
review+
|
Details |
https://treeherder.mozilla.org/logviewer.html#?job_id=81162457&repo=try (and all the other flavors of Linux e10s, https://treeherder.mozilla.org/#/jobs?repo=try&revision=ac7638bee4656510e7fc4865dd923122c1e17c24&group_state=expanded&filter-resultStatus=testfailed&filter-searchStr=linux%20e10s%20browser-chrome) - bug 1337162 forgot to tell a couple of tests which make sure sandboxing is disabled on non-nightly that it was enabling sandbox on non-nightly. [Tracking Requested - why for this release]: permaorange and a closed tree when we merge to aurora on Monday.
Reporter | ||
Updated•7 years ago
|
Flags: needinfo?(gpascutto)
Comment 1•7 years ago
|
||
Removes the Linux-specifc todo() calls in sandbox filesystem and syscall tests. These were added to cause failures when Linux sandboxing was ready to ride the trains so we'd make sure to enable the sandbox for !nightly releases. Haven't tested these yet. Wanted to post the patch sooner rather than later to not hold up the integration.
Comment hidden (mozreview-request) |
Comment 5•7 years ago
|
||
mozreview-review |
Comment on attachment 8843261 [details] Bug 1344106 - Remove Linux todos() now that Linux sandboxing is riding the trains. https://reviewboard.mozilla.org/r/117060/#review118732
Attachment #8843261 -
Flags: review?(haftandilian) → review+
Updated•7 years ago
|
Flags: needinfo?(gpascutto)
Pushed by gpascutto@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/c2aac9615aa4 Remove Linux todos() now that Linux sandboxing is riding the trains. r=haik
Comment 7•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/c2aac9615aa4
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla54
You need to log in
before you can comment on or make changes to this bug.
Description
•