Closed
Bug 1380110
Opened 7 years ago
Closed 7 years ago
--disable-sandbox broken in toolkit/xre/nsAppRunner.cpp
Categories
(Core :: Security: Process Sandboxing, enhancement)
Core
Security: Process Sandboxing
Tracking
()
RESOLVED
FIXED
mozilla56
Tracking | Status | |
---|---|---|
firefox56 | --- | fixed |
People
(Reporter: tjr, Assigned: tjr)
Details
(Whiteboard: sb+)
Attachments
(1 file)
Added a couple of lines without the if guard.
Comment hidden (mozreview-request) |
Comment 2•7 years ago
|
||
mozreview-review |
Comment on attachment 8885416 [details] Bug 1380110 Fix --disable-sandbox by adding #if guards https://reviewboard.mozilla.org/r/156264/#review161486 Its definitely missing the preprocessor directives but these should be MOZ_SANDBOX, not MOZ_CONTENT_SANDBOX. r+ with that change.
Attachment #8885416 -
Flags: review?(davidp99) → review+
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Assignee | ||
Updated•7 years ago
|
Keywords: checkin-needed
Comment 6•7 years ago
|
||
This doesn't meet the review requirements in MozReview for Autoland to push it. http://mozilla-version-control-tools.readthedocs.io/en/latest/mozreview/autoland.html#landing-commits
Keywords: checkin-needed
Updated•7 years ago
|
Flags: needinfo?(tom)
Whiteboard: sb+
Comment 7•7 years ago
|
||
mozreview-review |
Comment on attachment 8885416 [details] Bug 1380110 Fix --disable-sandbox by adding #if guards https://reviewboard.mozilla.org/r/156264/#review162082
Attachment #8885416 -
Flags: review+
Comment 8•7 years ago
|
||
mozreview-review |
Comment on attachment 8885416 [details] Bug 1380110 Fix --disable-sandbox by adding #if guards https://reviewboard.mozilla.org/r/156264/#review162086
Pushed by bobowencode@gmail.com: https://hg.mozilla.org/integration/autoland/rev/5f2d5cf0714f Fix --disable-sandbox by adding #if guards r=bobowen,handyman
Comment 10•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/5f2d5cf0714f
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox56:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla56
Assignee | ||
Updated•7 years ago
|
Flags: needinfo?(tom)
You need to log in
before you can comment on or make changes to this bug.
Description
•