Closed
Bug 1040939
Opened 10 years ago
Closed 10 years ago
Fix startup error "Couldn't load XPCOM" Package sandboxbroker.dll
Categories
(SeaMonkey :: Installer, defect)
Tracking
(seamonkey2.30+ fixed)
RESOLVED
FIXED
seamonkey2.30
People
(Reporter: philip.chee, Assigned: philip.chee)
References
Details
Attachments
(1 file)
1.91 KB,
patch
|
mcsmurf
:
review+
philip.chee
:
review+
|
Details | Diff | Splinter Review |
(from Bug 985252 comment #0) > For security reasons we want to sandbox the OpenH264 plug (which runs in > separate process so it should be straightforward). > Bug 985252 - Sandbox Gecko Media Plugins (including OpenH264) for Windows > - Build sandbox code regardless of whether --enable-content-sandbox/MOZ_CONTENT_SANDBOX is provided. > - Enable sandboxing of GMP plugins. > - Enable `SandboxBroker` to set different security policies for different process types. (From Bug 933947 comment #0) > sandboxbroker.dll should be checked to see if it is in use from the > installer and uninstaller.
Comment 1•10 years ago
|
||
Sorry, I don't follow exactly what this bug is describing. Did I break SeaMonkey with my patch in bug 985252? Is there something I can do to help here?
Assignee | ||
Comment 2•10 years ago
|
||
Bustage fix. STR: 1. make package or make installer. 2. Install (or unpackage). 3. Try to run seamonkey.exe Actual Results: Error message: "Couldn't load XPCOM" Expected Results: SeaMonkey starts up properly Requestees: bugzilla@mcsmurf.de, bugspam.Callek@gmail.com I'll take whoever reviews this first.
Attachment #8458915 -
Flags: review?(bugzilla)
Attachment #8458915 -
Flags: review?(bugspam.Callek)
Assignee | ||
Comment 3•10 years ago
|
||
(In reply to Tim Abraldes [:TimAbraldes] [:tabraldes] from comment #1) > Sorry, I don't follow exactly what this bug is describing. Did I break > SeaMonkey with my patch in bug 985252? Is there something I can do to help > here? Hi Tim, Sorry for the bug spam. SeaMonkey needs to port the packaging changes (browser/installer/*) from https://hg.mozilla.org/mozilla-central/rev/fbd06fa70b84 SeaMonkey patch in Comment 2. I suspect TB will also need the same fix.
Assignee | ||
Comment 4•10 years ago
|
||
Pushed comm-central changeset: 6266e28534c9 https://hg.mozilla.org/comm-central/rev/6266e28534c9
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
status-seamonkey2.30:
--- → fixed
tracking-seamonkey2.30:
--- → +
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.30
Assignee | ||
Comment 5•10 years ago
|
||
Comment on attachment 8458915 [details] [diff] [review] Patch v1.0 Package sandboxer.dll & Installer changes. [check-in comment 4] rs=ewong a=bustagefix over irc rs=ewong a=bustagefix over irc
Attachment #8458915 -
Attachment description: Patch v1.0 Package sandboxer.dll & Installer changes. → Patch v1.0 Package sandboxer.dll & Installer changes. [check-in comment 4] rs=ewong a=bustagefix over irc
Attachment #8458915 -
Flags: review+
Comment 6•10 years ago
|
||
Comment on attachment 8458915 [details] [diff] [review] Patch v1.0 Package sandboxer.dll & Installer changes. [check-in comment 4] rs=ewong a=bustagefix over irc Post-landing review, looks fine.
Attachment #8458915 -
Flags: review?(bugzilla)
Attachment #8458915 -
Flags: review?(bugspam.Callek)
Attachment #8458915 -
Flags: review+
You need to log in
before you can comment on or make changes to this bug.
Description
•