Closed Bug 1615139 Opened 6 months ago Closed 5 months ago

We run RestoreImportDirectory for a sandbox process twice

Categories

(Firefox :: Launcher Process, task, P3)

Unspecified
Windows
task

Tracking

()

RESOLVED FIXED
Firefox 76
Tracking Status
firefox76 --- fixed

People

(Reporter: toshi, Assigned: toshi)

Details

Attachments

(1 file)

In the browser process, firefox!mozilla::RestoreImportDirectory is called twice: from SandboxBroker::LaunchApp first, and then from InitializeDllBlocklistOOP inside dllSvc->InitDllBlocklistOOP. We can remove the first one.

Since InitializeDllBlocklistOOP does RestoreImportDirectory,
SandboxBroker::LaunchApp does not have to call it before
calling dllSvc->InitDllBlocklistOOP.

Pushed by cbrindusan@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/2b63cf3ee009
Remove RestoreImportDirectory from SandboxBroker::LaunchApp. r=aklotz
Status: NEW → RESOLVED
Closed: 5 months ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 76
You need to log in before you can comment on or make changes to this bug.