Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2042)

Unified Diff: content/common/sandbox_policy.cc

Issue 10832236: Prevent cmsetac.dll from loading in GPU process. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 8 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/common/sandbox_policy.cc
===================================================================
--- content/common/sandbox_policy.cc (revision 150857)
+++ content/common/sandbox_policy.cc (working copy)
@@ -120,6 +120,12 @@
L"CLRGL.ax" // Cyberlink Camera helper.
};
+// The DLLs listed here are known (or under strong suspicion) of causing crashes
+// when they are loaded in the GPU process.
+const wchar_t* const kTroublesomeGpuDlls[] = {
+ L"cmsetac.dll", // Unknown (suspected malware).
+};
+
// Adds the policy rules for the path and path\ with the semantic |access|.
// If |children| is set to true, we need to add the wildcard rules to also
// apply the rule to the subfiles and subfolders.
@@ -242,6 +248,14 @@
BlacklistAddOneDll(kTroublesomePluginDlls[ix], false, policy);
}
+// Same as AddGenericDllEvictionPolicy but specifically for the GPU process.
+// In this we add the blacklisted dlls even if they are not loaded in this
+// process.
+void AddGpuDllEvictionPolicy(sandbox::TargetPolicy* policy) {
+ for (int ix = 0; ix != arraysize(kTroublesomeGpuDlls); ++ix)
+ BlacklistAddOneDll(kTroublesomeGpuDlls[ix], false, policy);
+}
+
// Returns the object path prepended with the current logon session.
string16 PrependWindowsSessionPath(const char16* object) {
// Cache this because it can't change after process creation.
@@ -405,6 +419,7 @@
#endif
AddGenericDllEvictionPolicy(policy);
+ AddGpuDllEvictionPolicy(policy);
#endif
return true;
}
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698