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

Unified Diff: chrome_elf/blacklist/blacklist.cc

Issue 2163803003: [chrome_elf] Removing blacklist finch for dynamic dll changes. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Review fixes. Created 4 years, 5 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 | « chrome_elf/blacklist/blacklist.h ('k') | chrome_elf/blacklist/test/blacklist_test.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome_elf/blacklist/blacklist.cc
diff --git a/chrome_elf/blacklist/blacklist.cc b/chrome_elf/blacklist/blacklist.cc
index 0fdf91d274276396cf9578a67deb37005c7754ae..2b441a6a16efd0885de17d5bd0e42221b6c996b5 100644
--- a/chrome_elf/blacklist/blacklist.cc
+++ b/chrome_elf/blacklist/blacklist.cc
@@ -347,25 +347,7 @@ bool Initialize(bool force) {
VirtualProtect(&g_thunk_storage, sizeof(g_thunk_storage),
PAGE_EXECUTE_READ, &old_protect);
- AddDllsFromRegistryToBlacklist();
-
return NT_SUCCESS(ret) && page_executable;
}
-void AddDllsFromRegistryToBlacklist() {
- std::vector<std::wstring> dlls;
-
- if (!nt::QueryRegValueMULTISZ(nt::HKCU, kRegistryFinchListPath,
- kRegistryFinchListValueName, &dlls) ||
- dlls.empty())
- return;
-
- // Add each DLL to the BL in memory
- for (auto name : dlls) {
- AddDllToBlacklist(name.c_str());
- }
-
- return;
-}
-
} // namespace blacklist
« no previous file with comments | « chrome_elf/blacklist/blacklist.h ('k') | chrome_elf/blacklist/test/blacklist_test.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698