Closed Bug 1417959 Opened 2 years ago Closed 2 years ago

Remove alternate desktop from Level 4

Categories

(Core :: Security: Process Sandboxing, defect, P1)

All
Windows
defect

Tracking

()

RESOLVED FIXED
mozilla59
Tracking Status
firefox58 --- fixed
firefox59 --- fixed

People

(Reporter: bobowen, Assigned: bobowen)

References

Details

(Whiteboard: sb+)

Attachments

(1 file)

We're getting a load of crashes for bug 1400637 again on Beta, so we need to remove alternate desktop from level 4.
Attachment #8929039 - Flags: review?(jmathies) → review+
Pushed by bobowencode@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/82305e302758
Bump Alternate Desktop to Level 5 and make that the Default on Nightly. r=jimm
Comment on attachment 8929039 [details] [diff] [review]
Bump Alternate Desktop to Level 5 and make that the Default on Nightly

Approval Request Comment
[Feature/Bug causing the regression]:
Alternate desktop in content sandbox (bug 1229829)

[User impact if declined]:
People with certain anti-virus applications get intermittent or constant crashes of the content process (bug 1400637).

[Is this code covered by automated tests?]:
This is a trivial logic change to remove the alternate desktop but keep some other sandboxing measures.

[Has the fix been verified in Nightly?]:
This will not affect Nightly.

[Needs manual test from QE? If yes, steps to reproduce]:
No, as no change on Nightly and on Beta we can rely on the massive reduction in the crash rate.

[List of other uplifts needed for the feature/fix]:
None

[Is the change risky?]:
No

[Why is the change risky/not risky?]:
Very simple logic change.

[String changes made/needed]:
None
Attachment #8929039 - Flags: approval-mozilla-beta?
https://hg.mozilla.org/mozilla-central/rev/82305e302758
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla59
Comment on attachment 8929039 [details] [diff] [review]
Bump Alternate Desktop to Level 5 and make that the Default on Nightly

Let's take this and see how it goes in Beta58. Beta58+.
Attachment #8929039 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
You need to log in before you can comment on or make changes to this bug.