Closed
Bug 1371759
Opened 9 years ago
Closed 9 years ago
Port bug 1323188 to C-C [Don't use restricting SIDs in the sandbox access tokens when running from a network drive]
Categories
(MailNews Core :: Security, enhancement)
MailNews Core
Security
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird 55.0
People
(Reporter: Paenglab, Assigned: Paenglab)
Details
Attachments
(1 file, 1 obsolete file)
|
2.53 KB,
patch
|
jorgk-bmo
:
review+
|
Details | Diff | Splinter Review |
Simply port it.
|
Assignee | |
Comment 1•9 years ago
|
||
Port it.
|
||
Comment 2•9 years ago
|
||
Comment on attachment 8876232 [details] [diff] [review]
sandbox.patch
What about the other stuff that M-C has and we don't?
+ sandboxing::NetworkDriveCheck();
sandboxing::PermissionsService* permissionsService =
sandboxing::GetPermissionsService();
#if defined(MOZ_CONTENT_SANDBOX)
if (!brokerServices) {
Output("Couldn't initialize the broker services.\n");
return 255;
}
#endif
When was that landed? Should we port this as well?
|
|
Assignee | |
Comment 3•9 years ago
|
||
Uhh, my bad. I inserted it after the wrong line.
Attachment #8876232 -
Attachment is obsolete: true
Attachment #8876232 -
Flags: review?(jorgk)
Attachment #8876238 -
Flags: review?(jorgk)
|
|
||
Comment 4•9 years ago
|
||
Comment on attachment 8876238 [details] [diff] [review]
sandbox.patch
That looks more like it ;-)
Attachment #8876238 -
Flags: review?(jorgk) → review+
|
|
Assignee | |
Updated•9 years ago
|
Keywords: checkin-needed
|
|
||
Comment 5•9 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 55.0
You need to log in
before you can comment on or make changes to this bug.
Description
•