Closed Bug 1277968 Opened 8 years ago Closed 8 years ago

Use the firefox executable for non-plugin child processes on Linux

Categories

(Core :: Security: Process Sandboxing, defect)

Unspecified
Linux
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla53
Tracking Status
firefox49 --- affected
firefox53 --- fixed

People

(Reporter: jld, Assigned: jld)

References

Details

(Whiteboard: sb+)

Attachments

(1 file)

This is a followup for bug 1114647 — using the "firefox" executable to run content processes (instead of the confusingly named plugin-container) can't work on Linux currently because of how the sandbox code makes various things depend on each other, combined with firefox (with libmozsandbox statically linked into it, on desktop) not being able to link directly against libxul the way plugin-container can.

See bug 1268733 comment #9 for more context and an idea about how to deal with this: replace the symbol dependencies between libmozsandbox and libxul by passing appropriate pointers as part of XREChildData, and then linkage won't matter (and, as a side effect, the weak symbol hacks can go away).
Whiteboard: sb+
Bug 1268733 is reopened; adjusting bug summary accordingly.
Assignee: nobody → jld
Summary: Change Linux sandbox build so that the firefox executable can be a content process → Use the firefox executable for non-plugin child processes on Linux
Comment on attachment 8813775 [details]
Bug 1277968 - Use `firefox -contentproc` on sandbox-enabled Linux.

https://reviewboard.mozilla.org/r/95152/#review99432
Attachment #8813775 - Flags: review?(ted) → review+
Pushed by jedavis@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/383136208ab0
Use `firefox -contentproc` on sandbox-enabled Linux. r=ted
https://hg.mozilla.org/mozilla-central/rev/383136208ab0
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla53
Blocks: 1333511
No longer blocks: 1333511
Depends on: 1333511
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: