1746290 - Remove IPDL `intr` from the PRemoteSandboxBroker protocol

Status: RESOLVED FIXED

(Core :: Security: Process Sandboxing, task)

Description

[Jed Davis [:jld] ⟨⏰|UTC-8⟩ ⟦he/him⟧]

PRemoteSandboxBroker::LaunchApp is similar to PGMP::StartPlugin: it's basically a sync message but it's in the parent-to-child direction, which isn't allowed for normal sync messages. However, nested sync messages can support that; intr isn't required.

Comment 1

[Jed Davis [:jld] ⟨⏰|UTC-8⟩ ⟦he/him⟧]

Attachment: Bug 1746290 - Remove `intr` from the `PRemoteSandboxBroker` protocol.

PRemoteSandboxBroker::LaunchApp is similar to PGMP::StartPlugin:
it's basically a sync message but it's in the parent-to-child direction,
which isn't allowed for normal sync messages. However, nested sync
messages can support that; intr isn't required.

Comment 2

[BugBot [:suhaib / :marco/ :calixte]]

There's a r+ patch which didn't land and no activity in this bug for 2 weeks.
:jld, could you have a look please?
For more information, please visit auto_nag documentation.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/c68bbade4f81
Remove `intr` from the `PRemoteSandboxBroker` protocol. r=nika,bobowen

Comment 4

[Alexandru Michis [:malexandru]]

https://hg.mozilla.org/mozilla-central/rev/c68bbade4f81