Closed Bug 1187031 Opened 4 years ago Closed 4 years ago
[EME] Investigate moving back to a USER
_LOCKDOWN access token level
MozReview Request: Bug 1187031: Move back to using USER_LOCKDOWN for the GMP sandbox policy on Windows. r?aklotz
40 bytes, text/x-review-board-request
The fix for bug 1184333 to ensure that we properly handle Unicode paths, may have fixed the problem that caused us to move from USER_LOCKDOWN back to USER_RESTRICTED. The first thing would be to re-test against Windows 10. This is where we first experienced a similar issue, even though the paths didn't obviously contain Unicode.
Did a quick test with my Windows 10 (build 10162). This still fails with USER_LOCKDOWN even with the Unicode path changes.
The change to remove the side by side assembly manifest from the CDM seems to have fixed this, as expected (see bug 1197007 comment 20). So we should be able to move back to USER_LOCKDOWN on all versions now. I'll remove the old work around code for creating the activation context at the same time. We suspect that that work around failed in some cases and it always failed on Windows 10 anyway.
Assignee: nobody → bobowen.code
Status: NEW → ASSIGNED
Priority P2 because we would like this for the CDM sandbox but it is not a release blocker.
Priority: -- → P2
Bug 1187031: Move back to using USER_LOCKDOWN for the GMP sandbox policy on Windows. r?aklotz This also removes turning off optimization for the Load function. That was an attempt to fix the side-by-side loading. It may also have helped with ensuring that the memsets were not optimized, but that has been fixed by Bug 1208892.
Attachment #8674947 - Flags: review?(aklotz)
The widePath was now only being used further down, so I moved the creation of it down to there. It makes the diff a little more confusing, but I think it's worth it to get rid of a #ifdef. I've tested the try version from comment 5 on Windows 10 against clearkey. I've also tested using Nightly and the sandboxbroker.dll from that try build on Netflix, so that would only have the policy change not the code change in GMPLoader. We can only test that once this is landed because of the voucher.
Comment on attachment 8674947 [details] MozReview Request: Bug 1187031: Move back to using USER_LOCKDOWN for the GMP sandbox policy on Windows. r?aklotz https://reviewboard.mozilla.org/r/22287/#review20187 mmmkay
Attachment #8674947 - Flags: review?(aklotz) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/8481c9619cc4d4098b7acdd3f77f7d1f8d3174c1 Bug 1187031: Move back to using USER_LOCKDOWN for the GMP sandbox policy on Windows. r=aklotz
You need to log in before you can comment on or make changes to this bug.