Closed Bug 776648 Opened 9 years ago Closed 8 years ago

Pass needed fds to child process through fd remap after fork()


(Core :: IPC, defect)

Gonk (Firefox OS)
Not set



blocking-kilimanjaro +
blocking-basecamp -


(Reporter: cjones, Assigned: kang)


(Blocks 1 open bug)


(Whiteboard: [WebAPI:P0][LOE:M])

There are some files that no-rights content processes need access to, like omni.ja.  When we fork the content process, we should hand it these fds beforehand, before we cut off their access to the file system.

Assigning to Guillaume because he has most of the code to do this already.
blocking-basecamp: --- → ?
Whiteboard: [blocked-on-input]
I forget why we're blocked on input here.  Can anyone enlighten me?
blocking-basecamp: ? → +
Whiteboard: [blocked-on-input] → [blocked-on-input] [WebAPI:P0]
For omni.jar (grepOmni and appOmni) i only found it in XRE_InitCommandLine() at gecko/toolkit/xre/nsAppRunner.cpp.  Ie, that's opened from the child process instead of passing a FD.

I initialize the sandbox (which cuts off access to the filesystem) inside ContentProcess::Init(), after mContent.InitXPCOM(), which, as far as I understand, is run after XRE_InitCommandLine()

If not, the sandbox that cuts off file system access can be initialized elsewhere, too. Once initialized we just can't go back.

We can also initialize it before exec'ing the child process (the restrictions are inherited), but that means all fds need to be opened and passed before hand, which might be a quite large code modification.

Additionally, apps like the camera attempt to open:
/vendor/lib/hw/ and /vendor/lib/ (Nexus S)

I'm auditing them with:

strace -p <PID>  -xx -s64 -e trace=file

And the regular:
ls -l /proc/<PID>/fd/
For open fds.
Whiteboard: [blocked-on-input] [WebAPI:P0] → [blocked-on-input] [WebAPI:P0][LOE:M]
Clearing blocked-on-input since I have no idea why I put it there :)
Whiteboard: [blocked-on-input] [WebAPI:P0][LOE:M] → [WebAPI:P0][LOE:M]
discussed - "Not going to make v1, k9o blockers".
blocking-basecamp: + → -
blocking-kilimanjaro: --- → +
This is needed for OS level strong sandboxing, which won't make v1.
this is AFAIK no longer needed.
the privileges are set after the process has been started and initialized, effectively giving us a similar security level.

see also bug 862082 and bug 790923

as usual, feel free to reopen if this becomes an issue, or if desktop sandboxing for other platforms needs it (in this case, please change the assignee as well)
Closed: 8 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.