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

Unified Diff: chrome/installer/util/compat_checks.cc

Issue 300593002: Make omaha, gcapi and uninstall registry accesses use Wow6432Node on 64-bit (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: move the uninstall registry access to 32-bit Created 6 years, 7 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
Index: chrome/installer/util/compat_checks.cc
diff --git a/chrome/installer/util/compat_checks.cc b/chrome/installer/util/compat_checks.cc
index ef50f61ef01c781392b3d0c8f6667cd66217509a..ae54a6458e274ea4ed2562d5666451ebc8a6b2c5 100644
--- a/chrome/installer/util/compat_checks.cc
+++ b/chrome/installer/util/compat_checks.cc
@@ -15,7 +15,8 @@ namespace {
std::wstring GetSEPVersion() {
const wchar_t kProductKey[] =
L"SOFTWARE\\Symantec\\Symantec Endpoint Protection\\SMC";
- base::win::RegKey key(HKEY_LOCAL_MACHINE, kProductKey, KEY_READ);
+ base::win::RegKey key(HKEY_LOCAL_MACHINE, kProductKey,
+ KEY_READ | KEY_WOW64_32KEY);
grt (UTC plus 2) 2014/05/27 16:42:08 won't this miss if Symantec ever ships a 64-bit SE
Will Harris 2014/05/27 19:25:10 We're only checking for anything before 11MR3, whi
grt (UTC plus 2) 2014/05/27 22:06:19 groovy. sounds like good info for a comment in the
Will Harris 2014/05/27 22:29:28 Done.
std::wstring version_str;
key.ReadValue(L"ProductVersion", &version_str);
return version_str;

Powered by Google App Engine
This is Rietveld 408576698