Closed
Bug 1605867
Opened 4 years ago
Closed 4 years ago
Improve performance when failing to launch sandboxed process.
Categories
(Core :: Security: Process Sandboxing, enhancement, P1)
Tracking
()
RESOLVED
FIXED
mozilla73
People
(Reporter: bobowen, Assigned: bobowen)
Details
Attachments
(2 files)
47 bytes,
text/x-phabricator-request
|
lizzard
:
approval-mozilla-beta+
|
Details | Review |
3.67 KB,
text/plain
|
jcristau
:
approval-mozilla-esr68+
|
Details |
While investigating another performance bug, I noticed that we pointlessly duplicate the sandbox IPC memory when we might fail to launch.
Assignee | ||
Comment 1•4 years ago
|
||
Assignee | ||
Comment 2•4 years ago
|
||
Pushed by bobowencode@gmail.com: https://hg.mozilla.org/integration/autoland/rev/dc6f6fedd1a1 Don't duplicate IPC shared memory when we might fail to launch the process correctly. r=handyman
Assignee | ||
Comment 4•4 years ago
|
||
Comment on attachment 9117818 [details]
Bug 1605867: Don't duplicate IPC shared memory when we might fail to launch the process correctly. r=handyman
Beta/Release Uplift Approval Request
- User impact if declined: Uplift requested via IRC
- Is this code covered by automated tests?: Yes
- Has the fix been verified in Nightly?: No
- Needs manual test from QE?: No
- If yes, steps to reproduce:
- List of other uplifts needed: None
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): Just moves some code after other checks that might mean the process failed to launch correctly.
- String changes made/needed: None
Attachment #9117818 -
Flags: approval-mozilla-beta?
Assignee | ||
Comment 5•4 years ago
|
||
Comment on attachment 9117819 [details]
ESR68 patch Bug 1605867: Don't duplicate IPC shared memory when we might fail to launch the process correctly. r=handyman
ESR Uplift Approval Request
- If this is not a sec:{high,crit} bug, please state case for ESR consideration: Uplift requested on IRC
- User impact if declined: Uplift requested on IRC
- Fix Landed on Version: 73
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): Just moves some code after other checks that might mean the process failed to launch correctly.
- String or UUID changes made by this patch: None
Attachment #9117819 -
Flags: approval-mozilla-esr68?
Comment 6•4 years ago
|
||
Comment on attachment 9117818 [details]
Bug 1605867: Don't duplicate IPC shared memory when we might fail to launch the process correctly. r=handyman
OK for uplift for beta 11.
Attachment #9117818 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Comment 7•4 years ago
|
||
bugherder uplift |
status-firefox72:
--- → fixed
Comment 8•4 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
status-firefox73:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla73
Comment 9•4 years ago
|
||
Comment on attachment 9117819 [details] ***ESR68 patch*** Bug 1605867: Don't duplicate IPC shared memory when we might fail to launch the process correctly. r=handyman approved for 68.4
Attachment #9117819 -
Flags: approval-mozilla-esr68? → approval-mozilla-esr68+
Updated•4 years ago
|
status-firefox71:
--- → wontfix
status-firefox-esr68:
--- → affected
tracking-firefox-esr68:
--- → 72+
Comment 10•4 years ago
|
||
bugherder uplift |
You need to log in
before you can comment on or make changes to this bug.
Description
•